Opened 19 years ago

Closed 18 years ago

Last modified 18 years ago

#645 closed enhancement (fixed)

[patch] Replace _module_-prefix and module_constants with "class MODULE"

Reported by: Sune Kirkeby <sune.kirkeby@…> Owned by: Adrian Holovaty
Component: Core (Other) Version:
Severity: normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

As discussed in the thread ​Get rid of model modules? on django-devel, it
would be nice to get rid of the _module_-prefix and module_constants, and have module-level constants and functions
in a class MODULE ala class META.

Attachments (1)

class-module.patch​ (27.2 KB ) - added by Sune Kirkeby <sune.kirkeby@…> 19 years ago.

Download all attachments as: .zip

Change History (5)

by Sune Kirkeby <sune.kirkeby@…>, 19 years ago

Attachment: class-module.patch​ added

comment:1 by Sune Kirkeby <sune.kirkeby@…>, 19 years ago

Component: Admin interface → Core framework

The patch is here now :) It removes support for the _module_-prefix and module_constants,
I don't know if we want to keep them for a short while? That should be a pretty simple fix.

comment:2 by Adrian Holovaty, 19 years ago

Status: new → assigned

Thanks for the reminder on this, Sune! And for the patch, of course...

comment:3 by rjwittams, 19 years ago

I had planned to do this, subclassing, and removing core fields in a shortish branch after new-admin merges, to lump all the model changes into one go. Thoughts?

comment:4 by Adrian Holovaty, 18 years ago

Resolution: → fixed
Status: assigned → closed

This has been fixed in magic-removal branch, via the managers concept. See RemovingTheMagic.

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