{"action":"created","issue":{"url":"https://api.github.com/repos/caolan/async/issues/1122","repository_url":"https://api.github.com/repos/caolan/async","labels_url":"https://api.github.com/repos/caolan/async/issues/1122/labels{/name}","comments_url":"https://api.github.com/repos/caolan/async/issues/1122/comments","events_url":"https://api.github.com/repos/caolan/async/issues/1122/events","html_url":"https://github.com/caolan/async/issues/1122","id":150182648,"number":1122,"title":"Feature request - autoInjectReturnResults","user":{"login":"darrin","id":73746,"avatar_url":"https://avatars.githubusercontent.com/u/73746?v=3","gravatar_id":"","url":"https://api.github.com/users/darrin","html_url":"https://github.com/darrin","followers_url":"https://api.github.com/users/darrin/followers","following_url":"https://api.github.com/users/darrin/following{/other_user}","gists_url":"https://api.github.com/users/darrin/gists{/gist_id}","starred_url":"https://api.github.com/users/darrin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darrin/subscriptions","organizations_url":"https://api.github.com/users/darrin/orgs","repos_url":"https://api.github.com/users/darrin/repos","events_url":"https://api.github.com/users/darrin/events{/privacy}","received_events_url":"https://api.github.com/users/darrin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-04-21T20:28:08Z","updated_at":"2016-04-22T15:10:39Z","closed_at":null,"body":"First - thanks for async - it's just awesome! I think I love you guys.\r\n\r\nWhat's more - I'm loving the autoInject function.  I have some generic code where I build the set of tasks dynamically - it all works swimmingly with autoInject. However - the code doesn't need a single result or a small subset. Nope, I need the full set of task results - ideally in a map just like `auto` would return.\r\n\r\nI'm currently forking async ([darrin/async](https://github.com/darrin/async)) to accomplish this. I've created a new function called `autoInjectReturnResults` that does what I want. The only difference between it and `autoInject` is that I pass the callback straight-through to `auto` after the `forOwn` block [here](https://github.com/caolan/async/blob/master/lib/autoInject.js#L131) with this:\r\n\r\n    \tauto(newTasks, callback);\r\n\r\nUnless I was under the influence I think an early implementation of autoInject as working exactly this way. I'm certain it was changed because this is a bit inconsistent and maybe non-intuitive unless you really know you want this behavior.\r\n\r\nAnyone have thoughts on this? Am I the only one that needs to know the full set of task results without knowing apriori what the task names will be? \r\n\r\nThanks in advance,\r\n-Darrin"},"comment":{"url":"https://api.github.com/repos/caolan/async/issues/comments/213467435","html_url":"https://github.com/caolan/async/issues/1122#issuecomment-213467435","issue_url":"https://api.github.com/repos/caolan/async/issues/1122","id":213467435,"user":{"login":"darrin","id":73746,"avatar_url":"https://avatars.githubusercontent.com/u/73746?v=3","gravatar_id":"","url":"https://api.github.com/users/darrin","html_url":"https://github.com/darrin","followers_url":"https://api.github.com/users/darrin/followers","following_url":"https://api.github.com/users/darrin/following{/other_user}","gists_url":"https://api.github.com/users/darrin/gists{/gist_id}","starred_url":"https://api.github.com/users/darrin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darrin/subscriptions","organizations_url":"https://api.github.com/users/darrin/orgs","repos_url":"https://api.github.com/users/darrin/repos","events_url":"https://api.github.com/users/darrin/events{/privacy}","received_events_url":"https://api.github.com/users/darrin/received_events","type":"User","site_admin":false},"created_at":"2016-04-22T15:10:38Z","updated_at":"2016-04-22T15:10:38Z","body":"@jdalton @megawac - Yeah I realize that we could have a religious war about use of param names by `autoInject` (and my suggested variant of same). Since `autoInject` is already part of the latest version of async I was hoping to steer clear of this debate. I just want to get the full set of results for what is otherwise a mechanism that fits my use case perfectly.\r\n\r\n@megawac thanks for the suggestion I'll take a look! The hurdle will be that our nodejs project doesn't currently use ES6. We've been thinking of moving in that direction for a variety of reasons. I don't think this will push us over the cusp but it'll continue to build the case for it."}}