Landing pull request 459. Do not allow assumed cache[id] in jQuery.data. Fixes #8235.

rwaldron · timmywil · commit 130a29e78666 · 2011-08-17T17:18:58.000-04:00
More Details: - jquery#459 - http://bugs.jquery.com/ticket/8235
diff --git a/src/data.js b/src/data.js
@@ -51,7 +51,7 @@ jQuery.extend({
 
 		// Avoid doing any more work than we need to when trying to get data on an
 		// object that has no data at all
-		if ( (!id || (pvt && id && !cache[ id ][ internalKey ])) && getByName && data === undefined ) {
+		if ( (!id || (pvt && id && (cache[ id ] && !cache[ id ][ internalKey ]))) && getByName && data === undefined ) {
 			return;
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ jQuery.extend({`
`51`	`51`
`52`	`52`	`// Avoid doing any more work than we need to when trying to get data on an`
`53`	`53`	`// object that has no data at all`
`54`		`- if ( (!id \|\| (pvt && id && !cache[ id ][ internalKey ])) && getByName && data === undefined ) {`
	`54`	`+ if ( (!id \|\| (pvt && id && (cache[ id ] && !cache[ id ][ internalKey ]))) && getByName && data === undefined ) {`
`55`	`55`	`return;`
`56`	`56`	`}`
`57`	`57`