(no commit message)

diff --git a/lib/sqlalchemy/mapper.py b/lib/sqlalchemy/mapper.py
index d7e3b9c34da45a4be4523f220cfde77fd6ae8e1a..548ca612b4ea3fdf57387a5cb8bfe7a4a6aef7ac 100644 (file)
--- a/lib/sqlalchemy/mapper.py
+++ b/lib/sqlalchemy/mapper.py
@@ -460,6 +460,7 @@ class ForeignKeySetter(sql.ClauseVisitor):
 
     def visit_binary(self, binary):
         if binary.operator == '=':
+            # TODO: gotta use ColumnProperties to get/set these fields
             if binary.left.table == self.primarytable and binary.right.table == self.secondarytable:
                 setattr(self.child, binary.left.key, getattr(self.obj, binary.right.key))
             elif binary.right.table == self.primarytable and binary.left.table == self.secondarytable:

diff --git a/test/mapper.py b/test/mapper.py
index 14e3f760df3b0297830a3b12ffcef8444fec017b..e9c6eb8024f0a0cdf762698476025b49d611517d 100644 (file)
--- a/test/mapper.py
+++ b/test/mapper.py
@@ -306,5 +306,11 @@ class SaveTest(PersistTest):
         self.assert_(addresstable[0].row == (a.address_id, u.user_id, 'one2many@test.org'))
         self.assert_(addresstable[1].row == (a2.address_id, u.user_id, 'lala@test.org'))
 
+        a2.email_address = 'somethingnew@foo.com'
+        m.save(u)
+        addresstable = engine.ResultProxy(addresses.select(addresses.c.address_id == a2.address_id).execute()).fetchall()
+        self.assert_(addresstable[0].row == (a2.address_id, u.user_id, 'somethingnew@foo.com'))
+
+
 if __name__ == "__main__":
     unittest.main()