[Checkins] SVN: zope.sqlalchemy/trunk/src/zope/sqlalchemy/ update to use session.add instead of deprecated session.save

Laurence Rowe l at lrowe.co.uk
Tue Jan 20 16:08:19 EST 2009


Log message for revision 94880:
  update to use session.add instead of deprecated session.save

Changed:
  U   zope.sqlalchemy/trunk/src/zope/sqlalchemy/README.txt
  U   zope.sqlalchemy/trunk/src/zope/sqlalchemy/tests.py

-=-
Modified: zope.sqlalchemy/trunk/src/zope/sqlalchemy/README.txt
===================================================================
--- zope.sqlalchemy/trunk/src/zope/sqlalchemy/README.txt	2009-01-20 20:46:47 UTC (rev 94879)
+++ zope.sqlalchemy/trunk/src/zope/sqlalchemy/README.txt	2009-01-20 21:08:19 UTC (rev 94880)
@@ -95,7 +95,7 @@
 We can now create a new user and commit the changes using Zope's transaction
 machinary, just as Zope's publisher would.
 
-    >>> session.save(User(name='bob'))
+    >>> session.add(User(name='bob'))
     >>> transaction.commit()
 
 Engine level connections are outside the scope of the transaction integration.

Modified: zope.sqlalchemy/trunk/src/zope/sqlalchemy/tests.py
===================================================================
--- zope.sqlalchemy/trunk/src/zope/sqlalchemy/tests.py	2009-01-20 20:46:47 UTC (rev 94879)
+++ zope.sqlalchemy/trunk/src/zope/sqlalchemy/tests.py	2009-01-20 21:08:19 UTC (rev 94880)
@@ -202,8 +202,8 @@
         rows = query.all()
         self.assertEqual(len(rows), 0)
                
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         session.flush()
         
         rows = query.order_by(User.id).all()
@@ -220,7 +220,7 @@
         
     def testRelations(self):
         session = Session()
-        session.save(User(id=1,firstname='foo', lastname='bar'))
+        session.add(User(id=1,firstname='foo', lastname='bar'))
 
         user = session.query(User).filter_by(firstname='foo')[0]
         user.skills.append(Skill(id=1, name='Zope'))
@@ -232,7 +232,7 @@
         self.failIf([r for r in t._resources if isinstance(r, tx.SessionDataManager)],
              "Joined transaction too early")
         session = Session()
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
         t = transaction.get()
         # Expect this to fail with SQLAlchemy 0.4
         self.failUnless([r for r in t._resources if isinstance(r, tx.SessionDataManager)],
@@ -256,12 +256,12 @@
             return # sqlite databases do not support savepoints
         
         s1 = t.savepoint()
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
         session.flush()
         self.failUnless(len(query.all())==1, "Users table should have one row")
         
         s2 = t.savepoint()
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         session.flush()
         self.failUnless(len(query.all())==2, "Users table should have two rows")
         
@@ -284,8 +284,8 @@
         session = Session()
         query = session.query(User)
 
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         session.flush()
 
         rows = query.order_by(User.id).all()
@@ -297,8 +297,8 @@
         rows = query.order_by(User.id).all()
         self.assertEqual(len(rows), 0)
         
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         session.flush()
         rows = query.order_by(User.id).all()
         row1 = rows[0]
@@ -321,8 +321,8 @@
         if engine.url.drivername in tx.NO_SAVEPOINT_SUPPORT:
             return
         session = Session()
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         session.flush()
         transaction.commit()
         
@@ -345,8 +345,8 @@
         session = Session()
         if not session.twophase:
             return
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         session.flush()
         transaction.commit()
         
@@ -392,8 +392,8 @@
                 rows = query.all()
                 self.assertEqual(len(rows), 0)
 
-                session.save(User(id=1, firstname='udo', lastname='juergens'))
-                session.save(User(id=2, firstname='heino', lastname='n/a'))
+                session.add(User(id=1, firstname='udo', lastname='juergens'))
+                session.add(User(id=2, firstname='heino', lastname='n/a'))
                 session.flush()
 
                 rows = query.order_by(User.id).all()
@@ -414,8 +414,8 @@
 
     def testBulkDelete(self):
         session = Session()
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         transaction.commit()
         session = Session()
         session.query(User).delete()
@@ -425,8 +425,8 @@
 
     def testBulkUpdate(self):
         session = Session()
-        session.save(User(id=1, firstname='udo', lastname='juergens'))
-        session.save(User(id=2, firstname='heino', lastname='n/a'))
+        session.add(User(id=1, firstname='udo', lastname='juergens'))
+        session.add(User(id=2, firstname='heino', lastname='n/a'))
         transaction.commit()
         session = Session()
         session.query(User).update(dict(lastname="smith"))
@@ -450,8 +450,8 @@
 
     def testTwoEngines(self):
         session = UnboundSession()
-        session.save(TestOne(id=1))
-        session.save(TestTwo(id=2))
+        session.add(TestOne(id=1))
+        session.add(TestTwo(id=2))
         session.flush()
         transaction.commit()
         session = UnboundSession()



More information about the Checkins mailing list