Opened 16 years ago

Closed 15 years ago

Last modified 13 years ago

#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)

preview_9070.diff (1.1 KB ) - added by Bob Thomas 16 years ago.
Add a process_preview method to FormPreview
process_preview_doc.diff (881 bytes ) - added by thalin 15 years ago.
Basic docs for process_preview; indicating new in 1.2

Download all attachments as: .zip

Change History (7)

by Bob Thomas, 16 years ago

Attachment: preview_9070.diff added

Add a process_preview method to FormPreview

comment:1 by Bob Thomas, 16 years ago

Needs documentation: set

As a bonus, this doesn't break backwards compatibility :)

comment:2 by Jacob, 16 years ago

milestone: 1.1
Triage Stage: UnreviewedAccepted

comment:3 by Jacob, 16 years ago

milestone: 1.11.2

by thalin, 15 years ago

Attachment: process_preview_doc.diff added

Basic docs for process_preview; indicating new in 1.2

comment:4 by jbronn, 15 years ago

Resolution: fixed
Status: newclosed

(In [12486]) Fixed #9147 -- Added FormPreview.process_preview customization hook. Thanks, bthomas and thalin.

comment:5 by Jacob, 13 years ago

milestone: 1.2

Milestone 1.2 deleted

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