#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 , 17 years ago
| Attachment: | preview_9070.diff added |
|---|
comment:1 by , 17 years ago
| Needs documentation: | set |
|---|
As a bonus, this doesn't break backwards compatibility :)
comment:2 by , 17 years ago
| milestone: | → 1.1 |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:3 by , 17 years ago
| milestone: | 1.1 → 1.2 |
|---|
by , 16 years ago
| Attachment: | process_preview_doc.diff added |
|---|
Basic docs for process_preview; indicating new in 1.2
comment:4 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Add a process_preview method to FormPreview