fixes involving when child object from list items is None - onetoone must pass, objectstore.testbackwardsmanipulations with postgres passes

1 files changed, 7 insertions, 5 deletions

diff --git a/lib/sqlalchemy/mapping/properties.py b/lib/sqlalchemy/mapping/properties.py
index 9eb97aa0c..a58db251f 100644
--- a/lib/sqlalchemy/mapping/properties.py
+++ b/lib/sqlalchemy/mapping/properties.py
@@ -516,11 +516,10 @@ class PropertyLoader(MapperProperty):
                 childlist = getlist(obj, passive=True)
                 if childlist is None: continue
                 for child in childlist.added_items():
-                    if child is not None:
-                        self._synchronize(obj, child, None, False)
-                        if self.direction == PropertyLoader.ONETOMANY:
-                            # for a cyclical task, this registration is handled by the objectstore
-                            uowcommit.register_object(child)
+                    self._synchronize(obj, child, None, False)
+                    if self.direction == PropertyLoader.ONETOMANY and child is not None:
+                        # for a cyclical task, this registration is handled by the objectstore
+                        uowcommit.register_object(child)
                 if self.direction != PropertyLoader.MANYTOONE or len(childlist.added_items()) == 0:
                     for child in childlist.deleted_items():
                         if not self.private:
@@ -540,6 +539,9 @@ class PropertyLoader(MapperProperty):
             source = None
             dest = associationrow
 
+        if dest is None:
+            return
+
         for rule in self.syncrules:
             localsource = source
             (smapper, scol, dmapper, dcol) = rule