Data: camelCasing should not ignore case

Fixes gh-2070
jquery · May 4, 2015 · 172cad8 · 172cad8
1 parent 0e79098
commit 172cad8
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/core.js b/src/core.js
@@ -27,7 +27,7 @@ var
 
 	// Matches dashed string for camelizing
 	rmsPrefix = /^-ms-/,
-	rdashAlpha = /-([a-z])/gi,
+	rdashAlpha = /-([a-z])/g,
 
 	// Used by jQuery.camelCase as callback to replace()
 	fcamelCase = function( all, letter ) {

diff --git a/test/unit/data.js b/test/unit/data.js
@@ -615,6 +615,16 @@ test(".data always sets data with the camelCased key (gh-2257)", function() {
 	});
 });
 
+test( ".data should not strip more than one hyphen when camelCasing (gh-2070)", function() {
+	expect( 3 );
+	var div = jQuery( "<div data-nested-single='single' data-nested--double='double' data-nested---triple='triple'></div>" ).appendTo( "#qunit-fixture" ),
+		allData = div.data();
+
+	equal( allData.nestedSingle, "single", "Key is correctly camelCased" );
+	equal( allData[ "nested-Double" ], "double", "Key with double hyphens is correctly camelCased" );
+	equal( allData[ "nested--Triple" ], "triple", "Key with triple hyphens is correctly camelCased" );
+});
+
 test(".data supports interoperable hyphenated/camelCase get/set of properties with arbitrary non-null|NaN|undefined values", function() {
 
 	var div = jQuery("<div/>", { id: "hyphened" }).appendTo("#qunit-fixture"),