Closed (fixed)
Project:
Class
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Reporter:
Created:
8 Sep 2007 at 11:04 UTC
Updated:
22 Sep 2007 at 11:21 UTC
Class requires node_copy because it uses the node_copy_extra_properties_add()
node copy should be a completely separate module.
Comments
Comment #1
regx commentedI duplicated the function int the class module.
node_clone_extra_properties_add() is just a helper function that makes it easy to add extra elements to a node, and groups the elements together on the node form. Since class properties should be in a separate group anyway duplicating the function makes sense.
Class_note and other class modules need this as well and shouldn't require node_copy to function. Since these modules all require the class module, this seemed like the best place to put it.
If the extra properties are all copy related, then this should use the copy module to add properties only if it is loaded.
Comment #2
(not verified) commented