From d0e8a2452ebd30abe403d402e813513cef675694 Mon Sep 17 00:00:00 2001 From: John Resig Date: Mon, 8 Jan 2007 01:17:28 +0000 Subject: [PATCH] Fixed a bug in the jQuery.prop() addition and fixed the test cases to represent the current set of features. --- src/jquery/coreTest.js | 10 +++------- src/jquery/jquery.js | 2 +- 2 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/jquery/coreTest.js b/src/jquery/coreTest.js index 4b29c8f8..73621cd1 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("
Hello cruel world!
")[0].innerHTML == "<div><b>Hello</b> cruel world!</div>", "Check escaped text" ); ok( $("#foo").text("
Hello cruel world!
", 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 50549bb1..4f1306ae 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: {