Skip to content
Permalink
Browse files

Merge pull request #456 from rwldrn/9318

Improves support for arbitrary numbers in data keys. Fixes #9318
  • Loading branch information
dmethvin committed Aug 5, 2011
2 parents 37254ba + e7af6a2 commit b22c9046529852c7ce567df13397849e11e2b9cc
Showing with 8 additions and 5 deletions.
  1. +2 −2 src/core.js
  2. +6 −3 test/unit/data.js
@@ -46,11 +46,11 @@ var jQuery = function( selector, context ) {
rmozilla = /(mozilla)(?:.*? rv:([\w.]+))?/,

// Matches dashed string for camelizing
rdashAlpha = /-([a-z])/ig,
rdashAlpha = /-([a-z]|[0-9])/ig,

// Used by jQuery.camelCase as callback to replace()
fcamelCase = function( all, letter ) {
return letter.toUpperCase();
return ( letter + "" ).toUpperCase();
},

// Keep a UserAgent string for use with jQuery.browser
@@ -538,10 +538,13 @@ test("jQuery.data supports interoperable hyphenated/camelCase get/set of propert
"an-object": {},
"bool-true": true,
"bool-false": false,
"some-json": '{ "foo": "bar" }'
"some-json": '{ "foo": "bar" }',
"num-1-middle": true,
"num-end-2": true,
"2-num-start": true
};

expect( 18 );
expect( 24 );

jQuery.each( datas, function( key, val ) {
div.data( key, val );
@@ -578,4 +581,4 @@ test("jQuery.data supports interoperable removal of hyphenated/camelCase propert
equal( div.data( key ), undefined, "get: " + key );

});
});
});

0 comments on commit b22c904

Please sign in to comment.
You can’t perform that action at this time.