Jacopo,
I agree that ownerPartyId should be required for createFacility.
Si
Begin forwarded message:
From: [EMAIL PROTECTED]
Date: August 4, 2006 1:47:53 AM PDT
To: [email protected]
Subject: svn commit: r428665 - in /incubator/ofbiz/trunk/
applications/product: script/org/ofbiz/product/inventory/
InventoryServices.xml servicedef/secas.xml
Reply-To: [email protected]
Author: jacopoc
Date: Fri Aug 4 01:47:52 2006
New Revision: 428665
URL: http://svn.apache.org/viewvc?rev=428665&view=rev
Log:
I have fixed two issues:
1) when an inventory item is updated using the
"updateInventoryItem" service, the output parameter
"oldOwnerPartyId" is mandatory.
So, if the ownerPartyId field is not set in the InventoryItem
(for example, it is not set in the OFBiz demo data) then an error
is returned. I've fixed it by retrieveng the owner from the facility.
Should we make the ownerPartyId a mandatory parameter for the
createFacility service? If you agree I'll commit a patch for this.
2) when an inventory transfer is completed, the service that
balances the inventory reservations must be called; it is now done
using a seca
Modified:
incubator/ofbiz/trunk/applications/product/script/org/ofbiz/
product/inventory/InventoryServices.xml
incubator/ofbiz/trunk/applications/product/servicedef/secas.xml
Modified: incubator/ofbiz/trunk/applications/product/script/org/
ofbiz/product/inventory/InventoryServices.xml
URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/
applications/product/script/org/ofbiz/product/inventory/
InventoryServices.xml?rev=428665&r1=428664&r2=428665&view=diff
======================================================================
========
--- incubator/ofbiz/trunk/applications/product/script/org/ofbiz/
product/inventory/InventoryServices.xml (original)
+++ incubator/ofbiz/trunk/applications/product/script/org/ofbiz/
product/inventory/InventoryServices.xml Fri Aug 4 01:47:52 2006
@@ -151,6 +151,10 @@
<make-value entity-name="InventoryItem" value-
name="lookupPKMap"/>
<set-pk-fields map-name="parameters" value-
name="lookupPKMap"/>
<find-by-primary-key map-name="lookupPKMap" value-
name="lookedUpValue"/>
+ <if-empty field-name="lookedUpValue.ownerPartyId">
+ <get-related-one value-name="lookedUpValue" relation-
name="Facility" to-value-name="oldFacility"/>
+ <set field="lookedUpValue.ownerPartyId" from-
field="oldFacility.ownerPartyId"/>
+ </if-empty>
<field-to-result field-name="lookedUpValue.ownerPartyId"
result-name="oldOwnerPartyId"/>
<set-nonpk-fields map-name="parameters" value-
name="lookedUpValue"/>
<store-value value-name="lookedUpValue"/>
Modified: incubator/ofbiz/trunk/applications/product/servicedef/
secas.xml
URL: http://svn.apache.org/viewvc/incubator/ofbiz/trunk/
applications/product/servicedef/secas.xml?
rev=428665&r1=428664&r2=428665&view=diff
======================================================================
========
--- incubator/ofbiz/trunk/applications/product/servicedef/secas.xml
(original)
+++ incubator/ofbiz/trunk/applications/product/servicedef/secas.xml
Fri Aug 4 01:47:52 2006
@@ -26,6 +26,7 @@
<eca service="createInventoryTransfer" event="commit">
<condition field-name="statusId" operator="equals"
value="IXF_COMPLETE"/>
<action service="completeInventoryTransfer" mode="sync"/>
+ <action service="balanceInventoryItems" mode="sync"/>
</eca>
<eca service="updateInventoryTransfer" event="invoke">
@@ -35,6 +36,7 @@
<eca service="updateInventoryTransfer" event="commit">
<condition field-name="statusId" operator="equals"
value="IXF_COMPLETE"/>
<action service="completeInventoryTransfer" mode="sync"/>
+ <action service="balanceInventoryItems" mode="sync"/>
</eca>
<!-- inventory variance -->
