SlatePermutate Changeset - a58b788f2a55 · protofusion repositories

Changeset - a58b788f2a55

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Nathan Brink (binki) - 15 years ago 2011-02-09 17:06:35
ohnobinki@ohnopublishing.net

Don't use title attribute to store course index, use jQuery().data() instead.

1 file changed with 8 insertions and 4 deletions:

scripts/scheduleInput.js

0 comments (0 inline, 0 general)

scripts/scheduleInput.js

➞

Show inline comments

@@ @@ -287,7 +287,10 @@ function add_sections(cnum, data) @@
 	function add_class_n(name)
+	{
 		sectionsOfClass[classNum] = 0; // Initialize at 0
 		jQuery('#jsrows').append('<tr title="' + classNum + '" class="class class' + classNum + ' pclass' + classNum + '"><td class="nameTip"><input type="text" id="input-course-' + classNum + '" class="classRequired defText className'+classNum+' className" title="Class Name" name="postData[' + classNum + '][name]" value="' + name + '" /></td><td colspan="10"></td><td class="tdInput"><div class="addSection"><input type="button" value="+" class="gray" /></div></td><td class="tdInput"><div class="deleteClass"><input type="button" value="Remove" class="gray" /></div></td></tr>');
 		jQuery('#jsrows').append('<tr id="tr-course-' + classNum + '" class="class class' + classNum + ' pclass' + classNum + '"><td class="nameTip"><input type="text" id="input-course-' + classNum + '" class="classRequired defText className'+classNum+' className" title="Class Name" name="postData[' + classNum + '][name]" value="' + name + '" /></td><td colspan="10"></td><td class="tdInput"><div class="addSection"><input type="button" value="+" class="gray" /></div></td><td class="tdInput"><div class="deleteClass"><input type="button" value="Remove" class="gray" /></div></td></tr>');
 		/* store classNum as course_i into the <tr />: */
 		jQuery('#tr-course-' + classNum).data({course_i: classNum});
 		var class_elem = jQuery('.className' + classNum);
 		class_elem.autocomplete({ source: "auto.php" });
@@ @@ -404,7 +407,7 @@ jQuery(document).ready(function() { @@
 	//--------------------------------------------------
 	jQuery('.deleteClass').live('click', function() {
 		if(confirm('Delete class and all sections of this class?')) {
-			jQuery('.class'+ jQuery(this).parent().parent().attr("title")).remove();
+		    jQuery('.class' + jQuery(this).parent().parent().data('course_i')).remove();
+		}
 	});
@@ @@ -413,7 +416,7 @@ jQuery(document).ready(function() { @@
 	//--------------------------------------------------
 	jQuery('.deleteSection').live('click', function() {
 	  // Decreases the total number of classes
-	  sectionsOfClass[jQuery(this).parent().parent().attr("title")]--;
+		sectionsOfClass[jQuery(this).parent().parent().data('course_i')]--;
 	  // Find the ID cell of the row we're in
 	  var row = jQuery(this).parent().parent().find(".sectionIdentifier");
@@ @@ -438,7 +441,8 @@ jQuery(document).ready(function() { @@
 	// Bind the section-adding method
 	//--------------------------------------------------
 	jQuery('.addSection').live('click', function() {
 		add_section(jQuery(this).parent().parent().attr("title"), sectionsOfClass[jQuery(this).parent().parent().attr("title")]);
 		var course_i = jQuery(this).parent().parent().data('course_i');
 		add_section(course_i, sectionsOfClass[course_i]);
 	});
 	//--------------------------------------------------

0 comments (0 inline, 0 general)