User tests: Successful: Unsuccessful:
Create a multilanguage site.
Set the languagefilter to
1. Items Associations: No
2. Use Automatic Language Change: Yes
3. Set a frontend registered user to use a specific language in frontend
4. Display the site in another language
5. Login with the user
before patch you will get (see #4079 ) "PHP Warning: Illegal string offset '*' etc.)
Patch and test again
Labels |
Added:
?
|
Category | ⇒ | Multilanguage |
Test on 3.3.6 : "PHP warning" in PHP error log file.
Cleaning PHP error log, patch, test: no more error in log file.
Thanks.
@joomdonation
You are right. Will update
Thanks JM :).
One more test to commit
@test: OK
Before patch, there are warning in error logs like this
[Tue Oct 28 19:12:28.602154 2014] [:error] [pid 1852:tid 916] [client ::1:60257] PHP Warning: Illegal string offset '*' in E:\www\testinstallation\plugins\system\languagefilter\languagefilter.php on line 535, referer: http://localhost/testinstallation/index.php/fr/
[Tue Oct 28 19:12:28.602154 2014] [:error] [pid 1852:tid 916] [client ::1:60257] PHP Notice: Trying to get property of non-object in E:\www\testinstallation\plugins\system\languagefilter\languagefilter.php on line 535, referer: http://localhost/testinstallation/index.php/fr/
After path, no warnings anymore.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-10-29 06:14:06 |
Thanks for testing.
Also ported to 2.5.x
Hello all,
after the small fix i get this notices on my site:
Notice: Undefined index: language in /plugins/system/languagefilter/languagefilter.php on line 506
Notice: Undefined property: stdClass::$id in /plugins/system/languagefilter/languagefilter.php on line 535
After i log me in, the site redirect me to the english site and not back to my german site.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4944.
We use at the moment Joomla! 3.3.6 Stable.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4944.
You should never patch only partly a former version.
Please test locally with your database on staging, as there were other changes too.
https://github.com/joomla/joomla-cms/archive/staging.zip
Thanks JM. From a quick look at the code, I think we can improve the code alitle (in the file plugins/system/languagefilter/languagefilter.php). I think we don't have to definine new property $homepages of PlgSystemLanguageFilter class, just use the old $homes property would be enough. Actually, in PlgSystemLanguageFilter, I think we only need to change one line of code:
From
To
and it should work well. No need for other changes like you did in the PR (I might be wrong, will test and review the code carefully when I got home)