Skip to content
Browse files

[soc2009/admin-ui] Fix for ticket #11720: "When an OrderField is on a…

…n inline and there are empty inlines, saving throws an error"

If the only things changed in an empty form are OrderField(s), it is treated as untouched.

git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
1 parent 687a5af commit 3a99684fe2bca8a3745943aee5bd480874dc51ce @zain zain committed
Showing with 12 additions and 1 deletion.
  1. +12 −1 django/forms/
13 django/forms/
@@ -9,7 +9,7 @@
from django.utils.encoding import StrAndUnicode, smart_unicode, force_unicode
from django.utils.safestring import mark_safe
-from fields import Field, FileField
+from fields import Field, FileField, OrderField
from widgets import Media, media_property, TextInput, Textarea
from util import flatatt, ErrorDict, ErrorList, ValidationError
@@ -271,6 +271,7 @@ def has_changed(self):
def _get_changed_data(self):
if self._changed_data is None:
self._changed_data = []
+ order_fields = []
# XXX: For now we're asking the individual widgets whether or not the
# data has changed. It would probably be more efficient to hash the
# initial data, store it in a hidden field, and compare a hash of the
@@ -288,7 +289,17 @@ def _get_changed_data(self):
initial_value = hidden_widget.value_from_datadict(, self.files, initial_prefixed_name)
if field.widget._has_changed(initial_value, data_value):
+ if isinstance(field, OrderField):
+ order_fields.append(name)
+ # if this is an empty form and only the OrderField is changed, we want to treat
+ # it as untouched
+ if self.empty_permitted and order_fields != []:
+ difference = filter(lambda x: x not in self._changed_data, order_fields)
+ if difference == []:
+ self._changed_data = []
return self._changed_data
changed_data = property(_get_changed_data)

0 comments on commit 3a99684

Please sign in to comment.
Something went wrong with that request. Please try again.