You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Andrew Zeneski <an...@hotwaxmedia.com> on 2007/05/21 22:00:56 UTC
Re: svn commit: r540162 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderListState.java
Jacques,
The hasFilter() method you removed here is being used in the
orderlist BSH file. Now, that this method is gone, the order list
screen blows up. Can you look at this?
Andrew
On May 21, 2007, at 11:05 AM, jleroux@apache.org wrote:
> Author: jleroux
> Date: Mon May 21 08:05:42 2007
> New Revision: 540162
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=540162
> Log:
> The orderFiltersStateId field does not exist in the entity
> OrderHeader, hence this lines are at least unuseful.
> I guess it was my mistake adding this lines when reviewing Leon's
> patch
>
> Modified:
> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> OrderListState.java
>
> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> OrderListState.java
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
> src/org/ofbiz/order/order/OrderListState.java?
> view=diff&rev=540162&r1=540161&r2=540162
> ======================================================================
> ========
> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> OrderListState.java (original)
> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> OrderListState.java Mon May 21 08:05:42 2007
> @@ -190,7 +190,6 @@
>
> public boolean hasStatus(String param) { return ("Y".equals
> (orderStatusState.get(param))); }
> public boolean hasType(String param) { return ("Y".equals
> (orderTypeState.get(param))); }
> - public boolean hasFilter(String param) { return ("Y".equals
> (orderFilterState.get(param))); }
>
> public boolean hasAllStatus() {
> for (Iterator iter = orderStatusState.values().iterator();
> iter.hasNext(); ) {
> @@ -229,11 +228,6 @@
> typeConditions.add( new EntityExpr("orderTypeId",
> EntityOperator.EQUALS, parameterToOrderTypeId.get(type)) );
> }
> List filterConditions = new ArrayList();
> - for (Iterator iter = orderFilterState.keySet().iterator();
> iter.hasNext(); ) {
> - String type = (String) iter.next();
> - if (!hasType(type)) continue;
> - filterConditions.add( new EntityExpr
> ("orderFiltersStateId", EntityOperator.EQUALS,
> parameterToFilterId.get(type)) );
> - }
>
> EntityCondition statusConditionsList = new
> EntityConditionList(statusConditions, EntityOperator.OR);
> EntityCondition typeConditionsList = new
> EntityConditionList(typeConditions, EntityOperator.OR);
> @@ -241,9 +235,6 @@
> if ((typeConditions.size() > 0) && (statusConditions.size
> () > 0)) {
> allConditions.add(statusConditionsList);
> allConditions.add(typeConditionsList);
> - }
> - if (filterConditions.size() > 0) {
> - allConditions.add(filterConditionsList);
> }
>
> EntityCondition queryConditionsList = new
> EntityConditionList(allConditions, EntityOperator.AND);
>
>
Re: svn commit: r540162 - /ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderListState.java
Posted by Andrew Zeneski <an...@hotwaxmedia.com>.
Works great, thanks!
On May 21, 2007, at 4:43 PM, Jacques Le Roux wrote:
> Andy,
>
> Sorry, corrected in trunk, doing it in release4.0.
>
> Thanks for the alert.
>
> Jacques
>
> De : "Andrew Zeneski" <an...@hotwaxmedia.com>
>> Jacques,
>>
>> The hasFilter() method you removed here is being used in the
>> orderlist BSH file. Now, that this method is gone, the order list
>> screen blows up. Can you look at this?
>>
>> Andrew
>>
>> On May 21, 2007, at 11:05 AM, jleroux@apache.org wrote:
>>
>>> Author: jleroux
>>> Date: Mon May 21 08:05:42 2007
>>> New Revision: 540162
>>>
>>> URL: http://svn.apache.org/viewvc?view=rev&rev=540162
>>> Log:
>>> The orderFiltersStateId field does not exist in the entity
>>> OrderHeader, hence this lines are at least unuseful.
>>> I guess it was my mistake adding this lines when reviewing Leon's
>>> patch
>>>
>>> Modified:
>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>> OrderListState.java
>>>
>>> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>> OrderListState.java
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>> src/org/ofbiz/order/order/OrderListState.java?
>>> view=diff&rev=540162&r1=540161&r2=540162
>>>
> ======================================================================
>>> ========
>>> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>> OrderListState.java (original)
>>> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>> OrderListState.java Mon May 21 08:05:42 2007
>>> @@ -190,7 +190,6 @@
>>>
>>> public boolean hasStatus(String param) { return ("Y".equals
>>> (orderStatusState.get(param))); }
>>> public boolean hasType(String param) { return ("Y".equals
>>> (orderTypeState.get(param))); }
>>> - public boolean hasFilter(String param) { return ("Y".equals
>>> (orderFilterState.get(param))); }
>>>
>>> public boolean hasAllStatus() {
>>> for (Iterator iter = orderStatusState.values().iterator();
>>> iter.hasNext(); ) {
>>> @@ -229,11 +228,6 @@
>>> typeConditions.add( new EntityExpr("orderTypeId",
>>> EntityOperator.EQUALS, parameterToOrderTypeId.get(type)) );
>>> }
>>> List filterConditions = new ArrayList();
>>> - for (Iterator iter = orderFilterState.keySet().iterator();
>>> iter.hasNext(); ) {
>>> - String type = (String) iter.next();
>>> - if (!hasType(type)) continue;
>>> - filterConditions.add( new EntityExpr
>>> ("orderFiltersStateId", EntityOperator.EQUALS,
>>> parameterToFilterId.get(type)) );
>>> - }
>>>
>>> EntityCondition statusConditionsList = new
>>> EntityConditionList(statusConditions, EntityOperator.OR);
>>> EntityCondition typeConditionsList = new
>>> EntityConditionList(typeConditions, EntityOperator.OR);
>>> @@ -241,9 +235,6 @@
>>> if ((typeConditions.size() > 0) && (statusConditions.size
>>> () > 0)) {
>>> allConditions.add(statusConditionsList);
>>> allConditions.add(typeConditionsList);
>>> - }
>>> - if (filterConditions.size() > 0) {
>>> - allConditions.add(filterConditionsList);
>>> }
>>>
>>> EntityCondition queryConditionsList = new
>>> EntityConditionList(allConditions, EntityOperator.AND);
>>>
>>>
>>
>>
>
Re: svn commit: r540162 -
/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderListState.java
Posted by Jacques Le Roux <ja...@les7arts.com>.
Andy,
Sorry, corrected in trunk, doing it in release4.0.
Thanks for the alert.
Jacques
De : "Andrew Zeneski" <an...@hotwaxmedia.com>
> Jacques,
>
> The hasFilter() method you removed here is being used in the
> orderlist BSH file. Now, that this method is gone, the order list
> screen blows up. Can you look at this?
>
> Andrew
>
> On May 21, 2007, at 11:05 AM, jleroux@apache.org wrote:
>
> > Author: jleroux
> > Date: Mon May 21 08:05:42 2007
> > New Revision: 540162
> >
> > URL: http://svn.apache.org/viewvc?view=rev&rev=540162
> > Log:
> > The orderFiltersStateId field does not exist in the entity
> > OrderHeader, hence this lines are at least unuseful.
> > I guess it was my mistake adding this lines when reviewing Leon's
> > patch
> >
> > Modified:
> > ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> > OrderListState.java
> >
> > Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> > OrderListState.java
> > URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
> > src/org/ofbiz/order/order/OrderListState.java?
> > view=diff&rev=540162&r1=540161&r2=540162
> >
======================================================================
> > ========
> > --- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> > OrderListState.java (original)
> > +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> > OrderListState.java Mon May 21 08:05:42 2007
> > @@ -190,7 +190,6 @@
> >
> > public boolean hasStatus(String param) { return ("Y".equals
> > (orderStatusState.get(param))); }
> > public boolean hasType(String param) { return ("Y".equals
> > (orderTypeState.get(param))); }
> > - public boolean hasFilter(String param) { return ("Y".equals
> > (orderFilterState.get(param))); }
> >
> > public boolean hasAllStatus() {
> > for (Iterator iter = orderStatusState.values().iterator();
> > iter.hasNext(); ) {
> > @@ -229,11 +228,6 @@
> > typeConditions.add( new EntityExpr("orderTypeId",
> > EntityOperator.EQUALS, parameterToOrderTypeId.get(type)) );
> > }
> > List filterConditions = new ArrayList();
> > - for (Iterator iter = orderFilterState.keySet().iterator();
> > iter.hasNext(); ) {
> > - String type = (String) iter.next();
> > - if (!hasType(type)) continue;
> > - filterConditions.add( new EntityExpr
> > ("orderFiltersStateId", EntityOperator.EQUALS,
> > parameterToFilterId.get(type)) );
> > - }
> >
> > EntityCondition statusConditionsList = new
> > EntityConditionList(statusConditions, EntityOperator.OR);
> > EntityCondition typeConditionsList = new
> > EntityConditionList(typeConditions, EntityOperator.OR);
> > @@ -241,9 +235,6 @@
> > if ((typeConditions.size() > 0) && (statusConditions.size
> > () > 0)) {
> > allConditions.add(statusConditionsList);
> > allConditions.add(typeConditionsList);
> > - }
> > - if (filterConditions.size() > 0) {
> > - allConditions.add(filterConditionsList);
> > }
> >
> > EntityCondition queryConditionsList = new
> > EntityConditionList(allConditions, EntityOperator.AND);
> >
> >
>
>