Mercurial > hg > lukkari

changeset 44:57af36f0864e

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Rename some functions + reindentation cosmetics.
author Matti Hamalainen <ccr@tnsp.org>
date Mon, 27 Aug 2012 18:35:20 +0300
parents 7621bfc47491
children d8a1e85b8dda
files index.php
diffstat 1 files changed, 56 insertions(+), 52 deletions(-) [+]
line wrap: on
line diff
--- a/index.php	Mon Aug 27 07:56:07 2012 +0300
+++ b/index.php	Mon Aug 27 18:35:20 2012 +0300
@@ -51,7 +51,7 @@
     $id = $m[1];
     return TRUE;
   } else {
-    lukErrorMsg("Virhe! Luokan täytyy olla muotoa <b>XXXnXXX</b>, käytetään vakioarvoa <b>".$luokkaDefault."</b>.");
+    errorMsg("Virhe! Luokan täytyy olla muotoa <b>XXXnXXX</b>, käytetään vakioarvoa <b>".$luokkaDefault."</b>.");
     $id = $luokkaDefault;
     return FALSE;
   }
@@ -121,7 +121,7 @@
   $luokka = $_COOKIE["lukluokka"];
   lukCheckClassID($luokka);
 } else {
-  lukErrorMsg("Luokkaa ei asetettu, käytetään vakioarvoa <b>".$luokkaDefault."</b>.");
+  errorMsg("Luokkaa ei asetettu, käytetään vakioarvoa <b>".$luokkaDefault."</b>.");
   $luokka = $luokkaDefault;
 }
 
@@ -176,12 +176,14 @@
   }
   fclose($fp);
 } else {
-  lukErrorMsg("Luokkien listaa ei löytynyt. Kokeile ladata sivu uudelleen hetken kuluttua.");
+  errorMsg("Luokkien listaa ei löytynyt. Kokeile ladata sivu uudelleen hetken kuluttua.");
 }
 
 
 if (!file_exists($dataFile)) {
-  lukErrorMsg("Luokan ".lukChEntities($luokka)." tietoja ei löytynyt! Jos luokkakoodi on uusi, ilmestyy se järjestelmään seuraavan päivityksen aikana. Luokkatiedot päivitetään 2 kertaa vuorokaudessa.");
+  errorMsg("Luokan ".lukChEntities($luokka)." tietoja ei löytynyt! ".
+  "Jos luokkakoodi on uusi, ilmestyy se järjestelmään seuraavan ".
+  "päivityksen aikana. Luokkatiedot päivitetään kerran viikossa.");
   $haveData = FALSE;
 } else {
   require($dataFile);
@@ -189,7 +191,7 @@
 }
 
 
-function printHourInfo($dayHours, $showDays)
+function lukPrintHourInfo($dayHours, $showDays)
 {
   $out = " <tr>\n  <td>Tunteja<br />(<b>ryhmä/vv</b>)</td>\n";
   for ($day = 0; $day < $showDays; $day++) {
@@ -213,7 +215,7 @@
 }
 
 
-function findNextHour(&$j, &$i, $day, $hour, $chk)
+function lukFindNextActiveHour(&$j, &$i, $day, $hour, $chk)
 {
   global $classInfo, $classHourTable;
 //  for ($j = $day; $j < $classInfo["maxdays"]; $j++)
@@ -230,28 +232,28 @@
 }
 
 
-function getHourInfo($isSplit, $d)
+function lukGetHourInfo($isSplit, $d)
 {
   if ($isSplit) {
     $out = "<table>".
     "<tr><td>".lukMatchCourse($d[0])."</td><td>".lukMatchCourse($d[1])."</td></tr>";
 
-    for ($j = 2; $j < count($d); $j += 2)
-      $out .= "<tr><td>".lukChEntities($d[$j])."</td><td>".(isset($d[$j+1]) ? lukChEntities($d[$j+1]) : "")."</td></tr>";
+    for ($i = 2; $i < count($d); $i += 2)
+      $out .= "<tr><td>".lukChEntities($d[$i])."</td><td>".(isset($d[$i+1]) ? lukChEntities($d[$i+1]) : "")."</td></tr>";
     
     return $out."</table>";
   } else {
     $out = lukMatchCourse($d[0])."<br />";
 
-    for ($j = 1; $j < count($d); $j++)
-      $out .= lukChEntities($d[$j])."<br />";
+    for ($i = 1; $i < count($d); $i++)
+      $out .= lukChEntities($d[$i])."<br />";
 
     return $out;
   }
 }
 
 
-function getHourInfoData($id)
+function lukGetHourInfoData($id)
 {
   global $classDefs;
 
@@ -259,7 +261,7 @@
     if ($id >= 0 && isset($classDefs[$id])) {
       $i = $classDefs[$id];
       $isSplit = preg_match("/^[A-Z]\d{6}$/", $i["data"][1]);
-      return getHourInfo($isSplit, $i["data"]);
+      return lukGetHourInfo($isSplit, $i["data"]);
     } else
       return "<p>Ei tunteja.</p>";
   } else
@@ -269,6 +271,8 @@
 }
 
 
+$out = "";
+
 if ($miniMode && $haveData) {
   $stamp = time();
   $aika = getdate($stamp);
@@ -283,7 +287,7 @@
     $id = $classHourTable[$hour][$day];
     $out .=
     "<h2>".$dayNames[$day]." ".date("d.n.Y", $stamp)." - klo ".date("H:i", $stamp)."</h2>\n".
-    getHourInfoData($id)."\n";
+    lukGetHourInfoData($id)."\n";
   }
   
 
@@ -309,7 +313,7 @@
   }
 
 //  echo "day=$day, hour=$hour, id=$id<br />\n";
-  $found = findNextHour($nday, $nhour, $day, $hour, $id);
+  $found = lukFindNextActiveHour($nday, $nhour, $day, $hour, $id);
 
 //  echo "nday=$nday, nhour=$nhour, id=$found<br />\n";
 
@@ -318,7 +322,7 @@
     $stamp = lukMakeHourStamp($nhour, 0, 0, $aika["mon"], $aika["mday"] + $nday - $day, $aika["year"]);
     $aika = getdate($stamp);
     $out .= ": ".$dayNames[$nday]." ".date("d.n.Y", $stamp)." - klo ".date("H:i", $stamp)."</h2>\n".
-    getHourInfoData($found)."\n";
+    lukGetHourInfoData($found)."\n";
   } else {
     $out .= "</h2>\n".
     "<p>Ei mitään</p>\n";
@@ -362,46 +366,46 @@
   }
   $out .= " </tr>\n";
 
-for ($hour = 0; $hour < $classInfo["maxhours"]; $hour++) {
-  $out .= " <tr>\n".
-  "  <th class=\"hours\">".getHour($hour)."</th>\n";
-  for ($day = 0; $day < $classInfo["maxdays"]; $day++) {
-    if (isset($classHourTable[$hour][$day])) {
-      $h = $classHourTable[$hour][$day];
-        
-      if ($h < 1) {
-        if ($h > -9999) {
-          $n = lukResolveHours($hour, $day, $h, TRUE);
-          $out .= "  <td rowspan=\"".$n."\" class=\"clnothing\"></td>\n";
-        }
-      } else
-      if (isset($classDefs[$h])) {
-        if (!isset($classDefs[$h]["done"])) {
-          $classDefs[$h]["done"] = true;
-          $i = $classDefs[$h];
-          $d = $classDefs[$h]["data"];
+  for ($hour = 0; $hour < $classInfo["maxhours"]; $hour++) {
+    $out .= " <tr>\n".
+    "  <th class=\"hours\">".getHour($hour)."</th>\n";
+    for ($day = 0; $day < $classInfo["maxdays"]; $day++) {
+      if (isset($classHourTable[$hour][$day])) {
+        $h = $classHourTable[$hour][$day];
           
-          $isSplit = preg_match("/^[A-Z]\d{6}$/", $d[1]);
-          
-          $hours = lukResolveHours($hour, $day, $h, FALSE);
-          if ($hours != $i["hours"]) {
-            lukErrorMsg("Internal error, cell $hour / $day : hour id $h hours ($hours) do not match ".$i["hours"]."!");
+        if ($h < 1) {
+          if ($h > -9999) {
+            $n = lukResolveHours($hour, $day, $h, TRUE);
+            $out .= "  <td rowspan=\"".$n."\" class=\"clnothing\"></td>\n";
           }
-          
-          $out .= "  <td rowspan=\"".$hours.
-          "\" class=\"".($isSplit || $i["grouped"] ? "clgrouped" : "clnormal")."\">".
-          getHourInfo($isSplit, $d).
-          "<div class=\"nhours\"><span>".$hours."h</span></div></td>\n";
-        }
+        } else
+        if (isset($classDefs[$h])) {
+          if (!isset($classDefs[$h]["done"])) {
+            $classDefs[$h]["done"] = true;
+            $i = $classDefs[$h];
+            $d = $classDefs[$h]["data"];
+            
+            $isSplit = preg_match("/^[A-Z]\d{6}$/", $d[1]);
+            
+            $hours = lukResolveHours($hour, $day, $h, FALSE);
+            if ($hours != $i["hours"]) {
+              errorMsg("Internal error, cell $hour / $day : hour id $h hours ($hours) do not match ".$i["hours"]."!");
+            }
+            
+            $out .= "  <td rowspan=\"".$hours.
+            "\" class=\"".($isSplit || $i["grouped"] ? "clgrouped" : "clnormal")."\">".
+            lukGetHourInfo($isSplit, $d).
+            "<div class=\"nhours\"><span>".$hours."h</span></div></td>\n";
+          }
+        } else
+          errorMsg("Internal error cell $hour / $day : hour id $h does not exist!");
       } else
-        lukErrorMsg("Internal error cell $hour / $day : hour id $h does not exist!");
-    } else
-      lukErrorMsg("Internal error, cell $hour / $day does not exist.");
+        errorMsg("Internal error, cell $hour / $day does not exist.");
+    }
+    $out .= " </tr>\n";
   }
-  $out .= " </tr>\n";
-}
 
-  $out .= printHourInfo($dayHours, $classInfo["maxdays"]).
+  $out .= lukPrintHourInfo($dayHours, $classInfo["maxdays"]).
   "</table>\n";
 
 } // haveData
@@ -448,7 +452,7 @@
 // Show error messages
 if ($errorSet) {
   echo "<ul>\n";
-  foreach ($lukErrorMsgs as $msg)
+  foreach ($errorMsgs as $msg)
     echo "<li>$msg</li>\n";
   echo "</ul>\n";
 }