You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Erwan de FERRIERES <er...@nereide.biz> on 2009/12/04 10:34:33 UTC
Fwd: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl
specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
specialpurpose/ecommerce/widget/CommonScreens.xml
Hi Jacques,
I think there is still a problem with the locale and the calendar.
If you are running a very fresh OFBiz copy, you will see that the
lastLocale is empty and then the calendar is using its default locale
which is en.
If then go choose your langage, the locale is well defined and you've
got your calendar translated.
Maybe it will be useful to use the initialLocale if the lastLocale is empty.
Cheers,
-------- Message original --------
Sujet: svn commit: r886723 - in /ofbiz/trunk:
framework/common/webcommon/includes/lookup.ftl
specialpurpose/cmssite/template/cms/HtmlHead.ftl
specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
specialpurpose/ecommerce/widget/CommonScreens.xml
Date : Thu, 03 Dec 2009 09:45:36 -0000
De : jleroux@apache.org
Répondre à : dev@ofbiz.apache.org
Pour : commits@ofbiz.apache.org
Author: jleroux
Date: Thu Dec 3 09:45:33 2009
New Revision: 886723
URL: http://svn.apache.org/viewvc?rev=886723&view=rev
Log:
Fix a bug introduced with r886143 (when a language.js was missing no
default was given)
Modified:
ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl
ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl
ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml
Modified: ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl?rev=886723&r1=886722&r2=886723&view=diff
==============================================================================
--- ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl (original)
+++ ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl Thu Dec
3 09:45:33 2009
@@ -30,8 +30,8 @@
<script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
<script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
<script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
- <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
- <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
+ <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
+ <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
<#if layoutSettings.javaScripts?has_content>
<#--layoutSettings.javaScripts is a list of java scripts. -->
<#-- use a Set to make sure each javascript is declared only
once, but iterate the list to maintain the correct order -->
Modified: ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl?rev=886723&r1=886722&r2=886723&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl (original)
+++ ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl Thu Dec
3 09:45:33 2009
@@ -27,7 +27,7 @@
<script language="javascript" src="/images/fieldlookup.js"
type="text/javascript"></script>
<script language="javascript" src="/images/selectall.js"
type="text/javascript"></script>
<script language="javascript"
src="/images/calendarDateSelect/calendar_date_select.js"
type="text/javascript"></script>
- <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
+ <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
<link rel="stylesheet" href="/images/ecommain.css" type="text/css"/>
<link rel="stylesheet" href="/ecommerce/images/blog.css"
type="text/css"/>
<link rel="stylesheet" href="/content/images/contentForum.css"
type="text/css"/>
Modified:
ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl?rev=886723&r1=886722&r2=886723&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
(original)
+++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
Thu Dec 3 09:45:33 2009
@@ -25,7 +25,7 @@
<script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
<script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
<script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
- <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
+ <script language="javascript"
src="<@o...@ofbizContentUrl>"
type="text/javascript"></script>
<link rel='stylesheet'
href='<@o...@ofbizContentUrl>'
type='text/css'>
<link rel='stylesheet'
href='<@o...@ofbizContentUrl>'
type='text/css'>
<link rel='stylesheet'
href='<@o...@ofbizContentUrl>'
type='text/css'>
Modified: ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml
URL:
http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml?rev=886723&r1=886722&r2=886723&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml Thu
Dec 3 09:45:33 2009
@@ -246,7 +246,7 @@
<set field="layoutSettings.javaScripts[]"
value="/images/fieldlookup.js" global="true"/>
<set field="layoutSettings.javaScripts[]"
value="/images/selectall.js" global="true"/>
<set field="layoutSettings.javaScripts[]"
value="/images/calendarDateSelect/calendar_date_select.js" global="true"/>
- <set field="layoutSettings.javaScripts[]"
value="/images/calendarDateSelect/locale/${parameters.userLogin.lastLocale}.js"
global="true"/>
+ <set field="layoutSettings.javaScripts[]"
value="/images/calendarDateSelect/locale/${parameters.userLogin.lastLocale}.js"
default-value="en.js" global="true"/>
<set field="layoutSettings.javaScripts[+0]"
value="/images/prototypejs/effects.js" global="true"/>
<set field="layoutSettings.javaScripts[+0]"
value="/images/prototypejs/validation.js" global="true"/>
<set field="layoutSettings.javaScripts[+0]"
value="/images/prototypejs/prototype.js" global="true"/>
Re: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl specialpurpose/ecommerce/widget/CommonScreens.xml
Posted by Jacques Le Roux <ja...@les7arts.com>.
Done at r887454, insomnias have some good aspects
Jacques
From: "Jacques Le Roux" <ja...@les7arts.com>
> From: "Erwan de FERRIERES" <er...@nereide.biz>
>> Le 04/12/2009 15:49, Jacques Le Roux a écrit :
>>> Ewan,
>>>
>>> Yes, I saw that there, how do you envision to use it ?
>>
>> myabe use this locale is case of the lastLocale is empty ?
>
> OK, it's the same than what I did for FTLs, I will do later for commonScreenS (syntactically a bit harder in screen)
>
> Thanks
>
> Jacques
>
>>>
>>> Jacques
>>>
>>> From: "Erwan de FERRIERES" <er...@nereide.biz>
>>>> Jacques,
>>>>
>>>> I was speaking of the initialLocale, which is generated from the
>>>> client locale, in the visitHandler.java, line 121.
>>>>
>>>> Cheers,
>>>>
>>>> Le 04/12/2009 13:20, Jacques Le Roux a écrit :
>>>>> Thanks Erwan,
>>>>>
>>>>> Good catch, I wondered about that also. But this stuff is not easy to
>>>>> test.
>>>>> Which initialLocale are you speaking about?
>>>>> Nevertheless I have followed your ideas and commited a fix for FTL files
>>>>> at r887166. This works well for lookups (I only tested there)
>>>>> I will soon use the same kind of fix for CommonScreens files
>>>>>
>>>>> Jacques
>>>>
>>>> ../..
>>>> --
>>>> Erwan
>>>>
>>>
>>>
>>>
>>
>> --
>> Erwan
>>
>
>
Re: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl specialpurpose/ecommerce/widget/CommonScreens.xml
Posted by Jacques Le Roux <ja...@les7arts.com>.
From: "Erwan de FERRIERES" <er...@nereide.biz>
> Le 04/12/2009 15:49, Jacques Le Roux a écrit :
>> Ewan,
>>
>> Yes, I saw that there, how do you envision to use it ?
>
> myabe use this locale is case of the lastLocale is empty ?
OK, it's the same than what I did for FTLs, I will do later for commonScreenS (syntactically a bit harder in screen)
Thanks
Jacques
>>
>> Jacques
>>
>> From: "Erwan de FERRIERES" <er...@nereide.biz>
>>> Jacques,
>>>
>>> I was speaking of the initialLocale, which is generated from the
>>> client locale, in the visitHandler.java, line 121.
>>>
>>> Cheers,
>>>
>>> Le 04/12/2009 13:20, Jacques Le Roux a écrit :
>>>> Thanks Erwan,
>>>>
>>>> Good catch, I wondered about that also. But this stuff is not easy to
>>>> test.
>>>> Which initialLocale are you speaking about?
>>>> Nevertheless I have followed your ideas and commited a fix for FTL files
>>>> at r887166. This works well for lookups (I only tested there)
>>>> I will soon use the same kind of fix for CommonScreens files
>>>>
>>>> Jacques
>>>
>>> ../..
>>> --
>>> Erwan
>>>
>>
>>
>>
>
> --
> Erwan
>
Re: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl
specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
specialpurpose/ecommerce/widget/CommonScreens.xml
Posted by Erwan de FERRIERES <er...@nereide.biz>.
Le 04/12/2009 15:49, Jacques Le Roux a écrit :
> Ewan,
>
> Yes, I saw that there, how do you envision to use it ?
myabe use this locale is case of the lastLocale is empty ?
>
> Jacques
>
> From: "Erwan de FERRIERES" <er...@nereide.biz>
>> Jacques,
>>
>> I was speaking of the initialLocale, which is generated from the
>> client locale, in the visitHandler.java, line 121.
>>
>> Cheers,
>>
>> Le 04/12/2009 13:20, Jacques Le Roux a écrit :
>>> Thanks Erwan,
>>>
>>> Good catch, I wondered about that also. But this stuff is not easy to
>>> test.
>>> Which initialLocale are you speaking about?
>>> Nevertheless I have followed your ideas and commited a fix for FTL files
>>> at r887166. This works well for lookups (I only tested there)
>>> I will soon use the same kind of fix for CommonScreens files
>>>
>>> Jacques
>>
>> ../..
>> --
>> Erwan
>>
>
>
>
--
Erwan
Re: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl specialpurpose/ecommerce/widget/CommonScreens.xml
Posted by Jacques Le Roux <ja...@les7arts.com>.
Ewan,
Yes, I saw that there, how do you envision to use it ?
Jacques
From: "Erwan de FERRIERES" <er...@nereide.biz>
> Jacques,
>
> I was speaking of the initialLocale, which is generated from the client locale, in the visitHandler.java, line 121.
>
> Cheers,
>
> Le 04/12/2009 13:20, Jacques Le Roux a écrit :
>> Thanks Erwan,
>>
>> Good catch, I wondered about that also. But this stuff is not easy to test.
>> Which initialLocale are you speaking about?
>> Nevertheless I have followed your ideas and commited a fix for FTL files
>> at r887166. This works well for lookups (I only tested there)
>> I will soon use the same kind of fix for CommonScreens files
>>
>> Jacques
>
> ../..
> --
> Erwan
>
Re: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl
specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
specialpurpose/ecommerce/widget/CommonScreens.xml
Posted by Erwan de FERRIERES <er...@nereide.biz>.
Jacques,
I was speaking of the initialLocale, which is generated from the client
locale, in the visitHandler.java, line 121.
Cheers,
Le 04/12/2009 13:20, Jacques Le Roux a écrit :
> Thanks Erwan,
>
> Good catch, I wondered about that also. But this stuff is not easy to test.
> Which initialLocale are you speaking about?
> Nevertheless I have followed your ideas and commited a fix for FTL files
> at r887166. This works well for lookups (I only tested there)
> I will soon use the same kind of fix for CommonScreens files
>
> Jacques
../..
--
Erwan
Re: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl specialpurpose/ecommerce/widget/CommonScreens.xml
Posted by Jacques Le Roux <ja...@les7arts.com>.
Thanks Erwan,
Good catch, I wondered about that also. But this stuff is not easy to test.
Which initialLocale are you speaking about?
Nevertheless I have followed your ideas and commited a fix for FTL files at r887166. This works well for lookups (I only tested
there)
I will soon use the same kind of fix for CommonScreens files
Jacques
From: "Erwan de FERRIERES" <er...@nereide.biz>
> Hi Jacques,
>
> I think there is still a problem with the locale and the calendar.
> If you are running a very fresh OFBiz copy, you will see that the lastLocale is empty and then the calendar is using its default
> locale which is en.
> If then go choose your langage, the locale is well defined and you've got your calendar translated.
>
> Maybe it will be useful to use the initialLocale if the lastLocale is empty.
>
> Cheers,
>
> -------- Message original --------
> Sujet: svn commit: r886723 - in /ofbiz/trunk: framework/common/webcommon/includes/lookup.ftl
> specialpurpose/cmssite/template/cms/HtmlHead.ftl specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
> specialpurpose/ecommerce/widget/CommonScreens.xml
> Date : Thu, 03 Dec 2009 09:45:36 -0000
> De : jleroux@apache.org
> Répondre à : dev@ofbiz.apache.org
> Pour : commits@ofbiz.apache.org
>
> Author: jleroux
> Date: Thu Dec 3 09:45:33 2009
> New Revision: 886723
>
> URL: http://svn.apache.org/viewvc?rev=886723&view=rev
> Log:
> Fix a bug introduced with r886143 (when a language.js was missing no default was given)
>
> Modified:
> ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl
> ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl
> ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
> ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml
>
> Modified: ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl?rev=886723&r1=886722&r2=886723&view=diff
> ==============================================================================
> --- ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl (original)
> +++ ofbiz/trunk/framework/common/webcommon/includes/lookup.ftl Thu Dec 3 09:45:33 2009
> @@ -30,8 +30,8 @@
> <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> - <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> - <script language="javascript"
> src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> + <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> + <script language="javascript"
> src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> <#if layoutSettings.javaScripts?has_content>
> <#--layoutSettings.javaScripts is a list of java scripts. -->
> <#-- use a Set to make sure each javascript is declared only once, but iterate the list to maintain the correct order -->
>
> Modified: ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl?rev=886723&r1=886722&r2=886723&view=diff
> ==============================================================================
> --- ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl (original)
> +++ ofbiz/trunk/specialpurpose/cmssite/template/cms/HtmlHead.ftl Thu Dec 3 09:45:33 2009
> @@ -27,7 +27,7 @@
> <script language="javascript" src="/images/fieldlookup.js" type="text/javascript"></script>
> <script language="javascript" src="/images/selectall.js" type="text/javascript"></script>
> <script language="javascript" src="/images/calendarDateSelect/calendar_date_select.js" type="text/javascript"></script>
> - <script language="javascript"
> src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> + <script language="javascript"
> src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> <link rel="stylesheet" href="/images/ecommain.css" type="text/css"/>
> <link rel="stylesheet" href="/ecommerce/images/blog.css" type="text/css"/>
> <link rel="stylesheet" href="/content/images/contentForum.css" type="text/css"/>
>
> Modified: ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl?rev=886723&r1=886722&r2=886723&view=diff
> ==============================================================================
> --- ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl (original)
> +++ ofbiz/trunk/specialpurpose/ecommerce/webapp/ecommerce/blog/main.ftl Thu Dec 3 09:45:33 2009
> @@ -25,7 +25,7 @@
> <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> <script language="javascript" src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> - <script language="javascript"
> src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> + <script language="javascript"
> src="<@o...@ofbizContentUrl>"
> type="text/javascript"></script>
> <link rel='stylesheet' href='<@o...@ofbizContentUrl>' type='text/css'>
> <link rel='stylesheet' href='<@o...@ofbizContentUrl>' type='text/css'>
> <link rel='stylesheet' href='<@o...@ofbizContentUrl>' type='text/css'>
>
> Modified: ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml?rev=886723&r1=886722&r2=886723&view=diff
> ==============================================================================
> --- ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml (original)
> +++ ofbiz/trunk/specialpurpose/ecommerce/widget/CommonScreens.xml Thu Dec 3 09:45:33 2009
> @@ -246,7 +246,7 @@
> <set field="layoutSettings.javaScripts[]" value="/images/fieldlookup.js" global="true"/>
> <set field="layoutSettings.javaScripts[]" value="/images/selectall.js" global="true"/>
> <set field="layoutSettings.javaScripts[]" value="/images/calendarDateSelect/calendar_date_select.js"
> global="true"/>
> - <set field="layoutSettings.javaScripts[]"
> value="/images/calendarDateSelect/locale/${parameters.userLogin.lastLocale}.js" global="true"/>
> + <set field="layoutSettings.javaScripts[]"
> value="/images/calendarDateSelect/locale/${parameters.userLogin.lastLocale}.js" default-value="en.js" global="true"/>
> <set field="layoutSettings.javaScripts[+0]" value="/images/prototypejs/effects.js" global="true"/>
> <set field="layoutSettings.javaScripts[+0]" value="/images/prototypejs/validation.js" global="true"/>
> <set field="layoutSettings.javaScripts[+0]" value="/images/prototypejs/prototype.js" global="true"/>
>
>
>