Changeset 9569

Timestamp:

12/04/08 22:22:00 (1 month ago)

Author:

mtredinnick

Message:

Added a way to iterate over hidden/visible fields in a form. Useful for manual
form layout.

Files:

• django/trunk/django/forms/forms.py (modified) (2 diffs)
• django/trunk/docs/topics/forms/index.txt (modified) (1 diff)
• django/trunk/tests/regressiontests/forms/forms.py (modified) (1 diff)

django/trunk/django/forms/forms.py

@@ -304 +304 @@
-        return False
-
+    def hidden_fields(self):
+        """
+        Returns a list of all the BoundField objects that correspond to hidden
+        fields in the HTML output. Useful for manual form layout in templates.
+        """
+        return [field for field in self if field.is_hidden]
+
+    def visible_fields(self):
+        """
+        Returns a lits of BoundField objects that do not correspond to hidden
+        fields. The opposite of the hidden_fields() method.
+        """
+        return [field for field in self if not field.is_hidden]
+
-class Form(BaseForm):
-    "A collection of Fields, plus their associated data."
@@ -364 +378 @@
-            name = self.html_initial_name
-        return widget.render(name, data, attrs=attrs)
-
+
-    def as_text(self, attrs=None, **kwargs):
-        """

django/trunk/docs/topics/forms/index.txt

@@ -293 +293 @@
-        message.
-
+    ``field.is_hidden``
+        This attribute is ``True`` is the form field is a hidden field and
+        ``False`` otherwise. It's not particularly useful as a template
+        variable, but could be useful in conditional tests such as::
+
+            {% if field.is_hidden %}
+               {# Do something special #}
+            {% endif %}
+
+Looping over hidden and visible fields
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+If you are manually laying out a form in a template, you will often want to
+work with any hidden fields in a single loop and then treat the visible fields
+differently. For example, since hidden fields don't display anything, putting
+their error messages "next to" the field isn't going to be very clear to the
+reader. So you need to handle errors for those fields a bit differently.
+
+Django provides two methods on a form that allow you to loop over the hidden
+and visible fields independently: ``hidden_fields()`` and
+``visible_fields()``. In a template, you might use these like this (this is a
+modification of an earlier example)::
+
+    <form action="/contact/" method="POST">
+        {% for field in form.visible_fields %}
+            <div class="fieldWrapper">
+
+                {# Include the hidden fields in the form #}
+                {% if forloop.first %}
+                    {% for hidden in form.hidden_fields %}
+                    {{ field }}
+                    {% endfor %}
+                {% endif %}
+
+                {{ field.errors }}
+                {{ field.label_tag }}: {{ field }}
+            </div>
+        {% endfor %}
+        <p><input type="submit" value="Send message" /></p>
+    </form>
+
+This example does not handle any errors in the hidden fields. Usually, an
+error in a hidden field is a sign of form tampering, since normal form
+interaction won't alter them. However, you could easily insert some error
+displays for those form errors as well.
+
+.. versionadded:: 1.1
+    The ``hidden_fields`` and ``visible_fields`` methods are new in Django
+    1.1.
+
-Reusable form templates
------------------------

django/trunk/tests/regressiontests/forms/forms.py

@@ -1758 +1758 @@
-True
-
+# Extracting hidden and visible fields ######################################
+
+>>> class SongForm(Form):
+...     token = CharField(widget=HiddenInput)
+...     artist = CharField()
+...     name = CharField()
+>>> form = SongForm()
+>>> [f.name for f in form.hidden_fields()]
+['token']
+>>> [f.name for f in form.visible_fields()]
+['artist', 'name']
+
-"""