diff --git a/django/contrib/admin/widgets.py b/django/contrib/admin/widgets.py
index f501ffb1a85..68575794f6a 100644
--- a/django/contrib/admin/widgets.py
+++ b/django/contrib/admin/widgets.py
@@ -235,7 +235,6 @@ class RelatedFieldWidgetWrapper(forms.Widget):
admin interface.
"""
def __init__(self, widget, rel, admin_site, can_add_related=None):
- self.is_hidden = widget.is_hidden
self.needs_multipart_form = widget.needs_multipart_form
self.attrs = widget.attrs
self.choices = widget.choices
@@ -256,6 +255,10 @@ class RelatedFieldWidgetWrapper(forms.Widget):
memo[id(self)] = obj
return obj
+ @property
+ def is_hidden(self):
+ return self.widget.is_hidden
+
@property
def media(self):
return self.widget.media
diff --git a/django/forms/widgets.py b/django/forms/widgets.py
index 5c50ca049f5..a9754bbc4ee 100644
--- a/django/forms/widgets.py
+++ b/django/forms/widgets.py
@@ -166,7 +166,6 @@ class SubWidget(object):
class Widget(six.with_metaclass(MediaDefiningClass)):
- is_hidden = False # Determines whether this corresponds to an .
needs_multipart_form = False # Determines does this widget need multipart form
is_localized = False
is_required = False
@@ -183,6 +182,18 @@ class Widget(six.with_metaclass(MediaDefiningClass)):
memo[id(self)] = obj
return obj
+ @property
+ def is_hidden(self):
+ return self.input_type == 'hidden' if hasattr(self, 'input_type') else False
+
+ @is_hidden.setter
+ def is_hidden(self, *args):
+ warnings.warn(
+ "`is_hidden` property is now read-only (and checks `input_type`). "
+ "Please update your code.",
+ DeprecationWarning, stacklevel=2
+ )
+
def subwidgets(self, name, value, attrs=None, choices=()):
"""
Yields all "subwidgets" of this widget. Used only by RadioSelect to
@@ -286,7 +297,6 @@ class PasswordInput(TextInput):
class HiddenInput(Input):
input_type = 'hidden'
- is_hidden = True
class MultipleHiddenInput(HiddenInput):
@@ -778,6 +788,10 @@ class MultiWidget(Widget):
self.widgets = [w() if isinstance(w, type) else w for w in widgets]
super(MultiWidget, self).__init__(attrs)
+ @property
+ def is_hidden(self):
+ return all(w.is_hidden for w in self.widgets)
+
def render(self, name, value, attrs=None):
if self.is_localized:
for widget in self.widgets:
@@ -865,10 +879,7 @@ class SplitHiddenDateTimeWidget(SplitDateTimeWidget):
"""
A Widget that splits datetime input into two inputs.
"""
- is_hidden = True
-
def __init__(self, attrs=None, date_format=None, time_format=None):
super(SplitHiddenDateTimeWidget, self).__init__(attrs, date_format, time_format)
for widget in self.widgets:
widget.input_type = 'hidden'
- widget.is_hidden = True
diff --git a/docs/releases/1.7.txt b/docs/releases/1.7.txt
index c0cacb2ec73..4d1890b3d95 100644
--- a/docs/releases/1.7.txt
+++ b/docs/releases/1.7.txt
@@ -1100,6 +1100,9 @@ Miscellaneous
This widget now respects the form field's ``is_required`` attribute like
other widgets.
+* ``Widget.is_hidden`` is now a read-only property, getting its value by
+ introspecting the presence of ``input_type == 'hidden'``.
+
* :meth:`~django.db.models.query.QuerySet.select_related` now chains in the
same way as other similar calls like ``prefetch_related``. That is,
``select_related('foo', 'bar')`` is equivalent to