From: David Serduke <davidserduke@gmail.com>
Date: Wed, 28 Nov 2007 16:38:34 +0000 (+0000)
Subject: Fixed #1714 by adding a default empty string if the value is falsey.
X-Git-Url: http://git.asbjorn.it/?a=commitdiff_plain;h=5c6c99361f315bfbc5829be3af94499c3a107f5d;p=jquery.git

Fixed #1714 by adding a default empty string if the value is falsey.
---

diff --git a/src/core.js b/src/core.js
index ce26519..0ebc61d 100644
--- a/src/core.js
+++ b/src/core.js
@@ -396,7 +396,7 @@ jQuery.fn = jQuery.prototype = {
 					
 				// Everything else, we just grab the value
 				} else
-					return this[0].value.replace(/\r/g, "");
+					return (this[0].value || "").replace(/\r/g, "");
 
 			}
 
diff --git a/test/unit/core.js b/test/unit/core.js
index cea8c61..e71de2d 100644
--- a/test/unit/core.js
+++ b/test/unit/core.js
@@ -881,9 +881,11 @@ test("$.extend(Object, Object)", function() {
 });
 
 test("val()", function() {
-	expect(2);
+	expect(3);
 	ok( $("#text1").val() == "Test", "Check for value of input element" );
 	ok( !$("#text1").val() == "", "Check for value of input element" );
+	// ticket #1714 this caused a JS error in IE
+	ok( $("#first").val() == "", "Check a paragraph element to see if it has a value" );
 });
 
 test("val(String)", function() {