Changeset 1001

Timestamp:

Mar 27, 2009, 3:48:30 PM (14 years ago)

Author:

Jari Häkkinen

Message:

Minor changes to debug_option.

File:

• plugins/base2/net.sf.basedb.pluginutilities/trunk/src/net/sf/basedb/plugins/AbstractRunBinaryPlugin.java (modified) (2 diffs)

plugins/base2/net.sf.basedb.pluginutilities/trunk/src/net/sf/basedb/plugins/AbstractRunBinaryPlugin.java

@@ -287 +287 @@
       runBinary();
       storeResult(dc);
-//      dc.commit();
-      response.setDone("Plug-in ended successfully");
-      if (debug_output) System.err.println("run succesful");
-      response.setError("All is okay, just make sure that BASE produces error.",
-                        null);
+      if (debug_output)
+      {
+        System.err.println("run succesful");
+        response.setError("All is okay, just forcing a BASE error.", null);
+      }
+      else
+      {
+        dc.commit();
+        response.setDone("Plug-in ended successfully");
+      }
     }
     catch (Throwable e) {
@@ -366 +371 @@
 
   // debug settings. Note, if debug information is on, the exec
-  // directory is not removed after plug-in execution. The file name
-  // is modified in the constructor to contain the java temp
-  // directory, i.e. debug file is written into the java temp
-  // directory. The file name is (somewhat) arbitrarily chosen and may
-  // overwrite existing file.
+  // directory is not removed after plug-in execution, nor is the
+  // transformation committed into the database. The file name
+  // (debug_outputfile) is modified in the constructor to contain the
+  // java temp directory, i.e., the debug file is written into the
+  // java temp directory. The file name is (somewhat) arbitrarily
+  // chosen and may overwrite existing file.
   protected Boolean debug_output=true; 
   private String debug_outputfile="ARBPdebug.txt";