There's absolutely no reason for a Session().commit object mirrored onto Session...

Thanks for catching, Elrond :)

diff --git a/mediagoblin/db/open.py b/mediagoblin/db/open.py
index bd629909c84caa8b9b9f0e11766bdffef16cddf9..8f81c8d910fb40586b58008d802203df21707450 100644 (file)
--- a/mediagoblin/db/open.py
+++ b/mediagoblin/db/open.py
@@ -97,9 +97,6 @@ else:
             #####################################
             # Functions to emulate DatabaseMaster
             #####################################
-            def commit():
-                session.commit()
-
             def save(obj):
                 session.add(obj)
                 session.flush()
@@ -116,7 +113,6 @@ else:
                 session.remove()
 
             # now attach
-            session.commit = commit
             session.save = save
             session.check_session_clean = check_session_clean
             session.reset_after_request = reset_after_request