#8275 closed (fixed)
Unused and unnecessary imports in auth views
Reported by: | Julien Phalip | Owned by: | nobody |
---|---|---|---|
Component: | contrib.auth | Version: | dev |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
Attached patch removes those.
Attachments (1)
Change History (9)
by , 16 years ago
Attachment: | 8275.unused_imports.diff added |
---|
comment:1 by , 16 years ago
comment:2 by , 16 years ago
Yes, but settings
is already imported at the top of the file, so all the imports inside the views are unnecessary.
comment:3 by , 16 years ago
milestone: | → 1.0 |
---|---|
Triage Stage: | Unreviewed → Accepted |
comment:4 by , 16 years ago
milestone: | 1.0 → 1.0 maybe |
---|
Since this is cosmetic, I'm dropping it back to "maybe".
comment:7 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
The latest SVN version doesn't have the extra import statements. Fixed by r8386.
comment:8 by , 16 years ago
Oops, sorry -- I ought to have read the rules carefully, first. I should have set this to worksforme, since I'm certainly no core developer. My mistake!
Note:
See TracTickets
for help on using tickets.
Below each of the lines where you removed the settings import is a line that uses the settings.