SlatePermutate Changeset - ed49fbf4f2e1 · protofusion repositories

Changeset - ed49fbf4f2e1

Parent rev.

Child rev.

[Not reviewed]

default

0 4 0

Nathan Brink (binki) - 15 years ago 2010-10-19 19:15:49
ohnobinki@ohnopublishing.net

Slowly reindent some javascript, hiddenly slip the meeting location into the final table.

4 files changed with 21 insertions and 17 deletions:

class.schedule.php

input.php

process.php

scripts/scheduleInput.js

0 comments (0 inline, 0 general)

class.schedule.php

➞

Show inline comments

@@ @@ -66,13 +66,13 @@ class Schedule @@
     $this->nclasses++;
+  }
   //--------------------------------------------------
   // Adds a section to the desired class.
   //--------------------------------------------------
-  function addSection($course_name, $letter, $time_start, $time_end, $days, $synonym = NULL, $faculty = NULL, $room = NULL)
+  function addSection($course_name, $letter, $time_start, $time_end, $days, $synonym = NULL, $faculty = NULL, $location = NULL)
+  {
     $found = false;
     $counter = 0;
     while(!$found && ($counter < $this->nclasses))
+      {
@@ @@ -93,13 +93,13 @@ class Schedule @@
       $section = $this->classStorage[$counter]->section_get($letter);
       if (!$section)
+	{
 	  $section = new Section($letter, array(), $synonym, $faculty);
 	  $this->classStorage[$counter]->section_add($section);
+	}
-      $section->meeting_add(new SectionMeeting($days, $time_start, $time_end, $room));
+      $section->meeting_add(new SectionMeeting($days, $time_start, $time_end, $location));
+    }
+  }
   //--------------------------------------------------
   // Finds all of the possible permutations and stores
   // the results in the storage array.

input.php

➞

Show inline comments

@@ @@ -41,13 +41,14 @@ if ($sch) @@
+	    {
 	      $my_hc .= '    add_section_n(class_last, \'' . htmlentities($section->getLetter(), ENT_QUOTES) . '\', \''
 		. htmlentities($section->getSynonym(), ENT_QUOTES) . '\', \''
 		. $meeting->getStartTime() . '\', \''
 		. $meeting->getEndTime() . '\', '
 		. json_encode(array('m' => $meeting->getDay(0), 't' => $meeting->getDay(1), 'w' => $meeting->getDay(2), 'h' => $meeting->getDay(3), 'f' => $meeting->getDay(4))) . ', \''
 		. htmlentities($section->getProf(), ENT_QUOTES) . "');\n";
 		. htmlentities($section->getProf(), ENT_QUOTES) . '\', \''
 		. htmlentities($meeting->getLocation(), ENT_QUOTES) . "');\n";
+	    }
+	}
+    }
   $my_hc .= '  });
 ';
   $inputPage->headcode_add('scheduleInput', $inputPage->script_wrap($my_hc), TRUE);

process.php

➞

Show inline comments

@@ @@ -118,13 +118,13 @@ if(!$DEBUG) @@
 				$allClasses->addClass($class['name']);
 				foreach($class as $section)
 				  /* Skip the section name, which isn't a section */
 					if(is_array($section))
+					{
 					  $allClasses->addSection($class['name'], $section['letter'], $section['start'], $section['end'], arrayToDays($section['days'], 'alpha'), NULL, $section['professor']);
+					  $allClasses->addSection($class['name'], $section['letter'], $section['start'], $section['end'], arrayToDays($section['days'], 'alpha'), NULL, $section['professor'], $section['location']);
+					}
+			}
+		}
 		$allClasses->findPossibilities();
 		if (!isset($_SESSION['saved']))
 		  $_SESSION['saved'] = array();

scripts/scheduleInput.js

➞

Show inline comments

@@ @@ -62,22 +62,23 @@ var sectionsOfClass = new Array(); @@
     //--------------------------------------------------
     // General Input Functions
     //--------------------------------------------------
 	//--------------------------------------------------
 	// Returns the common inputs for each new section.
 	//--------------------------------------------------
 /**
  * \brief
  * Returns the common inputs for each new section.
  */
         function genSectionHtml(cnum)
+	{
-		genSectionHtml_n(cnum,'','','','','','');
+    genSectionHtml_n(cnum, '', '', '', '', '', '', '');
+	}
 	/* @TODO: This should select & set items based on args, if the args != '' */
