handle numeric literal members referring to like string keys; fixes d…

…chester#44
spbast · Nov 23, 2016 · c469537 · c469537
1 parent d10c6c4
commit c469537
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/lib/handlers.js b/lib/handlers.js
@@ -48,6 +48,9 @@ Handlers.prototype._fns = {
   'subscript-child-numeric_literal':
     _descend(function(key, value, ref) { return key === ref }),
 
+  'member-child-numeric_literal':
+    _descend(function(key, value, ref) { return String(key) === String(ref) }),
+
   'subscript-descendant-numeric_literal':
     _traverse(function(key, value, ref) { return key === ref }),
 
@@ -150,9 +153,6 @@ Handlers.prototype._fns = {
 Handlers.prototype._fns['subscript-child-string_literal'] =
 	Handlers.prototype._fns['member-child-identifier'];
 
-Handlers.prototype._fns['member-child-numeric_literal'] =
-	Handlers.prototype._fns['subscript-child-numeric_literal'];
-
 Handlers.prototype._fns['member-descendant-numeric_literal'] =
     Handlers.prototype._fns['subscript-descendant-string_literal'] =
     Handlers.prototype._fns['member-descendant-identifier'];

diff --git a/test/query.js b/test/query.js
@@ -166,6 +166,12 @@ suite('query', function() {
     assert.deepEqual(results, [ { path: [ '$', 'store', 'book', 0 ], value: data.store.book[0] } ]);
   });
 
+  test('member numeric literal matches string-numeric key', function() {
+    var data = { authors: { '1': 'Herman Melville', '2': 'J. R. R. Tolkien' } };
+    var results = jp.nodes(data, '$.authors.1');
+    assert.deepEqual(results, [ { path: [ '$', 'authors', 1 ], value: 'Herman Melville' } ]);
+  });
+
   test('descendant numeric literal gets first element', function() {
     var results = jp.nodes(data, '$.store.book..0');
     assert.deepEqual(results, [ { path: [ '$', 'store', 'book', 0 ], value: data.store.book[0] } ]);