| Issue 34: | refactor to abstract models | |
| 6 people starred this issue and may be notified of changes. | Back to list |
refactor django-mptt to be a set of abstract models that can be extended. I think this would be a more elegant solution thant the current monkey patching.
Dec 5, 2008
Yep, this is good idea. Also I want mptt to attach methods on model instances like Model.mptt.is_root(), not Model.is_root(). For clearer code and stability (avoid name collisions in large projects).
Jul 1, 2009
(No comment was entered for this change.)
Labels:
-Milestone-Release0.3 Milestone-Release0.4
Sep 2, 2010
I'm definitely +1 for this. Will either be 0.4 or 0.5
Owner:
craig.ds
Sep 6, 2010
(No comment was entered for this change.)
Labels:
-Milestone-Release0.4
Sep 12, 2010
Created a branch for this: http://github.com/django-mptt/django-mptt/tree/abstract-models-refactor This is a major refactor, may not be sane anytime soon.
Sep 17, 2010
All the tests now pass on the new branch. Backwards incompatible changes: * mptt.register() has been removed * Field names and order_insertion_by are now specified via a MPTTMeta inner class
Sep 20, 2010
Issue 52 has been merged into this issue.
Sep 23, 2010
Merged to master in [http://github.com/django-mptt/django-mptt/commit/a7876509f4cbc35b25a76abd860e10eb6ac7ed11 a7876509f]
Status:
Fixed
Labels: Milestone-Release0.4
Sep 27, 2010
Some discussion in FeinCMS project, around using multiple inheritance for MPTTModel subclasses: http://github.com/matthiask/feincms/issues/#issue/105 |
Owner: jonathan.buchanan
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-High Milestone-Release0.3