SlatePermutate Changeset - 0a851eb741f7 · protofusion repositories

Changeset - 0a851eb741f7

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Nathan Brink (binki) - 15 years ago 2011-03-22 19:30:21
ohnobinki@ohnopublishing.net

Use an index more reliable than a section's possibly absent synonym when building a list of sections in a schedule permutation, fixes issues with cedarville's chapel.

1 file changed with 2 insertions and 2 deletions:

inc/class.schedule.php

0 comments (0 inline, 0 general)

inc/class.schedule.php

➞

Show inline comments

@@ @@ -527,17 +527,17 @@ class Schedule @@
 					. '<span class="prof block">' . htmlentities($section->getProf(), ENT_QUOTES) . "</span>\n"
 					. '<span class="location block">' . htmlentities($current_meeting->getLocation(), ENT_QUOTES) . "</span>\n"
 					. '<span class="synonym block">' . htmlentities($section->getSynonym(), ENT_QUOTES) . "</span>\n"
 					. "</td>\n";
 				      /* for the ``Registration Codes'' dialogue: */
-				      if (empty($permutations_courses[$section->getSynonym()]))
+				      if (empty($permutations_courses[$j]))
+					{
 					  $singleton_course = new Course($course->getName());
 					  $singleton_course->section_add($section);
-					  $permutation_courses[$section->getSynonym()] = $singleton_course->to_json_array();
+					  $permutation_courses[$j] = $singleton_course->to_json_array();
+					}
 				      $filled = TRUE;
+				    }
+			}
+		    }

0 comments (0 inline, 0 general)