Changes between Initial Version and Version 1 of ImprovementsForDjangoForms

Timestamp:

Jun 3, 2010, 6:38:06 AM (14 years ago)

Author:

Petr Marhoun <petr.marhoun@…>

Comment:

--

ImprovementsForDjangoForms

@@ +1 @@
+= Proposal: Improvements for django.forms =
+
+I would like to propose several (quite independent) improvements for django.forms. Their common goal is: '''It should be simple to write powerful and flexible forms.'''
+
+These proposals have some common assumptions:
+ * They try to expand possibilities in definitions of forms and API for views and templates. They do not try to change methods as_table, as_ul and as_li of forms - I think in Django there are templates for writing of HTML code and these helper methods should be used only for simple forms.
+ * They assume that forms are used with generic views and generic templates. (But it does not mean that they could not be useful in other situations.)
+ * They try to improve core Django forms - it is decision of contributed and external applications to use or not to use new possibilities.
+
+My proposals are not only ideas - all forms in my code are subclasses of django.forms' subclasses which use most of these possible improvements. But these subclasses are not nice, they assume to much about internals of Django forms and they are sometimes broken by changes in Django.
+
+And sorry for my English.
+
+== Fieldsets ==
+
+I would like to propose new attribute for inner meta class of forms - "fieldsets". It would be list of dictionaries with two possible items - "fields" (list of fields) and "legend" (but I also propose items "attrs" and "template" in another proposal). Example:
+
+{{{
+class UserForm(forms.ModelForm):
+    class Meta:
+         model = User
+         fieldsets = (
+             {'fields': ('username', 'first_name', 'last_name', 'email')},
+             {'fields': ('is_active', 'is_staff', 'is_superuser'), 'legend': _('Permissions')},
+         )
+}}}
+
+Forms would have two new methods - has_fieldsets and fieldsets. The second one would iterate all fieldsets - each item would be a dictionary with items "fields" (list of bound fields) and "legend".
+
+Fieldsets are useful also for common (non-model) forms - so this attribute should be defined for them. But I think it would be nice to move also "fields" and "exclude" attribute to common forms - it is useful for reusing of existing forms.
+
+== Inlines ==
+
+I would like to propose inlines - forms with inner formsets for editing of 1..n relations. They could be defined with attribute "fieldsets" or "inlines". The second one would be the special case of the first one, "inlines" would be appended to "fieldsets". Example:
+
+{{{
+class UserForm(forms.ModelForm):
+    class Meta:
+         model = User
+         fieldsets = (
+             {'fields': ('username', 'first_name', 'last_name', 'email')},
+             forms.inlineformset_factory(User, MyModelWithLinkToUser),
+             {'fields': ('is_active', 'is_staff', 'is_superuser'), 'legend': _('Permissions')},
+         )
+         inlines = (forms.inlineformset_factory(User, AnotherModelWithLinkToUser), forms.inlineformset_factory(User, LastModelWithLinkToUser))
+}}}
+
+Method has_fieldsets would be changed to return True if there are fieldsets or inlines. Method fieldsets would be changed to iterate dictionaries which could have item "inline" which would be relevant inline formset instance - or not, it means that it is a fieldset. Each item in dictionaries would have items "counter" and "counter0" (maybe others variables from tag "for") with indexes local to type (inline or fieldset). So the example would like as:
+
+{{{
+(
+    ('counter0': 0, 'counter': 1, 'fields': (...)),
+    ('counter0': 0, 'counter': 1, 'inline': ...),
+    ('counter0': 1, 'counter': 2, 'fields': (...), 'legend': 'Permission'),
+    ('counter0': 1, 'counter': 2, 'inline': ...),
+    ('counter0': 2, 'counter': 3, 'inline': ...),
+)
+}}}   
+
+All relevant methods of forms would be changed to iterate through inlines: is_valid, save, is_multipart and media.
+
+== Better API for formsets ==
+
+Forms in formsets can be classified as:
+ * based on existing object or new,
+ * filled or not filled,
+ * valid or not valid,
+ * marked or not marked for deletion - if can_delete is True,
+ * ordered or not ordered - if can_order is True.
+
+So there are many combinations of states - but problem is that not all formset methods are consistent. For example, method is_valid does not check validity if form is marked for deletion, but cleaned_data property returns cleaned_data from all forms, possible not existing attribute. More generally, all forms are iterated in four methods - _deleted_forms, _ordered_forms, is_valid and full_clean - with different rules (if I really understand code, method is_valid iterates forms, in the first form checks errors property which calls methods full_clean which iterates all forms). And what I see as the main problem - there is no property of unordered formset which give me all filled and valid forms and nothing more (something as ordered_forms for ordered formset).
+
+So I would like to propose:
+ * Move all validation to method full_clean - this method sets attributes _is_valid, _errors, _non_form_errors, _cleaned_forms (if valid) and _deleted_forms (if valid and can_delete is True). If can_order is True, _cleaned_forms are ordered.
+ * Method is_valid only checks if full_clean has been run (if _errors is not None) and returns _is_valid.
+ * Property cleaned_forms only checks if full_clean has been run (if _errors is not None) and returns _cleaned_forms (it could raise !AttributeError if formset is not valid).
+ * Property deleted_forms only checks if full_clean has been run (if _errors is not None) and returns _deleted_forms (it could raise !AttributeError if formset is not valid).
+ * Property cleaned_data is deprecated - it should not be used because it could raise !AttributeError for valid formsets (and it is possible to iterate attribute forms for advanced use).
+ * Property ordered_forms is deprecated name for cleaned_forms.
+ * If form is marked for deletion (and can_delete is True) - it does not matter if is filled or valid. These forms are added to _deleted_forms.
+ * If form is not marked for deletion (or can_delete is False) and based on existing object, it has to be filled and valid, otherwise formset is not valid. These forms are added to _cleaned_forms.
+ * If form is not marked for deletion (or can_delete is False) and new, it has to be not filled or valid, otherwise formset is not valid. These forms are added to _cleaned_forms if filled.
+
+== Model fields with not-default values ==
+
+I thing that this idea has been proposed - but I do not remember refusing, only not accepting. So I would like to propose attribute of inner meta class of model forms which could be called for example fields_kwargs. It would be dictionaries of redefined kwargs for fields. Example:
+
+{{{
+class UserForm(forms.ModelForm):
+    class Meta:
+         model = User
+         fields_kwargs = {
+             'password': {'help_text': 'At least six characters.'},
+             'username': {'label': 'Name', 'help_text': 'Name for login.'},
+         }
+}}}
+
+Some people propose something as:
+
+{{{
+class UserForm(forms.ModelForm):
+    password = User._meta.get_field('password').formfield(help_text='At least six characters.')
+    username = helper_function(User, 'username', label='Name', help_text='Name for login.')
+
+    class Meta:
+         model = User
+}}}
+
+I do not like this. It is not DRY - it is necessary to write name of attribute twice. And what is more important - it hides indentions of author, it seems that fields are replaced but they are only changed a little.
+
+== Parameters "template" and "attrs" for forms, formsets, fieldsets, forms, (bound) fields and widgets ==
+
+There are two requirements which seem to be conflicting:
+ * It should be possible to write generic templates which are able to render all forms and formsets.
+ * It should be possible to customize each part of form rendering - but only these parts which are necessary to customize.
+
+I would like to propose to add attributes "template" and "attrs" to all parts of forms - forms, formsets, fieldsets, (bound) fields and widgets. Attribute "attrs" should be dictionary of attributes for HTML code, attribute "template" should be name of template. These attributes would not be used in Django in any way, they would be only saved for template authors. They could write something as:
+
+{{{
+{% for field in form %}
+    {% if field.template %}
+        {% include field.template %}
+    {% else %}
+        <div {{ field.attrs|flatatt }}>{{ field.errors }} {{ field.label_tag }}: {{ field }}</div>
+    {% endif %}
+{% endfor %}
+}}}
+
+I do not propose how exactly these attributes should be set (it could be in inner meta class or in init or as class attributes or in factories), it is the next step if this idea would be accepted.
+
+== Support for (X)HTML5 and legacy HTML ==
+
+This is quite new idea and I did not think about it so much as about other ones - but it is logical extension of the previous one. Django should support HTML5 forms in the future - but not all people would like to start using them in the same time (it would also break some tests). And others would like to use legacy HTML, not XTHML.
+
+What if widget rendering would not be hard-coded in Python code - but there would be only names of templates? And there should be some prepared templates - one set for HTML5, one set as in current Django. Everybody could decide which set to use or to write own set.
+
+I have no answer for these possible problems:
+ * How to guarantee that at least one set of templates is usable and form rendering is not broken?
+ * What about performance? Now caching loaders should be used in production - but is it enough?
+
+== Request in forms ==
+
+(This proposal is rather request for design decision: I expect answer "no".)
+
+Quite often it is useful to use request in forms methods, especially in init. And it is also useful and simple to use generic views for forms. So it would be nice to have some generic mechanism for setting of request to form. My proposal is to add variable "request" to init method of forms and formsets with default value None - this variable would be only saved, not used in any way.
+
+This change could be again Django philosophy of "loose coupling" - forms should not know about requests. Requests in forms could be bad for testing of forms or for using of forms in management commands. But in reality many forms know about requests.
+
+(And I naturally know some possibilities how to solve this problem - for example request as thread local variable or factory method for form. But I do not know any nice and simple solution - maybe there is one.)