Changeset 25109

Timestamp:

Aug 18, 2009, 11:56:59 AM (17 years ago)

Author:

bills

Message:

allow listing of requests by name as well as req_id allow job type 'none'
to help reporting errors

File:

• trunk/ippTools/src/pstamptool.c (modified) (2 diffs)

trunk/ippTools/src/pstamptool.c

@@ -315 +315 @@
     PS_ASSERT_PTR_NON_NULL(config, false);
 
-    PXOPT_LOOKUP_S64(req_id,  config->args, "-req_id", false, false);
-    PXOPT_LOOKUP_STR(name,  config->args, "-name", false, false);
+    psMetadata *where = psMetadataAlloc();
+    PXOPT_COPY_S64(config->args, where, "-req_id", "req_id", "==");
+    PXOPT_COPY_STR(config->args, where, "-name", "name", "==");
+
     PXOPT_LOOKUP_U64(limit,   config->args, "-limit",  false, false);
     PXOPT_LOOKUP_BOOL(simple, config->args, "-simple", false);
 
-    if (!req_id && !name) {
+    if (!psListLength(where->list)) {
         psError(PS_ERR_UNKNOWN, true, "-req_id or -name must be supplied");
         return false;
     }
 
-    psString query = NULL;
-    psStringAppend(&query, "SELECT * from pstampRequest WHERE req_id = %" PRId64, req_id);
+    psString query = psStringCopy("SELECT * from pstampRequest");
+
+    psString whereClause = psDBGenerateWhereConditionSQL(where, NULL);
+    psStringAppend(&query, " WHERE %s", whereClause);
+    psFree(whereClause);
+    psFree(where);
     
     // treat limit == 0 as "no limit"
@@ -523 +529 @@
 
     // default value for job_type is defined in pstamptoolConfig.c
-    if (!strcmp(job_type, "get_image") || !strcmp(job_type, "detect_query")) {
+    if (!strcmp(job_type, "get_image") || !strcmp(job_type, "detect_query") || !strcmp(job_type, "none")) {
         stampJob = false;
     } else if (!strcmp(job_type, "stamp")) {