Changeset 13967 for branches/backtrack/ippTools/src/regtool.c

Timestamp:

Jun 25, 2007, 10:19:18 AM (19 years ago)

Author:

jhoblitt

Message:

overhaul new* -> raw* table transistions to keep the complete history

File:

• branches/backtrack/ippTools/src/regtool.c (modified) (6 diffs)

branches/backtrack/ippTools/src/regtool.c

@@ -282 +282 @@
                 return false;
             }
+            psFree(object);
             // insert the rawImfile object into the database
             if (!rawImfileInsertObject(config->dbh, imfile)) {
@@ -290 +291 @@
                 psError(PS_ERR_UNKNOWN, false, "failed to insert row into the database");
                 psFree(imfile);
-                psFree(object);
                 psFree(output);
                 return false;
             }
             psFree(imfile);
-            // remove the neImfile object from the database
-            if (!newImfileDeleteObject(config->dbh, object)) {
-                // rollback
-                if (!psDBRollback(config->dbh)) {
-                    psError(PS_ERR_UNKNOWN, false, "database error");
-                }
-                psError(PS_ERR_UNKNOWN, false, "failed to delete row from the database");
-                psFree(object);
-                psFree(output);
-                return false;
-            }
-            psFree(object);
         }
 
@@ -573 +561 @@
     }
     if (!psArrayLength(output)) {
-        psTrace("regtool", PS_LOG_INFO, "no rows found");
+        psError(PS_ERR_UNKNOWN, false, "no pending newExp rows found");
         psFree(output);
-        return true;
-    }
-
-    // start a transaction so we don't end up with an exp in both rawExp &
-    // newExp
+        return false;
+    }
+
     if (!psDBTransaction(config->dbh)) {
         psError(PS_ERR_UNKNOWN, false, "database error");
@@ -603 +589 @@
             return false;
         }
+        psFree(newExp);
 
         // insert the rawExp object into the database
@@ -612 +599 @@
             psError(PS_ERR_UNKNOWN, false, "database error");
             psFree(rawExp);
-            psFree(newExp);
             psFree(output);
             return false;
         }
         psFree(rawExp);
-
-        // delete the newExp object from the database
-        if (!newExpDeleteObject(config->dbh, newExp)) {
-            // rollback
-            if (!psDBRollback(config->dbh)) {
-                psError(PS_ERR_UNKNOWN, false, "database error");
-            }
-            psError(PS_ERR_UNKNOWN, false, "database error");
-            psFree(newExp);
-            psFree(output);
-            return false;
-        }
-
-        psFree(newExp);
 
         // if this is a detrend image don't put it in the chip queue (and we're
         // done)
         if (detrend) {
+            // set the state for the newExp to stop 
+            if (!pxnewExpSetState(config, exp_tag, "stop")) {
+                // rollback
+                if (!psDBRollback(config->dbh)) {
+                    psError(PS_ERR_UNKNOWN, false, "database error");
+                }
+                psError(PS_ERR_UNKNOWN, false, "failed to change newExp.state for exp_tag: %s", exp_tag);
+                psFree(output);
+                return false;
+            }
             continue;
         }
@@ -648 +630 @@
             return false;
         }
-    }
-
+
+        // set the state for the newExp to stop
+        if (!pxnewExpSetState(config, exp_tag, "stop")) {
+            // rollback
+            if (!psDBRollback(config->dbh)) {
+                psError(PS_ERR_UNKNOWN, false, "database error");
+            }
+            psError(PS_ERR_UNKNOWN, false, "failed to change newExp.state for exp_tag: %s", exp_tag);
+            psFree(output);
+            return false;
+        }
+    }
     psFree(output);