From d11038acb2ea2f59a1d00a41b553f578b5f2c59c Mon Sep 17 00:00:00 2001 From: Claude Paroz Date: Mon, 31 Dec 2012 10:18:59 +0100 Subject: [PATCH] Fixed #19537 -- Made CheckboxInput._has_changed handle 'False' string Thanks dibrovsd@gmail.com for the report. --- django/forms/widgets.py | 3 +++ tests/regressiontests/forms/tests/widgets.py | 2 ++ 2 files changed, 5 insertions(+) diff --git a/django/forms/widgets.py b/django/forms/widgets.py index c761ea857d..4782b99117 100644 --- a/django/forms/widgets.py +++ b/django/forms/widgets.py @@ -533,6 +533,9 @@ class CheckboxInput(Widget): def _has_changed(self, initial, data): # Sometimes data or initial could be None or '' which should be the # same thing as False. + if initial == 'False': + # show_hidden_initial may have transformed False to 'False' + initial = False return bool(initial) != bool(data) class Select(Widget): diff --git a/tests/regressiontests/forms/tests/widgets.py b/tests/regressiontests/forms/tests/widgets.py index 31c0e65c7c..f9dc4a7ec8 100644 --- a/tests/regressiontests/forms/tests/widgets.py +++ b/tests/regressiontests/forms/tests/widgets.py @@ -240,6 +240,8 @@ class FormsWidgetTestCase(TestCase): self.assertTrue(w._has_changed(False, 'on')) self.assertFalse(w._has_changed(True, 'on')) self.assertTrue(w._has_changed(True, '')) + # Initial value may have mutated to a string due to show_hidden_initial (#19537) + self.assertTrue(w._has_changed('False', 'on')) def test_select(self): w = Select()