simpl-schema - npm Package Compare versions

Comparing version 1.5.0 to 1.5.1

dist/doValidation.js

@@ -121,26 +121,5 @@ 'use strict';
       addValidationErrors: function addValidationErrors(errors) {
-        var _iteratorNormalCompletion = true;
-        var _didIteratorError = false;
-        var _iteratorError = undefined;
-
-        try {
-          for (var _iterator = errors[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
-            var error = _step.value;
-
-            fieldValidationErrors.push(error);
-          }
-        } catch (err) {
-          _didIteratorError = true;
-          _iteratorError = err;
-        } finally {
-          try {
-            if (!_iteratorNormalCompletion && _iterator.return) {
-              _iterator.return();
-            }
-          } finally {
-            if (_didIteratorError) {
-              throw _iteratorError;
-            }
-          }
-        }
+        errors.forEach(function (error) {
+          return fieldValidationErrors.push(error);
+        });
       },
@@ -147,0 +126,0 @@ field: function field(fName) {

dist/SimpleSchema.js

@@ -1080,2 +1080,6 @@ 'use strict';
 
+    // Handle the case when pulling an object from an array the object contains a field
+    // which has a defaultValue. We don't wan't the default value to be returned in this case
+    if (this.operator === '$pull') return;
+
     // Handle the case where we are $pushing an object into an array of objects and we
@@ -1082,0 +1086,0 @@ // want any fields missing from that object to be added if they have default values

dist/ValidationContext.js

@@ -27,2 +27,4 @@ 'use strict';
   function ValidationContext(ss) {
+    var _this = this;
+
     _classCallCheck(this, ValidationContext);
@@ -41,26 +43,5 @@
       this._depsAny = new tracker.Dependency();
-      var _iteratorNormalCompletion = true;
-      var _didIteratorError = false;
-      var _iteratorError = undefined;
-
-      try {
-        for (var _iterator = this._schemaKeys[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
-          var key = _step.value;
-
-          this._deps[key] = new tracker.Dependency();
-        }
-      } catch (err) {
-        _didIteratorError = true;
-        _iteratorError = err;
-      } finally {
-        try {
-          if (!_iteratorNormalCompletion && _iterator.return) {
-            _iterator.return();
-          }
-        } finally {
-          if (_didIteratorError) {
-            throw _iteratorError;
-          }
-        }
-      }
+      this._schemaKeys.forEach(function (key) {
+        _this._deps[key] = new tracker.Dependency();
+      });
     }
@@ -78,29 +59,10 @@ }
     value: function _markKeysChanged(keys) {
+      var _this2 = this;
+
       if (!keys || !Array.isArray(keys) || !keys.length) return;
 
-      var _iteratorNormalCompletion2 = true;
-      var _didIteratorError2 = false;
-      var _iteratorError2 = undefined;
+      keys.forEach(function (key) {
+        return _this2._markKeyChanged(key);
+      });
 
-      try {
-        for (var _iterator2 = keys[Symbol.iterator](), _step2; !(_iteratorNormalCompletion2 = (_step2 = _iterator2.next()).done); _iteratorNormalCompletion2 = true) {
-          var key = _step2.value;
-
-          this._markKeyChanged(key);
-        }
-      } catch (err) {
-        _didIteratorError2 = true;
-        _iteratorError2 = err;
-      } finally {
-        try {
-          if (!_iteratorNormalCompletion2 && _iterator2.return) {
-            _iterator2.return();
-          }
-        } finally {
-          if (_didIteratorError2) {
-            throw _iteratorError2;
-          }
-        }
-      }
-
       this._depsAny && this._depsAny.changed();
@@ -127,2 +89,4 @@ }
     value: function addValidationErrors(errors) {
+      var _this3 = this;
+
       var newValidationErrors = errors.map(function (o) {
@@ -132,29 +96,7 @@ return o.name;
 
-      var _iteratorNormalCompletion3 = true;
-      var _didIteratorError3 = false;
-      var _iteratorError3 = undefined;
+      errors.forEach(function (error) {
+        return _this3._validationErrors.push(error);
+      });
 
-      try {
-        for (var _iterator3 = errors[Symbol.iterator](), _step3; !(_iteratorNormalCompletion3 = (_step3 = _iterator3.next()).done); _iteratorNormalCompletion3 = true) {
-          var error = _step3.value;
-
-          this._validationErrors.push(error);
-        }
-
-        // Mark all new as changed
-      } catch (err) {
-        _didIteratorError3 = true;
-        _iteratorError3 = err;
-      } finally {
-        try {
-          if (!_iteratorNormalCompletion3 && _iterator3.return) {
-            _iterator3.return();
-          }
-        } finally {
-          if (_didIteratorError3) {
-            throw _iteratorError3;
-          }
-        }
-      }
-
+      // Mark all new as changed
       this._markKeysChanged(newValidationErrors);
@@ -243,9 +185,9 @@ }
         // we should keep these errors.
-        var _iteratorNormalCompletion4 = true;
-        var _didIteratorError4 = false;
-        var _iteratorError4 = undefined;
+        var _iteratorNormalCompletion = true;
+        var _didIteratorError = false;
+        var _iteratorError = undefined;
 
         try {
           var _loop = function _loop() {
-            var error = _step4.value;
+            var error = _step.value;
 
@@ -258,16 +200,16 @@ var wasValidated = keysToValidate.some(function (key) {
 
-          for (var _iterator4 = this._validationErrors[Symbol.iterator](), _step4; !(_iteratorNormalCompletion4 = (_step4 = _iterator4.next()).done); _iteratorNormalCompletion4 = true) {
+          for (var _iterator = this._validationErrors[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
             _loop();
           }
         } catch (err) {
-          _didIteratorError4 = true;
-          _iteratorError4 = err;
+          _didIteratorError = true;
+          _iteratorError = err;
         } finally {
           try {
-            if (!_iteratorNormalCompletion4 && _iterator4.return) {
-              _iterator4.return();
+            if (!_iteratorNormalCompletion && _iterator.return) {
+              _iterator.return();
             }
           } finally {
-            if (_didIteratorError4) {
-              throw _iteratorError4;
+            if (_didIteratorError) {
+              throw _iteratorError;
             }
@@ -274,0 +216,0 @@ }

package.json

 {
   "name": "simpl-schema",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "A schema validation package that supports direct validation of MongoDB update modifier objects.",
@@ -17,4 +17,4 @@ "author": "Eric Dobbertin <[e-mail]>",
     "dist",
-    "LICENSE",
-    "README.md"
+    "../LICENSE",
+    "../README.md"
   ],
@@ -21,0 +21,0 @@ "main": "./dist/main.js",

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

Worsened metrics

Total package byte prevSize

118259

decreased by-1.92%

Lines of code

2608

decreased by-2.61%

No dependency changes