lukkari: index.php comparison

comparison index.php @ 44:57af36f0864e

Rename some functions + reindentation cosmetics.

author	Matti Hamalainen <ccr@tnsp.org>
date	Mon, 27 Aug 2012 18:35:20 +0300
parents	7621bfc47491
children	d8a1e85b8dda

comparison

equal deleted inserted replaced

-:7621bfc47491
+:57af36f0864e
 global $luokkaDefault;
 if (preg_match("#^([A-Z]{3}\d[A-Za-z0-9_]{1,6}|ccr|Ryh_[A-Z]{3}\d[A-Za-z0-9_]{1,6})$#", $id, $m)) {
 $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;
 }
 }
 } else
 if (isset($_COOKIE["lukluokka"])) {
 $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;
 }
 if (isset($_GET["tila"])) {
 }
 flock($fp, LOCK_UN);
 }
 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);
 $haveData = isset($classDefs);
 }
-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++) {
 $out .=  "  <td>".$dayHours[$day]["total"]."h (<b>".$dayHours[$day]["grouped"]."h</b>)</td>\n";
 }
 break;
 return $n;
 }
-function findNextHour(&$j, &$i, $day, $hour, $chk)
+function lukFindNextActiveHour(&$j, &$i, $day, $hour, $chk)
 {
 global $classInfo, $classHourTable;
 //  for ($j = $day; $j < $classInfo["maxdays"]; $j++)
 $j = $day;
 for ($i = $hour; $i < $classInfo["maxhours"]; $i++) {
 return -1;
 }
-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)
+for ($i = 2; $i < count($d); $i += 2)
-$out .= "<tr><td>".lukChEntities($d[$j])."</td><td>".(isset($d[$j+1]) ? lukChEntities($d[$j+1]) : "")."</td></tr>";
+$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++)
+for ($i = 1; $i < count($d); $i++)
-$out .= lukChEntities($d[$j])."<br />";
+$out .= lukChEntities($d[$i])."<br />";
 return $out;
 }
 }
-function getHourInfoData($id)
+function lukGetHourInfoData($id)
 {
 global $classDefs;
 if (isset($id)) {
 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
 return "<p>Ei mitään.</p>";
 return "";
 }
+$out = "";
 if ($miniMode && $haveData) {
 $stamp = time();
 $aika = getdate($stamp);
 $day = $aika["wday"] - 1;
 if ($hour >= 0 && $day >= 0)
 {
 $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";
 }
 if (isset($id) && $id >= 0 && isset($classDefs[$id])) {
 $i = $classDefs[$id];
 $hour = 0;
 $id = -1;
 }
 //  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";
 $out .= "<h2>Seuraavaksi";
 if ($found > 0) {
 $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";
 }
 $out .=  "  <th style=\"width: ".(100 / $classInfo["maxdays"]).
 "%;\" class=\"days\">".$dayNames[$day]."</th>\n";
 }
 $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"];
+$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
-if (isset($classDefs[$h])) {
+errorMsg("Internal error, cell $hour / $day does not exist.");
-if (!isset($classDefs[$h]["done"])) {
+}
-$classDefs[$h]["done"] = true;
+$out .= " </tr>\n";
-$i = $classDefs[$h];
+}
-$d = $classDefs[$h]["data"];
+$out .= lukPrintHourInfo($dayHours, $classInfo["maxdays"]).
-$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"]."!");
-}
-$out .= "  <td rowspan=\"".$hours.
-"\" class=\"".($isSplit || $i["grouped"] ? "clgrouped" : "clnormal")."\">".
-getHourInfo($isSplit, $d).
-"<div class=\"nhours\"><span>".$hours."h</span></div></td>\n";
-}
-} else
-lukErrorMsg("Internal error cell $hour / $day : hour id $h does not exist!");
-} else
-lukErrorMsg("Internal error, cell $hour / $day does not exist.");
-}
-$out .= " </tr>\n";
-}
-$out .= printHourInfo($dayHours, $classInfo["maxdays"]).
 "</table>\n";
 } // haveData
 echo "<h1>".$pageTitle."</h1>\n";
 // Show error messages
 if ($errorSet) {
 echo "<ul>\n";
-foreach ($lukErrorMsgs as $msg)
+foreach ($errorMsgs as $msg)
 echo "<li>$msg</li>\n";
 echo "</ul>\n";
 }
 echo $out;

Mercurial > hg > lukkari

comparison index.php @ 44:57af36f0864e