Add `add_form_template` option to ModelAdmin

[David Cramer]

In some use cases we've found that we need to slightly tweak our change templates. Typically this is because the change view isn't directly changing the object, but it's handling it in some other way.

One small problem that came across was there's no way to specify the add page's template versus the change page's template.

I propose simply modifying render_change_form, to check for add=True, and if it's present, check for add_form_template. If add_form_template is set in the ModelAdmin to a value it should look for that template first. If it's set to None, it should use the default templates (and not change_form_template).

[j.c.sackett]

Patch to add the requested behavior

[j.c.sackett]

I've added a patch to add this behavior.

The patch also changes the regression tests to verify use of the custom change form on an existing instance, and the custom add form when adding. Previously the regression test used the add view to check the custom change form.

All tests appear to be passing on python 2.5.

[Karen Tracey]

Replying to jcsackett:

Sorry I did not get a chance to look at this during the sprint. A couple of things need to be fixed before this can be checked in. First the new template file for the test isn't in the patch, so the test fails. To get svn diff to include it you first need to svn add the file. (I'm guessing you used svn based on the patch, but if you're using another vcs likely you need to do something similar for the diff to pick up the added file.)

Also, the description of ModelAdmin.change_form_template in docs/ref/contrib/admin/index.txt (online here: ​http://docs.djangoproject.com/en/1.1/ref/contrib/admin/#django.contrib.admin.ModelAdmin.change_form_template) needs to be updated to reflect the change, and a new bit need to be added to describe the new add_form_tempalte option.

If you can do both of those, upload a new diff, and update the needs doc and patch needs improvement flags when its done I'll take another look.

[j.c.sackett]

Updated patch including missing template for tests and updated documentation.

[j.c.sackett]

.patch version of the updated patch, just to keep them in sync.

[j.c.sackett]

Updated patch includes modified documentation detailing the add_form_template and changing the documentation of the change_form_template attribute.

Added the add_form.html template for the tests (forgot to svn add it before, kmtracey was right).

I think this should have everything now.

[Jannis Leidel]

(In [12216]) Fixed #10640 - Adds a add_form_template option to ModelAdmin. Thanks jcsackett.

[Jacob]

Milestone 1.2 deleted