#5126: Populate `initial_data` of forms from model instance
------------------------------------------------+---------------------------
Reporter: Christopher Lenz <[EMAIL PROTECTED]> | Owner:
adrian
Status: new | Component:
django.newforms
Version: SVN | Resolution:
Keywords: | Stage: Design
decision needed
Has_patch: 1 | Needs_docs: 0
Needs_tests: 1 | Needs_better_patch: 0
------------------------------------------------+---------------------------
Comment (by Christopher Lenz <[EMAIL PROTECTED]>):
Replying to [comment:2 Christopher Lenz <[EMAIL PROTECTED]>]:
> 2. Finally, an overridden constructor as initially proposed by this
ticket:
>
> {{{
> #!python
> form = MyForm(myinstance)
> }}}
>
> (A variant of this last form would to accept the model instance as value
of the `initial` keyword argument.)
Actually, that's a much better approach. Using the first parameter as in
the first two patches sucks for the case where you're not interested in
the initial data. So make that example:
{{{
#!python
form = MyForm(initial=myinstance)
}}}
This is implemented by the [attachment:ticket5126_3.diff third patch],
which I think is really good and unobtrusive, and fully backwards
compatible.
--
Ticket URL: <http://code.djangoproject.com/ticket/5126#comment:3>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
