r13442 - Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js

Author: rmattb
Date: Mon Jan 31 01:55:34 2011
New Revision: 13442

Modified:
   Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js
Log:
Renamed unused variables that were possible to pass in from the js call, which were confusingly called options by the original writers, even though each item in the select is also called an option. Refs #11017.

Modified: Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js
==============================================================================
--- Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js	(original)
+++ Products.Archetypes/trunk/Products/Archetypes/skins/archetypes/widgets/js/keywordmultiselect.js	Mon Jan 31 01:55:34 2011
@@ -79,7 +79,7 @@
 	}
 	
 	// render the html for the options/optgroups
-	function renderOptions(id, options, o, selectName)
+	function renderOptions(id, options, selectName)
 	{
 		var html = "";
 		for(var i = 0; i < options.length; i++) {
@@ -93,7 +93,6 @@
 	{
 		var multiSelectA = $(this);
 		var multiSelectOptions = multiSelectA.next('.multiSelectOptions');
-		var o = multiSelectA.data("config");
 		
 		// Help text here is only relevant when there are many tags, 
 		// so putting that in documentation, rather than here.
@@ -105,7 +104,7 @@
 		var html = "";
 
 		// generate the html for the new options
-		html += renderOptions(multiSelectA.attr('id'), options, o, multiSelectA.attr('name'));
+		html += renderOptions(multiSelectA.attr('id'), options, multiSelectA.attr('name'));
 		
 		multiSelectOptions.html(html);
 		
@@ -347,8 +346,6 @@
 	
 	$.extend($.fn, {
 		multiSelect: function(o) {
-			// Default options
-			if( !o ) o = {};
 
 			// Initialize each multiSelectA
 			$(this).each( function() {
@@ -364,9 +361,6 @@
 				var multiSelectA = $(select).next('.multiSelectA');
 				var multiSelectOptions = multiSelectA.next('.multiSelectOptions');
 				
-				// Attach the config options to the multiSelectA
-				multiSelectA.data("config", o);
-				
 				// Serialize the select options into json options
 				var options = [];
 				$(select).children().each( function() {

------------------------------------------------------------------------------
Special Offer-- Download ArcSight Logger for FREE (a $49 USD value)!
Finally, a world-class log management solution at an even better price-free!
Download using promo code Free_Logger_4_Dev2Dev. Offer expires 
February 28th, so secure your free ArcSight Logger TODAY! 
http://p.sf.net/sfu/arcsight-sfd2d