OFBiz › OFBiz - Dev

Re: svn commit: r441292 - Si Chen - /applications/product/widget/facility/FacilityScreens.xml

_‹ Previous Topic Next Topic _›

Classic

List

Threaded

4 messages Options

Ray Barlow

Oct 23, 2006; 12:01pm

Re: svn commit: r441292 - Si Chen - /applications/product/widget/facility/FacilityScreens.xml

202 posts

Hi Si,

Any reason in this commit you commented out the picklistActiveList
condition? It was committed a bit over a month ago but I've just hit an
error on the PicklistManage.ftl page where line 113 is trying to use the
picklistActiveList value and crashing out. I wasn't sure if you meant to
commit the  tags as you have also added the PACKED status
condition and there is no comment about why you've commented it out,
just a bit confusing really.

I've not really dug deep on this one but I think it occurs if you have
an assigned/picked list and then also add a new one in the input status.

Thanks,

Ray

[hidden email] wrote:

> Author: sichen
> Date: Thu Sep 7 16:07:19 2006
> New Revision: 441292
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=441292
> Log:
> Fixed bug where accepted shipment route segments would show up on list of shipments to be scheduled
>
> Modified:
> incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
>
> Modified: incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
> URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml?view=diff&rev=441292&r1=441291&r2=441292
> ==============================================================================
> --- incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml (original)
> +++ incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml Thu Sep 7 16:07:19 2006
> @@ -546,6 +546,7 @@
> <condition-list combine="and">
> <condition-expr field-name="facilityId" operator="equals" env-name="parameters.facilityId"/>
> <condition-expr field-name="statusId" operator="not-equals" value="PICKLIST_PICKED"/>
> + <condition-expr field-name="statusId" operator="not-equals" value="PICKLIST_PACKED"/>
> <condition-expr field-name="statusId" operator="not-equals" value="PICKLIST_CANCELLED"/>
> </condition-list>
> <order-by field-name="picklistDate"/>
> @@ -577,14 +578,17 @@
> <entity-condition entity-name="PartyRoleAndPartyDetail" list-name="partyRoleAndPartyDetailList" use-cache="true">
> <condition-expr field-name="roleTypeId" value="PICKER"/>
> </entity-condition>
> + 
> </actions>
> <widgets>
> <decorator-screen name="CommonFacilityDecorator">
> @@ -720,6 +724,7 @@
> <condition-expr field-name="originFacilityId" operator="equals" env-name="parameters.facilityId"/>
> <condition-expr field-name="statusId" operator="equals" value="SHIPMENT_PACKED"/>
> <condition-expr field-name="carrierServiceStatusId" operator="not-equals" value="SHRSCS_CONFIRMED"/>
> + <condition-expr field-name="carrierServiceStatusId" operator="not-equals" value="SHRSCS_ACCEPTED"/>
> </condition-list>
> <order-by field-name="shipmentId DESC"/>
> </entity-condition>
>
>
>
>

... [show rest of quote]

Si Chen-2

Oct 23, 2006; 7:23pm

Re: svn commit: r441292 - Si Chen - /applications/product/widget/facility/FacilityScreens.xml

1414 posts

I probably made a mistake with that. Would it work if you remove the
?

On Oct 23, 2006, at 4:01 AM, Ray Barlow wrote:

