@hig/utils - npm Package Compare versions

Comparing version 0.2.0 to 0.2.1

build/index.es.js

 /**
- * Combines multiple event handlers into one
- * @param {...function(...any)} handlers the event handler functions
- * @returns {function(...any)} An event handler to add to an element
+ * @param {Array<function(...any): void>} handlers the event handler functions
+ * @returns {function(...any): void} An event handler to add to an element
  */
-function combineEventHandlers() {
-  for (var _len = arguments.length, handlers = Array(_len), _key = 0; _key < _len; _key++) {
-    handlers[_key] = arguments[_key];
-  }
-
-  return function () {
-    for (var _len2 = arguments.length, args = Array(_len2), _key2 = 0; _key2 < _len2; _key2++) {
-      args[_key2] = arguments[_key2];
+function createAggregateEventHandler(handlers) {
+  /**
+   * This function needed to be moved out of `combineEventHandlers`
+   * since babel doesn't handle nested functions using rest operators
+   * properly.
+   */
+  return function aggregateEventHandler() {
+    for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
+      args[_key] = arguments[_key];
     }
 
     handlers.forEach(function (handler) {
-      handler.apply(undefined, args);
+      if (handler) handler.apply(undefined, args);
     });
@@ -23,2 +23,15 @@ };
 /**
+ * Combines multiple event handlers into one
+ * @param {...function(...any): void} handlers the event handler functions
+ * @returns {function(...any): void} An event handler to add to an element
+ */
+function combineEventHandlers() {
+  for (var _len2 = arguments.length, handlers = Array(_len2), _key2 = 0; _key2 < _len2; _key2++) {
+    handlers[_key2] = arguments[_key2];
+  }
+
+  return createAggregateEventHandler(handlers);
+}
+
+/**
  * Generates a unique ID
@@ -25,0 +38,0 @@ * @param {string} key

build/index.js

@@ -6,18 +6,18 @@ 'use strict';
 /**
- * Combines multiple event handlers into one
- * @param {...function(...any)} handlers the event handler functions
- * @returns {function(...any)} An event handler to add to an element
+ * @param {Array<function(...any): void>} handlers the event handler functions
+ * @returns {function(...any): void} An event handler to add to an element
  */
-function combineEventHandlers() {
-  for (var _len = arguments.length, handlers = Array(_len), _key = 0; _key < _len; _key++) {
-    handlers[_key] = arguments[_key];
-  }
-
-  return function () {
-    for (var _len2 = arguments.length, args = Array(_len2), _key2 = 0; _key2 < _len2; _key2++) {
-      args[_key2] = arguments[_key2];
+function createAggregateEventHandler(handlers) {
+  /**
+   * This function needed to be moved out of `combineEventHandlers`
+   * since babel doesn't handle nested functions using rest operators
+   * properly.
+   */
+  return function aggregateEventHandler() {
+    for (var _len = arguments.length, args = Array(_len), _key = 0; _key < _len; _key++) {
+      args[_key] = arguments[_key];
     }
 
     handlers.forEach(function (handler) {
-      handler.apply(undefined, args);
+      if (handler) handler.apply(undefined, args);
     });
@@ -28,2 +28,15 @@ };
 /**
+ * Combines multiple event handlers into one
+ * @param {...function(...any): void} handlers the event handler functions
+ * @returns {function(...any): void} An event handler to add to an element
+ */
+function combineEventHandlers() {
+  for (var _len2 = arguments.length, handlers = Array(_len2), _key2 = 0; _key2 < _len2; _key2++) {
+    handlers[_key2] = arguments[_key2];
+  }
+
+  return createAggregateEventHandler(handlers);
+}
+
+/**
  * Generates a unique ID
@@ -30,0 +43,0 @@ * @param {string} key

CHANGELOG.md

@@ -0,1 +1,9 @@
+# [@hig/utils-v0.2.1](https://github.com/Autodesk/hig/compare/@hig/utils@0.2.0...@hig/utils@0.2.1) (2018-08-24)
+
+
+### Bug Fixes
+
+* make `combineEventHandlers` more reliable ([cdece6e](https://github.com/Autodesk/hig/commit/cdece6e))
+* **behavior:** bump [@hig](https://github.com/hig)/utils dependents to v0.2.1 ([48b74d0](https://github.com/Autodesk/hig/commit/48b74d0))
+
 <a name="@hig/utils-v0.2.0"></a>
@@ -2,0 +10,0 @@ # [@hig/utils-v0.2.0](https://github.com/Autodesk/hig/compare/@hig/utils@0.1.0...@hig/utils@0.2.0) (2018-07-26)

package.json

 {
   "name": "@hig/utils",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Non-component specific utility functions for HIG components",
@@ -5,0 +5,0 @@ "author": "Autodesk Inc.",

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

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

4620

increased by39.96%

Lines of code

83

increased by40.68%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes