Opened 6 years ago

Closed 6 years ago

#19433 closed Cleanup/optimization (fixed)

Allow declaration form_list and condition_dict in WizzardView subclass, not as params to as_view

Reported by: migajek Owned by: steph
Component: contrib.formtools Version: 1.4
Severity: Normal Keywords: cbv wizzard wizzardview
Cc: Triage Stage: Ready for checkin
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


every standard django CBV allows for declaring some attributesin the subclass to avoid passing them to as_view function in module.
Unfortunately this does not apply to WizzardView, which does not allow a declaration of form_list nor condition_dict in a WizzardView subclass - each of these must be passed as an argument to as_view function.

Since we need to subclass the view anyway (to define form processing function) AND the condition_dict contains a view logics, I believe the proper place for those attributes would be - WizardView subclass, NOT - as_view params.

example code provided here

Change History (5)

comment:1 Changed 6 years ago by Florian Apolloner

Triage Stage: UnreviewedAccepted

Can you write a pull request for this?

comment:2 Changed 6 years ago by steph

I just wrote some code to make it possible to pass the form lists and condition dicts as class attributes instead of as_view arguments. It would be cool if someone reviews

comment:3 Changed 6 years ago by steph

Owner: changed from nobody to steph
Status: newassigned

comment:4 Changed 6 years ago by Jannis Leidel

Triage Stage: AcceptedReady for checkin

Other than versionchanged additions in the docs this looks good to me..

comment:5 Changed 6 years ago by Jannis Leidel <jannis@…>

Resolution: fixed
Status: assignedclosed

In f2c55d2ee1b161b8ea96bed446deb45c63039e0a:

Merge pull request #943 from stephrdev/trac-19433

Fixed #19433 -- Added some class attributes to pass initial form lists to the WizardView.

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