Opened 17 years ago

Closed 17 years ago

#6178 closed (invalid)

ModelForm too many restrictions.

Reported by: bear330 Owned by: nobody
Component: Forms Version: dev
Severity: Keywords: ModelForm,
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


Originally, I used form_for_model for form generation from model and worked fine.

After form_for_model deprecated, I try to use ModelForm instead, but some problems I occurred.

First, I never use method to save my model instance. My preferred way is to get data from form, do some required processes, and set it to my model instance then save it.

ModelForm implementation forces me to pass a model instance in form's constructor. This is not backward compatible. All of my view must pass a useless model instance to it.

I don't mind to change my code to follow django's new development version, but this parameter is not necessary for people who don't need to use feature.
I just want to generate a form from declared model. That's all I need..

ModelForm implementation tied 'generate a form from model' feature and 'save a form which generated from model' feature together. This truly bothered me.

For the same reason, the ModelForm implementation restricts multiple inheritance from two or more form generated from model. All of that are for method.

I think the is a great feature in newforms library, but it shouldn't restrict the people who don't need it. All validations can be done in method and can raise exceptions if form is not fulfill the requirement but in form generation.


Change History (2)

comment:1 by Karen Tracey <kmtracey@…>, 17 years ago

Note #6162 is getting rid of the required instance parameter for ModelForm. Not sure if it also addresses your concern about multiple inheritance, maybe you could take a look and see? There's also a pointer in that ticket to the thread on the dev list that has relevant discussion.

comment:2 by James Bennett, 17 years ago

Resolution: invalid
Status: newclosed

You can already get the data out of a ModelForm without saving a model object; the commit=False argument to save() works exactly the same way.

Another ticket is already dealing with the constructor's argument signature and required instance.

Given that, I don't see an actual issue here.

Note: See TracTickets for help on using tickets.
Back to Top