Changeset 26039 for trunk/ippTools/src/magictool.c

Timestamp:

Nov 4, 2009, 7:14:15 PM (17 years ago)

Author:

bills

Message:

in -definebyquery -rerun don't queue if a run with the given label exists
(previous change didn't work)

File:

• trunk/ippTools/src/magictool.c (modified) (3 diffs)

trunk/ippTools/src/magictool.c

@@ -151 +151 @@
         // what if no skycells for the diff run completed?
 
-        psStringAppend(&queryWhereStr, "\n%s (magic_id IS NULL", queryWhereStr ? "AND" : "WHERE");
+        psStringAppend(&queryWhereStr, "\n%s magic_id IS NULL", queryWhereStr ? "AND" : "WHERE");
+        psString rerunWhereStr = NULL;
         if (rerun) {
-            psStringAppend(&queryWhereStr, " OR (magicRun.label != '%s')", label);
-        }
-        psStringAppend(&queryWhereStr, ") ");
+            psStringAppend(&rerunWhereStr, "\n WHERE magicRun.label = '%s'", label);
+        }
 
         // now add the user specified qualifiers
@@ -178 +178 @@
             queryWhereStr = psStringCopy("");
         }
-
-        if (!p_psDBRunQueryF(config->dbh, query, diffWhereStr, diffWhereStr, queryWhereStr)) {
+        if (!rerunWhereStr) {
+            rerunWhereStr = psStringCopy("");
+        }
+
+        if (!p_psDBRunQueryF(config->dbh, query, diffWhereStr, diffWhereStr, rerunWhereStr, queryWhereStr)) {
             psError(PS_ERR_UNKNOWN, false, "database error");
             psFree(diffWhereStr);
             psFree(queryWhereStr);
+            psFree(rerunWhereStr);
             psFree(query);
             return false;
@@ -188 +192 @@
         psFree(diffWhereStr);
         psFree(queryWhereStr);
+        psFree(rerunWhereStr);
         psFree(query);
     }