core-decorators - npm Package Compare versions

Comparing version 0.13.0 to 0.14.0

lib/autobind.js

@@ -13,2 +13,3 @@ 'use strict';
 var defineProperty = Object.defineProperty;
+var getPrototypeOf = Object.getPrototypeOf;
 
@@ -51,31 +52,13 @@ function bind(fn, context) {
   var descs = (0, _privateUtils.getOwnPropertyDescriptors)(klass.prototype);
+  var keys = (0, _privateUtils.getOwnKeys)(descs);
 
-  var _iteratorNormalCompletion = true;
-  var _didIteratorError = false;
-  var _iteratorError = undefined;
+  for (var i = 0, l = keys.length; i < l; i++) {
+    var key = keys[i];
+    var desc = descs[key];
 
-  try {
-    for (var _iterator = (0, _privateUtils.getOwnKeys)(descs)[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
-      var key = _step.value;
+    if (typeof desc.value !== 'function' || key === 'constructor') {
+      continue;
+    }
 
-      var desc = descs[key];
-      if (typeof desc.value !== 'function' || key === 'constructor') {
-        continue;
-      }
-
-      defineProperty(klass.prototype, key, autobindMethod(klass.prototype, key, desc));
-    }
-  } catch (err) {
-    _didIteratorError = true;
-    _iteratorError = err;
-  } finally {
-    try {
-      if (!_iteratorNormalCompletion && _iterator['return']) {
-        _iterator['return']();
-      }
-    } finally {
-      if (_didIteratorError) {
-        throw _iteratorError;
-      }
-    }
+    defineProperty(klass.prototype, key, autobindMethod(klass.prototype, key, desc));
   }
@@ -111,3 +94,3 @@ }
       // i.e. Class.prototype.hasOwnProperty(key) == false && key in Class.prototype
-      if (this.constructor !== constructor && Object.getPrototypeOf(this).constructor === constructor) {
+      if (this.constructor !== constructor && getPrototypeOf(this).constructor === constructor) {
         return fn;
@@ -114,0 +97,0 @@ }

lib/decorate.js

@@ -14,2 +14,4 @@ 'use strict';
 
+var defineProperty = Object.defineProperty;
+
 function handleDescriptor(target, key, descriptor, _ref) {
@@ -49,3 +51,3 @@ var _ref2 = _toArray(_ref);
 
-        Object.defineProperty(this, key, desc);
+        defineProperty(this, key, desc);
 
@@ -52,0 +54,0 @@ return value;

lib/extendDescriptor.js

@@ -13,5 +13,8 @@ 'use strict';
 
+var getPrototypeOf = Object.getPrototypeOf;
+var getOwnPropertyDescriptor = Object.getOwnPropertyDescriptor;
+
 function handleDescriptor(target, key, descriptor) {
-  var superKlass = Object.getPrototypeOf(target);
-  var superDesc = Object.getOwnPropertyDescriptor(superKlass, key);
+  var superKlass = getPrototypeOf(target);
+  var superDesc = getOwnPropertyDescriptor(superKlass, key);
 
@@ -18,0 +21,0 @@ return _extends({}, superDesc, {

lib/lazy-initialize.js

@@ -10,2 +10,4 @@ 'use strict';
 
+var defineProperty = Object.defineProperty;
+
 function handleDescriptor(target, key, descriptor) {
@@ -30,3 +32,3 @@ var configurable = descriptor.configurable;
 
-      Object.defineProperty(this, key, {
+      defineProperty(this, key, {
         configurable: configurable,
@@ -33,0 +35,0 @@ enumerable: enumerable,

lib/mixin.js

@@ -47,28 +47,10 @@ 'use strict';
     var descs = (0, _privateUtils.getOwnPropertyDescriptors)(mixins[i]);
+    var keys = (0, _privateUtils.getOwnKeys)(descs);
 
-    var _iteratorNormalCompletion = true;
-    var _didIteratorError = false;
-    var _iteratorError = undefined;
+    for (var j = 0, k = keys.length; j < k; j++) {
+      var key = keys[j];
 
-    try {
-      for (var _iterator = (0, _privateUtils.getOwnKeys)(descs)[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
-        var key = _step.value;
-
-        if (!hasProperty(key, target.prototype)) {
-          defineProperty(target.prototype, key, descs[key]);
-        }
+      if (!hasProperty(key, target.prototype)) {
+        defineProperty(target.prototype, key, descs[key]);
       }
-    } catch (err) {
-      _didIteratorError = true;
-      _iteratorError = err;
-    } finally {
-      try {
-        if (!_iteratorNormalCompletion && _iterator['return']) {
-          _iterator['return']();
-        }
-      } finally {
-        if (_didIteratorError) {
-          throw _iteratorError;
-        }
-      }
     }
@@ -75,0 +57,0 @@ }

lib/override.js

@@ -250,28 +250,9 @@ 'use strict';
 function findPossibleAlternatives(superKlass, key) {
-  var _iteratorNormalCompletion = true;
-  var _didIteratorError = false;
-  var _iteratorError = undefined;
+  for (var i = 0, l = suggestionTransforms.length; i < l; i++) {
+    var fn = suggestionTransforms[i];
+    var suggestion = fn(key);
 
-  try {
-    for (var _iterator = suggestionTransforms[Symbol.iterator](), _step; !(_iteratorNormalCompletion = (_step = _iterator.next()).done); _iteratorNormalCompletion = true) {
-      var fn = _step.value;
-
-      var suggestion = fn(key);
-      if (suggestion in superKlass) {
-        return suggestion;
-      }
+    if (suggestion in superKlass) {
+      return suggestion;
     }
-  } catch (err) {
-    _didIteratorError = true;
-    _iteratorError = err;
-  } finally {
-    try {
-      if (!_iteratorNormalCompletion && _iterator['return']) {
-        _iterator['return']();
-      }
-    } finally {
-      if (_didIteratorError) {
-        throw _iteratorError;
-      }
-    }
   }
@@ -278,0 +259,0 @@

lib/suppress-warnings.js

@@ -18,7 +18,11 @@ 'use strict';
 function applyWithoutWarnings(context, fn, args) {
-  var nativeWarn = console.warn;
-  console.warn = suppressedWarningNoop;
-  var ret = fn.apply(context, args);
-  console.warn = nativeWarn;
-  return ret;
+  if (typeof console === 'object') {
+    var nativeWarn = console.warn;
+    console.warn = suppressedWarningNoop;
+    var ret = fn.apply(context, args);
+    console.warn = nativeWarn;
+    return ret;
+  } else {
+    return fn.apply(context, args);
+  }
 }
@@ -25,0 +29,0 @@

package.json

 {
   "name": "core-decorators",
-  "version": "0.13.0",
+  "version": "0.14.0",
   "description": "Library of JavaScript stage-0 decorators (aka ES2016/ES7 decorators but that's not accurate!) inspired by languages that come with built-ins like @​override, @​deprecate, @​autobind, @​mixin and more! Works great with React/Angular/more!",
@@ -5,0 +5,0 @@ "main": "lib/core-decorators.js",

src/autobind.js

@@ -1,3 +0,4 @@
-import { decorate, createDefaultSetter, getOwnPropertyDescriptors, getOwnKeys } from './private/utils';
-const { defineProperty } = Object;
+import { decorate, createDefaultSetter,
+  getOwnPropertyDescriptors, getOwnKeys } from './private/utils';
+const { defineProperty, getPrototypeOf } = Object;
 
@@ -43,5 +44,8 @@ function bind(fn, context) {
   const descs = getOwnPropertyDescriptors(klass.prototype);
+  const keys = getOwnKeys(descs);
 
-  for (const key of getOwnKeys(descs)) {
+  for (let i = 0, l = keys.length; i < l; i++) {
+    const key = keys[i];
     const desc = descs[key];
+
     if (typeof desc.value !== 'function' || key === 'constructor') {
@@ -78,3 +82,3 @@ continue;
       // i.e. Class.prototype.hasOwnProperty(key) == false && key in Class.prototype
-      if (this.constructor !== constructor && Object.getPrototypeOf(this).constructor === constructor) {
+      if (this.constructor !== constructor && getPrototypeOf(this).constructor === constructor) {
         return fn;
@@ -81,0 +85,0 @@ }

src/decorate.js

 import { decorate as _decorate, createDefaultSetter } from './private/utils';
+const { defineProperty } = Object;
 
@@ -28,3 +29,3 @@ function handleDescriptor(target, key, descriptor, [decorator, ...args]) {
 
-        Object.defineProperty(this, key, desc);
+        defineProperty(this, key, desc);
 
@@ -31,0 +32,0 @@ return value;

src/extendDescriptor.js

 import { decorate } from './private/utils';
+const { getPrototypeOf, getOwnPropertyDescriptor } = Object;
 
 function handleDescriptor(target, key, descriptor) {
-  const superKlass = Object.getPrototypeOf(target);
-  const superDesc = Object.getOwnPropertyDescriptor(superKlass, key);
+  const superKlass = getPrototypeOf(target);
+  const superDesc = getOwnPropertyDescriptor(superKlass, key);
 
@@ -7,0 +8,0 @@ return {

src/lazy-initialize.js

 import { decorate, createDefaultSetter } from './private/utils';
+const { defineProperty } = Object;
 
@@ -18,3 +19,3 @@ function handleDescriptor(target, key, descriptor) {
 
-      Object.defineProperty(this, key, {
+      defineProperty(this, key, {
         configurable,
@@ -21,0 +22,0 @@ enumerable,

src/mixin.js

@@ -39,4 +39,7 @@ import { getOwnPropertyDescriptors, getOwnKeys } from './private/utils';
     const descs = getOwnPropertyDescriptors(mixins[i]);
+    const keys = getOwnKeys(descs);
 
-    for (const key of getOwnKeys(descs)) {
+    for (let j = 0, k = keys.length; j < k; j++) {
+      const key = keys[j];
+
       if (!(hasProperty(key, target.prototype))) {
@@ -43,0 +46,0 @@ defineProperty(target.prototype, key, descs[key]);

src/override.js

@@ -212,4 +212,6 @@ import { decorate } from './private/utils';
 function findPossibleAlternatives(superKlass, key) {
-  for (const fn of suggestionTransforms) {
+  for (let i = 0, l = suggestionTransforms.length; i < l; i++) {
+    const fn = suggestionTransforms[i];
     const suggestion = fn(key);
+
     if (suggestion in superKlass) {
@@ -216,0 +218,0 @@ return suggestion;

src/suppress-warnings.js

@@ -8,7 +8,11 @@ import { decorate } from './private/utils';
 function applyWithoutWarnings(context, fn, args) {
-  const nativeWarn = console.warn;
-  console.warn = suppressedWarningNoop;
-  const ret = fn.apply(context, args);
-  console.warn = nativeWarn;
-  return ret;
+  if (typeof console === 'object') {
+    const nativeWarn = console.warn;
+    console.warn = suppressedWarningNoop;
+    const ret = fn.apply(context, args);
+    console.warn = nativeWarn;
+    return ret;
+  } else {
+    return fn.apply(context, args);
+  }
 }
@@ -15,0 +19,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

Worsened metrics

Total package byte prevSize

87285

decreased by-0.98%

Lines of code

1856

decreased by-1.59%

No dependency changes