github/jquery
1
1
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2026-02-01 01:36:16 -05:00
Code Issues Packages Projects Releases Wiki Activity

Data: Combine register and cache methods

Browse Source 
Closes gh-2553
This commit is contained in:
Jason Bedard
2015-08-16 20:03:27 -07:00
committed by Michał Gołębiowski
parent ce3b4a6242
commit b5f7c9e2d5
1 changed files with 26 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

58
src/data/Data.js
View File

@@ -12,44 +12,38 @@ Data.uid = 1;
Data.prototype = {
register: function( owner ) {
var value = {};
// If it is a node unlikely to be stringify-ed or looped over
// use plain assignment
if ( owner.nodeType ) {
owner[ this.expando ] = value;
// Otherwise secure it in a non-enumerable property
// configurable must be true to allow the property to be
// deleted when data is removed
} else {
Object.defineProperty( owner, this.expando, {
value: value,
configurable: true
} );
}
return owner[ this.expando ];
},
cache: function( owner ) {
// We can accept data for non-element nodes in modern browsers,
// but we should not, see #8335.
// Always return an empty object.
if ( !acceptData( owner ) ) {
return {};
}
// Check if the owner object already has a cache
var cache = owner[ this.expando ];
var value = owner[ this.expando ];
// If so, return it
if ( cache ) {
return cache;
// If not, create one
if ( !value ) {
value = {};
// We can accept data for non-element nodes in modern browsers,
// but we should not, see #8335.
// Always return an empty object.
if ( acceptData( owner ) ) {
// If it is a node unlikely to be stringify-ed or looped over
// use plain assignment
if ( owner.nodeType ) {
owner[ this.expando ] = value;
// Otherwise secure it in a non-enumerable property
// configurable must be true to allow the property to be
// deleted when data is removed
} else {
Object.defineProperty( owner, this.expando, {
value: value,
configurable: true
} );
}
}
}
// If not, register one
return this.register( owner );
return value;
},
set: function( owner, data, value ) {
var prop,