Great, looks better this time. I think those lanugage.ftl files are the only
ones out of the thirty-some-odd in the search results that needed to be changed.
-David
[EMAIL PROTECTED] wrote:
Author: jleroux
Date: Mon Sep 3 08:56:51 2007
New Revision: 572375
URL: http://svn.apache.org/viewvc?rev=572375&view=rev
Log:
A patch from Adrian Crum for an issue from Bilgin Ibryam"Changing the language in
party manager is broken" (https://issues.apache.org/jira/browse/OFBIZ-1223)
This patch définitively corrects the problem David raised + some cosmetic UI
changes
Modified:
ofbiz/trunk/applications/ecommerce/webapp/ecommerce/includes/language.ftl
ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonEvents.java
ofbiz/trunk/framework/common/webcommon/includes/language.ftl
ofbiz/trunk/framework/common/webcommon/includes/listLocales.ftl
Modified:
ofbiz/trunk/applications/ecommerce/webapp/ecommerce/includes/language.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/applications/ecommerce/webapp/ecommerce/includes/language.ftl?rev=572375&r1=572374&r2=572375&view=diff
==============================================================================
--- ofbiz/trunk/applications/ecommerce/webapp/ecommerce/includes/language.ftl
(original)
+++ ofbiz/trunk/applications/ecommerce/webapp/ecommerce/includes/language.ftl
Mon Sep 3 08:56:51 2007
@@ -23,7 +23,7 @@
</div>
<div class="screenlet-body" style="text-align: center;">
<form method="post" name="chooseLanguage"
action="<@ofbizUrl>setSessionLocale</@ofbizUrl>" style="margin: 0;">
- <select name="locale" class="selectBox" style="width:95%">
+ <select name="newLocale" class="selectBox" style="width:95%">
<#list availableLocales as availableLocale>
<#assign langAttr = availableLocale.toString()?replace("_", "-")>
<#assign langDir = "ltr">
Modified: ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonEvents.java
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonEvents.java?rev=572375&r1=572374&r2=572375&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonEvents.java
(original)
+++ ofbiz/trunk/framework/common/src/org/ofbiz/common/CommonEvents.java Mon Sep
3 08:56:51 2007
@@ -164,9 +164,10 @@
return "success";
}
- /** Simple event to set the users per-session locale setting */
+ /** Simple event to set the users per-session locale setting. The user's
locale
+ * setting should be passed as a "newLocale" request parameter. */
public static String setSessionLocale(HttpServletRequest request,
HttpServletResponse response) {
- String localeString = request.getParameter("locale");
+ String localeString = request.getParameter("newLocale");
if (UtilValidate.isNotEmpty(localeString)) {
UtilHttp.setLocale(request, localeString);
@@ -243,6 +244,8 @@
return "success";
}
}
+
+
Modified: ofbiz/trunk/framework/common/webcommon/includes/language.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/includes/language.ftl?rev=572375&r1=572374&r2=572375&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/webcommon/includes/language.ftl (original)
+++ ofbiz/trunk/framework/common/webcommon/includes/language.ftl Mon Sep 3
08:56:51 2007
@@ -37,7 +37,7 @@
<tr>
<td align='center'>
<form method="post" name="chooseLanguage"
action="<@ofbizUrl>setSessionLocale</@ofbizUrl>" style="margin: 0;">
- <select name="locale" class="selectBox">
+ <select name="newLocale" class="selectBox">
<#assign initialDisplayName = locale.getDisplayName(locale)>
<#if 18 < initialDisplayName?length>
<assign initialDisplayName = initialDisplayName[0..15] +
"...">
Modified: ofbiz/trunk/framework/common/webcommon/includes/listLocales.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/includes/listLocales.ftl?rev=572375&r1=572374&r2=572375&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/webcommon/includes/listLocales.ftl (original)
+++ ofbiz/trunk/framework/common/webcommon/includes/listLocales.ftl Mon Sep 3
08:56:51 2007
@@ -50,7 +50,7 @@
</#if>
<tr <#if altRow>class="alternate-row"</#if>>
<td lang="${langAttr}" dir="${langDir}">
- <a
href="<@ofbizUrl>setSessionLocale</@ofbizUrl>?locale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
+ <a
href="<@ofbizUrl>setSessionLocale</@ofbizUrl>?newLocale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
</td>
</tr>
</#list>