From: John Resig <jeresig@gmail.com>
Date: Sun, 9 Sep 2007 19:00:56 +0000 (+0000)
Subject: So, apparently, I never committed the second half of the new field value code. This... 
X-Git-Tag: 1.2~7
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=b59c94d5c23c0b1ec4b2506be0aed6b0578f19f3;p=jquery.git

So, apparently, I never committed the second half of the new field value code. This allows you to set values on groups of checkboxes, radio buttons, and selects.
---

diff --git a/src/core.js b/src/core.js
index ac45d6275..67cb147c7 100644
--- a/src/core.js
+++ b/src/core.js
@@ -292,31 +292,60 @@ jQuery.fn = jQuery.prototype = {
 	
 	val: function( val ) {
 		if ( val == undefined ) {
-		    if ( this.length ) {
-		    	var elem = this[0];
+			if ( this.length ) {
+				var elem = this[0];
 		    	
-		    	// We need to handle select boxes special
				if ( jQuery.nodeName(elem, "select") ) {
					var index = elem.selectedIndex,
+				// We need to handle select boxes special
+				if ( jQuery.nodeName(elem, "select") ) {
+					var index = elem.selectedIndex,
 						a = [],
 						options = elem.options,
 						one = elem.type == "select-one";
 					
-					// Nothing was selected
					if ( index < 0 )
-						return null;
-
-					// Loop through all the selected options
					for ( var i = one ? index : 0, max = one ? index + 1 : options.length; i < max; i++ ) {
-						var option = options[i];
						if ( option.selected ) {
							// Get the specifc value for the option
							var val = jQuery.browser.msie && !option.attributes["value"].specified ? option.text : option.value;
+					// Nothing was selected
+					if ( index < 0 )
+						return null;
+
+					// Loop through all the selected options
+					for ( var i = one ? index : 0, max = one ? index + 1 : options.length; i < max; i++ ) {
+						var option = options[i];
+						if ( option.selected ) {
+							// Get the specifc value for the option
+							var val = jQuery.browser.msie && !option.attributes["value"].specified ? option.text : option.value;
 							
-							// We don't need an array for one selects
							if ( one )
+							// We don't need an array for one selects
+							if ( one )
 								return val;
 							
-							// Multi-Selects return an array
							a.push(val);
						}
					}
-					
					return a;
+							// Multi-Selects return an array
+							a.push(val);
+						}
+					}
+					
+					return a;
 					
-				// Everything else, we just grab the value
				} else
+				// Everything else, we just grab the value
+				} else
 					return this[0].value.replace(/\r/g, "");
 			}
 		} else
-			return this.attr( "value", val );
+			return this.each(function(){
+				if ( val.constructor == Array && /radio|checkbox/.test(this.type) )
+					this.checked = (jQuery.inArray(this.value, val) >= 0 ||
+						jQuery.inArray(this.name, val) >= 0);
+				else if ( jQuery.nodeName(this, "select") ) {
+					var tmp = val.constructor == Array ? val : [val];
+
+					jQuery("option", this).each(function(){
+						this.selected = (jQuery.inArray(this.value, tmp) >= 0 ||
+						jQuery.inArray(this.text, tmp) >= 0);
+					});
+
+					if ( !tmp.length )
+						this.selectedIndex = -1;
+				} else
+					this.value = val;
+			});
 	},
 	
 	html: function( val ) {