> Hi Si,
>
> Any reason in this commit you commented out the picklistActiveList
> condition? It was committed a bit over a month ago but I've just
> hit an error on the PicklistManage.ftl page where line 113 is
> trying to use the picklistActiveList value and crashing out. I
> wasn't sure if you meant to commit the  tags as you have
> also added the PACKED status condition and there is no comment
> about why you've commented it out, just a bit confusing really.
>
> I've not really dug deep on this one but I think it occurs if you
> have an assigned/picked list and then also add a new one in the
> input status.
>
> Thanks,
>
> Ray
>
>
> [hidden email] wrote:
>> Author: sichen
>> Date: Thu Sep 7 16:07:19 2006
>> New Revision: 441292
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=441292
>> Log:
>> Fixed bug where accepted shipment route segments would show up on
>> list of shipments to be scheduled
>>
>> Modified:
>> incubator/ofbiz/trunk/applications/product/widget/facility/
>> FacilityScreens.xml
>>
>> Modified: incubator/ofbiz/trunk/applications/product/widget/
>> facility/FacilityScreens.xml
>> URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/
>> applications/product/widget/facility/FacilityScreens.xml?
>> view=diff&rev=441292&r1=441291&r2=441292
>> =====================================================================
>> =========
>> --- incubator/ofbiz/trunk/applications/product/widget/facility/
>> FacilityScreens.xml (original)
>> +++ incubator/ofbiz/trunk/applications/product/widget/facility/
>> FacilityScreens.xml Thu Sep 7 16:07:19 2006
>> @@ -546,6 +546,7 @@
>> <condition-list combine="and">
>> <condition-expr field-name="facilityId"
>> operator="equals" env-name="parameters.facilityId"/>
>> <condition-expr field-name="statusId"
>> operator="not-equals" value="PICKLIST_PICKED"/>
>> + <condition-expr field-name="statusId"
>> operator="not-equals" value="PICKLIST_PACKED"/>
>> <condition-expr field-name="statusId"
>> operator="not-equals" value="PICKLIST_CANCELLED"/>
>> </condition-list>
>> <order-by field-name="picklistDate"/>
>> @@ -577,14 +578,17 @@
>> <entity-condition entity-
>> name="PartyRoleAndPartyDetail" list-
>> name="partyRoleAndPartyDetailList" use-cache="true">
>> <condition-expr field-name="roleTypeId"
>> value="PICKER"/>
>> </entity-condition>
>> + 
>> </actions>
>> <widgets>
>> <decorator-screen name="CommonFacilityDecorator">
>> @@ -720,6 +724,7 @@
>> <condition-expr field-
>> name="originFacilityId" operator="equals" env-
>> name="parameters.facilityId"/>
>> <condition-expr field-name="statusId"
>> operator="equals" value="SHIPMENT_PACKED"/>
>> <condition-expr field-
>> name="carrierServiceStatusId" operator="not-equals"
>> value="SHRSCS_CONFIRMED"/>
>> + <condition-expr field-
>> name="carrierServiceStatusId" operator="not-equals"
>> value="SHRSCS_ACCEPTED"/>
>> </condition-list>
>> <order-by field-name="shipmentId DESC"/>
>> </entity-condition>
>>
>>
>>
>>

... [show rest of quote]

Best Regards,

Si
[hidden email]

Ray Barlow

Oct 23, 2006; 11:21pm

Re: svn commit: r441292 - Si Chen - /applications/product/widget/facility/FacilityScreens.xml

202 posts

I'll check tomorrow. For my immediate fix I went the
safest route and just put an if_exists clause in the
ftl file as I didn't really need the feature, just
needed to get the pick list displaying.

I'll let you know,

Ray

--- Si Chen <[hidden email]> wrote:

> I probably made a mistake with that. Would it work
> if you remove the
> ?
>
> On Oct 23, 2006, at 4:01 AM, Ray Barlow wrote:
>
> > Hi Si,
> >
> > Any reason in this commit you commented out the
> picklistActiveList
> > condition? It was committed a bit over a month ago
> but I've just
> > hit an error on the PicklistManage.ftl page where
> line 113 is
> > trying to use the picklistActiveList value and
> crashing out. I
> > wasn't sure if you meant to commit the 
> tags as you have
> > also added the PACKED status condition and there
> is no comment
> > about why you've commented it out, just a bit
> confusing really.
> >
> > I've not really dug deep on this one but I think
> it occurs if you
> > have an assigned/picked list and then also add a
> new one in the
> > input status.
> >
> > Thanks,
> >
> > Ray

... [show rest of quote]

Ray Barlow

Oct 24, 2006; 11:23am

Re: svn commit: r441292 - Si Chen - /applications/product/widget/facility/FacilityScreens.xml

202 posts

