fix: support camel-case escaping in jquery.get search param

mu-lib · Apr 25, 2017 · fa69836 · fa69836
1 parent c4c5eed
commit fa69836
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/jquery.get.js b/jquery.get.js
@@ -12,9 +12,9 @@
     var $ = me.constructor;
     var values = {};
 
-    search = $.expando + "#" + (search || "");
+    search = $.expando + "#" + $.camelCase(search || "");
 
-    return $.map(me, function(element) {
+    return $.map(me, function (element) {
       return $.map($.data(element), function (value, key) {
         return values.hasOwnProperty(key) ? undefined : values[key] = key.startsWith(search) ? value : undefined;
       });

diff --git a/tests/jquery.get.js b/tests/jquery.get.js
@@ -72,4 +72,12 @@
 
     assert.deepEqual(get.call($elements), ["test1", "test2"], "should match data");
   });
+
+  QUnit.test("camelCase search", function (assert) {
+    assert.expect(1);
+
+    var $element = $("<div>").data(expando + "#camel-case", "test");
+
+    assert.deepEqual(get.call($element, "camel-case"), ["test"], "should match data");
+  });
 });