From 947f8e3485b1406d18ea062fd275c083d7ffbed5 Mon Sep 17 00:00:00 2001 From: Carlton Gibson Date: Thu, 21 Nov 2019 20:53:31 +0100 Subject: [3.0.x] Fixed #31012 -- Reverted "Fixed #29056 -- Fixed HTML5 validation of required SelectDateWidget." This reverts commit f038214d917c982613f5a15db8dfe325b1f7479b. The initial issue was incorrect. Django 2.2, and before, did not generate invalid HTML as reported. With f03821 in place invalid HTML was generated. Thanks to Kevin Brown for follow-up report and investigation. Backport of ee4a19053a32d41cdd79e087b1968980804ce658 from master --- .../widget_tests/test_selectdatewidget.py | 40 ---------------------- 1 file changed, 40 deletions(-) (limited to 'tests') diff --git a/tests/forms_tests/widget_tests/test_selectdatewidget.py b/tests/forms_tests/widget_tests/test_selectdatewidget.py index 29f2f2399d..ea1c31b487 100644 --- a/tests/forms_tests/widget_tests/test_selectdatewidget.py +++ b/tests/forms_tests/widget_tests/test_selectdatewidget.py @@ -314,46 +314,6 @@ class SelectDateWidgetTest(WidgetTest): self.assertFalse(GetNotRequiredDate().fields['mydate'].widget.is_required) self.assertTrue(GetRequiredDate().fields['mydate'].widget.is_required) - def test_selectdate_required_placeholder(self): - for required in (True, False): - field = DateField(widget=SelectDateWidget(years=('2018', '2019')), required=required) - self.check_html(field.widget, 'my_date', '', html=( - """ - - - - """ % { - 'days_options': '\n'.join( - '' % (i, i) for i in range(1, 32) - ), - 'm_placeholder': 'placeholder="Month"' if required else '', - 'd_placeholder': 'placeholder="Day"' if required else '', - 'y_placeholder': 'placeholder="Year"' if required else '', - 'empty': '' if required else '', - } - )) - def test_selectdate_empty_label(self): w = SelectDateWidget(years=('2014',), empty_label='empty_label') -- cgit v1.3