In reply to this post by Si Chen-2

Si,

Yes removing the comments works fine, no adverse side effects and the
menus are populated OK.

Adding the if_exists:
<#list picklistActiveList?if_exists as
picklistActive>
to the PicklistManage.ftl file is probably worth doing as well for
protection against killing the page.

Sorry can't do a patch at the moment don't have access to the code or
tools at the moment, not much to change anyway but if you want one let
me know.

Ray

Si Chen wrote:

> I probably made a mistake with that. Would it work if you remove the
> ?
>
> On Oct 23, 2006, at 4:01 AM, Ray Barlow wrote:
>
>> Hi Si,
>>
>> Any reason in this commit you commented out the picklistActiveList
>> condition? It was committed a bit over a month ago but I've just hit
>> an error on the PicklistManage.ftl page where line 113 is trying to
>> use the picklistActiveList value and crashing out. I wasn't sure if
>> you meant to commit the  tags as you have also added the
>> PACKED status condition and there is no comment about why you've
>> commented it out, just a bit confusing really.
>>
>> I've not really dug deep on this one but I think it occurs if you
>> have an assigned/picked list and then also add a new one in the input
>> status.
>>
>> Thanks,
>>
>> Ray
>>
>>
>> [hidden email] wrote:
>>> Author: sichen
>>> Date: Thu Sep 7 16:07:19 2006
>>> New Revision: 441292
>>>
>>> URL: http://svn.apache.org/viewvc?view=rev&rev=441292
>>> Log:
>>> Fixed bug where accepted shipment route segments would show up on
>>> list of shipments to be scheduled
>>>
>>> Modified:
>>>
>>> incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
>>>
>>>
>>> Modified:
>>> incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
>>>
>>> URL:
>>> http://svn.apache.org/viewvc/incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml?view=diff&rev=441292&r1=441291&r2=441292
>>>
>>> ==============================================================================
>>>
>>> ---
>>> incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
>>> (original)
>>> +++
>>> incubator/ofbiz/trunk/applications/product/widget/facility/FacilityScreens.xml
>>> Thu Sep 7 16:07:19 2006
>>> @@ -546,6 +546,7 @@
>>> <condition-list combine="and">
>>> <condition-expr field-name="facilityId"
>>> operator="equals" env-name="parameters.facilityId"/>
>>> <condition-expr field-name="statusId"
>>> operator="not-equals" value="PICKLIST_PICKED"/>
>>> + <condition-expr field-name="statusId"
>>> operator="not-equals" value="PICKLIST_PACKED"/>
>>> <condition-expr field-name="statusId"
>>> operator="not-equals" value="PICKLIST_CANCELLED"/>
>>> </condition-list>
>>> <order-by field-name="picklistDate"/>
>>> @@ -577,14 +578,17 @@
>>> <entity-condition
>>> entity-name="PartyRoleAndPartyDetail"
>>> list-name="partyRoleAndPartyDetailList" use-cache="true">
>>> <condition-expr field-name="roleTypeId"
>>> value="PICKER"/>
>>> </entity-condition>
>>> + 
>>> </actions>
>>> <widgets>
>>> <decorator-screen name="CommonFacilityDecorator">
>>> @@ -720,6 +724,7 @@
>>> <condition-expr
>>> field-name="originFacilityId" operator="equals"
>>> env-name="parameters.facilityId"/>
>>> <condition-expr field-name="statusId"
>>> operator="equals" value="SHIPMENT_PACKED"/>
>>> <condition-expr
>>> field-name="carrierServiceStatusId" operator="not-equals"
>>> value="SHRSCS_CONFIRMED"/>
>>> + <condition-expr
>>> field-name="carrierServiceStatusId" operator="not-equals"
>>> value="SHRSCS_ACCEPTED"/>
>>> </condition-list>
>>> <order-by field-name="shipmentId DESC"/>
>>> </entity-condition>
>>>
>>>
>>>
>>>
>
> Best Regards,
>
> Si
> [hidden email]
>
>
>
>

... [show rest of quote]