Opened 17 years ago

Last modified 13 years ago

#6735 closed

Class-based generic views — at Version 9

Reported by: jkocherhans Owned by: nobody
Component: Generic views Version:
Severity: Keywords:
Cc: Daniel Duan, floguy@…, David Larlet, Carl Meyer, Jannis Leidel, sciyoshi@…, vbmendes@…, anossov@…, mjmalone@…, taavi@…, narma.nsk@…, Oliver Beattie, Aaron C. de Bruyn, Jari Pennanen, brian@…, ssteinerX@…, drbob@…, ben@…, Alexander Koshelev, Chris Chambers, Brandon Konkle, Henrique Romano Triage Stage: Accepted
Has patch: yes Needs documentation: yes
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description (last modified by jkocherhans)

Rewrite generic views to be class based, but leave the existing generic views as-is for now.

Change History (14)

by jkocherhans, 17 years ago

Attachment: new-generic-views.diff​ added

comment:1 by jkocherhans, 17 years ago

This ticket refs #3639 use newforms in generic create_update view

comment:2 by jkocherhans, 17 years ago

Needs documentation: set
Needs tests: set

The tests need to be fleshed out a little more, and we need a plan for backwards compatibility, but the classes themselves are fairly close, I think.

comment:3 by floguy, 17 years ago

Cc: floguy@… added

comment:5 by Jacob, 17 years ago

Triage Stage: Unreviewed → Design decision needed

by jkocherhans, 17 years ago

Attachment: new-generic-views.2.diff​ added

by jkocherhans, 17 years ago

Attachment: new-generic-views.3.diff​ added

comment:6 by jkocherhans, 17 years ago

post_save is now on_success and I've added hook to return the rendered template.

comment:7 by jkocherhans, 17 years ago

Needs tests: unset
Triage Stage: Design decision needed → Accepted

Sorry about the mess of patch files. I forgot to check the "replace me" box. Anyhow, Adrian and Jacob are both in favor after a discussion today, so I'm changing status to Accepted.

by jkocherhans, 17 years ago

Attachment: generic-views.diff​ added

Now list_detail views can play too.

comment:8 by jkocherhans, 17 years ago

Description: modified (diff)
Summary: Class-based generic create/update/delete views → Class-based generic views

comment:9 by jkocherhans, 17 years ago

Description: modified (diff)

by jkocherhans, 17 years ago

Attachment: generic-views.2.diff​ added
Note: See TracTickets for help on using tickets.
Back to Top