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

Fix a fatal error for PHP 5.3 and a multilanguage side #16966

Merged
merged 2 commits into from Jul 4, 2017

Conversation

rdeutz
Copy link
Contributor

@rdeutz rdeutz commented Jul 4, 2017

Pull Request for Issue #16965.

Summary of Changes

This reverts the last change form #13257

Testing Instructions

Test a multilanguage side under PHP 5.3.29. After applying the patch the fatal error should be gone.

@SniperSister
Copy link
Contributor

I have tested this item ✅ successfully on c13c78a


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16966.

@@ -636,7 +636,7 @@ protected function processBuildRules(&$uri, $stage = self::PROCESS_DURING)

foreach ($this->_rules['build' . $stage] as $rule)
{
$rule($this, $uri);
- call_user_func_array($rule, array(&$this, &$uri));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove - and tabs.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks

@mbabker mbabker merged commit 13182c3 into joomla:staging Jul 4, 2017
@rdeutz rdeutz deleted the fix_16965 branch February 5, 2018 19:20
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