Opened 11 years ago

Closed 11 years ago

Last modified 8 years ago

#7657 closed (invalid)

newforms-admin: django/contrib/auth/ should be using wraps

Reported by: simon Owned by:
Component: contrib.auth Version: newforms-admin
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


wraps and update_wrapper are imported but not used. This is probably going to break things further down the line e.g. URL reversing which can get confused by decorators.

Change History (2)

comment:1 Changed 11 years ago by simon

Resolution: invalid
Status: newclosed

On further inspection, the _CheckLogin class in that module appears to be using update_wrapper and doing the right thing. Closed as invalid.

comment:2 Changed 8 years ago by Jacob

milestone: 1.0 beta

Milestone 1.0 beta deleted

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