Changeset 4312

Timestamp:

01/12/07 23:08:07 (2 years ago)

Author:

adrian

Message:

Fixed #3293 -- newforms: Changed IntegerField?.clean() to return None if field is not required and empty. Thanks, Honza Kral

Files:

• django/trunk/django/newforms/fields.py (modified) (1 diff)
• django/trunk/tests/regressiontests/forms/tests.py (modified) (1 diff)

django/trunk/django/newforms/fields.py

@@ -115 +115 @@
-        """
-        Validates that int() can be called on the input. Returns the result
-
+ Returns None for empty values.
-        """
-        super(IntegerField, self).clean(value)
-        if not self.required and value in EMPTY_VALUES:
-u''
+None
-        try:
-            value = int(value)

django/trunk/tests/regressiontests/forms/tests.py

@@ -811 +811 @@
->>> f = IntegerField(required=False)
->>> f.clean('')
-
+
+
->>> f.clean(None)
-
+
+
->>> f.clean('1')
-1