#645 closed enhancement (fixed)
[patch] Replace _module_-prefix and module_constants with "class MODULE"
| Reported by: | 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)
Change History (5)
by , 19 years ago
| Attachment: | class-module.patch added |
|---|
comment:1 by , 19 years ago
| Component: | Admin interface → Core framework |
|---|
comment:2 by , 19 years ago
| Status: | new → assigned |
|---|
Thanks for the reminder on this, Sune! And for the patch, of course...
comment:3 by , 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 , 19 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.
The patch is here now :) It removes support for the
_module_-prefix andmodule_constants,I don't know if we want to keep them for a short while? That should be a pretty simple fix.