#7657 closed (invalid)
newforms-admin: django/contrib/auth/decorators.py 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 |
Description
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.
Note:
See TracTickets
for help on using tickets.
On further inspection, the _CheckLogin class in that module appears to be using update_wrapper and doing the right thing. Closed as invalid.