SlatePermutate Changeset - ff180c328353 · protofusion repositories

Changeset - ff180c328353

Parent rev.

Child rev.

[Not reviewed]

default

0 3 0

Nathan Brink (binki) - 14 years ago 2012-02-15 23:22:23
ohnobinki@ohnopublishing.net

Add support for displaying the friendly name of the department or course in the autocomplete.

3 files changed with 109 insertions and 11 deletions:

auto.php

inc/admin.inc

inc/class.semester.inc

0 comments (0 inline, 0 general)

auto.php

➞

Show inline comments

@@ @@ -80,29 +80,38 @@ if (!$school['crawled']) { @@
  */
 $term_strlen = strlen($term);
 $dept_strlen = strlen($term_parts['department']);
 $dept = $term_parts['department'];
 if (!$getsections && count($term_parts) == 1 && $term_strlen == strlen($dept))
+  {
     $dept_file = $cache_dir . '-depts';
     if (!file_exists($dept_file)) {
       clean_empty_exit();
+    }
     $departments = unserialize(file_get_contents($dept_file));
     $json_depts = array();
     foreach ($departments as $key => $department) {
       if (!strncmp($department, $dept, $dept_strlen)) {
 	$json_depts[] = $department;
     if (!empty($departments) && is_array($departments[0]))
+      {
 	/* New format with department names/labels */
 	foreach ($departments as $department)
 	  if (!strncmp($department['value'], $dept, $dept_strlen))
 	    $json_depts[] = $department;
+      }
+    }
     else
+      {
 	/* Old format with just department id. */
 	foreach ($departments as $department)
 	  if (!strncmp($department, $dept, $dept_strlen))
 	    $json_depts[] = $department;
+      }
     echo json_encode($json_depts);
     exit(0);
+  }
 if ($getsections)
+  {
     if (!isset($term_parts['course']))
+      {
 	/* user didn't give us enough information */
 	header('HTTP/1.1 404: Nof found');
 	header('Content-Type: text/plain; encoding=utf-8');
@@ @@ -130,29 +139,37 @@ if ($getsections) @@
  */
 $classes_file = $cache_dir . $dept . '.sects';
 if (file_exists($classes_file))
+  {
     $classes = unserialize(file_get_contents($classes_file));
     $class_start = '';
     if (count($term_parts) > 1)
       $class_start = $term_parts['course'];
     $class_start_strlen = strlen($class_start);
     /* reduce/create resultset */
     $json_classes = array();
     foreach ($classes as $class)
       if (!strncmp($class, $class_start, $class_start_strlen))
+	{
 	  $json_classes[] = $dept . '-' . $class;
+	}
     if (!empty($classes) && is_array($classes[0]))
+      {
 	foreach ($classes as $course)
 	  if (!strncmp($course['value'], $class_start, $class_start_strlen))
 	    $json_classes[] = $course;
+      }
     else
+      {
 	/* Old format with just course id. */
 	foreach ($classes as $class)
 	  if (!strncmp($class, $class_start, $class_start_strlen))
 	    $json_classes[] = $dept . '-' . $class;
+      }
     echo json_encode($json_classes);
     exit(0);
+  }
 /**
  * Nothing caught
  */
 clean_empty_exit();
 /**
  * \brief

inc/admin.inc

➞

Show inline comments

@@ @@ -289,34 +289,61 @@ function school_crawl(array &$school, Pa @@
        * Write out this semester's cache now that we're here.
        */
       $cache_auto_school_semester_dir_name = $cache_auto_school_dir_name . $semester->id() . DIRECTORY_SEPARATOR;
 	      if (!is_dir($cache_auto_school_semester_dir_name))
+		{
 		  if (!mkdir($cache_auto_school_semester_dir_name, 0755, TRUE))
 		    error_log('Unable to create needed directory: `' . $cache_auto_school_semester_dir_name . '\'');
+		}
 	      $departments = $semester->departments_get();
 	      sort($departments);
 	      /*
 	       * Create friendly department labels (like `MATH
 	       * (Mathematics)') if the crawler has given us that
 	       * data.
 	       */
 	      $has_department_names = FALSE;
 	      $department_names = array();
 	      foreach ($semester->department_names_get() as $dept => $dept_name)
+		{
 		  $department_names[] = array('value' => $dept, 'label' => $dept . ' (' . $dept_name . ')');
 		  if (!$has_department_names && strcmp($dept, $dept_name))
 		    $has_department_names = TRUE;
+		}
 	      $dept_file = fopen($cache_auto_school_semester_dir_name . '-depts', 'wb');
 	      fwrite($dept_file, serialize($departments));
+	      fwrite($dept_file, serialize($has_department_names ? $department_names : $departments));
 	      fclose($dept_file);
 	      /* now per-department autocomplete */
 	      foreach ($departments as $department)
+		{
 		  $classes = $semester->department_classes_get($department);
 		  $courses_json = array();
 		  foreach ($classes as $course_id)
+		    {
 		      $course = $semester->class_get($department, $course_id);
 		      $course_json = array(
 			'value' => $course->getName(),
 			'label' => $course->getName(),
 		      );
 		      if (strlen($course_title = $course->title_get()))
 			$course_json['label'] .= ' (' . $course_title . ')';
 		      $courses_json[] = $course_json;
+		    }
 		  $classes_file = fopen($cache_auto_school_semester_dir_name . $department . '.sects', 'wb');
-		  fwrite($classes_file, serialize($classes));
+		  fwrite($classes_file, serialize($courses_json));
 		  fclose($classes_file);
 		  /* now individual section informations, pre-JSON-ized */
 		  foreach ($classes as $class)
+		    {
 		      if (!is_dir($cache_auto_school_semester_dir_name . $department))
 			mkdir($cache_auto_school_semester_dir_name . $department);
 		      $class_file = fopen($cache_auto_school_semester_dir_name . $department . DIRECTORY_SEPARATOR . $class, 'wb');
 		      fwrite($class_file, json_encode($semester->class_get($department, $class)->to_json_array()));
 		      fclose($class_file);
+		    }
+		}

inc/class.semester.inc

➞

Show inline comments

@@ @@ -70,24 +70,25 @@ class Semester @@
+  {
     $this->time_start = 0;
     $this->time_end = 0;
     $this->time_starts = array();
     $this->time_ends = array();
     $this->season = $season;
     if (strlen($year) != 4 || !is_numeric($year))
       throw new ErrorException('Attempt to construct a Semester with an invalid year. The given year is `' . $year . '\'');
     $this->year = $year;
     $this->departments = array();
     $this->department_names = array();
+  }
   /**
    * \brief
    *   Add a class to this Semester.
+   *
    * \param $class
    *   The class/course to add.
    */
   public function class_add(Course $class)
+  {
     $class_parts = Course::parse($class->getName());
@@ @@ -122,24 +123,77 @@ class Semester @@
   /**
    * \brief
    *   Gets a list of departments available in this semester.
    */
   public function departments_get()
+  {
     return array_keys($this->departments);
+  }
   /**
    * \brief
    *   Set a department's human-friendly name.
+   *
    * \param $dept
    *   The department.
    * \param $name
    *   The human-friendly name of the department.
    */
   public function department_name_set($dept, $name)
+  {
     $this->department_names[$dept] = $name;
+  }
   /**
    * \brief
    *   Determine if a particular department has a human-oriented name.
+   *
    * \param $dept
    *   The department to check.
    * \return
    *   TRUE if this department has been assigned a human-friendly
    *   name.
    */
   public function department_name_has($dept)
+  {
     return !empty($this->department_names[$dept]);
+  }
   /**
    * \brief
    *   Get a list of departments and long names, if available.
+   *
    * This function only returns departments for which this Semester
    * has courses.
+   *
    * \return
    *   An array where keys are department identifiers and values are
    *   human-friendly names (if available). If human-friendly names
    *   are not available, department identifiers are used instead of
    *   the human-friendly ones.
    */
   public function department_names_get()
+  {
     $department_names = array();
     foreach ($this->departments_get() as $dept)
       if (empty($this->department_names[$dept]))
         $department_names[$dept] = $dept;
       else
         $department_names[$dept] = $this->department_names[$dept];
     return $department_names;
+  }
   /**
    * \brief
    *   Gets a list of class/course numbers available for a particular
    *   department.
    */
   public function department_classes_get($dept)
+  {
     if (!isset($this->departments[$dept]))
       throw new ErrorException('I was asked for a department I don\'t own: ' . $dept);
     return array_keys($this->departments[$dept]);
+  }
   /**

0 comments (0 inline, 0 general)