jsonpath - npm Package Compare versions

Comparing version 0.2.7 to 0.2.8

lib/handlers.js

@@ -68,3 +68,3 @@ var aesprim = require('./aesprim');
     if (is_array(partial.value)) {
-      var args = component.expression.value.split(':');
+      var args = component.expression.value.split(':').map(_parse_nullable_int);
       var values = partial.value.map(function(v, i) { return { value: v, path: partial.path.concat(i) } });
@@ -253,2 +253,7 @@ return slice.apply(null, [values].concat(args));
 
+function _parse_nullable_int(val) {
+  var sval = String(val);
+  return sval.match(/^-?[0-9]+$/) ? parseInt(sval) : null;
+}
+
 module.exports = Handlers;

lib/slice.js

 module.exports = function(arr, start, end, step) {
 
+  if (typeof start == 'string') throw new Error("start cannot be a string");
+  if (typeof end == 'string') throw new Error("end cannot be a string");
+  if (typeof step == 'string') throw new Error("step cannot be a string");
+
   var len = arr.length;
@@ -4,0 +8,0 @@

package.json

 {
   "name": "jsonpath",
   "description": "Query JavaScript objects with JSONPath expressions. Robust / safe JSONPath engine for Node.js.",
-  "version": "0.2.7",
+  "version": "0.2.8",
   "author": "[e-mail]",
@@ -6,0 +6,0 @@ "scripts": {

test/lessons.js

@@ -32,3 +32,8 @@ var assert = require('assert');
 
+  test('negative slices', function() {
+    var results = jp.query(data, "$..book[-1:].title");
+    assert.deepEqual(results, ['The Lord of the Rings']);
+  });
+
 });
 

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

389937

increased by0.31%

Lines of code

7747

increased by0.23%

No dependency changes