[Checkins] SVN: zope.app.generations/trunk/src/zope/app/generations/interfaces.py Added note that methods should not commit.

Jim Fulton jim at zope.com
Thu Aug 2 15:38:20 EDT 2007


Log message for revision 78550:
  Added note that methods should not commit.
  

Changed:
  U   zope.app.generations/trunk/src/zope/app/generations/interfaces.py

-=-
Modified: zope.app.generations/trunk/src/zope/app/generations/interfaces.py
===================================================================
--- zope.app.generations/trunk/src/zope/app/generations/interfaces.py	2007-08-02 18:45:40 UTC (rev 78549)
+++ zope.app.generations/trunk/src/zope/app/generations/interfaces.py	2007-08-02 19:38:20 UTC (rev 78550)
@@ -58,6 +58,12 @@
         the database.  A `context` argument is passed rather than
         a connection to make it possible to provide additional
         information later, if it becomes necessary.
+
+        This method should *not* commit a transaction.  The
+        transaction will be committed by the caller if there is no
+        error.  It is acceptable to commit a transaction if there are no
+        subsequent operations.  The method may create savepoints.
+        
         """
 
     def getInfo(generation):
@@ -76,5 +82,10 @@
         The application has never had the application installed
         before.  The schema manager should bring the database to the
         current generation.
+
+        This method should *not* commit a transaction.  The
+        transaction will be committed by the caller if there is no
+        error.  It is acceptable to commit a transaction if there are no
+        subsequent operations.  The method may create savepoints.
         
         """



More information about the Checkins mailing list