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 2008/08/21 11:49:36 UTC

svn commit: r687675 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Author: jleroux
Date: Thu Aug 21 02:49:36 2008
New Revision: 687675

URL: http://svn.apache.org/viewvc?rev=687675&view=rev
Log:
A patch from Bilgin Ibryam on dev ML

Modified:
    ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Modified: ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java?rev=687675&r1=687674&r2=687675&view=diff
==============================================================================
--- ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (original)
+++ ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java Thu Aug 21 02:49:36 2008
@@ -340,7 +340,8 @@
 
         }
         if (listSize > 0) {
-            writer.append("          <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " of " + listSize + "</span> \n");
+            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
+            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " " + listSize + "</span> \n");
         }
         if (highIndex < listSize) {
             writer.append(" <a href=\"");



Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Adam Heath <do...@brainfood.com>.
Scott Gray wrote:
> Done in r688212

Yeah, that looks good.

Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Scott Gray <le...@gmail.com>.
Done in r688212

2008/8/23 Scott Gray <le...@gmail.com>:
> I'll go ahead and recommit it, I've run through a few processes that
> were having issues and they seem fine now.
>
> Regards
> Scott
>
> 2008/8/23 Scott Gray <le...@gmail.com>:
>> Oops not r687675 but r687578, I was looking at the subject of this thread.
>>
>> 2008/8/23 Scott Gray <le...@gmail.com>:
>>> Adam, I reverted 687675 yesterday a few hours before you committed
>>> your last fix, I think it's related to that.  The base compare method
>>> is still expecting a comparable rhs.
>>>
>>> 2008/8/23 Adam Heath <do...@brainfood.com>:
>>>> Jacques Le Roux wrote:
>>>>>
>>>>> Yes, + clean and even a fresh DB. To reproduce : create a fin account,
>>>>> then try to update it
>>>>
>>>> Please don't leave out any steps.  Tell me exactly what you did, all of it.
>>>>
>>>>
>>>>
>>>
>>
>

Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Scott Gray <le...@gmail.com>.
I'll go ahead and recommit it, I've run through a few processes that
were having issues and they seem fine now.

Regards
Scott

2008/8/23 Scott Gray <le...@gmail.com>:
> Oops not r687675 but r687578, I was looking at the subject of this thread.
>
> 2008/8/23 Scott Gray <le...@gmail.com>:
>> Adam, I reverted 687675 yesterday a few hours before you committed
>> your last fix, I think it's related to that.  The base compare method
>> is still expecting a comparable rhs.
>>
>> 2008/8/23 Adam Heath <do...@brainfood.com>:
>>> Jacques Le Roux wrote:
>>>>
>>>> Yes, + clean and even a fresh DB. To reproduce : create a fin account,
>>>> then try to update it
>>>
>>> Please don't leave out any steps.  Tell me exactly what you did, all of it.
>>>
>>>
>>>
>>
>

Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Scott Gray <le...@gmail.com>.
Oops not r687675 but r687578, I was looking at the subject of this thread.

2008/8/23 Scott Gray <le...@gmail.com>:
> Adam, I reverted 687675 yesterday a few hours before you committed
> your last fix, I think it's related to that.  The base compare method
> is still expecting a comparable rhs.
>
> 2008/8/23 Adam Heath <do...@brainfood.com>:
>> Jacques Le Roux wrote:
>>>
>>> Yes, + clean and even a fresh DB. To reproduce : create a fin account,
>>> then try to update it
>>
>> Please don't leave out any steps.  Tell me exactly what you did, all of it.
>>
>>
>>
>

Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Scott Gray <le...@gmail.com>.
Adam, I reverted 687675 yesterday a few hours before you committed
your last fix, I think it's related to that.  The base compare method
is still expecting a comparable rhs.

2008/8/23 Adam Heath <do...@brainfood.com>:
> Jacques Le Roux wrote:
>>
>> Yes, + clean and even a fresh DB. To reproduce : create a fin account,
>> then try to update it
>
> Please don't leave out any steps.  Tell me exactly what you did, all of it.
>
>
>

Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Jacques Le Roux <ja...@les7arts.com>.
From: "Adam Heath" <do...@brainfood.com>
To: <de...@ofbiz.apache.org>
Sent: Friday, August 22, 2008 10:51 PM
Subject: Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java


> Jacques Le Roux wrote:
>> Yes, + clean and even a fresh DB. To reproduce : create a fin account, then try to update it
>
> Please don't leave out any steps.  Tell me exactly what you did, all of it.

Create a fin account of type service credit for party admin. View admin profile (1st error, random : pass it by refeshing screen). 
Try to "Apply service credit" this one is reproductible...

Jacques 


Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Adam Heath <do...@brainfood.com>.
Jacques Le Roux wrote:
> Yes, + clean and even a fresh DB. To reproduce : create a fin account, 
> then try to update it

Please don't leave out any steps.  Tell me exactly what you did, all of it.



Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Jacques Le Roux <ja...@les7arts.com>.
Yes, + clean and even a fresh DB. To reproduce : create a fin account, then try to update it

Jacques

From: "Adam Heath" <do...@brainfood.com>
> Jacques Le Roux wrote:
>> Thanks to Bilgin's help, I commited something in revision 688167.
>> It works well but doint this I found an error which ends by
>>    java.lang.UnsupportedOperationException: = (=)
>> You can easily repreoduce this error on the demo server while using 
>> Party Profile, but it's a random error.
> 
> Did you update first?  I committed something that should fix this in 
> 687936; I found a case of this exception, that then went away when I 
> made the fix.
>

Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Adam Heath <do...@brainfood.com>.
Jacques Le Roux wrote:
> Thanks to Bilgin's help, I commited something in revision 688167.
> It works well but doint this I found an error which ends by
>    java.lang.UnsupportedOperationException: = (=)
> You can easily repreoduce this error on the demo server while using 
> Party Profile, but it's a random error.

Did you update first?  I committed something that should fix this in 
687936; I found a case of this exception, that then went away when I 
made the fix.

Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Jacques Le Roux <ja...@les7arts.com>.
Here is a local log for the error appearing on FinAccounts screenlet in Party Profile. We get the same trying to update a Fin 
Account

2008-08-22 20:42:15,468 (http-0.0.0.0-18443-1) [       SimpleMethod.java:926:ERROR]
---- runtime exception report --------------------------------------------------
Error in simple-method operation [<store-value/>]: java.lang.UnsupportedOperationException: =
Exception: java.lang.UnsupportedOperationException
Message: =
---- stack trace ---------------------------------------------------------------
java.lang.UnsupportedOperationException: =
org.ofbiz.entity.condition.EntityComparisonOperator.compare(EntityComparisonOperator.java:131)
org.ofbiz.entity.condition.EntityComparisonOperator.mapMatches(EntityComparisonOperator.java:158)
org.ofbiz.entity.condition.EntityExpr.mapMatches(EntityExpr.java:167)
org.ofbiz.entity.condition.EntityJoinOperator.mapMatches(EntityJoinOperator.java:142)
org.ofbiz.entity.condition.EntityConditionListBase.mapMatches(EntityConditionListBase.java:99)
org.ofbiz.entity.cache.AbstractEntityConditionCache.storeHook(AbstractEntityConditionCache.java:200)
org.ofbiz.entity.cache.AbstractEntityConditionCache.storeHook(AbstractEntityConditionCache.java:166)
org.ofbiz.entity.cache.AbstractEntityConditionCache.storeHook(AbstractEntityConditionCache.java:141)
org.ofbiz.entity.cache.Cache.remove(Cache.java:114)
org.ofbiz.entity.GenericDelegator.clearCacheLine(GenericDelegator.java:2810)
org.ofbiz.entity.GenericDelegator.clearCacheLine(GenericDelegator.java:2795)
org.ofbiz.entity.GenericDelegator.store(GenericDelegator.java:1213)
org.ofbiz.minilang.method.entityops.StoreValue.exec(StoreValue.java:74)
org.ofbiz.minilang.SimpleMethod.runSubOps(SimpleMethod.java:921)
org.ofbiz.minilang.SimpleMethod.exec(SimpleMethod.java:749)
org.ofbiz.minilang.SimpleMethod.runSimpleMethod(SimpleMethod.java:149)
org.ofbiz.minilang.SimpleMethod.runSimpleService(SimpleMethod.java:131)
org.ofbiz.minilang.SimpleServiceEngine.serviceInvoker(SimpleServiceEngine.java:76)
org.ofbiz.minilang.SimpleServiceEngine.runSync(SimpleServiceEngine.java:51)
org.ofbiz.service.ServiceDispatcher.runSync(ServiceDispatcher.java:384)
org.ofbiz.service.ServiceDispatcher.runSync(ServiceDispatcher.java:213)
org.ofbiz.service.GenericDispatcher.runSync(GenericDispatcher.java:149)
org.ofbiz.webapp.event.ServiceEventHandler.invoke(ServiceEventHandler.java:332)
org.ofbiz.webapp.control.RequestHandler.runEvent(RequestHandler.java:444)
org.ofbiz.webapp.control.RequestHandler.doRequest(RequestHandler.java:276)
org.ofbiz.webapp.control.ControlServlet.doGet(ControlServlet.java:198)
org.ofbiz.webapp.control.ControlServlet.doPost(ControlServlet.java:78)
javax.servlet.http.HttpServlet.service(HttpServlet.java:710)
javax.servlet.http.HttpServlet.service(HttpServlet.java:803)
org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:290)
org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
org.ofbiz.webapp.control.ContextFilter.doFilter(ContextFilter.java:259)
org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:235)
org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:206)
org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:233)
org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:175)
org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:128)
org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:102)
org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:109)
org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:568)
org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:286)
org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:844)
org.apache.coyote.http11.Http11Protocol$Http11ConnectionHandler.process(Http11Protocol.java:583)
org.apache.tomcat.util.net.JIoEndpoint$Worker.run(JIoEndpoint.java:447)
java.lang.Thread.run(Thread.java:595)

