Changeset 28224

Timestamp:

Jun 4, 2010, 2:22:12 PM (16 years ago)

Author:

bills

Message:

don't advance pstampRequest.state to run unless parsing succeeds. All user triggered errors
should be handled now, so if parser returns an error it is a programming error or system problem.

File:

• trunk/pstamp/scripts/pstamp_parser_run.pl (modified) (3 diffs)

trunk/pstamp/scripts/pstamp_parser_run.pl

@@ -198 +198 @@
 $parse_cmd .= " --verbose" if $verbose;
 
-my $newState = "run";
+my $newState;
 my $fault;
 {
@@ -224 +224 @@
     }
 
-    unless ($success) {
+    if ($success) {
+        $newState = 'run';
+    } else {
         $fault = $error_code >> 8;
-#        $newState = "stop";
     }
 }
@@ -234 +235 @@
 #
 {
-    my $command = "$pstamptool -updatereq -req_id $req_id -set_state $newState";
+    my $command = "$pstamptool -updatereq -req_id $req_id";
+    $command   .= " -set_state $newState" if $newState;
     $command   .= " -set_outdir $outdir";
     $command   .= " -set_reqType $reqType" if $reqType;