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

Fixes #16912 #16913

Merged
merged 1 commit into from Jun 30, 2017
Merged

Fixes #16912 #16913

merged 1 commit into from Jun 30, 2017

Conversation

frankmayer
Copy link
Contributor

@frankmayer frankmayer commented Jun 29, 2017

Pull Request for Issue #16912.

This PR reverts all but one replacement of call_user_func_array() in PR #13257 as the other ones "may" also cause similar problems.

I mistakenly assumed that the callable in those cases would be a string, but it might be an array that potentially can have many forms. Therefore I reverted other ones that might also cause issues, too.

Summary of Changes

Reversal of some changes of PR #13257

Testing Instructions

Check, that problem reported in issue #16912 has been resolved

Expected result

See issue #16912

Actual result

See issue #16912

Documentation Changes Required

None

This PR reverts all but one replacement of call_user_func_array() in PR #13257 as the other ones "may" also cause similar problems.
@rdeutz rdeutz added this to the Joomla 3.7.3 milestone Jun 30, 2017
@rdeutz rdeutz merged commit 60e8083 into joomla:staging Jun 30, 2017
@zero-24
Copy link
Contributor

zero-24 commented Jun 30, 2017

Thanks for the quick response 👍

@ggppdk
Copy link
Contributor

ggppdk commented Jun 30, 2017

@frankmayer thanks 👍

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

Successfully merging this pull request may close these issues.

None yet

5 participants