Commit44c56f8

authored

Core: Fix regression in jQuery.text() on HTMLDocument objects

1 parent13a870b commit44c56f8Copy full SHA for 44c56f8

File tree

+12

-4

lines changed

+12

-4

lines changed

Lines changed: 7 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -289,9 +289,14 @@ jQuery.extend( {`
`289`	`289`	`// Do not traverse comment nodes`
`290`	`290`	`ret+=jQuery.text(node);`
`291`	`291`	`}`
`292`		`-}elseif(nodeType===1\|\|nodeType===9\|\|nodeType===11){`
	`292`	`+}`
	`293`	`+if(nodeType===1\|\|nodeType===11){`
`293`	`294`	`returnelem.textContent;`
`294`		`-}elseif(nodeType===3\|\|nodeType===4){`
	`295`	`+}`
	`296`	`+if(nodeType===9){`
	`297`	`+returnelem.documentElement.textContent;`
	`298`	`+}`
	`299`	`+if(nodeType===3\|\|nodeType===4){`
`295`	`300`	`returnelem.nodeValue;`
`296`	`301`	`}`
`297`	`302`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -30,9 +30,9 @@ function manipulationFunctionReturningObj( value ) {`
`30`	`30`
`31`	`31`	`QUnit.test("text()",function(assert){`
`32`	`32`
`33`		`-assert.expect(5);`
	`33`	`+assert.expect(6);`
`34`	`34`
`35`		`-varexpected,frag,$newLineTest;`
	`35`	`+varexpected,frag,$newLineTest,doc;`
`36`	`36`
`37`	`37`	`expected="This link has class=\"blog\": Simon Willison's Weblog";`
`38`	`38`	`assert.equal(jQuery("#sap").text(),expected,"Check for merged text of more then one element.");`
`@@ -52,6 +52,9 @@ QUnit.test( "text()", function( assert ) {`
`52`	`52`	`assert.equal($newLineTest.text(),"test\ntesty","text() does not remove new lines (trac-11153)");`
`53`	`53`
`54`	`54`	`$newLineTest.remove();`
	`55`	`+`
	`56`	`+doc=newDOMParser().parseFromString("<span>example</span>","text/html");`
	`57`	`+assert.equal(jQuery(doc).text(),"example","text() on HTMLDocument (gh-5264)");`
`55`	`58`	`});`
`56`	`59`
`57`	`60`	`QUnit.test("text(undefined)",function(assert){`

Comments

(0)