-	function genSectionHtml_n(cnum,name,synonym,stime,etime,days,prof)
+function genSectionHtml_n(cnum, name, synonym, stime, etime, days, prof, location)
+	{
 		var snum = sectionsOfClass[cnum];
 		var result = '<tr class="section class' + cnum + '"><td class="none"></td>';
 	        result = result + '<td class="sectionIdentifier center"><input type="hidden" name="postData[' + cnum + '][' + snum + '][synonym]" value="' + synonym + '" /><input type="text" size="1" class="required" name="postData[' + cnum + '][' + snum + '][letter]" value="' + name + '" /></td>';
 		result = result + '<td class="professor center"><input type="text" size="10" class="required" name="postData[' + cnum + ']['+ snum + '][professor]" value="' + prof + '" /></td>';
@@ @@ -133,13 +134,14 @@ var sectionsOfClass = new Array(); @@
 		result = result + '</select></td>\
 			<td class="cbrow"><input type="checkbox" class="daysRequired" name="postData[' + cnum + '][' + snum + '][days][0]" value="1" ' + (days.m ? 'checked="checked"' : '') + ' /></td> \
 			<td class="cbrow"><input type="checkbox" class="daysRequired" name="postData[' + cnum + '][' + snum + '][days][1]" value="1" ' + (days.t ? 'checked="checked"' : '') + ' /></td>\
 			<td class="cbrow"><input type="checkbox" class="daysRequired" name="postData[' + cnum + '][' + snum + '][days][2]" value="1" ' + (days.w ? 'checked="checked"' : '') + ' /></td>\
 			<td class="cbrow"><input type="checkbox" class="daysRequired" name="postData[' + cnum + '][' + snum + '][days][3]" value="1" ' + (days.h ? 'checked="checked"' : '') + ' /></td>\
 			<td class="cbrow"><input type="checkbox" class="daysRequired" name="postData[' + cnum + '][' + snum + '][days][4]" value="1" ' + (days.f ? 'checked="checked"' : '') + ' /></td>';
 		result = result + '<td><div class="deleteSection"><input type="button" value="x" class="gray" /></div></td><td></td></tr>';
 		result = result + '<td><div class="deleteSection"><input type="button" value="x" class="gray" /></div></td><td>'
 		    + '<input type="hidden" name="postData[' + cnum + '][' + snum + '][location]" value="' + location + '" />'+ '</td></tr>';
 		return result;
+	}
 /**
  * Outputs an <option/> element. It will inlcude selected="selected"
  * if the value param equals the test_value param.
@@ @@ -149,23 +151,24 @@ function genOptionHtml(value, content, t @@
     var selected = ' selected="selected"';
     if (value != test_value)
 	selected = '';
     return '<option value="' + value + '"' + selected + '>' + content + "</option>\n";
+}
         //--------------------------------------------------
         // Add a section to a class
         //--------------------------------------------------
 	function add_section_n(cnum,name,synonym,stime,etime,days,prof)
 /**
  * \brief
  *   Add a section to a class.
  */
 function add_section_n(cnum, name, synonym, stime, etime, days, prof, location)
+	{
-		 jQuery('.pclass'+cnum).after(genSectionHtml_n(cnum,name,synonym,stime,etime,days,prof));
+    jQuery('.pclass'+cnum).after(genSectionHtml_n(cnum, name, synonym, stime, etime, days, prof, location));
 		 sectionsOfClass[cnum] ++;
+	}
         function add_section(cnum)
+	{
-	    return add_section_n(cnum, '', '', '', '', {'m':false, 't':false, 'w':false, 'h':false, 'f':false}, '');
+    return add_section_n(cnum, '', '', '', '', {'m':false, 't':false, 'w':false, 'h':false, 'f':false}, '', '');
+        }
 /**
  * Add a list of sections gotten via an AJAX call.
  */
 function add_sections(cnum, data)
@@ @@ -173,13 +176,13 @@ function add_sections(cnum, data) @@
     var i;
     if (!data.sections)
 	return;
     for (i = data.sections.length - 1; i >= 0; i --)
+	{
 	    section = data.sections[i];
 	    add_section_n(cnum, section.section, section.synonym, section.time_start, section.time_end, section.days, section.prof);
+	    add_section_n(cnum, section.section, section.synonym, section.time_start, section.time_end, section.days, section.prof, section.location);
+	}
+}
 	//--------------------------------------------------
 	// Adds a new class to the input.
 	//--------------------------------------------------

0 comments (0 inline, 0 general)