#9147 closed (fixed)
FormPreview needs easier extensibility
Reported by: | Bob Thomas | Owned by: | nobody |
---|---|---|---|
Component: | contrib.formtools | Version: | 1.0 |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | yes |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
In addition to #7222 and #8690, I have to override more parts of FormPreview than I'd like to do any additional processing before displaying the preview page. FormPreview should have a hook for additional processing like FormWizard's process_step() method so I don't have to override all of preview_post or put unwanted logic in my template.
Attachments (2)
Change History (7)
by , 16 years ago
Attachment: | preview_9070.diff added |
---|
comment:1 by , 16 years ago
Needs documentation: | set |
---|
As a bonus, this doesn't break backwards compatibility :)
comment:2 by , 16 years ago
milestone: | → 1.1 |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:3 by , 16 years ago
milestone: | 1.1 → 1.2 |
---|
by , 15 years ago
Attachment: | process_preview_doc.diff added |
---|
Basic docs for process_preview; indicating new in 1.2
comment:4 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Add a process_preview method to FormPreview