Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#27736 closed Cleanup/optimization (fixed)

Prefer decorators for lookup-registration

Reported by: Mads Jensen Owned by: nobody
Component: Core (Other) Version: dev
Severity: Normal 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

django/db/models/lookups.py can be made a bit more compact by using decorators for registrering the lookups. The file has an inconsistent style for it.

Attachments (1)

django-db-models-lookups.patch​ (5.2 KB ) - added by Mads Jensen 7 years ago.

Download all attachments as: .zip

Change History (5)

by Mads Jensen, 7 years ago

Attachment: django-db-models-lookups.patch​ added

comment:1 by Josh Smeaton, 7 years ago

Triage Stage: Unreviewed → Accepted

Yep a cleanup here looks good. Can you turn your patch into a PR please?

comment:2 by Mads Jensen, 7 years ago

​PR

comment:3 by Tim Graham <timograham@…>, 7 years ago

Resolution: → fixed
Status: new → closed

In 0bc59978:

Fixed #27736 -- Used decorators for lookup registration.

comment:4 by Tim Graham <timograham@…>, 7 years ago

In d47de2e:

Refs #27736 -- Used decorators for GIS lookup registration.

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