Ticket #7132: 7132.selectmultiple-fix.2.diff

a/django/contrib/admin/widgets.py

@@ -7 +7 @@
-from django.utils.text import capfirst, truncate_words
-from django.utils.translation import ugettext as _
-from django.utils.safestring import mark_safe
+from django.utils.encoding import force_unicode
-from django.conf import settings
-
-class FilteredSelectMultiple(forms.SelectMultiple):
@@ -135 +136 @@
-        if value:
-            return [value]
-        return None
+    
+    def _has_changed(self, initial, data):
+        if data is None:
+            data = []
+        if len(initial) != len(data):
+            return True
+        for pk1, pk2 in zip(initial, data):
+            if force_unicode(pk1) != force_unicode(pk2):
+                return True
+        return False
-
-class RelatedFieldWidgetWrapper(object):
-    """

a/django/newforms/widgets.py

@@ -401 +401 @@
-        if isinstance(data, MultiValueDict):
-            return data.getlist(name)
-        return data.get(name, None)
+    
+    def _has_changed(self, initial, data):
+        if data is None:
+            data = []
+        if len(initial) != len(data):
+            return True
+        for pk1, pk2 in zip(initial, data):
+            if force_unicode(pk1) != force_unicode(pk2):
+                return True
+        return False
-
-class RadioInput(StrAndUnicode):
-    """

a/tests/regressiontests/admin_widgets/models.py

@@ -66 +66 @@
->>> w = ManyToManyRawIdWidget(rel)
->>> print conditional_escape(w.render('test', [m1.pk, m2.pk], attrs={}))
-<input type="text" name="test" value="1,2" class="vManyToManyRawIdAdminField" /><a href="../../../admin_widgets/member/" class="related-lookup" id="lookup_id_test" onclick="return showRelatedObjectLookupPopup(this);"> <img src="%(ADMIN_MEDIA_PREFIX)simg/admin/selector-search.gif" width="16" height="16" alt="Lookup"></a>
+>>> w._has_changed([], None)
+False
+>>> w._has_changed([1, 2], [u'1', u'2'])
+False
+>>> w._has_changed([1, 2], [u'1'])
+True
+>>> w._has_changed([1, 2], [u'1', u'3'])
+True
-
-""" % {
-    'ADMIN_MEDIA_PREFIX': settings.ADMIN_MEDIA_PREFIX,

a/tests/regressiontests/forms/widgets.py

@@ -612 +612 @@
->>> w.render('nums', ['ŠĐĆŜćşšđ'], choices=[('ŠĐĆŜćşšđ', 'ŠĐabcĆŜćşšđ'), ('ćşšđ', 'abcćşšđ')])
-u'<select multiple="multiple" name="nums">\n<option value="1">1</option>\n<option value="2">2</option>\n<option value="3">3</option>\n<option value="\u0160\u0110\u0106\u017d\u0107\u017e\u0161\u0111" selected="selected">\u0160\u0110abc\u0106\u017d\u0107\u017e\u0161\u0111</option>\n<option value="\u0107\u017e\u0161\u0111">abc\u0107\u017e\u0161\u0111</option>\n</select>'
-
+# Test the usage of _has_changed
+>>> w._has_changed([], None)
+False
+>>> w._has_changed([1, 2], [u'1', u'2'])
+False
+>>> w._has_changed([1, 2], [u'1'])
+True
+>>> w._has_changed([1, 2], [u'1', u'3'])
+True
+
-# RadioSelect Widget ##########################################################
-
->>> w = RadioSelect()
@@ -914 +924 @@
->>> w.render('nums', ['ŠĐĆŜćşšđ'], choices=[('ŠĐĆŜćşšđ', 'ŠĐabcĆŜćşšđ'), ('ćşšđ', 'abcćşšđ')])
-u'<ul>\n<li><label><input type="checkbox" name="nums" value="1" /> 1</label></li>\n<li><label><input type="checkbox" name="nums" value="2" /> 2</label></li>\n<li><label><input type="checkbox" name="nums" value="3" /> 3</label></li>\n<li><label><input checked="checked" type="checkbox" name="nums" value="\u0160\u0110\u0106\u017d\u0107\u017e\u0161\u0111" /> \u0160\u0110abc\u0106\u017d\u0107\u017e\u0161\u0111</label></li>\n<li><label><input type="checkbox" name="nums" value="\u0107\u017e\u0161\u0111" /> abc\u0107\u017e\u0161\u0111</label></li>\n</ul>'
-
+>>> w._has_changed([], None)
+False
+>>> w._has_changed([1, 2], [u'1', u'2'])
+False
+>>> w._has_changed([1, 2], [u'1'])
+True
+>>> w._has_changed([1, 2], [u'1', u'3'])
+True
+
-# MultiWidget #################################################################
-
->>> class MyMultiWidget(MultiWidget):