User tests: Successful: Unsuccessful:
This PR fixes the issue #4907 reported by @brianteeman. Please see #4907 for details description of the issue, he explained it very well.
Make sure your site has more than one language available.
Apply this PR
Login to back-end of your website, then go to Users -> Users Manager, click on an existing users to edit. Look at Basic Settings tab, there are two params "Backend language" and "Frontend language", please check and make sure the languages displayed in these languages dropdown are sorted by locale.
Labels |
Added:
?
|
This issue is the same with #4679 which was fixed with the PR #4690 . This issue rarely happens (seems it only happens with some special conditions), so it is difficult for us to replicate it.
So if we really want to test it, just make sure the languages are sorted in locale (expected behavior) and it should be OK.
We also need @brianteeman to confirm that this PR fixed the issue on the site he is seeing the issue as well before merging it.
For me, no change before or after, so we can merge if solved on the said site indeed.
Yes, there is no change for me as well (the same as in PR #4690). But It can happen (atleast @brianteeman and @dkanchev saw it).
So if @brianteeman confirms that this PR fixes the issue he is seeing on that site, we can merge it.
While you are here, could you help looking at #4916 as well ? It would be nice to have the issue mentioned on that PR fixed.
I am trying to get Daniel to confirm
On 26 October 2014 09:54, Tuan Pham Ngoc notifications@github.com wrote:
Yes, there is no change for me as well (the same as in PR #4690
#4690). But It can happen
(atleast @brianteeman https://github.com/brianteeman and @dkanchev
https://github.com/dkanchev saw it).So if @brianteeman https://github.com/brianteeman confirms that this PR
fixes the issue he is seeing on that site, we can merge it.While you are here, could you help looking at #4916
#4916 as well ? It would be
nice to have the issue mentioned on that PR fixed.—
Reply to this email directly or view it on GitHub
#4917 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
That would be great. Thanks Brian
Hi guys, sorry for the late reply! I tested the proposed patch - it works as expected :) 10x for the solution!
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-10-27 07:43:22 |
Thanks.
Thanks All :)
How to really test?
Here, before and after, the languages order is alpha based on the tags which are ascii.
Below: de-DE, en-GB, es-ES, etc.