[Commits] (vajda) fixed bug 1710

[commits at osafoundation.org]

Commit by: vajda
Modified files:
chandler/repository/item/Item.py 1.149 1.150
chandler/repository/item/ItemRef.py 1.84 1.85

Log message:
fixed bug 1710

ViewCVS links:
http://cvs.osafoundation.org/index.cgi/chandler/repository/item/Item.py.diff?r1=text&tr1=1.149&r2=text&tr2=1.150
http://cvs.osafoundation.org/index.cgi/chandler/repository/item/ItemRef.py.diff?r1=text&tr1=1.84&r2=text&tr2=1.85

Index: chandler/repository/item/ItemRef.py
diff -u chandler/repository/item/ItemRef.py:1.84 chandler/repository/item/ItemRef.py:1.85
--- chandler/repository/item/ItemRef.py:1.84	Sun Aug  1 04:14:40 2004
+++ chandler/repository/item/ItemRef.py	Tue Aug  3 17:10:24 2004
@@ -1,6 +1,6 @@
 
-__revision__  = "$Revision: 1.84 $"
-__date__      = "$Date: 2004/08/01 11:14:40 $"
+__revision__  = "$Revision: 1.85 $"
+__date__      = "$Date: 2004/08/04 00:10:24 $"
 __copyright__ = "Copyright (c) 2002 Open Source Applications Foundation"
 __license__   = "http://osafoundation.org/Chandler_0.1_license_terms.htm"
 
@@ -100,8 +100,10 @@
 
     def reattach(self, item, name, old, new, otherName):
 
-        self.detach(item, name, old, otherName)
-        self.attach(item, name, new, otherName)
+        if old is not new:
+            self.detach(item, name, old, otherName)
+            self.attach(item, name, new, otherName)
+            item.setDirty(attribute=name)
 
     def _unload(self, item):
 

Index: chandler/repository/item/Item.py
diff -u chandler/repository/item/Item.py:1.149 chandler/repository/item/Item.py:1.150
--- chandler/repository/item/Item.py:1.149	Tue Aug  3 04:25:02 2004
+++ chandler/repository/item/Item.py	Tue Aug  3 17:10:24 2004
@@ -1,6 +1,6 @@
 
-__revision__  = "$Revision: 1.149 $"
-__date__      = "$Date: 2004/08/03 11:25:02 $"
+__revision__  = "$Revision: 1.150 $"
+__date__      = "$Date: 2004/08/04 00:10:24 $"
 __copyright__ = "Copyright (c) 2002 Open Source Applications Foundation"
 __license__   = "http://osafoundation.org/Chandler_0.1_license_terms.htm"
 
@@ -367,7 +367,6 @@
                             # reattaching on original endpoint
                             old.reattach(self, name, old.other(self), value,
                                          self._kind.getOtherName(name))
-                            self.setDirty(attribute=name)
                             return value
                     elif isRef:
                         # reattaching on other endpoint,