I had a quick look, but did not find anything so far...

Jacques

From: "Jacques Le Roux" <ja...@les7arts.com>
> Thanks to Bilgin's help, I commited something in revision 688167.
> It works well but doint this I found an error which ends by
>    java.lang.UnsupportedOperationException: = (=)
> You can easily repreoduce this error on the demo server while using Party Profile, but it's a random error.
>
> Actually trying to reproduce it I got
> org.ofbiz.widget.screen.ScreenRenderException: Error rendering screen 
> [component://party/widget/partymgr/PartyScreens.xml#findparty]: java.lang.IllegalArgumentException: Error calling service with 
> name findParty:         org.ofbiz.service.GenericServiceException: Service [findParty] target threw an unexpected exception (=) 
> (Error calling service with name findParty: org.ofbiz.service.GenericServiceException: Service [findParty] target threw an 
> unexpected exception (=))
>
> Also trying to create a fin account I got
>    Error in simple-method operation []: java.lang.UnsupportedOperationException: = null
> But not always...
>
> No logs sorry... I suspect something related to cache...
>
> Jacques
>
> From: "Bilgin Ibryam" <bi...@iguanait.com>
>> On Fri, 2008-08-22 at 14:24 +0200, Jacques Le Roux wrote:
>>> I wrote this
>>>
>>> Index: framework/common/config/CommonUiLabels.xml
>>> ===================================================================
>>> --- framework/common/config/CommonUiLabels.xml (revision 687584)
>>> +++ framework/common/config/CommonUiLabels.xml (working copy)
>>> @@ -1705,6 +1705,10 @@
>>>          <value xml:lang="zh_CN">�用</value>
>>>          <value xml:lang="zh">æ— æ•ˆ</value>
>>>      </property>
>>> +    <property key="CommonDisplaying">
>>> +        <value xml:lang="en">Displaying ${lowCount} - ${highCount} of ${total}</value>
>>> +        <value xml:lang="fr">Affichage de ${lowCount} Ã  ${highCount} sur ${total}</value>
>>> +    </property>
>>>      <property key="CommonDistance">
>>>          <value xml:lang="ar">مساÙ?Ø©</value>
>>>          <value xml:lang="de">Distanz</value>
>>> Index: framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
>>> ===================================================================
>>> --- framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (revision 687675)
>>> +++ framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (working copy)
>>> @@ -340,8 +340,11 @@
>>>
>>>          }
>>>          if (listSize > 0) {
>>> -            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
>>> -            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + 
>>> " "
>>> + listSize + "</span> \n");
>>> +            @SuppressWarnings("unused") int lowCount = lowIndex + 1;
>>> +            @SuppressWarnings("unused") int highCount = lowIndex + actualPageSize;
>>> +            @SuppressWarnings("unused") int total = listSize;
>>> +            String commonDisplaying = UtilProperties.getMessage("CommonUiLabels", "CommonDisplaying", (Locale)
>>> context.get("locale"));
>>> +            writer.append(" <span class=\"tabletext\">" + commonDisplaying + "</span> \n");
>>>          }
>>>          if (highIndex < listSize) {
>>>              writer.append(" <a href=\"");
>>>
>>> But I'm still unsure how to test it. Bilgin spoke about party manager -> fin accounts but then you are in accouting and I guess 
>>> I
>>> miss some data he has.
>>>
>>> Jacques
>>
>>
>> Jacques,
>>
>> it is not very easy to see iterate section with page numbers.
>> you have to create few fin accounts for a party and then a screenlet with iterate section for fin accounts will appear in 
>> party -> profile screen.
>> But even then you can not see the page numbers. To see them you have to add the following attributes to iterate-section element: 
>> paginate="true" , view-size, paginate-target
>> (I will create a jira issue to change DEFAULT_PAGE_SIZE from 100 to 10, as it is in form pagination).
>>
>> Finally you have to modify your code as shown bellow.
>> Thanks for working on this.
>>
>>
>> Index: framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
>> ===================================================================
>> --- framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (revision 688079)
>> +++ framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (working copy)
>> @@ -33,6 +33,7 @@
>>
>> import org.ofbiz.base.util.Debug;
>> import org.ofbiz.base.util.GeneralException;
>> +import org.ofbiz.base.util.UtilMisc;
>> import org.ofbiz.base.util.UtilProperties;
>> import org.ofbiz.base.util.UtilValidate;
>> import org.ofbiz.base.util.UtilXml;
>> @@ -340,8 +341,9 @@
>>
>>         }
>>         if (listSize > 0) {
>> -            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
>> -            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + 
>> " " + listSize + "</span> \n");
>> +            Map messageMap = UtilMisc.toMap("lowCount", Integer.valueOf(lowIndex + 1), "highCount", Integer.valueOf(lowIndex + 
>> actualPageSize), "total", Integer.valueOf(listSize));
>> +            String commonDisplaying = UtilProperties.getMessage("CommonUiLabels", "CommonDisplaying", messageMap, (Locale) 
>> context.get("locale"));
>> +            writer.append(" <span class=\"tabletext\">" + commonDisplaying + "</span> \n");
>>         }
>>         if (highIndex < listSize) {
>>             writer.append(" <a href=\"");
>>
>>
>>
>>
> 


Re: svn commit: r687675 -/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Jacques Le Roux <ja...@les7arts.com>.
Thanks to Bilgin's help, I commited something in revision 688167.
It works well but doint this I found an error which ends by
    java.lang.UnsupportedOperationException: = (=)
You can easily repreoduce this error on the demo server while using Party Profile, but it's a random error.

Actually trying to reproduce it I got
org.ofbiz.widget.screen.ScreenRenderException: Error rendering screen 
[component://party/widget/partymgr/PartyScreens.xml#findparty]: java.lang.IllegalArgumentException: Error calling service with name 
findParty:         org.ofbiz.service.GenericServiceException: Service [findParty] target threw an unexpected exception (=) (Error 
calling service with name findParty: org.ofbiz.service.GenericServiceException: Service [findParty] target threw an unexpected 
exception (=))

Also trying to create a fin account I got
    Error in simple-method operation []: java.lang.UnsupportedOperationException: = null
But not always...

No logs sorry... I suspect something related to cache...

Jacques

From: "Bilgin Ibryam" <bi...@iguanait.com>
> On Fri, 2008-08-22 at 14:24 +0200, Jacques Le Roux wrote:
>> I wrote this
>>
>> Index: framework/common/config/CommonUiLabels.xml
>> ===================================================================
>> --- framework/common/config/CommonUiLabels.xml (revision 687584)
>> +++ framework/common/config/CommonUiLabels.xml (working copy)
>> @@ -1705,6 +1705,10 @@
>>          <value xml:lang="zh_CN">�用</value>
>>          <value xml:lang="zh">æ— æ•ˆ</value>
>>      </property>
>> +    <property key="CommonDisplaying">
>> +        <value xml:lang="en">Displaying ${lowCount} - ${highCount} of ${total}</value>
>> +        <value xml:lang="fr">Affichage de ${lowCount} Ã  ${highCount} sur ${total}</value>
>> +    </property>
>>      <property key="CommonDistance">
>>          <value xml:lang="ar">مساÙ?Ø©</value>
>>          <value xml:lang="de">Distanz</value>
>> Index: framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
>> ===================================================================
>> --- framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (revision 687675)
>> +++ framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (working copy)
>> @@ -340,8 +340,11 @@
>>
>>          }
>>          if (listSize > 0) {
>> -            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
>> -            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + 
>> " "
>> + listSize + "</span> \n");
>> +            @SuppressWarnings("unused") int lowCount = lowIndex + 1;
>> +            @SuppressWarnings("unused") int highCount = lowIndex + actualPageSize;
>> +            @SuppressWarnings("unused") int total = listSize;
>> +            String commonDisplaying = UtilProperties.getMessage("CommonUiLabels", "CommonDisplaying", (Locale)
>> context.get("locale"));
>> +            writer.append(" <span class=\"tabletext\">" + commonDisplaying + "</span> \n");
>>          }
>>          if (highIndex < listSize) {
>>              writer.append(" <a href=\"");
>>
>> But I'm still unsure how to test it. Bilgin spoke about party manager -> fin accounts but then you are in accouting and I guess I
>> miss some data he has.
>>
>> Jacques
>
>
> Jacques,
>
> it is not very easy to see iterate section with page numbers.
> you have to create few fin accounts for a party and then a screenlet with iterate section for fin accounts will appear in party -> 
> profile screen.
> But even then you can not see the page numbers. To see them you have to add the following attributes to iterate-section element: 
> paginate="true" , view-size, paginate-target
> (I will create a jira issue to change DEFAULT_PAGE_SIZE from 100 to 10, as it is in form pagination).
>
> Finally you have to modify your code as shown bellow.
> Thanks for working on this.
>
>
> Index: framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
> ===================================================================
> --- framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (revision 688079)
> +++ framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (working copy)
> @@ -33,6 +33,7 @@
>
> import org.ofbiz.base.util.Debug;
> import org.ofbiz.base.util.GeneralException;
> +import org.ofbiz.base.util.UtilMisc;
> import org.ofbiz.base.util.UtilProperties;
> import org.ofbiz.base.util.UtilValidate;
> import org.ofbiz.base.util.UtilXml;
> @@ -340,8 +341,9 @@
>
>         }
>         if (listSize > 0) {
> -            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
> -            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " 
> " + listSize + "</span> \n");
> +            Map messageMap = UtilMisc.toMap("lowCount", Integer.valueOf(lowIndex + 1), "highCount", Integer.valueOf(lowIndex + 
> actualPageSize), "total", Integer.valueOf(listSize));
> +            String commonDisplaying = UtilProperties.getMessage("CommonUiLabels", "CommonDisplaying", messageMap, (Locale) 
> context.get("locale"));
> +            writer.append(" <span class=\"tabletext\">" + commonDisplaying + "</span> \n");
>         }
>         if (highIndex < listSize) {
>             writer.append(" <a href=\"");
>
>
>
> 


Re: svn commit: r687675 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Bilgin Ibryam <bi...@iguanait.com>.
On Fri, 2008-08-22 at 14:24 +0200, Jacques Le Roux wrote: 
> I wrote this
> 
> Index: framework/common/config/CommonUiLabels.xml
> ===================================================================
> --- framework/common/config/CommonUiLabels.xml (revision 687584)
> +++ framework/common/config/CommonUiLabels.xml (working copy)
> @@ -1705,6 +1705,10 @@
>          <value xml:lang="zh_CN">�用</value>
>          <value xml:lang="zh">æ— æ•ˆ</value>
>      </property>
> +    <property key="CommonDisplaying">
> +        <value xml:lang="en">Displaying ${lowCount} - ${highCount} of ${total}</value>
> +        <value xml:lang="fr">Affichage de ${lowCount} Ã  ${highCount} sur ${total}</value>
> +    </property>
>      <property key="CommonDistance">
>          <value xml:lang="ar">مساÙ?Ø©</value>
>          <value xml:lang="de">Distanz</value>
> Index: framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
> ===================================================================
> --- framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (revision 687675)
> +++ framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (working copy)
> @@ -340,8 +340,11 @@
> 
>          }
>          if (listSize > 0) {
> -            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
> -            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " "
> + listSize + "</span> \n");
> +            @SuppressWarnings("unused") int lowCount = lowIndex + 1;
> +            @SuppressWarnings("unused") int highCount = lowIndex + actualPageSize;
> +            @SuppressWarnings("unused") int total = listSize;
> +            String commonDisplaying = UtilProperties.getMessage("CommonUiLabels", "CommonDisplaying", (Locale)
> context.get("locale"));
> +            writer.append(" <span class=\"tabletext\">" + commonDisplaying + "</span> \n");
>          }
>          if (highIndex < listSize) {
>              writer.append(" <a href=\"");
> 
> But I'm still unsure how to test it. Bilgin spoke about party manager -> fin accounts but then you are in accouting and I guess I
> miss some data he has.
> 
> Jacques


Jacques, 

it is not very easy to see iterate section with page numbers.
you have to create few fin accounts for a party and then a screenlet with iterate section for fin accounts will appear in party -> profile screen.
But even then you can not see the page numbers. To see them you have to add the following attributes to iterate-section element:  paginate="true" , view-size, paginate-target
(I will create a jira issue to change DEFAULT_PAGE_SIZE from 100 to 10, as it is in form pagination).

Finally you have to modify your code as shown bellow.
Thanks for working on this.


Index: framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
===================================================================
--- framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java	(revision 688079)
+++ framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java	(working copy)
@@ -33,6 +33,7 @@
 
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
+import org.ofbiz.base.util.UtilMisc;
 import org.ofbiz.base.util.UtilProperties;
 import org.ofbiz.base.util.UtilValidate;
 import org.ofbiz.base.util.UtilXml;
@@ -340,8 +341,9 @@
 
         }
         if (listSize > 0) {
-            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
-            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " " + listSize + "</span> \n");
+            Map messageMap = UtilMisc.toMap("lowCount", Integer.valueOf(lowIndex + 1), "highCount", Integer.valueOf(lowIndex + actualPageSize), "total", Integer.valueOf(listSize)); 
+            String commonDisplaying = UtilProperties.getMessage("CommonUiLabels", "CommonDisplaying", messageMap, (Locale) context.get("locale"));
+            writer.append(" <span class=\"tabletext\">" + commonDisplaying + "</span> \n");
         }
         if (highIndex < listSize) {
             writer.append(" <a href=\"");





Re: svn commit: r687675 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Jacques Le Roux <ja...@les7arts.com>.
I wrote this

Index: framework/common/config/CommonUiLabels.xml
===================================================================
--- framework/common/config/CommonUiLabels.xml (revision 687584)
+++ framework/common/config/CommonUiLabels.xml (working copy)
@@ -1705,6 +1705,10 @@
         <value xml:lang="zh_CN">�用</value>
         <value xml:lang="zh">æ— æ•ˆ</value>
     </property>
+    <property key="CommonDisplaying">
+        <value xml:lang="en">Displaying ${lowCount} - ${highCount} of ${total}</value>
+        <value xml:lang="fr">Affichage de ${lowCount} Ã  ${highCount} sur ${total}</value>
+    </property>
     <property key="CommonDistance">
         <value xml:lang="ar">مساÙ?Ø©</value>
         <value xml:lang="de">Distanz</value>
Index: framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java
===================================================================
--- framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (revision 687675)
+++ framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java (working copy)
@@ -340,8 +340,11 @@

         }
         if (listSize > 0) {
-            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
-            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " "
+ listSize + "</span> \n");
+            @SuppressWarnings("unused") int lowCount = lowIndex + 1;
+            @SuppressWarnings("unused") int highCount = lowIndex + actualPageSize;
+            @SuppressWarnings("unused") int total = listSize;
+            String commonDisplaying = UtilProperties.getMessage("CommonUiLabels", "CommonDisplaying", (Locale)
context.get("locale"));
+            writer.append(" <span class=\"tabletext\">" + commonDisplaying + "</span> \n");
         }
         if (highIndex < listSize) {
             writer.append(" <a href=\"");

But I'm still unsure how to test it. Bilgin spoke about party manager -> fin accounts but then you are in accouting and I guess I
miss some data he has.

Jacques

From: "Jacques Le Roux" <ja...@les7arts.com>
>I agree that only sentences should be translated. There are still plenty of that in OFBiz :/ It would  need a full review. On a
>related topic I also mentionned the use <fail-message> in OFBiz OOTB recently
>
> I will write a bad translation practices in Wiki ! Any suggestions is/will be welcome
> I will also rewrite this one as suggested...
>
> Jacques
>
> From: "Adrian Crum" <ad...@hlmksw.com>
>> We have discussed this in the past. There should be a UI label like
>>
>> Displaying ${lowCount} - ${highCount} of ${total}
>>
>> -Adrian
>>
>> Adam Heath wrote:
>>> jleroux@apache.org wrote:
>>>> +            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
>>>> +            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel
>>>> + " " + listSize + "</span> \n");
>>>>          }
>>>>          if (highIndex < listSize) {
>>>>              writer.append(" <a href=\"");
>>>
>>> This is the wrong way to translate something.  You don't translate a word, but a phrase.  Some languages have different ordering
>>> of primary phrase components.
>>>
>>> A - B of Y -> "%1 - %2 of %3", with args {lowIndex + 1, lowIndex + actualPageSize, listSize}.
>>>
>>> I'd do that, but I don't know the translation system in ofbiz.
>>>
>>> ps: I'd probably go so far as to say that all single-word translation entries currently in ofbiz are probably wrong.
>>>
>>
>


Re: svn commit: r687675 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Jacques Le Roux <ja...@les7arts.com>.
I agree that only sentences should be translated. There are still plenty of that in OFBiz :/ 
It would  need a full review. 
On a related topic I also mentionned the use <fail-message> in OFBiz OOTB recently

I will write a bad translation practices in Wiki ! Any suggestions is/will be welcome
I will also rewrite this one as suggested...

Jacques

From: "Adrian Crum" <ad...@hlmksw.com>
> We have discussed this in the past. There should be a UI label like
> 
> Displaying ${lowCount} - ${highCount} of ${total}
> 
> -Adrian
> 
> Adam Heath wrote:
>> jleroux@apache.org wrote:
>>> +            String ofLabel = 
>>> UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) 
>>> context.get("locale"));
>>> +            writer.append(" <span class=\"tabletext\">" + (lowIndex + 
>>> 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " " + 
>>> listSize + "</span> \n");
>>>          }
>>>          if (highIndex < listSize) {
>>>              writer.append(" <a href=\"");
>> 
>> This is the wrong way to translate something.  You don't translate a 
>> word, but a phrase.  Some languages have different ordering of primary 
>> phrase components.
>> 
>> A - B of Y -> "%1 - %2 of %3", with args {lowIndex + 1, lowIndex + 
>> actualPageSize, listSize}.
>> 
>> I'd do that, but I don't know the translation system in ofbiz.
>> 
>> ps: I'd probably go so far as to say that all single-word translation 
>> entries currently in ofbiz are probably wrong.
>> 
>

Re: svn commit: r687675 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Adrian Crum <ad...@hlmksw.com>.
We have discussed this in the past. There should be a UI label like

Displaying ${lowCount} - ${highCount} of ${total}

-Adrian

Adam Heath wrote:
> jleroux@apache.org wrote:
>> +            String ofLabel = 
>> UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) 
>> context.get("locale"));
>> +            writer.append(" <span class=\"tabletext\">" + (lowIndex + 
>> 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " " + 
>> listSize + "</span> \n");
>>          }
>>          if (highIndex < listSize) {
>>              writer.append(" <a href=\"");
> 
> This is the wrong way to translate something.  You don't translate a 
> word, but a phrase.  Some languages have different ordering of primary 
> phrase components.
> 
> A - B of Y -> "%1 - %2 of %3", with args {lowIndex + 1, lowIndex + 
> actualPageSize, listSize}.
> 
> I'd do that, but I don't know the translation system in ofbiz.
> 
> ps: I'd probably go so far as to say that all single-word translation 
> entries currently in ofbiz are probably wrong.
> 

Re: svn commit: r687675 - /ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/IterateSectionWidget.java

Posted by Adam Heath <do...@brainfood.com>.
jleroux@apache.org wrote:
> +            String ofLabel = UtilProperties.getMessage("CommonUiLabels", "CommonOf", (Locale) context.get("locale"));
> +            writer.append(" <span class=\"tabletext\">" + (lowIndex + 1) + " - " + (lowIndex + actualPageSize) + " "+ ofLabel + " " + listSize + "</span> \n");
>          }
>          if (highIndex < listSize) {
>              writer.append(" <a href=\"");

This is the wrong way to translate something.  You don't translate a 
word, but a phrase.  Some languages have different ordering of primary 
phrase components.

A - B of Y -> "%1 - %2 of %3", with args {lowIndex + 1, lowIndex + 
actualPageSize, listSize}.

I'd do that, but I don't know the translation system in ofbiz.

ps: I'd probably go so far as to say that all single-word translation 
entries currently in ofbiz are probably wrong.