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

@@ @@ -178,25 +178,37 @@ if ($qtips_always || !isset($_SESSION['s @@
 if ($school['id'] != 'default'
     && empty($_REQUEST['selectschool'])
     && empty($_REQUEST['selectsemester']))
+  {
     /*
      * If we have chosen a school, set the canonical URL so that it
      * 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();
 /*
  * Force a student to choose a school or declare he's a generic
  * student before displaying the input form. To do this, we need
  * another variable in $_SESSION: $_SESSION['school_chosen'].
  */
 if (!empty($_REQUEST['school']) && !empty($_SESSION['school']) && !strcmp($_REQUEST['school'], $_SESSION['school']))
   $_SESSION['school_chosen'] = TRUE;

0 comments (0 inline, 0 general)