SlatePermutate Changeset - 150d5bddeadc · protofusion repositories

Changeset - 150d5bddeadc

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

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

Fix automatic maintenance of no more than one empty course row when adding and removing sections from an otherwise empty course.

1 file changed with 14 insertions and 10 deletions:

scripts/scheduleInput.js

0 comments (0 inline, 0 general)

scripts/scheduleInput.js

➞

Show inline comments

@@ @@ -195,16 +195,21 @@ function add_section_n(cnum, name, synon @@
     /* store course_i in a place the newly added section will look for it */
     jQuery('.pclass' + cnum).next().data({course_i: cnum});
     /* unhide the saturday columns if it's used by autocomplete data */
     if (days.s)
 	jQuery('#jsrows col.saturday').removeClass('collapsed');
     return last_section_i - 1;
+}
 function add_section(cnum)
+{
     return add_section_n(cnum, '', '', '', '', {m: false, t: false, w: false, h: false, f: false, s: false}, '', '', '');
     var section_i = add_section_n(cnum, '', '', '', '', {m: false, t: false, w: false, h: false, f: false, s: false}, '', '', '');
     if (cnum == slate_permutate_course_free)
 	course_free_check(cnum);
     return section_i;
+}
 /**
  * Add a list of sections gotten via an AJAX call.
  */
 function add_sections(cnum, data)
@@ @@ -383,27 +388,26 @@ function course_has_sections(course_i) @@
  *   Figure out whether or not an empty course entry has become filled
  *   or whether a full course has become emptied and react.
+ *
  * This mainly ensures that there is always exactly one course entry
  * spot, eliminating the need of an ``Add class'' button.
+ *
- * \param that
+ * \param course_i
  *   If this is not being called as a 'change' or 'keyup' event
  *   handler for a <input class="className"/>, then that may refer to
  *   a jQuery object representing the <input class="className"/> to
  *   inspect.
  *   handler for a <input class="className"/>, then course_i may refer to
  *   a the course_i to check.
  */
-function course_free_check(that)
+function course_free_check(course_i)
+{
     var me;
     if (jQuery.type(that) == 'undefined')
 	me = that;
     if (jQuery.type(course_i) == 'number')
 	me = jQuery('.pclass' + course_i + ' .className');
     else
 	me = jQuery(this);
-    var course_i = me.parent().parent().data('course_i');
     course_i = me.parent().parent().data('course_i');
     if (course_i == slate_permutate_course_free && (me.val().length || course_has_sections(course_i)))
+	{
 	    /* I am no longer the empty course entry */
 	    slate_permutate_course_free = -1;
 	    add_class();
+	}
@@ @@ -471,13 +475,12 @@ jQuery(document).ready(function() { @@
 	// Deletes the selected section from the input
 	//--------------------------------------------------
 	jQuery('.deleteSection').live('click', function() {
 	  // Decreases the total number of classes
 		var course_i = jQuery(this).parent().parent().data('course_i');
 		sectionsOfClass[course_i]--;
 		course_free_check(jQuery('.pclass' + course_i + ' .className'));
 	  // Find the ID cell of the row we're in
 	  var row = jQuery(this).parent().parent().find(".sectionIdentifier");
 	  // The first input is the one containing the section ID
 	  var toMatch = jQuery(row).find("input").val();
@@ @@ -489,12 +492,13 @@ jQuery(document).ready(function() { @@
 	  jQuery(classClass).each( function() {
 	    // If this section has the same course ID as the item clicked, remove it.
 	    if(jQuery(this).find("input").val() == toMatch){
 		jQuery(this).remove();
+	    }
 	  });
 	  course_free_check(course_i);
 	});
 	jQuery('.className').live('change', course_free_check).live('keyup', course_free_check);
 	//--------------------------------------------------
 	// Bind the section-adding method

0 comments (0 inline, 0 general)