feature request: allow bindActionCreators to accept array

[jimbolla]

Currently, bindActionCreators accepts a single function or an object of functions. It would be useful if it accepted an array as well, to aid this code pattern:

import * as ac from './actionCreators';

// later in file...
const [doThis, doSomethingElse, doAnotherThing] = bindActionCreators(
  [ac.doThis, ac.doSomethingElse, ac.doAnotherThing
], dispatch);
// ...

I like to specify my creator names explicitly because any mistakes are caught immediately by eslint-plugin-import/namespace as an error.

I believe this is the necessary change:

diff --git a/src/bindActionCreators.js b/src/bindActionCreators.js
index e70e270..6050c32 100644
--- a/src/bindActionCreators.js
+++ b/src/bindActionCreators.js
@@ -28,6 +28,10 @@ export default function bindActionCreators(actionCreators, dispatch) {
     return bindActionCreator(actionCreators, dispatch)
   }

+  if (Array.isArray(actionCreators)) {
+    return actionCreators.map(ac => bindActionCreator(ac, dispatch));
+  }
+
   if (typeof actionCreators !== 'object' || actionCreators === null) {
     throw new Error(
       `bindActionCreators expected an object or a function, instead received ${actionCreators === null ? 'null' : typeof actionCreators}. ` +

[aweary]

I think making bindActionCreator more polymorphic would be too much, how about just creating a wrapping function that does this for you?

function mapBindActionCreators(actions, dispatch) {
  return actions.map(action => bindActionCreator(action, dispatch)
}

[kennetpostigo]

If you have an array of them you could reduce() the array into an object and pass that to bindActionCreator

[gaearon]

Yea, I think we’d like to avoid supporting every possible permutation here.