[Uml-devel] kdesdk/umbrello/umbrello

[Brian Thomas]

CVS commit by thomas: 

humph. I dont know how it happened, but I submitted the patched code in the wrong method..setAttributesOnNode instead of (correctly) puting in setAttributesFromNode. Now fixed. No kidding.


  M +9 -9      codeclassfield.cpp   1.8


--- kdesdk/umbrello/umbrello/codeclassfield.cpp  #1.7:1.8
@@ -240,14 +240,7 @@ void CodeClassField::setAttributesOnNode
 {
 
-        // always disconnect
-        getParentObject()->disconnect(this);
-
-        // superclass call.. may reset the parent object
+        // super class
         CodeParameter::setAttributesOnNode(doc,cfElem);
 
-        // make AFTER super-class call. This will reconnect to the parent 
-        // and re-check we have all needed child accessor methods and decl blocks
-        initFields( );
-
         // now set local attributes/fields
         cfElem.setAttribute("field_type",m_classFieldType);
@@ -273,6 +266,13 @@ void CodeClassField::setAttributesOnNode
 void CodeClassField::setAttributesFromNode ( QDomElement & root) {
 
-        // super class
+        // always disconnect
+        getParentObject()->disconnect(this);
+
+        // superclass call.. may reset the parent object
         CodeParameter::setAttributesFromNode(root);
+
+        // make AFTER super-class call. This will reconnect to the parent 
+        // and re-check we have all needed child accessor methods and decl blocks
+        initFields( );
 
         setWriteOutMethods(root.attribute("writeOutMethods","true") == "true" ? true : false);