id	summary	reporter	owner	description	type	status	component	version	severity	resolution	keywords	cc	stage	has_patch	needs_docs	needs_tests	needs_better_patch	easy	ui_ux
5232	DecimalField max_digits counts negative sign as a digit.	Andrew Durdin <adurdin@…>	Adrian Holovaty	"In [5680], the validation of decimal values for DecimalField was changed to fix #4807, which also introduced a bug in validating negative decimals, in that it counts the - sign as one of the digits:

{{{
>>> import django.core.validators
>>> v = django.core.validators.IsValidDecimal(max_digits=5, decimal_places=2)
>>> v('999.99', {})
>>> v('-99.99', {})
>>> v('-999.99', {})  # This should succeed
Traceback (most recent call last):
...
django.core.validators.ValidationError: [u'Please enter a valid decimal number with at most 5 total digits.']
}}}

This error affects django.core.validators.IsValidDecimal and django.newforms.fields.DecimalField.

The attached patch fixes the problem and adds a regression test for it"		closed	Validators	dev		fixed			Ready for checkin	1	0	0	0	0	0