Ticket #14158: sm_has_changed.diff

django/forms/widgets.py

@@ -527 +527 @@
             data = []
         if len(initial) != len(data):
             return True
-        for value1, value2 in zip(initial, data):
-            if force_unicode(value1) != force_unicode(value2):
-                return True
-        return False
+        initial_set = set([force_unicode(value) for value in initial])
+        data_set = set([force_unicode(value) for value in data])
+        if len(data_set - initial_set) == 0:
+            return False
+        else:
+            return True
 
 class RadioInput(StrAndUnicode):
     """

tests/regressiontests/forms/widgets.py

@@ -673 +673 @@
 True
 >>> w._has_changed([1, 2], [u'1', u'3'])
 True
+>>> w._has_changed([2, 1], [u'1', u'2'])
+False
 
 # Choices can be nested one level in order to create HTML optgroups:
 >>> w.choices = (('outer1', 'Outer 1'), ('Group "1"', (('inner1', 'Inner 1'), ('inner2', 'Inner 2'))))