Opened 15 years ago
Closed 14 years ago
#11935 closed New feature (fixed)
Add a get_context(self, request, step) method to FormWizard
Reported by: | Owned by: | nobody | |
---|---|---|---|
Component: | contrib.formtools | Version: | 1.1 |
Severity: | Normal | Keywords: | formwizard |
Cc: | treborhudson@… | Triage Stage: | Accepted |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
If you want to simply add context to a step of a FormWizard there are currently two documented ways, both of which have shortcomings...
- Override
render_template
. The problem with this is quite a bit is happening in the to the variables in the context. The user mostly has to copy/paste the code, get it right, and hope this method doesn't change out from underneath him for future releases, just to add context.
- Override
process_step
. If the above turns the user's stomach, this seems like the next best place. The problem, as noted in the documentation, is that "... this method is called every time a page is rendered for all submitted steps." If you are only adding context, not processing anything, this can result in a pretty heavy load, for example if you are doing database calls to get a list of objects to display on a certain step.
I'm proposing adding a method just for adding context to any given step, and make it called only when the step is rendered.
Change History (3)
comment:1 by , 15 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:2 by , 14 years ago
Severity: | → Normal |
---|---|
Type: | → New feature |
comment:3 by , 14 years ago
Easy pickings: | unset |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Superseded by #9200.
Note:
See TracTickets
for help on using tickets.
Not necessarily sold on adding a method as the solution, but the idea (allowing for extra_context) is certainly valid.