You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2007/09/03 17:56:52 UTC
svn commit: r572375 - in /ofbiz/trunk:
applications/ecommerce/webapp/ecommerce/includes/
framework/common/src/org/ofbiz/common/ framework/common/webcommon/includes/
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="<@o...@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="<@o...@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="<@o...@ofbizUrl>?locale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
+ <a href="<@o...@ofbizUrl>?newLocale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
</td>
</tr>
</#list>
Re: svn commit: r572375 - in /ofbiz/trunk: applications/ecommerce/webapp/ecommerce/includes/
framework/common/src/org/ofbiz/common/ framework/common/webcommon/includes/
Posted by David E Jones <jo...@undersunconsulting.com>.
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
jleroux@apache.org 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="<@o...@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="<@o...@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="<@o...@ofbizUrl>?locale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
> + <a href="<@o...@ofbizUrl>?newLocale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
> </td>
> </tr>
> </#list>
>
>
Re: svn commit: r572375 - in /ofbiz/trunk: applications/ecommerce/webapp/ecommerce/includes/
framework/common/src/org/ofbiz/common/ framework/common/webcommon/includes/
Posted by David E Jones <jo...@undersunconsulting.com>.
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
jleroux@apache.org 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="<@o...@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="<@o...@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="<@o...@ofbizUrl>?locale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
> + <a href="<@o...@ofbizUrl>?newLocale=${availableLocale.toString()}">${availableLocale.getDisplayName(availableLocale)}</a>
> </td>
> </tr>
> </#list>
>
>