association object dependency glitches

diff --git a/lib/sqlalchemy/mapping/objectstore.py b/lib/sqlalchemy/mapping/objectstore.py
index 0f633eb0c58d6211a367f5a8805d4c2ca7b5e438..a986697bcaff18ed65e1d995b6ddf8759bb11119 100644 (file)
--- a/lib/sqlalchemy/mapping/objectstore.py
+++ b/lib/sqlalchemy/mapping/objectstore.py
@@ -672,12 +672,12 @@ class UOWTask(object):
     def __repr__(self):
         if self.mapper is not None:
             if self.mapper.__class__.__name__ == 'Mapper':
-                name = self.mapper.primarytable.name
+                name = self.mapper.class_.__name__ + "/" + self.mapper.primarytable.name
             else:
                 name = repr(self.mapper)
         else:
             name = '(none)'
-        return ("UOWTask/%d Table: '%s'" % (id(self), name))
+        return ("UOWTask/%d Mapper: '%s'" % (id(self), name))
 
 class DependencySorter(topological.QueueDependencySorter):
     pass

diff --git a/lib/sqlalchemy/mapping/properties.py b/lib/sqlalchemy/mapping/properties.py
index fc2460e4ab4932bfee1942e8b453bd78a80f48d8..443f5eac86311b7d3aac98023d34fac628eddfd6 100644 (file)
--- a/lib/sqlalchemy/mapping/properties.py
+++ b/lib/sqlalchemy/mapping/properties.py
@@ -390,10 +390,24 @@ class PropertyLoader(MapperProperty):
             # association object.  our mapper is made to be dependent on our parent,
             # as well as the object we associate to.  when theyre done saving (or before they
             # are deleted), we will process child items off objects managed by our parent mapper.
-            uowcommit.register_dependency(self.parent, self.mapper)
-            uowcommit.register_dependency(self.association, self.mapper)
-            uowcommit.register_processor(self.parent, self, self.parent, False)
-            uowcommit.register_processor(self.parent, self, self.parent, True)
+            
+            # still working out how the dependencies should work here.
+            
+            # this seems to work:
+    #        uowcommit.register_dependency(self.association, self.parent)
+    #        uowcommit.register_dependency(self.parent, self.mapper)
+    #       #uowcommit.register_dependency(self.association, self.mapper)
+    #        uowcommit.register_processor(self.parent, self, self.parent, False)
+    #        uowcommit.register_processor(self.parent, self, self.parent, True)
+
+            # this seems to work too, using the "stub" as a marker
+            stub = PropertyLoader.MapperStub()
+            uowcommit.register_dependency(self.parent, stub)
+            uowcommit.register_dependency(self.association, stub)
+            uowcommit.register_dependency(stub, self.mapper)
+            uowcommit.register_processor(stub, self, self.parent, False)
+            uowcommit.register_processor(stub, self, self.parent, True)
+
         elif self.direction == PropertyLoader.CENTER:
             # many-to-many.  create a "Stub" mapper to represent the
             # "middle table" in the relationship.  This stub mapper doesnt save
@@ -495,16 +509,19 @@ class PropertyLoader(MapperProperty):
                 if childlist is None: continue
                 uowcommit.register_saved_list(childlist)
                 
+                #print "DIRECTION", self.direction
                 d = {}
                 for child in childlist:
                     self._synchronize(obj, child, None, False)
                     key = self.mapper.instance_key(child)
+                    #print "SYNCHRONIZED", child, "INSTANCE KEY", key
                     d[key] = child
                     uowcommit.unregister_object(child)
 
                 for child in childlist.added_items():
                     uowcommit.register_object(child)
                     key = self.mapper.instance_key(child)
+                    #print "ADDED, INSTANCE KEY", key
                     d[key] = child
                     
                 for child in childlist.unchanged_items():
@@ -514,11 +531,13 @@ class PropertyLoader(MapperProperty):
                     
                 for child in childlist.deleted_items():
                     key = self.mapper.instance_key(child)
+                    #print "DELETED, INSTANCE KEY", key
                     if d.has_key(key):
                         o = d[key]
                         o._instance_key = key
                         uowcommit.unregister_object(child)
                     else:
+                        #print "DELETE ASSOC OBJ", repr(child)
                         uowcommit.register_object(child, isdelete=True)
         else:
             for obj in deplist:
@@ -567,6 +586,7 @@ class PropertyLoader(MapperProperty):
             if dest is associationrow:
                 associationrow[dcol.key] = value
             else:
+                #print "SYNC VALUE", value, "TO", dest
                 dmapper._setattrbycolumn(dest, dcol, value)
 
     def execute(self, instance, row, identitykey, imap, isnew):