From: John Resig <jeresig@gmail.com>
Date: Mon, 8 Jan 2007 01:17:28 +0000 (+0000)
Subject: Fixed a bug in the jQuery.prop() addition and fixed the test cases to represent the... 
X-Git-Tag: 1.1a~3
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=d0e8a2452ebd30abe403d402e813513cef675694;p=jquery.git

Fixed a bug in the jQuery.prop() addition and fixed the test cases to represent the current set of features.
---

diff --git a/src/jquery/coreTest.js b/src/jquery/coreTest.js
index 4b29c8f86..73621cd17 100644
--- a/src/jquery/coreTest.js
+++ b/src/jquery/coreTest.js
@@ -67,13 +67,9 @@ test("attr(String)", function() {
 	ok( $('#form').attr('action').indexOf("formaction") >= 0, 'Check for action attribute' );
 });
 
-test("attr(String, Function|String)", function() {
+test("attr(String, Function)", function() {
+	expect(1);
 	ok( $('#text1').attr('value', function() { return this.id })[0].value == "text1", "Set value from id" );
-	ok( $('#text2').attr('value', "${this.id}")[0].value == "text2", "Set value from id" );
-	reset();
-	$('#text1, #text2').attr({value: "${this.id + 'foobar'}"});
-	ok( $('#text1')[0].value == "text1foobar", "Set value from id" );
-	ok( $('#text2')[0].value == "text2foobar", "Set value from id" );
 });
 
 test("attr(Hash)", function() {
@@ -440,4 +436,4 @@ test("removeAttr(String", function() {
 test("text(String, Boolean)", function() {
 	ok( $("#foo").text("<div><b>Hello</b> cruel world!</div>")[0].innerHTML == "&lt;div&gt;&lt;b&gt;Hello&lt;/b&gt; cruel world!&lt;/div&gt;", "Check escaped text" );
 	ok( $("#foo").text("<div><b>Hello</b> cruel world!</div>", true)[0].innerHTML == "Hello cruel world!", "Check stripped text" );
-});
\ No newline at end of file
+});
diff --git a/src/jquery/jquery.js b/src/jquery/jquery.js
index 50549bb14..4f1306aeb 100644
--- a/src/jquery/jquery.js
+++ b/src/jquery/jquery.js
@@ -1235,7 +1235,7 @@ jQuery.extend({
 	prop: function(elem, key, value){
 		// Handle executable functions
 		return value.constructor == Function &&
-			value.call( elem, val ) || value;
+			value.call( elem ) || value;
 	},
 
 	className: {