aurelia-polyfills - npm Package Compare versions

Comparing version 1.0.0-beta.1.0.3 to 1.0.0-beta.1.0.4

bower.json

 {
   "name": "aurelia-polyfills",
-  "version": "1.0.0-beta.1.0.3",
+  "version": "1.0.0-beta.1.0.4",
   "description": "The minimal set of polyfills that the Aurelia platform needs to run on ES5 browsers.",
@@ -5,0 +5,0 @@ "keywords": [

dist/amd/aurelia-polyfills.js

@@ -470,3 +470,6 @@ define(['exports', 'aurelia-pal'], function (exports, _aureliaPal) {
   (function (O) {
-    if ('assign' in O) return;
+    if ('assign' in O) {
+      return;
+    }
+
     O.defineProperty(O, 'assign', {
@@ -483,2 +486,3 @@ configurable: true,
         };
+
         return function assign(where) {
@@ -493,6 +497,12 @@ if (gOPS && !(where instanceof O)) {
 
-          for (var arg, i = 1; i < arguments.length; i++) {
-            arg = arguments[i];
+          for (var i = 1, ii = arguments.length; i < ii; ++i) {
+            var arg = arguments[i];
+
+            if (arg === null || arg === undefined) {
+              continue;
+            }
+
             O.keys(arg).concat(filterOS(arg)).forEach(set);
           }
+
           return where;
@@ -499,0 +509,0 @@ };

dist/aurelia-polyfills.js

@@ -543,23 +543,21 @@ import {PLATFORM} from 'aurelia-pal';
 (function (O) {
-  if ('assign' in O) return;
-  O.defineProperty(
-    O,
-    'assign',
-    {
+  if ('assign' in O) {
+    return;
+  }
+
+  O.defineProperty(O, 'assign', {
       configurable: true,
       writable: true,
-      value: (function () {
-        var
-          gOPS = O.getOwnPropertySymbols,
-          // shortcut without explicitly passing through prototype
-          pIE = O.propertyIsEnumerable,
-          filterOS = gOPS ?
-            function (self) {
-              return gOPS(self).filter(pIE, self);
-            } :
-            function () {
-              // just empty Array won't do much within a .concat(...)
-              return Array.prototype;
-            }
-        ;
+      value: (function() {
+        var gOPS = O.getOwnPropertySymbols,
+            // shortcut without explicitly passing through prototype
+            pIE = O.propertyIsEnumerable,
+            filterOS = gOPS ?
+              function (self) {
+                return gOPS(self).filter(pIE, self);
+              } : function () {
+                // just empty Array won't do much within a .concat(...)
+                return Array.prototype;
+              };
+
         return function assign(where) {
@@ -573,3 +571,3 @@ // Object.create(null) and null objects in general
             console.warn('problematic Symbols', where);
-            // ... now this script does its bloody business !!!
+            // ... now this script does its business !!!
           }
@@ -581,18 +579,16 @@ // avoid JSHint "don't make function in loop"
           // the loop
-          for (var
-            arg,
-            i = 1; i < arguments.length; i++
-          ) {
-            arg = arguments[i];
-            O
-              .keys(arg)
-              .concat(filterOS(arg))
-              .forEach(set)
-            ;
+          for (var i = 1, ii = arguments.length; i < ii; ++i) {
+            var arg = arguments[i];
+
+            if (arg === null || arg === undefined) {
+              continue;
+            }
+
+            O.keys(arg).concat(filterOS(arg)).forEach(set);
           }
+
           return where;
         };
       }())
-    }
-  );
+    });
 }(Object));
@@ -599,0 +595,0 @@

dist/commonjs/aurelia-polyfills.js

@@ -471,3 +471,6 @@ 'use strict';
 (function (O) {
-  if ('assign' in O) return;
+  if ('assign' in O) {
+    return;
+  }
+
   O.defineProperty(O, 'assign', {
@@ -484,2 +487,3 @@ configurable: true,
       };
+
       return function assign(where) {
@@ -494,6 +498,12 @@ if (gOPS && !(where instanceof O)) {
 
-        for (var arg, i = 1; i < arguments.length; i++) {
-          arg = arguments[i];
+        for (var i = 1, ii = arguments.length; i < ii; ++i) {
+          var arg = arguments[i];
+
+          if (arg === null || arg === undefined) {
+            continue;
+          }
+
           O.keys(arg).concat(filterOS(arg)).forEach(set);
         }
+
         return where;
@@ -500,0 +510,0 @@ };

dist/es6/aurelia-polyfills.js

@@ -543,23 +543,21 @@ import {PLATFORM} from 'aurelia-pal';
 (function (O) {
-  if ('assign' in O) return;
-  O.defineProperty(
-    O,
-    'assign',
-    {
+  if ('assign' in O) {
+    return;
+  }
+
+  O.defineProperty(O, 'assign', {
       configurable: true,
       writable: true,
-      value: (function () {
-        var
-          gOPS = O.getOwnPropertySymbols,
-          // shortcut without explicitly passing through prototype
-          pIE = O.propertyIsEnumerable,
-          filterOS = gOPS ?
-            function (self) {
-              return gOPS(self).filter(pIE, self);
-            } :
-            function () {
-              // just empty Array won't do much within a .concat(...)
-              return Array.prototype;
-            }
-        ;
+      value: (function() {
+        var gOPS = O.getOwnPropertySymbols,
+            // shortcut without explicitly passing through prototype
+            pIE = O.propertyIsEnumerable,
+            filterOS = gOPS ?
+              function (self) {
+                return gOPS(self).filter(pIE, self);
+              } : function () {
+                // just empty Array won't do much within a .concat(...)
+                return Array.prototype;
+              };
+
         return function assign(where) {
@@ -573,3 +571,3 @@ // Object.create(null) and null objects in general
             console.warn('problematic Symbols', where);
-            // ... now this script does its bloody business !!!
+            // ... now this script does its business !!!
           }
@@ -581,18 +579,16 @@ // avoid JSHint "don't make function in loop"
           // the loop
-          for (var
-            arg,
-            i = 1; i < arguments.length; i++
-          ) {
-            arg = arguments[i];
-            O
-              .keys(arg)
-              .concat(filterOS(arg))
-              .forEach(set)
-            ;
+          for (var i = 1, ii = arguments.length; i < ii; ++i) {
+            var arg = arguments[i];
+
+            if (arg === null || arg === undefined) {
+              continue;
+            }
+
+            O.keys(arg).concat(filterOS(arg)).forEach(set);
           }
+
           return where;
         };
       }())
-    }
-  );
+    });
 }(Object));
@@ -599,0 +595,0 @@

dist/system/aurelia-polyfills.js

@@ -477,3 +477,6 @@ System.register(['aurelia-pal'], function (_export) {
       (function (O) {
-        if ('assign' in O) return;
+        if ('assign' in O) {
+          return;
+        }
+
         O.defineProperty(O, 'assign', {
@@ -490,2 +493,3 @@ configurable: true,
             };
+
             return function assign(where) {
@@ -500,6 +504,12 @@ if (gOPS && !(where instanceof O)) {
 
-              for (var arg, i = 1; i < arguments.length; i++) {
-                arg = arguments[i];
+              for (var i = 1, ii = arguments.length; i < ii; ++i) {
+                var arg = arguments[i];
+
+                if (arg === null || arg === undefined) {
+                  continue;
+                }
+
                 O.keys(arg).concat(filterOS(arg)).forEach(set);
               }
+
               return where;
@@ -506,0 +516,0 @@ };

doc/CHANGELOG.md

@@ -0,1 +1,9 @@
+### 1.0.0-beta.1.0.4 (2016-03-08)
+
+
+#### Bug Fixes
+
+* **object:** assign ignores null or undefined ([941a892f](http://github.com/aurelia/polyfills/commit/941a892f8a63bce8dea3566c97e911ee31622359))
+
+
 ### 1.0.0-beta.1.0.3 (2016-03-07)
@@ -2,0 +10,0 @@

package.json

 {
   "name": "aurelia-polyfills",
-  "version": "1.0.0-beta.1.0.3",
+  "version": "1.0.0-beta.1.0.4",
   "description": "The minimal set of polyfills that the Aurelia platform needs to run on ES5 browsers.",
@@ -5,0 +5,0 @@ "keywords": [

src/object.js

@@ -57,23 +57,21 @@ (function() {
 (function (O) {
-  if ('assign' in O) return;
-  O.defineProperty(
-    O,
-    'assign',
-    {
+  if ('assign' in O) {
+    return;
+  }
+
+  O.defineProperty(O, 'assign', {
       configurable: true,
       writable: true,
-      value: (function () {
-        var
-          gOPS = O.getOwnPropertySymbols,
-          // shortcut without explicitly passing through prototype
-          pIE = O.propertyIsEnumerable,
-          filterOS = gOPS ?
-            function (self) {
-              return gOPS(self).filter(pIE, self);
-            } :
-            function () {
-              // just empty Array won't do much within a .concat(...)
-              return Array.prototype;
-            }
-        ;
+      value: (function() {
+        var gOPS = O.getOwnPropertySymbols,
+            // shortcut without explicitly passing through prototype
+            pIE = O.propertyIsEnumerable,
+            filterOS = gOPS ?
+              function (self) {
+                return gOPS(self).filter(pIE, self);
+              } : function () {
+                // just empty Array won't do much within a .concat(...)
+                return Array.prototype;
+              };
+
         return function assign(where) {
@@ -87,3 +85,3 @@ // Object.create(null) and null objects in general
             console.warn('problematic Symbols', where);
-            // ... now this script does its bloody business !!!
+            // ... now this script does its business !!!
           }
@@ -95,18 +93,16 @@ // avoid JSHint "don't make function in loop"
           // the loop
-          for (var
-            arg,
-            i = 1; i < arguments.length; i++
-          ) {
-            arg = arguments[i];
-            O
-              .keys(arg)
-              .concat(filterOS(arg))
-              .forEach(set)
-            ;
+          for (var i = 1, ii = arguments.length; i < ii; ++i) {
+            var arg = arguments[i];
+
+            if (arg === null || arg === undefined) {
+              continue;
+            }
+
+            O.keys(arg).concat(filterOS(arg)).forEach(set);
           }
+
           return where;
         };
       }())
-    }
-  );
+    });
 }(Object));

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

230447

increased by0.22%

Worsened metrics

Lines of code

5081

decreased by-0.24%

No dependency changes