Opened 13 years ago

Closed 13 years ago

#15560 closed (fixed)

Redundant definitions of put() in BaseView classes

Reported by: maney@… Owned by: nobody
Component: Generic views Version: dev
Severity: Keywords: janitorial
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

ProcessFormView defines the PUT handler as a polymorphic proxy for the POST handler. BaseCreateView and BaseUpdateView contain identical, redundant definitions of put(), which may be removed.

I don't believe any new tests wold be called for; presumably this functionality is already tested.

Attachments (1)

generic-views.patch​ (1.1 KB ) - added by maney 13 years ago.
trivial patch included

Download all attachments as: .zip

Change History (2)

by maney, 13 years ago

Attachment: generic-views.patch​ added

trivial patch included

comment:1 by Russell Keith-Magee, 13 years ago

Resolution: → fixed
Status: new → closed

In [15763]:

Fixed #15560 -- Cleaned up some redundant post->put redirections in the class-based generic views. Thanks to maney for the report and patch.

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