Skip to content

fix for i18n stuff in IE10 and china firefox#723

Open
deanhiller wants to merge 973 commits intoplayframework:masterfrom
deanhiller:i18n
Open

fix for i18n stuff in IE10 and china firefox#723
deanhiller wants to merge 973 commits intoplayframework:masterfrom
deanhiller:i18n

Conversation

@deanhiller
Copy link
Copy Markdown

Added a unit test where in china, the request from firefox browser was resulting in displaying english which can frustrate users there who have zh_cn as their first language. The previous code picked the first lang match in the list in application.conf instead of the first in the list of the browsers request.

Erick Dovale and others added 30 commits August 17, 2012 11:19
…ons fail. This is helpful to prevent false positives when running on CI servers like jenkins.
Removed System.out.println in getLocalName(Object o)
Ensure that OAuth errors are logged correctly
ticket 1586: avoid NPE if template is not found and applicationClass is null
Fixing evolutions command to return a non-cero exit code when evolutions fail. This is helpful to prevent false positives when running on CI servers like jenkins.
Add check for existens of model file to load
…hread.currentThread().getContextClassLoader() is not Play.classloader
only implemented for unixoid oses, implementation for nt is missing
behavior on nt kept unchanged

[#1621] play refuses to start after crash (when pid_file contains an invalid pid) - windows implementation / changed messages
…-patch

[playframework#1558] i18n tag does not always include default messages
mkurz and others added 27 commits October 13, 2013 21:48
…eans

Conflicts:
	framework/src/play/data/binding/Unbinder.java
…late

 Add loadModels params in fixture.tag to be compatible with master version
 Fix parameters positions in fixtures for loading as template option
the result of "Time.parseDuration(duration)" was treated as an Integer, and this can exceed the limit of Integer.MAX_VALUE very easily, we can cast to Long to prevent this issue.
Used text data type can be set via "evolution.PLAY_EVOLUTIONS.textType"
…rt=<port number>

Use the framework id when parsing the jpda.port so that we can specify a separate fixed port to use for debugging per mode
Fixed bug where adding the option '-f' would actually always randomise the jpda_port
Add documentation for the following command line options: http.port, https.port, jpda.port and pid_file
…r webservice wanted messages of why the request was bad)
…aking our customers.....passwords now can have colon with this fix
Conflicts:
	framework/src/play/jobs/JobsPlugin.java
Conflicts:
	framework/src/play/db/DBPlugin.java
	framework/src/play/db/jpa/JPAPlugin.java
…lay1 into tazmaniax-lighthouse-1748-master-patch

Conflicts:
	framework/src/play/db/DBConfig.java
[#1735] add return to unbind method
@cloudbees-pull-request-builder
Copy link
Copy Markdown

play-1-3-x-pull-requests #144 FAILURE
Looks like there's a problem with this pull request

@xael-fry
Copy link
Copy Markdown
Member

Could you create an issue?
Could you fix the test
Map<String, Header> headers = new HashMap<>(); only compile for 1.7
Just change it to must be fine
Map<String, Header> headers = new HashMap<String, Header>();

Other remark, in your test the request want zh_cn, en or en_us.
So that means that the user want one of this exact match.
I your webiste is in zh_tw, showing this language instead of the requested zh_cn, is not good neither.
As all chinese from china (zh_cn : simplifed character) cannot read Traditional characters (zh_tw), so en will maybe better for them.
So as for me, I would prefer the previous behaviour.

But your example show some problem in Play, if the request contains "zh_cn,en;q=0.3,en_us;q=0.7"
and play has defined en-us and en, play.getLang should return en_us (as we should consider the q value "relative quality factor").

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.