Changeset 4121

Timestamp:

Feb 4, 2008, 12:17:38 PM (15 years ago)

Author:

Nicklas Nordborg

Message:

Fixes #916: Base1PluginExecuter commits transaction even in case of a failure

File:

• trunk/src/plugins/core/net/sf/basedb/plugins/Base1PluginExecuter.java (modified) (3 diffs)

trunk/src/plugins/core/net/sf/basedb/plugins/Base1PluginExecuter.java

@@ -448 +448 @@
     finally
     {
-      if (dc != null) 
-      {
-        dc.commit();
-        dc.close();
-      }
+      if (dc != null) dc.close();
     }
   }
@@ -731 +727 @@
           return;
         }
+        dc.commit();
       }
       finally
       {
-        if (dc != null)
-        {
-          dc.commit();
-          dc.close();
-        }
+        if (dc != null) dc.close();
       }
       
@@ -773 +766 @@
           }
         }
+        dc.commit();
       }
       finally
       {
-        if (dc != null)
-        {
-          dc.commit();
-          dc.close();
-        }
+        if (dc != null) dc.close();
       }
       response.setDone(msg);