Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drush_backend_batch_process doesn't support class static method for "finished" #3515

Closed
miteshmap opened this issue Apr 12, 2018 · 2 comments

Comments

@miteshmap
Copy link
Contributor

If you try to run a batch with drush_backend_batch_process(), using already existing static methods of a class, It does work for "operations", But it doesn't work for "finished".

It should have work without any issue But as it's called like:
$batch_set['finished']($batch_set['success'], $batch_set['results'], $operations, $elapsed), Instead it should have called usingcall_user_func_array

@weitzman
Copy link
Member

Merged both PRs. Thanks.

@miteshmap
Copy link
Contributor Author

👍 thanks @weitzman, That was real quick 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants