Opened 13 years ago

Closed 13 years ago

#429 closed enhancement (fixed)

Proposed minor cleanup of utils/

Reported by: pb@… Owned by: Adrian Holovaty
Component: Core (Other) Version:
Severity: minor Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no


The patch attached below makes minor changes to the variable names used for regular expression objects in utils/, for the sake of internal consistency and consistency with other Django modules.

The variable names in the patch are lowercase (as they are in core/ and core/ and they all now end with "_re" (this was not consistent in, but seems fairly consistent elsewhere).

No other modules in the Django distribution appear to reference these names directly.

I also edited the module docstring -- it contained what I think was the last reference to World Online in the codebase, and contained three variants of the word "use," which I thought excessive for a single sentence fragment :)

Attachments (1) (4.5 KB) - added by pb@… 13 years ago.

Download all attachments as: .zip

Change History (3)

Changed 13 years ago by pb@…

Attachment: added

comment:1 Changed 13 years ago by pb@…

Disregard -- use instead. (Trac server wouldn't let me replace the first attachment, but admins should feel free to delete it.)

comment:2 Changed 13 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

Fixed in [611].

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