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

Undefined variable lang_code in plugins/system/languagefilter/languagefilter.php on line 297 #6483

Merged
merged 1 commit into from Mar 18, 2015

Conversation

@Hackwar
Copy link
Member

@Hackwar Hackwar commented Mar 17, 2015

Fixes #6481. Please refer to there for testing instructions.

@richard67
Copy link
Member

@richard67 richard67 commented Mar 17, 2015

Just tested with success. Thanks @Hackwar 👍


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

@infograf768
Copy link
Member

@infograf768 infograf768 commented Mar 18, 2015

@test
SEF on: this solves the Notice.

We have an issue when SEF off and using associations;
Load a menu item with associations.
In this case, clicking on the languageswitcher flag for the language already loaded always redirects to the home page for that language instead of reloading the same page.
Associated items are loaded correctly for the other languages.
It does not look like related to this patch.

@Hackwar
Copy link
Member Author

@Hackwar Hackwar commented Mar 18, 2015

As far as I can see, the issue that you are describing is also present in 3.3

@infograf768
Copy link
Member

@infograf768 infograf768 commented Mar 18, 2015

It could be indeed.

wilsonge added a commit that referenced this pull request Mar 18, 2015
Undefined variable lang_code in plugins/system/languagefilter/languagefilter.php on line 297
@wilsonge wilsonge merged commit 9912054 into joomla:staging Mar 18, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@wilsonge wilsonge added this to the Joomla! 3.4.1 milestone Mar 18, 2015
@Hackwar Hackwar deleted the Hackwar:patch-46 branch Jan 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

5 participants