Einav Cohen has posted comments on this change. Change subject: engine,userportal,webadmin: enhance locale selection ......................................................................
Patch Set 5: (1 comment) http://gerrit.ovirt.org/#/c/27973/5/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/UnsupportedLocaleHelper.java File backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/UnsupportedLocaleHelper.java: Line 44: String[] localeKeys = locales.trim().split(" *, *"); //$NON-NLS-1$ Line 45: for (String localeKey: localeKeys) { Line 46: try { Line 47: //Check for valid locale. Line 48: org.apache.commons.lang.LocaleUtils.toLocale(localeKey); > No it only supported the _ variation. thanks; as I mentioned, I recommend doing a 'localeKey.replace("-", "_")', just to prevent undesired behavior in case the user filled locale(s) in the 'xx-YY' format. Line 49: result.add(localeKey); Line 50: } catch (IllegalArgumentException iae) { Line 51: //The locale passed in is not valid, don't add it to the list. Line 52: } -- To view, visit http://gerrit.ovirt.org/27973 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5b816694d6c34549360b189eb85c840688957bdb Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Einav Cohen <eco...@redhat.com> Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
