SlatePermutate Changeset - 7d36a6a1f3b6 · protofusion repositories

Changeset - 7d36a6a1f3b6

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Nathan Brink (binki) - 15 years ago 2011-02-05 11:40:48
ohnobinki@ohnopublishing.net

calvin: When crawling, add courses to the autocomplete cache even if they have no valid sections. This should reduce user confusion.

1 file changed with 26 insertions and 0 deletions:

school.d/calvin.crawl.inc

0 comments (0 inline, 0 general)

school.d/calvin.crawl.inc

➞

Show inline comments

@@ @@ -297,32 +297,39 @@ function calvin_crawl(Semester $semester @@
 	   * ('Lecture', 'Practicum', or some other unknown value),
 	   * the list of days of week the section meets, the start
 	   * time, the end time, and then the meeting location.
 	   */
 	  if (strpos($sec_meeting_info, 'Times to be Announced') !== FALSE
 	      || strpos($sec_meeting_info, 'Days to be Announced') !== FALSE)
+	    {
 	      if ($verbosity > 2)
 		error_log('Skipping class because of incomplete meeting time information: '
 			  . implode('-', $section_id) . ' has meeting info of `'
 			  . $sec_meeting_info . '\'');
 	      $skipped_sections['incomplete meeting info'] ++;
 	      /* Still add to have less confusing autocomplete */
 	      calvin_crawl_course_add($semester, $section_id['department'], $section_id['course']);
 	      continue;
+	    }
 	  if (!preg_match(';^([0-9]{2}/[0-9]{2}/[0-9]{4})-([0-9]{2}/[0-9]{2}/[0-9]{4}) (([^ ,]+ )+)([^0-9]+) ([^ ]+) - ([^ ]+), (.*)$;', $sec_meeting_info, $meeting_info_matches))
+	    {
 	      error_log('Unable to parse calvin section meeting info string into start/end/days information for '
 			. implode('-', $section_id) . ': ``' . $sec_meeting_info . '\'\'');
 	      $skipped_sections['invalid meeting info format'] ++;
 	      /*
 	       * Still add at least the course to the semester so that
 	       * it shows up in autocmoplete.
 	       */
 	      calvin_crawl_course_add($semester, $section_id['department'], $section_id['course']);
 	      continue;
+	    }
 	  $date_start = $meeting_info_matches[1];
 	  $date_end = $meeting_info_matches[2];
 	  /* e.g., 'Lecture', 'Practicum' */
 	  $meeting_type = strtolower(trim($meeting_info_matches[3]));
 	  $days = school_crawl_days_format(explode(', ', $meeting_info_matches[5]));
 	  $time_start = school_crawl_time_format(strptime($meeting_info_matches[6], '%I:%M%p'));
 	  $time_end = school_crawl_time_format(strptime($meeting_info_matches[7], '%I:%M%p'));
 	  $meeting_place = $meeting_info_matches[8];
@@ @@ -419,12 +426,31 @@ function dom_id_content($domdocument, $i @@
  * The WebAdvisor likes to put <noscript/> in a docs <head />, which
  * is quite bad invalid HTML so that DOM can't handle it.
+ *
  * \param $html
  *   The input HTML to filter.
  * \return
  *   The fixed HTML.
  */
 function calvin_crawl_noscript_filter($html)
+{
   return preg_replace(';\<(noscript)\>.*?\</\1\>;s', '', $html);
+}
 /**
  * \brief
  *   Add a course to a semester if that semester doesn't yet have this
  *   course.
+ *
  * \param $semester
  *   The semester to which the course should be appended.
  * \param $deparmtent
  *   The department of the course to add.
  * \param $course_id
  *   The course_id which, with the department string, forms a
  *   fully-qualified course_id.
  */
 function calvin_crawl_course_add(Semester $semester, $department, $course_id)
+{
   if ($semester->class_get($department, $course_id) == NULL)
     $semester->class_add(new Course($department . '-' . $course_id));
+}

0 comments (0 inline, 0 general)