property-expr - npm Package Compare versions

Comparing version 1.3.0 to 1.3.1

index.js

@@ -21,3 +21,3 @@ /**
   setter: function(path){
-    return setCache[path] || ( setCache[path] = new Function('data, value', expr(path, 'data') + ' = value'))
+    return setCache[path] || ( setCache[path] = new Function('data, value', 'return ' + expr(path, 'data') + ' = value, data;'))
   },
@@ -27,3 +27,3 @@
     var k = path + '_' + safe
-    return getCache[k] || ( getCache[k] = new Function('data', "return " + expr(path, safe, 'data') ))
+    return getCache[k] || ( getCache[k] = new Function('data', 'return ' + expr(path, safe, 'data') ))
   },
@@ -30,0 +30,0 @@

package.json

 {
   "name": "property-expr",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "tiny util for getting and setting deep object props safely",
@@ -5,0 +5,0 @@ "main": "index.js",

test.js

@@ -33,3 +33,5 @@ var a = require('assert')
 
-setter('foo.bar[1]')(obj, 'bot')
+a.strictEqual(
+    setter('foo.bar[1]')(obj, 'bot'), obj, 'setter must return the object')
+
 a.strictEqual(obj.foo.bar[1], 'bot')
@@ -36,0 +38,0 @@

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

9661

increased by0.83%

Lines of code

137

increased by0.74%

No dependency changes