Opened 2 years ago

Closed 2 years ago

#20957 closed New feature (duplicate)

Define form_list directly in the WizardView subclass

Reported by: BerislavLopac Owned by: nobody
Component: Generic views Version: 1.5
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Currently, the list of forms for WizardView is defined as the first parameter of the as_view method, called in the URLconf. It seems to me that it would be more Django-like if it was possible to define it in the view itself, for example as an optional attribute of the form. So it would be possible to define the view like this:

class MyWizardView(WizardView):

    form_list = [WizardForm1, WizardForm2]

    def done(self, form_list, **kwargs):
        return redirect('foo')

Stating forms in the URLconf seems to me plain wrong and non-Djangoish. Of course, it should still be possible to do it (for backward compatibility), in which case the attribute would be ignored.

Change History (1)

comment:1 Changed 2 years ago by timo

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to duplicate
  • Status changed from new to closed

In fact, this was added in Django 1.6 [b614c47f].

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