Changeset 18044 for trunk/FACT++/www/schedule/save.php

Timestamp:

12/29/14 22:06:33 (10 years ago)

Author:

tbretz

Message:

If only have of the schedule is trasmitted, also the first entry must be checked about the limit not just the order of all entries.

File:

• trunk/FACT++/www/schedule/save.php (modified) (3 diffs)

trunk/FACT++/www/schedule/save.php

@@ -96 +96 @@
 // Note that there is no sanity check yet, therefore the data and the
 // time variable must be consistent
+// FIXME: This should be 11:59:59 the prev day to allow for 12:00 being
+// the first possible entry, but this makes things below more complicated
 $time = isset($_POST['t']) ? $_POST['t'] : "12:00:00";
 
@@ -160 +162 @@
 // ----------------------------------------------------------------
 
+$last = $cut;
+
 foreach ($data as $row)
 {
@@ -183 +187 @@
     $t = $d['hour']<12 ? $nextDay." ".$t : $day." ".$t;
 
-    if (!isset($last))
-    {
-        $last = $t;
-    }
-    else
-    {
-        // Check all but the last task in a measurement whether
-        // the are not unlimited
-        if ($last==$t)
-        {
-            if ($measurements[$m]['fIsUnlimited']==true)
-                die("Unlimited task '".$measurements[$m]['fMeasurement']."' detected before end of observation\n[".$last."|".($id-1)."]");
-        }
-
-        if ($last>$t)
-            die("Times not sequential\n[".$last."|".$t."]");
-        $last = $t;
-    }
+    // Check all but the last task in a measurement whether
+    // the are not unlimited
+    if ($last==$t)
+    {
+        if ($measurements[$m]['fIsUnlimited']==true)
+            die("Unlimited task '".$measurements[$m]['fMeasurement']."' detected before end of observation\n[".$last."|".($id-1)."]");
+    }
+
+    if ($last>$t)
+        die("Times not sequential\n[".$last."|".$t."]");
+
+    $last = $t;
 
     $m = $row[1]; // measurement