SlatePermutate Changeset - 34aabb706e18 · protofusion repositories

Changeset - 34aabb706e18

Parent rev.

Child rev.

[Not reviewed]

default

0 2 0

Nathan Brink (binki) - 15 years ago 2011-03-22 00:32:41
ohnobinki@ohnopublishing.net

Fix some php errors by removing empty courses completely.

This case was not caught before because courses may appear to be nonempty during the parsing of POST data. Thus, an additional filtering process was added.

2 files changed with 14 insertions and 1 deletions:

inc/class.schedule.php

process.php

0 comments (0 inline, 0 general)

inc/class.schedule.php

➞

Show inline comments

@@ @@ -151,12 +151,25 @@ class Schedule @@
   //--------------------------------------------------
   // Finds all of the possible permutations and stores
   // the results in the storage array.
   //--------------------------------------------------
 	function findPossibilities()
+	{
 	  /*
 	   * Clean crud (completely empty courses) out of the
 	   * schedule. For some crud, it's much easier to detect that
 	   * it's crud now than during parsing of postData[].
 	   */
 	  foreach ($this->courses as $i => $course)
 	    if (!$course->getnsections())
+	      {
 		unset($this->courses[$i]);
 		$this->courses = array_values($this->courses);
 		return $this->findPossibilities();
+	      }
 		$this->possiblePermutations = 1;
 		/* special case: there is nothing entered into the schedule and thus there is one, NULL permutation */
 		if (!count($this->courses))
+		{
 			/* have an empty schedule */
 			$this->nPermutations = 1;

process.php

➞

Show inline comments

@@ @@ -175,13 +175,13 @@ if(!$DEBUG) @@
 				$allClasses->addCourse($class['name']);
 				foreach($class as $section)
 				  /* Skip the section name, which isn't a section */
 					if(is_array($section))
+					  {
 					    $error_string = $allClasses->addSection($class['name'], $section['letter'], $section['start'], $section['end'], arrayToDays($section['days'], 'alpha'), $section['synonym'], $section['professor'], $section['location'], $section['type']);
+					    $error_string = $allClasses->addSection($class['name'], $section['letter'], $section['start'], $section['end'], arrayToDays(empty($section['days']) ? array() : $section['days'], 'alpha'), $section['synonym'], $section['professor'], $section['location'], $section['type']);
 					    if ($error_string !== NULL)
 					      $errors[] = $error_string;
+					  }
+			}
+		}

0 comments (0 inline, 0 general)