Changeset 3350

Timestamp:

05/14/08 20:55:08 (2 months ago)

Author:

aaronr

Message:

Clean up to get the xml data model to use markAsDirty on all items that change the model. Should be good for cleaning up other areas of the code

Files:

• trunk/opus_gui/config/xmlmodelview/opusdatamodel.py (modified) (11 diffs)

trunk/opus_gui/config/xmlmodelview/opusdatamodel.py

@@ -289 +289 @@
                 if not domElement.isNull():
                     domElement.setTagName(value.toString())
-                    if self.dirty == False:
-                        wintitle = self.parentObj.windowTitle().replace(" - ", " - *")
-                        self.parentObj.setWindowTitle(wintitle)
-                    self.dirty = True
+                    self.markAsDirty()
         elif index.column() == 1:
             if domNode.hasChildNodes():
@@ -299 +296 @@
                     if children.item(x).isText():
                         children.item(x).setNodeValue(QString(value.toString()))
-                        if self.dirty == False:
-                            wintitle = self.parentObj.windowTitle().replace(" - ", " - *")
-                            self.parentObj.setWindowTitle(wintitle)                         
-                        self.dirty = True
+                        self.markAsDirty()
             else:
                 #print "New text node to be added"
@@ -308 +302 @@
                 newText = self.domDocument.createTextNode(QString(value.toString()))
                 domNode.appendChild(newText)
-                if self.dirty == False:
-                    wintitle = self.parentObj.windowTitle().replace(" - ", " - *")
-                    self.parentObj.setWindowTitle(wintitle)
-                self.dirty = True
+                self.markAsDirty()
         return True
 
@@ -334 +325 @@
         self.endInsertRows()
         #print "debug"
+        self.markAsDirty()
         return returnval
 
@@ -347 +339 @@
         parentItem.childItems.pop(row)
         self.endRemoveRows()
+        self.markAsDirty()
         return returnval
 
@@ -359 +352 @@
                 self.removeRow(currentRow,currentParent)
                 self.insertRow(currentRow-howmany,currentParent,clone)
+                self.markAsDirty()
 
     def moveDown(self,item,howmany=1):
@@ -374 +368 @@
                 self.removeRow(currentRow,currentParent)
                 self.insertRow(currentRow+howmany,currentParent,clone)
+                self.markAsDirty()
 
     def findElementIndexByName(self,name,parent,multiple=False):
@@ -451 +446 @@
         newNode.setAttribute(QString("type"),QString(type))
         newText = document.createTextNode(QString(value))
-        newNode.appendChild(newText)
-        
+        newNode.appendChild(newText)        
         if choices is not None:
             newNode.setAttribute(QString('choices'), QString(choices))
+        self.markAsDirty()
         return newNode
 
@@ -463 +458 @@
                 # remove the attribute
                 parentElement.removeAttribute(QString(attribute))
+                self.markAsDirty()
             rows = parent.childNodes().count()
             for x in xrange(0,rows,1):
@@ -472 +468 @@
                         # remove the attribute
                         childElement.removeAttribute(QString(attribute))
+                        self.markAsDirty()
                     # If this child has other children then we recurse
                     childRows = child.childNodes().count()
@@ -484 +481 @@
                 # remove the attribute
                 parentElement.removeAttribute(QString(attribute))
+                self.markAsDirty()
             grandParent = parent.parentNode()
             if not grandParent.isNull():