SlatePermutate Changeset - 85fd0c4b6efd · protofusion repositories

Changeset - 85fd0c4b6efd

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Nathan Brink (binki) - 14 years ago 2012-02-11 20:06:15
ohnobinki@ohnopublishing.net

Fix regression in d7044a583ccc where the semester could not be selected when editing a saved schedule.

1 file changed with 13 insertions and 1 deletions:

input.php

0 comments (0 inline, 0 general)

input.php

➞

Show inline comments

@@ @@ -184,13 +184,25 @@ if ($school['id'] != 'default' @@
      * contains the school and, optionall, the specified
      * schedule. This way, when Google caches the input.php page, the
      * <title/> it sees will reflect the selected school.
      */
     $query = array('school' => $school['id']);
     if ($sch)
       $query['s'] = $sch->id_get();
+      {
 	$query['s'] = $sch->id_get();
 	/*
 	 * When editing a schedule, also canonize on the
 	 * semester. Changes to the selected schedule which are made
 	 * when editing an existing schedule are not saved into the
 	 * SESSION. Thus, for the user to be able to edit a schedule
 	 * and load courses from an alternate semester, we must ensure
 	 * that semester remains in GET.
 	 */
 	if (!empty($semester))
 	  $query['semester'] = $semester['id'];
+      }
     $inputPage->canonize('input.php', $query);
+  }
 $inputPage->head();
 /*

0 comments (0 inline, 0 general)