You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by mo...@apache.org on 2009/03/19 14:58:09 UTC

svn commit: r756001 - in /ofbiz/trunk: applications/order/config/ applications/order/entitydef/ applications/order/servicedef/ applications/order/webapp/ordermgr/WEB-INF/ applications/order/webapp/ordermgr/order/ applications/product/script/org/ofbiz/p...

Author: mor
Date: Thu Mar 19 13:58:08 2009
New Revision: 756001

URL: http://svn.apache.org/viewvc?rev=756001&view=rev
Log:
Another new feature from Pranay and Co.
Now Order reservation can be changed manually by setting priority (new field priority on OrderHeader and OrderItemShipGrpInvRes) for the order. This way few orders can be
fulfilled while waiting for sufficient inventory to fulfill large orders.

Applied patch from OFBIZ-2224 (https://issues.apache.org/jira/browse/OFBIZ-2224).
Thanks Pratik Khera and Pranay Pandey.

i18n Note: This commit has three new labels (IMO, they can be moved to common component with names something like CommonHigh, CommonLow etc.)

Pratik, I have done minor changes in the service and they are
1) Used <if-empty field=""> instead of <if> <condition> <if-empty> as this makes the logic more clearer.
2) Removed <condition-list> because their is only one expression in the <entity-condition>
3) Added <clear-field> in the iteration, to make sure that always a new value will be set in the map that is passed to a service and <store-value>  


Modified:
    ofbiz/trunk/applications/order/config/OrderUiLabels.xml
    ofbiz/trunk/applications/order/entitydef/entitymodel.xml
    ofbiz/trunk/applications/order/servicedef/secas.xml
    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml
    ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
    ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml
    ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml
    ofbiz/trunk/applications/product/servicedef/services_facility.xml
    ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml

Modified: ofbiz/trunk/applications/order/config/OrderUiLabels.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderUiLabels.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/config/OrderUiLabels.xml (original)
+++ ofbiz/trunk/applications/order/config/OrderUiLabels.xml Thu Mar 19 13:58:08 2009
@@ -6271,6 +6271,15 @@
         <value xml:lang="zh">公开便笺 (已打印)</value>
         <value xml:lang="zh_CN">公开注释 (可打印)</value>
     </property>
+    <property key="OrderPriorityHigh">
+        <value xml:lang="en">High</value>
+    </property>
+    <property key="OrderPriorityLow">
+        <value xml:lang="en">Low</value>
+    </property>
+    <property key="OrderPriorityNormal">
+        <value xml:lang="en">Normal</value>
+    </property>    
     <property key="OrderProcessOrder">
         <value xml:lang="de">Auftrag verarbeiten</value>
         <value xml:lang="en">Process Order</value>

Modified: ofbiz/trunk/applications/order/entitydef/entitymodel.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/entitydef/entitymodel.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/entitydef/entitymodel.xml (original)
+++ ofbiz/trunk/applications/order/entitydef/entitymodel.xml Thu Mar 19 13:58:08 2009
@@ -337,6 +337,7 @@
       <field name="externalId" type="id"></field>
       <field name="salesChannelEnumId" type="id"></field>
       <field name="orderDate" type="date-time"></field>
+      <field name="priority" type="indicator"><description>Sets priority for Inventory Reservation</description></field>
       <field name="entryDate" type="date-time"></field>
       <field name="visitId" type="id"></field>
       <field name="statusId" type="id"></field>
@@ -888,6 +889,7 @@
       <field name="createdDatetime" type="date-time"></field>
       <field name="promisedDatetime" type="date-time"></field>
       <field name="currentPromisedDate" type="date-time"></field>
+      <field name="priority" type="indicator"><description>Sets priority for Inventory Reservation</description></field>
       <field name="sequenceId" type="numeric"></field> 
       <field name="oldPickStartDate" col-name="PICK_START_DATE" type="date-time"></field> <!-- this is no longer used and is here for migration or whatever if needed -->
       <prim-key field="orderId"/>

Modified: ofbiz/trunk/applications/order/servicedef/secas.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/secas.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/servicedef/secas.xml (original)
+++ ofbiz/trunk/applications/order/servicedef/secas.xml Thu Mar 19 13:58:08 2009
@@ -36,6 +36,9 @@
         <condition field-name="orderTypeId" operator="equals" value="PURCHASE_ORDER"/>
         <action service="setUnitPriceAsLastPrice" mode="sync"/>
     </eca>
+    <eca service="storeOrder" event="return">
+        <action service="setOrderReservationPriority" mode="sync"/>
+    </eca>
     <eca service="updateOrderItems" event="commit">
         <condition field-name="orderTypeId" operator="equals" value="PURCHASE_ORDER"/>
         <action service="setUnitPriceAsLastPrice" mode="sync"/>

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml Thu Mar 19 13:58:08 2009
@@ -1457,7 +1457,12 @@
         <response name="success" type="request-redirect" value="orderview"/>
         <response name="error" type="request-redirect" value="orderview"/>
     </request-map>
-    
+    <request-map uri="setOrderReservationPriority">
+        <security https="true" auth="true"/>
+        <event type="service" invoke="setOrderReservationPriority"/>
+        <response name="success" type="view" value="orderview"/>
+        <response name="error" type="view" value="orderview"/>
+    </request-map>
     <!-- Lookup request mappings -->
     <request-map uri="LookupPerson"><security https="true" auth="true"/><response name="success" type="view" value="LookupPerson"/></request-map>
     <request-map uri="LookupPartyGroup"><security https="true" auth="true"/><response name="success" type="view" value="LookupPartyGroup"/></request-map>

Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl (original)
+++ ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl Thu Mar 19 13:58:08 2009
@@ -179,6 +179,24 @@
               </td>
             </tr>
             </#if>
+            <#if "SALES_ORDER" == orderHeader.orderTypeId>
+              <form action="setOrderReservationPriority" method="post" name="setOrderReservationPriority">
+                <input type = "hidden" name="orderId" value="${orderId}"/> 
+                <tr><td colspan="3"><hr/></td></tr>
+                <tr>
+                  <td align="right" valign="top" width="15%" class="label">&nbsp;${uiLabelMap.FormFieldTitle_priority}</td>
+                  <td width="5%">&nbsp;</td>
+                  <td valign="top" width="80%">
+                    <select name="priority">
+                      <option value="1">${uiLabelMap.OrderPriorityHigh}</option>
+                      <option value="2"selected>${uiLabelMap.OrderPriorityNormal}</option> 
+                      <option value="3">${uiLabelMap.OrderPriorityLow}</option>
+                    </select>
+                    <input type="submit" class="smallSubmit" value="${uiLabelMap.FormFieldTitle_reserveInventory}"/>
+                  </td>
+                </tr>
+              </form>
+            </#if>
         </table>
     </div>
 </div>
\ No newline at end of file

Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml (original)
+++ ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml Thu Mar 19 13:58:08 2009
@@ -201,6 +201,7 @@
                             <set from-field="parameters.reservedDatetime" field="reserveOisgirMap.reservedDatetime"/>
                             <set from-field="promisedDatetime" field="reserveOisgirMap.promisedDatetime"/>
                             <set from-field="parameters.sequenceId" field="reserveOisgirMap.sequenceId"/>
+                            <set from-field="parameters.priority" field="reserveOisgirMap.priority"/>
                             <call-service service-name="reserveOrderItemInventory" in-map-name="reserveOisgirMap"/>
                             <clear-field field="reserveOisgirMap"/>
                         <else>
@@ -444,6 +445,7 @@
                         <set from-field="parameters.reservedDatetime" field="reserveOisgirMap.reservedDatetime"/>
                         <set from-field="parameters.deductAmount" field="reserveOisgirMap.quantity" type="BigDecimal"/>
                         <set from-field="promisedDatetime" field="reserveOisgirMap.promisedDatetime"/>
+                        <set from-field="parameters.priority" field="reserveOisgirMap.priority"/>
                         <if-not-empty field="parameters.sequenceId">
                             <set field="reserveOisgirMap.sequenceId" from-field="parameters.sequenceId"/>
                         </if-not-empty>
@@ -498,6 +500,7 @@
             <set-nonpk-fields map="parameters" value-field="newOisgirEntity"/>
             <now-timestamp field="nowTimestamp"/>
             <set from-field="nowTimestamp" field="newOisgirEntity.createdDatetime"/>
+            <set from-field="parameters.priority" field="newOisgirEntity.priority"/>
             <if-empty field="newOisgirEntity.reservedDatetime">
                 <set from-field="nowTimestamp" field="newOisgirEntity.reservedDatetime"/>
             </if-empty>

Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml (original)
+++ ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml Thu Mar 19 13:58:08 2009
@@ -814,6 +814,7 @@
                     </condition-list>
                 </condition-list>
             </condition-list>
+            <order-by field-name="priority"/>
             <order-by field-name="currentPromisedDate"/>
             <order-by field-name="reservedDatetime"/>
             <order-by field-name="sequenceId"/>
@@ -875,6 +876,9 @@
                     <log level="verbose" message="Adding ${oisgir.orderId} to touchedOrderIdMap"/>
                 </if-compare>
             </if-not-empty>
+            <entity-one entity-name="OrderHeader" value-field="orderHeader">
+                <field-map field-name="orderId" from-field="oisgir.orderId"/>
+            </entity-one>            
             
             <!-- require inventory is N because it had to be N to begin with to have a negative ATP -->
             <clear-field field="resMap"/>
@@ -888,6 +892,7 @@
             <set field="resMap.shipGroupSeqId" from-field="oisgir.shipGroupSeqId"/>
             <set field="resMap.sequenceId" from-field="oisgir.sequenceId"/>
             <set field="resMap.facilityId" from-field="parameters.facilityId"/>
+            <set field="resMap.priority" from-field="orderHeader.priority"/>
             <log level="info" message="Re-reserving product [${resMap.productId}] for order item [${resMap.orderId}:${resMap.orderItemSeqId}] quantity [${resMap.quantity}]; facility [${parameters.facilityId}]"/>
             <call-service service-name="reserveProductInventoryByFacility" in-map-name="resMap"/>
         </iterate>
@@ -1249,5 +1254,43 @@
             </if-compare>
         </if-not-empty>
     </simple-method>    
-    
-</simple-methods>
+    <simple-method method-name="setOrderReservationPriority" short-description="Sets priority of an order for Inventory Reservation, orders with HIGH priority would be served first.">
+        <set field="orderId" from-field="parameters.orderId"/>       
+        <entity-one entity-name="OrderHeader" value-field="orderHeader">
+            <field-map field-name="orderId" from-field="orderId"/>
+        </entity-one>
+        <set field="priority" from-field="parameters.priority"/>
+        <if-empty field="priority">
+            <entity-and list="oisgirs" entity-name="OrderItemShipGrpInvRes">
+                <field-map field-name="orderId" from-field="orderId"/>
+            </entity-and>
+            <iterate entry="oisgir" list="oisgirs">
+                <set field="oisgir.priority" default-value="2"/>
+                <store-value value-field="oisgir"/>       
+            </iterate> 
+            <set field="orderHeader.priority" default-value="2"/>
+            <store-value value-field="orderHeader"/>        
+        <else>
+            <set field="orderHeader.priority" from-field="priority"/>
+            <store-value value-field="orderHeader"/>
+            <entity-and list="oisgirs" entity-name="OrderItemShipGrpInvRes">
+                <field-map field-name="orderId" from-field="orderId"/>
+            </entity-and> 
+            <iterate entry="oisgir" list="oisgirs">
+                <set field="oisgir.priority" from-field="priority"/>
+                <store-value value-field="oisgir"/>
+                <clear-field field="oisgir"/>
+            </iterate>
+            <entity-condition entity-name="OrderItemAndShipGrpInvResAndItem" list="oisgirais">
+                <condition-expr field-name="orderId" operator="equals" from-field="orderId"/>
+            </entity-condition>
+            <iterate entry="oisgir" list="oisgirais">
+                <set field="reassignInventoryReservationsCtx.productId" from-field="oisgir.productId"/>
+                <set field="reassignInventoryReservationsCtx.facilityId" from-field="oisgir.facilityId"/>
+                <call-service service-name="reassignInventoryReservations" in-map-name="reassignInventoryReservationsCtx"/>
+                <clear-field field="reassignInventoryReservationsCtx"/>
+            </iterate>        
+        </else>
+        </if-empty>
+    </simple-method>
+</simple-methods>
\ No newline at end of file

Modified: ofbiz/trunk/applications/product/servicedef/services_facility.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services_facility.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/applications/product/servicedef/services_facility.xml (original)
+++ ofbiz/trunk/applications/product/servicedef/services_facility.xml Thu Mar 19 13:58:08 2009
@@ -262,6 +262,7 @@
         <attribute name="facilityId" type="String" mode="IN" optional="false"/>
         <attribute name="fromDate" type="Timestamp" mode="IN" optional="true"/>
         <attribute name="noLongerOnBackOrderIdSet" type="Set" mode="OUT" optional="true"/>
+        <attribute name="priority" type="String" mode="IN"  optional="true"/>   
     </service>
     <service name="balanceOrderItemsWithNegativeReservations" engine="simple" 
                 location="component://product/script/org/ofbiz/product/inventory/InventoryServices.xml" invoke="balanceOrderItemsWithNegativeReservations">
@@ -319,6 +320,7 @@
         <attribute name="reserveOrderEnumId" type="String" mode="IN" optional="true"/>
         <attribute name="sequenceId" type="Long" mode="IN" optional="true"/>
         <attribute name="quantityNotReserved" type="BigDecimal" mode="OUT" optional="false"/>
+        <attribute name="priority" type="String" mode="IN"  optional="true"/>    
     </service>
     <service name="reserveProductInventoryByContainer" engine="simple"
                 location="component://product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml" invoke="reserveProductInventory" auth="true">
@@ -345,6 +347,7 @@
         <auto-attributes mode="IN" entity-name="OrderItemShipGrpInvRes" include="nonpk" optional="true">
             <exclude field-name="createdDatetime"/>
         </auto-attributes>
+        <attribute name="priority" type="String" mode="IN"  optional="true"/>
         <override name="quantity" optional="false"/>
     </service>
     <service name="cancelOrderInventoryReservation" engine="simple"
@@ -873,4 +876,10 @@
         <permission-service service-name="facilityGenericPermission" main-action="DELETE"/>
         <auto-attributes include="pk" mode="IN" optional="false"/>
     </service>
+    <service name="setOrderReservationPriority" engine="simple" invoke="setOrderReservationPriority" 
+                location="component://product/script/org/ofbiz/product/inventory/InventoryServices.xml"  auth="true">
+        <description>set order priority</description>
+        <attribute name="priority" mode="IN" type="String" optional="true"/>
+        <attribute name="orderId" mode="IN" type="String" optional="false"/>
+    </service>
 </services>

Modified: ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml?rev=756001&r1=756000&r2=756001&view=diff
==============================================================================
--- ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml (original)
+++ ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml Thu Mar 19 13:58:08 2009
@@ -20,7 +20,7 @@
 
 <entity-engine-xml>
     <!--Demo data for a sales order -->
-    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER" orderName="" salesChannelEnumId="WEB_SALES_CHANNEL" orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23 16:49:27.392" visitId="10002" statusId="ORDER_APPROVED" createdBy="admin" currencyUom="USD" webSiteId="OrderEntry" productStoreId="9000" remainingSubTotal="12.45" grandTotal="12.45" lastUpdatedStamp="2008-04-23 16:49:33.196" lastUpdatedTxStamp="2008-04-23 16:49:33.118" createdStamp="2008-04-23 16:49:27.716" createdTxStamp="2008-04-23 16:49:27.349"/>
+    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER" orderName="" salesChannelEnumId="WEB_SALES_CHANNEL" orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23 16:49:27.392" priority="2" visitId="10002" statusId="ORDER_APPROVED" createdBy="admin" currencyUom="USD" webSiteId="OrderEntry" productStoreId="9000" remainingSubTotal="12.45" grandTotal="12.45" lastUpdatedStamp="2008-04-23 16:49:33.196" lastUpdatedTxStamp="2008-04-23 16:49:33.118" createdStamp="2008-04-23 16:49:27.716" createdTxStamp="2008-04-23 16:49:27.349"/>
     <OrderItem orderId="DEMO10090" orderItemSeqId="00001" orderItemTypeId="PRODUCT_ORDER_ITEM" productId="GZ-2644" prodCatalogId="DemoCatalog" isPromo="N" quantity="1.0" selectedAmount="0.0" unitPrice="38.4" unitListPrice="48.0" isModifiedPrice="N" itemDescription="Round Gizmo" correspondingPoId="" statusId="ITEM_APPROVED" lastUpdatedStamp="2008-04-23 16:49:33.514" lastUpdatedTxStamp="2008-04-23 16:49:33.484" createdStamp="2008-04-23 16:49:28.128" createdTxStamp="2008-04-23 16:49:27.349"/>
     <OrderItemPriceInfo orderItemPriceInfoId="9000" orderId="DEMO10090" orderItemSeqId="00001" productPriceRuleId="9000" productPriceActionSeqId="01" modifyAmount="-9.600000000000001" description="[PRODUCT_CATEGORY_IDIsPROMOTIONS] [list:48.0;avgCost:48.0;margin:0.0] [type:PRICE_POL]" lastUpdatedStamp="2008-04-23 16:49:28.682" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.682" createdTxStamp="2008-04-23 16:49:27.349"/>
     
@@ -36,7 +36,7 @@
     
     <OrderItemShipGroup orderId="DEMO10090" shipGroupSeqId="00001" shipmentMethodTypeId="NEXT_DAY" carrierPartyId="UPS" carrierRoleTypeId="CARRIER" contactMechId="9015" maySplit="N" giftMessage="" isGift="N" lastUpdatedStamp="2008-04-23 16:49:28.412" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.412" createdTxStamp="2008-04-23 16:49:27.349"/>
     <OrderItemShipGroupAssoc orderId="DEMO10090" orderItemSeqId="00001" shipGroupSeqId="00001" quantity="1.0" lastUpdatedStamp="2008-04-23 16:49:28.454" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.454" createdTxStamp="2008-04-23 16:49:27.349"/>
-    <OrderItemShipGrpInvRes orderId="DEMO10090" shipGroupSeqId="00001" orderItemSeqId="00001" inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC" quantity="1.0" quantityNotAvailable="0.0" reservedDatetime="2008-04-23 16:49:31.474" createdDatetime="2008-04-23 16:49:31.474" promisedDatetime="2008-05-08 16:49:27.392" lastUpdatedStamp="2008-04-23 16:49:31.474" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23 16:49:27.349"/>
+    <OrderItemShipGrpInvRes orderId="DEMO10090" shipGroupSeqId="00001" orderItemSeqId="00001" inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC" quantity="1.0" quantityNotAvailable="0.0" reservedDatetime="2008-04-23 16:49:31.474" createdDatetime="2008-04-23 16:49:31.474" promisedDatetime="2008-05-08 16:49:27.392" priority="2" lastUpdatedStamp="2008-04-23 16:49:31.474" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23 16:49:27.349"/>
     
     <OrderAdjustment orderAdjustmentId="9000" orderAdjustmentTypeId="PROMOTION_ADJUSTMENT" orderId="DEMO10090" orderItemSeqId="00001" shipGroupSeqId="_NA_" amount="-38.4" productPromoId="9016" productPromoRuleId="01" productPromoActionSeqId="01" createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin" lastUpdatedStamp="2008-04-23 16:49:28.271" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.271" createdTxStamp="2008-04-23 16:49:27.349"/>
     <OrderAdjustment orderAdjustmentId="9001" orderAdjustmentTypeId="SHIPPING_CHARGES" orderId="DEMO10090" orderItemSeqId="_NA_" shipGroupSeqId="00001" amount="12.45" createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin" lastUpdatedStamp="2008-04-23 16:49:28.431" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.431" createdTxStamp="2008-04-23 16:49:27.349"/>



Re: svn commit: r756001 - in /ofbiz/trunk: applications/order/config/ applications/order/entitydef/ applications/order/servicedef/ applications/order/webapp/ordermgr/WEB-INF/ applications/order/webapp/ordermgr/order/ applications/product/script/org/ofbiz/p

Posted by Vikas Mayur <vi...@hotwaxmedia.com>.
Thanks Jacques!

Vikas

On Mar 20, 2009, at 1:35 PM, Jacques Le Roux wrote:

> Thanks Vikas,
>
> Your patch is in trunk 756365
>
> From: "Vikas Mayur" <vi...@hotwaxmedia.com>
> no problemo!
>
> Uploaded the patch at https://issues.apache.org/jira/browse/OFBIZ-2245
>
> Vikas
>
> On Mar 20, 2009, at 1:19 AM, Jacques Le Roux wrote:
>
>> Sorry Vikas,
>>
>> But as you see I'd have to wrap the lines :/
>> And I have more interesting things to do :p
>> Just add a patch to the related Jira issue please
>>
>> Jacques
>>
>> From: "Vikas Mayur" <vi...@hotwaxmedia.com>
>>> Thanks Jacques!
>>>
>>> I have done few changes in rev. 756134 and to make them work,
>>> would  require to commit following changes. I would appreciate if
>>> you can  take a lead :) This is a small change, so not opening any
>>> new jira  issue.
>>>
>>> Index: framework/common/config/CommonUiLabels.xml
>>> ===================================================================
>>> --- framework/common/config/CommonUiLabels.xml (revision 756129)
>>> +++ framework/common/config/CommonUiLabels.xml (working copy)
>>> @@ -2877,6 +2877,9 @@
>>>        <value xml:lang="zh">隐藏域</value>
>>>        <value xml:lang="zh_CN">隐藏字段</value>
>>>    </property>
>>> +    <property key="CommonHigh">
>>> +        <value xml:lang="en">High</value>
>>> +    </property>
>>>    <property key="CommonHome">
>>>        <value xml:lang="en">Home</value>
>>>        <value xml:lang="fr">Point de départ</value>
>>> @@ -3445,6 +3448,9 @@
>>>        <value xml:lang="en">http://whitepages.com/find_person_results.pl?fid=p&amp;ac=$
>>> {telecomNumber.areaCode?if_exists}&amp;s=&amp;p=$
>>> {telecomNumber.contactNumber?if_exists}&amp;pt=b&amp;x=40&amp;y=9</
>>> value>
>>>        <value xml:lang="fr">http://www.pagesjaunes.fr/quidonc/aquiestcenumero.do
>>> </value>
>>>    </property>
>>> +    <property key="CommonLow">
>>> +        <value xml:lang="en">Low</value>
>>> +    </property>
>>>    <property key="CommonMain">
>>>        <value xml:lang="ar">الرئيسية</
>>> value>
>>>        <value xml:lang="cs">Úvodní stránka</value>
>>> @@ -3988,6 +3994,9 @@
>>>        <value xml:lang="zh">Âê¶</value>
>>>        <value xml:lang="zh_CN">不</value>
>>>    </property>
>>> +    <property key="CommonNormal">
>>> +        <value xml:lang="en">Normal</value>
>>> +    </property>
>>>    <property key="CommonNoAccess">
>>>        <value xml:lang="en">No Access to this function, contact
>>> your system administrator if you feel this is an error</value>
>>>        <value xml:lang="es">Acceso no autorizado. Póngase en
>>> contacto con su administrador de sistemas si piensa que esto es un
>>> error.</value>
>>>
>>>
>>> Thanks,
>>> Vikas
>>>
>>>
>>> On Mar 19, 2009, at 7:39 PM, Jacques Le Roux wrote:
>>>
>>>> +1 for CommonHigh, CommonLow etc.
>>>>
>>>> Jacques
>>>>
>>>> From: <mo...@apache.org>
>>>>> Author: mor
>>>>> Date: Thu Mar 19 13:58:08 2009
>>>>> New Revision: 756001
>>>>>
>>>>> URL: http://svn.apache.org/viewvc?rev=756001&view=rev
>>>>> Log:
>>>>> Another new feature from Pranay and Co.
>>>>> Now Order reservation can be changed manually by setting
>>>>> priority  (new field priority on OrderHeader and
>>>>> OrderItemShipGrpInvRes) for  the order. This way few orders can be
>>>>> fulfilled while waiting for sufficient inventory to fulfill
>>>>> large  orders.
>>>>>
>>>>> Applied patch from OFBIZ-2224 (https://issues.apache.org/jira/browse/OFBIZ-2224
>>>>> ).
>>>>> Thanks Pratik Khera and Pranay Pandey.
>>>>>
>>>>> i18n Note: This commit has three new labels (IMO, they can be
>>>>> moved  to common component with names something like CommonHigh,
>>>>> CommonLow  etc.)
>>>>>
>>>>> Pratik, I have done minor changes in the service and they are
>>>>> 1) Used <if-empty field=""> instead of <if> <condition> <if-
>>>>> empty>  as this makes the logic more clearer.
>>>>> 2) Removed <condition-list> because their is only one expression
>>>>> in  the <entity-condition>
>>>>> 3) Added <clear-field> in the iteration, to make sure that always
>>>>> a  new value will be set in the map that is passed to a service
>>>>> and  <store-value>
>>>>>
>>>>>
>>>>> Modified:
>>>>> ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>>> ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>>> ofbiz/trunk/applications/order/servicedef/secas.xml
>>>>> ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>>> controller.xml
>>>>> ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
>>>>> ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>>> inventory/InventoryReserveServices.xml
>>>>> ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>>> inventory/InventoryServices.xml
>>>>> ofbiz/trunk/applications/product/servicedef/services_facility.xml
>>>>> ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderUiLabels.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> =
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>>> (original)
>>>>> +++ ofbiz/trunk/applications/order/config/OrderUiLabels.xml Thu
>>>>> Mar  19 13:58:08 2009
>>>>> @@ -6271,6 +6271,15 @@
>>>>>      <value xml:lang="zh">å
>>>> ¬å¼€ä¾¿ç¬º (已打印)</value>
>>>>>      <value xml:lang="zh_CN">å
>>>> ¬å¼€æ³¨é‡Š (可打印)</value>
>>>>>  </property>
>>>>> +    <property key="OrderPriorityHigh">
>>>>> +        <value xml:lang="en">High</value>
>>>>> +    </property>
>>>>> +    <property key="OrderPriorityLow">
>>>>> +        <value xml:lang="en">Low</value>
>>>>> +    </property>
>>>>> +    <property key="OrderPriorityNormal">
>>>>> +        <value xml:lang="en">Normal</value>
>>>>> +    </property>
>>>>>  <property key="OrderProcessOrder">
>>>>>      <value xml:lang="de">Auftrag verarbeiten</value>
>>>>>      <value xml:lang="en">Process Order</value>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/entitydef/entitymodel.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> =
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>>> (original)
>>>>> +++ ofbiz/trunk/applications/order/entitydef/entitymodel.xml Thu
>>>>> Mar 19 13:58:08 2009
>>>>> @@ -337,6 +337,7 @@
>>>>>    <field name="externalId" type="id"></field>
>>>>>    <field name="salesChannelEnumId" type="id"></field>
>>>>>    <field name="orderDate" type="date-time"></field>
>>>>> +      <field name="priority" type="indicator"><description>Sets
>>>>> priority for Inventory Reservation</description></field>
>>>>>    <field name="entryDate" type="date-time"></field>
>>>>>    <field name="visitId" type="id"></field>
>>>>>    <field name="statusId" type="id"></field>
>>>>> @@ -888,6 +889,7 @@
>>>>>    <field name="createdDatetime" type="date-time"></field>
>>>>>    <field name="promisedDatetime" type="date-time"></field>
>>>>>    <field name="currentPromisedDate" type="date-time"></field>
>>>>> +      <field name="priority" type="indicator"><description>Sets
>>>>> priority for Inventory Reservation</description></field>
>>>>>    <field name="sequenceId" type="numeric"></field>
>>>>>    <field name="oldPickStartDate" col-name="PICK_START_DATE"
>>>>> type="date-time"></field> <!-- this is no longer used and is
>>>>> here  for migration or whatever if needed -->
>>>>>    <prim-key field="orderId"/>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/servicedef/secas.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/secas.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> =
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/order/servicedef/secas.xml (original)
>>>>> +++ ofbiz/trunk/applications/order/servicedef/secas.xml Thu Mar
>>>>> 19  13:58:08 2009
>>>>> @@ -36,6 +36,9 @@
>>>>>      <condition field-name="orderTypeId" operator="equals"
>>>>> value="PURCHASE_ORDER"/>
>>>>>      <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>>>  </eca>
>>>>> +    <eca service="storeOrder" event="return">
>>>>> +        <action service="setOrderReservationPriority"
>>>>> mode="sync"/>
>>>>> +    </eca>
>>>>>  <eca service="updateOrderItems" event="commit">
>>>>>      <condition field-name="orderTypeId" operator="equals"
>>>>> value="PURCHASE_ORDER"/>
>>>>>      <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>>> controller.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> =
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>>> controller.xml (original)
>>>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>>> controller.xml Thu Mar 19 13:58:08 2009
>>>>> @@ -1457,7 +1457,12 @@
>>>>>      <response name="success" type="request-redirect"
>>>>> value="orderview"/>
>>>>>      <response name="error" type="request-redirect"
>>>>> value="orderview"/>
>>>>>  </request-map>
>>>>> -
>>>>> +    <request-map uri="setOrderReservationPriority">
>>>>> +        <security https="true" auth="true"/>
>>>>> +        <event type="service"
>>>>> invoke="setOrderReservationPriority"/>
>>>>> +        <response name="success" type="view" value="orderview"/>
>>>>> +        <response name="error" type="view" value="orderview"/>
>>>>> +    </request-map>
>>>>>  <!-- Lookup request mappings -->
>>>>>  <request-map uri="LookupPerson"><security https="true"
>>>>> auth="true"/><response name="success" type="view"
>>>>> value="LookupPerson"/></request-map>
>>>>>  <request-map uri="LookupPartyGroup"><security https="true"
>>>>> auth="true"/><response name="success" type="view"
>>>>> value="LookupPartyGroup"/></request-map>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/
>>>>> orderinfo.ftl
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> =
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/order/
>>>>> orderinfo.ftl (original)
>>>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/order/
>>>>> orderinfo.ftl Thu Mar 19 13:58:08 2009
>>>>> @@ -179,6 +179,24 @@
>>>>>            </td>
>>>>>          </tr>
>>>>>          </#if>
>>>>> +            <#if "SALES_ORDER" == orderHeader.orderTypeId>
>>>>> +              <form action="setOrderReservationPriority"
>>>>> method="post" name="setOrderReservationPriority">
>>>>> +                <input type = "hidden" name="orderId" value="$
>>>>> {orderId}"/>
>>>>> +                <tr><td colspan="3"><hr/></td></tr>
>>>>> +                <tr>
>>>>> +                  <td align="right" valign="top" width="15%"
>>>>> class="label">&nbsp;${uiLabelMap.FormFieldTitle_priority}</td>
>>>>> +                  <td width="5%">&nbsp;</td>
>>>>> +                  <td valign="top" width="80%">
>>>>> +                    <select name="priority">
>>>>> +                      <option value="1">$
>>>>> {uiLabelMap.OrderPriorityHigh}</option>
>>>>> +                      <option value="2"selected>$
>>>>> {uiLabelMap.OrderPriorityNormal}</option>
>>>>> +                      <option value="3">$
>>>>> {uiLabelMap.OrderPriorityLow}</option>
>>>>> +                    </select>
>>>>> +                    <input type="submit" class="smallSubmit"
>>>>> value="${uiLabelMap.FormFieldTitle_reserveInventory}"/>
>>>>> +                  </td>
>>>>> +                </tr>
>>>>> +              </form>
>>>>> +            </#if>
>>>>>      </table>
>>>>>  </div>
>>>>> </div>
>>>>> \ No newline at end of file
>>>>>
>>>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/
>>>>> product/ inventory/InventoryReserveServices.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> =
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>>> inventory/InventoryReserveServices.xml (original)
>>>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>>> inventory/InventoryReserveServices.xml Thu Mar 19 13:58:08 2009
>>>>> @@ -201,6 +201,7 @@
>>>>>                          <set from-
>>>>> field="parameters.reservedDatetime"
>>>>> field="reserveOisgirMap.reservedDatetime"/>
>>>>>                          <set from-field="promisedDatetime"
>>>>> field="reserveOisgirMap.promisedDatetime"/>
>>>>>                          <set from-
>>>>> field="parameters.sequenceId"
>>>>> field="reserveOisgirMap.sequenceId"/>
>>>>> +                            <set from-
>>>>> field="parameters.priority"  field="reserveOisgirMap.priority"/>
>>>>>                          <call-service service-
>>>>> name="reserveOrderItemInventory" in-map-name="reserveOisgirMap"/>
>>>>>                          <clear-field field="reserveOisgirMap"/>
>>>>>                      <else>
>>>>> @@ -444,6 +445,7 @@
>>>>>                      <set from-
>>>>> field="parameters.reservedDatetime"
>>>>> field="reserveOisgirMap.reservedDatetime"/>
>>>>>                      <set from-field="parameters.deductAmount"
>>>>> field="reserveOisgirMap.quantity" type="BigDecimal"/>
>>>>>                      <set from-field="promisedDatetime"
>>>>> field="reserveOisgirMap.promisedDatetime"/>
>>>>> +                        <set from-field="parameters.priority"
>>>>> field="reserveOisgirMap.priority"/>
>>>>>                      <if-not-empty field="parameters.sequenceId">
>>>>>                          <set
>>>>> field="reserveOisgirMap.sequenceId"  from-
>>>>> field="parameters.sequenceId"/>
>>>>>                      </if-not-empty>
>>>>> @@ -498,6 +500,7 @@
>>>>>          <set-nonpk-fields map="parameters" value-
>>>>> field="newOisgirEntity"/>
>>>>>          <now-timestamp field="nowTimestamp"/>
>>>>>          <set from-field="nowTimestamp"
>>>>> field="newOisgirEntity.createdDatetime"/>
>>>>> +            <set from-field="parameters.priority"
>>>>> field="newOisgirEntity.priority"/>
>>>>>          <if-empty field="newOisgirEntity.reservedDatetime">
>>>>>              <set from-field="nowTimestamp"
>>>>> field="newOisgirEntity.reservedDatetime"/>
>>>>>          </if-empty>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/
>>>>> product/ inventory/InventoryServices.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> =
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>>> inventory/InventoryServices.xml (original)
>>>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>>> inventory/InventoryServices.xml Thu Mar 19 13:58:08 2009
>>>>> @@ -814,6 +814,7 @@
>>>>>                  </condition-list>
>>>>>              </condition-list>
>>>>>          </condition-list>
>>>>> +            <order-by field-name="priority"/>
>>>>>          <order-by field-name="currentPromisedDate"/>
>>>>>          <order-by field-name="reservedDatetime"/>
>>>>>          <order-by field-name="sequenceId"/>
>>>>> @@ -875,6 +876,9 @@
>>>>>                  <log level="verbose" message="Adding $
>>>>> {oisgir.orderId} to touchedOrderIdMap"/>
>>>>>              </if-compare>
>>>>>          </if-not-empty>
>>>>> +            <entity-one entity-name="OrderHeader" value-
>>>>> field="orderHeader">
>>>>> +                <field-map field-name="orderId" from-
>>>>> field="oisgir.orderId"/>
>>>>> +            </entity-one>
>>>>>
>>>>>          <!-- require inventory is N because it had to be N to
>>>>> begin with to have a negative ATP -->
>>>>>          <clear-field field="resMap"/>
>>>>> @@ -888,6 +892,7 @@
>>>>>          <set field="resMap.shipGroupSeqId" from-
>>>>> field="oisgir.shipGroupSeqId"/>
>>>>>          <set field="resMap.sequenceId" from-
>>>>> field="oisgir.sequenceId"/>
>>>>>          <set field="resMap.facilityId" from-
>>>>> field="parameters.facilityId"/>
>>>>> +            <set field="resMap.priority" from-
>>>>> field="orderHeader.priority"/>
>>>>>          <log level="info" message="Re-reserving product [$
>>>>> {resMap.productId}] for order item [${resMap.orderId}:$
>>>>> {resMap.orderItemSeqId}] quantity [${resMap.quantity}]; facility
>>>>> [$ {parameters.facilityId}]"/>
>>>>>          <call-service service-
>>>>> name="reserveProductInventoryByFacility" in-map-name="resMap"/>
>>>>>      </iterate>
>>>>> @@ -1249,5 +1254,43 @@
>>>>>          </if-compare>
>>>>>      </if-not-empty>
>>>>>  </simple-method>
>>>>> -
>>>>> -</simple-methods>
>>>>> +    <simple-method method-name="setOrderReservationPriority"
>>>>> short- description="Sets priority of an order for Inventory
>>>>> Reservation,  orders with HIGH priority would be served first.">
>>>>> +        <set field="orderId" from-field="parameters.orderId"/>
>>>>> +        <entity-one entity-name="OrderHeader" value-
>>>>> field="orderHeader">
>>>>> +            <field-map field-name="orderId" from- 
>>>>> field="orderId"/>
>>>>> +        </entity-one>
>>>>> +        <set field="priority" from-field="parameters.priority"/>
>>>>> +        <if-empty field="priority">
>>>>> +            <entity-and list="oisgirs" entity-
>>>>> name="OrderItemShipGrpInvRes">
>>>>> +                <field-map field-name="orderId" from-
>>>>> field="orderId"/>
>>>>> +            </entity-and>
>>>>> +            <iterate entry="oisgir" list="oisgirs">
>>>>> +                <set field="oisgir.priority" default-value="2"/>
>>>>> +                <store-value value-field="oisgir"/>
>>>>> +            </iterate>
>>>>> +            <set field="orderHeader.priority" default-value="2"/>
>>>>> +            <store-value value-field="orderHeader"/>
>>>>> +        <else>
>>>>> +            <set field="orderHeader.priority" from-
>>>>> field="priority"/>
>>>>> +            <store-value value-field="orderHeader"/>
>>>>> +            <entity-and list="oisgirs" entity-
>>>>> name="OrderItemShipGrpInvRes">
>>>>> +                <field-map field-name="orderId" from-
>>>>> field="orderId"/>
>>>>> +            </entity-and>
>>>>> +            <iterate entry="oisgir" list="oisgirs">
>>>>> +                <set field="oisgir.priority" from-
>>>>> field="priority"/>
>>>>> +                <store-value value-field="oisgir"/>
>>>>> +                <clear-field field="oisgir"/>
>>>>> +            </iterate>
>>>>> +            <entity-condition entity-
>>>>> name="OrderItemAndShipGrpInvResAndItem" list="oisgirais">
>>>>> +                <condition-expr field-name="orderId"
>>>>> operator="equals" from-field="orderId"/>
>>>>> +            </entity-condition>
>>>>> +            <iterate entry="oisgir" list="oisgirais">
>>>>> +                <set
>>>>> field="reassignInventoryReservationsCtx.productId" from-
>>>>> field="oisgir.productId"/>
>>>>> +                <set
>>>>> field="reassignInventoryReservationsCtx.facilityId" from-
>>>>> field="oisgir.facilityId"/>
>>>>> +                <call-service service-
>>>>> name="reassignInventoryReservations" in-map-
>>>>> name="reassignInventoryReservationsCtx"/>
>>>>> +                <clear-field
>>>>> field="reassignInventoryReservationsCtx"/>
>>>>> +            </iterate>
>>>>> +        </else>
>>>>> +        </if-empty>
>>>>> +    </simple-method>
>>>>> +</simple-methods>
>>>>> \ No newline at end of file
>>>>>
>>>>> Modified: ofbiz/trunk/applications/product/servicedef/
>>>>> services_facility.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services_facility.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> = 
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/applications/product/servicedef/
>>>>> services_facility.xml (original)
>>>>> +++ ofbiz/trunk/applications/product/servicedef/
>>>>> services_facility.xml Thu Mar 19 13:58:08 2009
>>>>> @@ -262,6 +262,7 @@
>>>>>      <attribute name="facilityId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>>      <attribute name="fromDate" type="Timestamp" mode="IN"
>>>>> optional="true"/>
>>>>>      <attribute name="noLongerOnBackOrderIdSet" type="Set"
>>>>> mode="OUT" optional="true"/>
>>>>> +        <attribute name="priority" type="String" mode="IN"
>>>>> optional="true"/>
>>>>>  </service>
>>>>>  <service name="balanceOrderItemsWithNegativeReservations"
>>>>> engine="simple"
>>>>>              location="component://product/script/org/ofbiz/
>>>>> product/inventory/InventoryServices.xml"
>>>>> invoke="balanceOrderItemsWithNegativeReservations">
>>>>> @@ -319,6 +320,7 @@
>>>>>      <attribute name="reserveOrderEnumId" type="String"
>>>>> mode="IN"  optional="true"/>
>>>>>      <attribute name="sequenceId" type="Long" mode="IN"
>>>>> optional="true"/>
>>>>>      <attribute name="quantityNotReserved" type="BigDecimal"
>>>>> mode="OUT" optional="false"/>
>>>>> +        <attribute name="priority" type="String" mode="IN"
>>>>> optional="true"/>
>>>>>  </service>
>>>>>  <service name="reserveProductInventoryByContainer"
>>>>> engine="simple"
>>>>>              location="component://product/script/org/ofbiz/
>>>>> product/inventory/InventoryReserveServices.xml"
>>>>> invoke="reserveProductInventory" auth="true">
>>>>> @@ -345,6 +347,7 @@
>>>>>      <auto-attributes mode="IN" entity-
>>>>> name="OrderItemShipGrpInvRes" include="nonpk" optional="true">
>>>>>          <exclude field-name="createdDatetime"/>
>>>>>      </auto-attributes>
>>>>> +        <attribute name="priority" type="String" mode="IN"
>>>>> optional="true"/>
>>>>>      <override name="quantity" optional="false"/>
>>>>>  </service>
>>>>>  <service name="cancelOrderInventoryReservation" engine="simple"
>>>>> @@ -873,4 +876,10 @@
>>>>>      <permission-service service-
>>>>> name="facilityGenericPermission"  main-action="DELETE"/>
>>>>>      <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>  </service>
>>>>> +    <service name="setOrderReservationPriority" engine="simple"
>>>>> invoke="setOrderReservationPriority"
>>>>> +                location="component://product/script/org/ofbiz/
>>>>> product/inventory/InventoryServices.xml"  auth="true">
>>>>> +        <description>set order priority</description>
>>>>> +        <attribute name="priority" mode="IN" type="String"
>>>>> optional="true"/>
>>>>> +        <attribute name="orderId" mode="IN" type="String"
>>>>> optional="false"/>
>>>>> +    </service>
>>>>> </services>
>>>>>
>>>>> Modified: ofbiz/trunk/specialpurpose/ecommerce/data/
>>>>> DemoOrderTestData.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>>> = = = = = = = = =
>>>>> =
>>>>> =
>>>>> = 
>>>>> ==================================================================
>>>>> --- ofbiz/trunk/specialpurpose/ecommerce/data/
>>>>> DemoOrderTestData.xml  (original)
>>>>> +++ ofbiz/trunk/specialpurpose/ecommerce/data/
>>>>> DemoOrderTestData.xml  Thu Mar 19 13:58:08 2009
>>>>> @@ -20,7 +20,7 @@
>>>>>
>>>>> <entity-engine-xml>
>>>>>  <!--Demo data for a sales order -->
>>>>> -    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"
>>>>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"
>>>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23
>>>>> 16:49:27.392" visitId="10002" statusId="ORDER_APPROVED"
>>>>> createdBy="admin" currencyUom="USD" webSiteId="OrderEntry"
>>>>> productStoreId="9000" remainingSubTotal="12.45"
>>>>> grandTotal="12.45" lastUpdatedStamp="2008-04-23 16:49:33.196"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"
>>>>> createdStamp="2008-04-23 16:49:27.716"
>>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>> +    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"
>>>>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"
>>>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23
>>>>> 16:49:27.392" priority="2" visitId="10002"
>>>>> statusId="ORDER_APPROVED" createdBy="admin" currencyUom="USD"
>>>>> webSiteId="OrderEntry" productStoreId="9000"
>>>>> remainingSubTotal="12.45" grandTotal="12.45"
>>>>> lastUpdatedStamp="2008-04-23 16:49:33.196"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"
>>>>> createdStamp="2008-04-23 16:49:27.716"
>>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>>  <OrderItem orderId="DEMO10090" orderItemSeqId="00001"
>>>>> orderItemTypeId="PRODUCT_ORDER_ITEM" productId="GZ-2644"
>>>>> prodCatalogId="DemoCatalog" isPromo="N" quantity="1.0"
>>>>> selectedAmount="0.0" unitPrice="38.4" unitListPrice="48.0"
>>>>> isModifiedPrice="N" itemDescription="Round Gizmo"
>>>>> correspondingPoId="" statusId="ITEM_APPROVED"
>>>>> lastUpdatedStamp="2008-04-23 16:49:33.514"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.484"
>>>>> createdStamp="2008-04-23 16:49:28.128" createdTxStamp="2008-04-23
>>>>> 16:49:27.349"/>
>>>>>  <OrderItemPriceInfo orderItemPriceInfoId="9000"
>>>>> orderId="DEMO10090" orderItemSeqId="00001"
>>>>> productPriceRuleId="9000" productPriceActionSeqId="01"
>>>>> modifyAmount="-9.600000000000001"
>>>>> description="[PRODUCT_CATEGORY_IDIsPROMOTIONS] [list:
>>>>> 48.0;avgCost: 48.0;margin:0.0] [type:PRICE_POL]"
>>>>> lastUpdatedStamp="2008-04-23  16:49:28.682"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>>> createdStamp="2008-04-23 16:49:28.682"
>>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>>
>>>>> @@ -36,7 +36,7 @@
>>>>>
>>>>>  <OrderItemShipGroup orderId="DEMO10090" shipGroupSeqId="00001"
>>>>> shipmentMethodTypeId="NEXT_DAY" carrierPartyId="UPS"
>>>>> carrierRoleTypeId="CARRIER" contactMechId="9015" maySplit="N"
>>>>> giftMessage="" isGift="N" lastUpdatedStamp="2008-04-23
>>>>> 16:49:28.412" lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>>> createdStamp="2008-04-23 16:49:28.412" createdTxStamp="2008-04-23
>>>>> 16:49:27.349"/>
>>>>>  <OrderItemShipGroupAssoc orderId="DEMO10090"
>>>>> orderItemSeqId="00001" shipGroupSeqId="00001" quantity="1.0"
>>>>> lastUpdatedStamp="2008-04-23 16:49:28.454"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>>> createdStamp="2008-04-23 16:49:28.454"
>>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>> -    <OrderItemShipGrpInvRes orderId="DEMO10090"
>>>>> shipGroupSeqId="00001" orderItemSeqId="00001"
>>>>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"
>>>>> quantity="1.0" quantityNotAvailable="0.0"
>>>>> reservedDatetime="2008-04-23 16:49:31.474"
>>>>> createdDatetime="2008-04-23 16:49:31.474"
>>>>> promisedDatetime="2008-05-08 16:49:27.392"
>>>>> lastUpdatedStamp="2008-04-23 16:49:31.474"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23
>>>>> 16:49:27.349"/>
>>>>> +    <OrderItemShipGrpInvRes orderId="DEMO10090"
>>>>> shipGroupSeqId="00001" orderItemSeqId="00001"
>>>>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"
>>>>> quantity="1.0" quantityNotAvailable="0.0"
>>>>> reservedDatetime="2008-04-23 16:49:31.474"
>>>>> createdDatetime="2008-04-23 16:49:31.474"
>>>>> promisedDatetime="2008-05-08 16:49:27.392" priority="2"
>>>>> lastUpdatedStamp="2008-04-23 16:49:31.474"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23
>>>>> 16:49:27.349"/>
>>>>>
>>>>>  <OrderAdjustment orderAdjustmentId="9000"
>>>>> orderAdjustmentTypeId="PROMOTION_ADJUSTMENT" orderId="DEMO10090"
>>>>> orderItemSeqId="00001" shipGroupSeqId="_NA_" amount="-38.4"
>>>>> productPromoId="9016" productPromoRuleId="01"
>>>>> productPromoActionSeqId="01" createdDate="2008-04-23
>>>>> 16:49:27.866"  createdByUserLogin="admin"
>>>>> lastUpdatedStamp="2008-04-23 16:49:28.271"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>>> createdStamp="2008-04-23 16:49:28.271" createdTxStamp="2008-04-23
>>>>> 16:49:27.349"/>
>>>>>  <OrderAdjustment orderAdjustmentId="9001"
>>>>> orderAdjustmentTypeId="SHIPPING_CHARGES" orderId="DEMO10090"
>>>>> orderItemSeqId="_NA_" shipGroupSeqId="00001" amount="12.45"
>>>>> createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin"
>>>>> lastUpdatedStamp="2008-04-23 16:49:28.431"
>>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>>> createdStamp="2008-04-23 16:49:28.431"
>>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>>
>>>>
>>>>
>>>
>>
>>
>
>
>


Re: svn commit: r756001 - in /ofbiz/trunk: applications/order/config/ applications/order/entitydef/ applications/order/servicedef/ applications/order/webapp/ordermgr/WEB-INF/ applications/order/webapp/ordermgr/order/ applications/product/script/org/ofbiz/p

Posted by Jacques Le Roux <ja...@les7arts.com>.
Thanks Vikas,

Your patch is in trunk 756365

From: "Vikas Mayur" <vi...@hotwaxmedia.com>
no problemo!

Uploaded the patch at https://issues.apache.org/jira/browse/OFBIZ-2245

Vikas

On Mar 20, 2009, at 1:19 AM, Jacques Le Roux wrote:

> Sorry Vikas,
>
> But as you see I'd have to wrap the lines :/
> And I have more interesting things to do :p
> Just add a patch to the related Jira issue please
>
> Jacques
>
> From: "Vikas Mayur" <vi...@hotwaxmedia.com>
>> Thanks Jacques!
>>
>> I have done few changes in rev. 756134 and to make them work,
>> would  require to commit following changes. I would appreciate if
>> you can  take a lead :) This is a small change, so not opening any
>> new jira  issue.
>>
>> Index: framework/common/config/CommonUiLabels.xml
>> ===================================================================
>> --- framework/common/config/CommonUiLabels.xml (revision 756129)
>> +++ framework/common/config/CommonUiLabels.xml (working copy)
>> @@ -2877,6 +2877,9 @@
>>         <value xml:lang="zh">隐藏域</value>
>>         <value xml:lang="zh_CN">隐藏字段</value>
>>     </property>
>> +    <property key="CommonHigh">
>> +        <value xml:lang="en">High</value>
>> +    </property>
>>     <property key="CommonHome">
>>         <value xml:lang="en">Home</value>
>>         <value xml:lang="fr">Point de départ</value>
>> @@ -3445,6 +3448,9 @@
>>         <value xml:lang="en">http://whitepages.com/find_person_results.pl?fid=p&amp;ac=$
>>  {telecomNumber.areaCode?if_exists}&amp;s=&amp;p=$
>> {telecomNumber.contactNumber?if_exists}&amp;pt=b&amp;x=40&amp;y=9</
>> value>
>>         <value xml:lang="fr">http://www.pagesjaunes.fr/quidonc/aquiestcenumero.do
>>  </value>
>>     </property>
>> +    <property key="CommonLow">
>> +        <value xml:lang="en">Low</value>
>> +    </property>
>>     <property key="CommonMain">
>>         <value xml:lang="ar">الرئيسية</
>> value>
>>         <value xml:lang="cs">Úvodní stránka</value>
>> @@ -3988,6 +3994,9 @@
>>         <value xml:lang="zh">Âê¶</value>
>>         <value xml:lang="zh_CN">不</value>
>>     </property>
>> +    <property key="CommonNormal">
>> +        <value xml:lang="en">Normal</value>
>> +    </property>
>>     <property key="CommonNoAccess">
>>         <value xml:lang="en">No Access to this function, contact
>> your system administrator if you feel this is an error</value>
>>         <value xml:lang="es">Acceso no autorizado. Póngase en
>> contacto con su administrador de sistemas si piensa que esto es un
>> error.</value>
>>
>>
>> Thanks,
>> Vikas
>>
>>
>> On Mar 19, 2009, at 7:39 PM, Jacques Le Roux wrote:
>>
>>> +1 for CommonHigh, CommonLow etc.
>>>
>>> Jacques
>>>
>>> From: <mo...@apache.org>
>>>> Author: mor
>>>> Date: Thu Mar 19 13:58:08 2009
>>>> New Revision: 756001
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=756001&view=rev
>>>> Log:
>>>> Another new feature from Pranay and Co.
>>>> Now Order reservation can be changed manually by setting
>>>> priority  (new field priority on OrderHeader and
>>>> OrderItemShipGrpInvRes) for  the order. This way few orders can be
>>>> fulfilled while waiting for sufficient inventory to fulfill
>>>> large  orders.
>>>>
>>>> Applied patch from OFBIZ-2224 (https://issues.apache.org/jira/browse/OFBIZ-2224
>>>>  ).
>>>> Thanks Pratik Khera and Pranay Pandey.
>>>>
>>>> i18n Note: This commit has three new labels (IMO, they can be
>>>> moved  to common component with names something like CommonHigh,
>>>> CommonLow  etc.)
>>>>
>>>> Pratik, I have done minor changes in the service and they are
>>>> 1) Used <if-empty field=""> instead of <if> <condition> <if-
>>>> empty>  as this makes the logic more clearer.
>>>> 2) Removed <condition-list> because their is only one expression
>>>> in  the <entity-condition>
>>>> 3) Added <clear-field> in the iteration, to make sure that always
>>>> a  new value will be set in the map that is passed to a service
>>>> and  <store-value>
>>>>
>>>>
>>>> Modified:
>>>>  ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>>  ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>>  ofbiz/trunk/applications/order/servicedef/secas.xml
>>>>  ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>> controller.xml
>>>>  ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
>>>>  ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>> inventory/InventoryReserveServices.xml
>>>>  ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>> inventory/InventoryServices.xml
>>>>  ofbiz/trunk/applications/product/servicedef/services_facility.xml
>>>>  ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
>>>>
>>>> Modified: ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderUiLabels.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>> (original)
>>>> +++ ofbiz/trunk/applications/order/config/OrderUiLabels.xml Thu
>>>> Mar  19 13:58:08 2009
>>>> @@ -6271,6 +6271,15 @@
>>>>       <value xml:lang="zh">å
>>> ¬å¼€ä¾¿ç¬º (已打印)</value>
>>>>       <value xml:lang="zh_CN">å
>>> ¬å¼€æ³¨é‡Š (可打印)</value>
>>>>   </property>
>>>> +    <property key="OrderPriorityHigh">
>>>> +        <value xml:lang="en">High</value>
>>>> +    </property>
>>>> +    <property key="OrderPriorityLow">
>>>> +        <value xml:lang="en">Low</value>
>>>> +    </property>
>>>> +    <property key="OrderPriorityNormal">
>>>> +        <value xml:lang="en">Normal</value>
>>>> +    </property>
>>>>   <property key="OrderProcessOrder">
>>>>       <value xml:lang="de">Auftrag verarbeiten</value>
>>>>       <value xml:lang="en">Process Order</value>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/entitydef/entitymodel.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>> (original)
>>>> +++ ofbiz/trunk/applications/order/entitydef/entitymodel.xml Thu
>>>> Mar 19 13:58:08 2009
>>>> @@ -337,6 +337,7 @@
>>>>     <field name="externalId" type="id"></field>
>>>>     <field name="salesChannelEnumId" type="id"></field>
>>>>     <field name="orderDate" type="date-time"></field>
>>>> +      <field name="priority" type="indicator"><description>Sets
>>>> priority for Inventory Reservation</description></field>
>>>>     <field name="entryDate" type="date-time"></field>
>>>>     <field name="visitId" type="id"></field>
>>>>     <field name="statusId" type="id"></field>
>>>> @@ -888,6 +889,7 @@
>>>>     <field name="createdDatetime" type="date-time"></field>
>>>>     <field name="promisedDatetime" type="date-time"></field>
>>>>     <field name="currentPromisedDate" type="date-time"></field>
>>>> +      <field name="priority" type="indicator"><description>Sets
>>>> priority for Inventory Reservation</description></field>
>>>>     <field name="sequenceId" type="numeric"></field>
>>>>     <field name="oldPickStartDate" col-name="PICK_START_DATE"
>>>> type="date-time"></field> <!-- this is no longer used and is
>>>> here  for migration or whatever if needed -->
>>>>     <prim-key field="orderId"/>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/servicedef/secas.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/secas.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/servicedef/secas.xml (original)
>>>> +++ ofbiz/trunk/applications/order/servicedef/secas.xml Thu Mar
>>>> 19  13:58:08 2009
>>>> @@ -36,6 +36,9 @@
>>>>       <condition field-name="orderTypeId" operator="equals"
>>>> value="PURCHASE_ORDER"/>
>>>>       <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>>   </eca>
>>>> +    <eca service="storeOrder" event="return">
>>>> +        <action service="setOrderReservationPriority"
>>>> mode="sync"/>
>>>> +    </eca>
>>>>   <eca service="updateOrderItems" event="commit">
>>>>       <condition field-name="orderTypeId" operator="equals"
>>>> value="PURCHASE_ORDER"/>
>>>>       <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>> controller.xml
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>> controller.xml (original)
>>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/
>>>> controller.xml Thu Mar 19 13:58:08 2009
>>>> @@ -1457,7 +1457,12 @@
>>>>       <response name="success" type="request-redirect"
>>>> value="orderview"/>
>>>>       <response name="error" type="request-redirect"
>>>> value="orderview"/>
>>>>   </request-map>
>>>> -
>>>> +    <request-map uri="setOrderReservationPriority">
>>>> +        <security https="true" auth="true"/>
>>>> +        <event type="service"
>>>> invoke="setOrderReservationPriority"/>
>>>> +        <response name="success" type="view" value="orderview"/>
>>>> +        <response name="error" type="view" value="orderview"/>
>>>> +    </request-map>
>>>>   <!-- Lookup request mappings -->
>>>>   <request-map uri="LookupPerson"><security https="true"
>>>> auth="true"/><response name="success" type="view"
>>>> value="LookupPerson"/></request-map>
>>>>   <request-map uri="LookupPartyGroup"><security https="true"
>>>> auth="true"/><response name="success" type="view"
>>>> value="LookupPartyGroup"/></request-map>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/
>>>> orderinfo.ftl
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/order/
>>>> orderinfo.ftl (original)
>>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/order/
>>>> orderinfo.ftl Thu Mar 19 13:58:08 2009
>>>> @@ -179,6 +179,24 @@
>>>>             </td>
>>>>           </tr>
>>>>           </#if>
>>>> +            <#if "SALES_ORDER" == orderHeader.orderTypeId>
>>>> +              <form action="setOrderReservationPriority"
>>>> method="post" name="setOrderReservationPriority">
>>>> +                <input type = "hidden" name="orderId" value="$
>>>> {orderId}"/>
>>>> +                <tr><td colspan="3"><hr/></td></tr>
>>>> +                <tr>
>>>> +                  <td align="right" valign="top" width="15%"
>>>> class="label">&nbsp;${uiLabelMap.FormFieldTitle_priority}</td>
>>>> +                  <td width="5%">&nbsp;</td>
>>>> +                  <td valign="top" width="80%">
>>>> +                    <select name="priority">
>>>> +                      <option value="1">$
>>>> {uiLabelMap.OrderPriorityHigh}</option>
>>>> +                      <option value="2"selected>$
>>>> {uiLabelMap.OrderPriorityNormal}</option>
>>>> +                      <option value="3">$
>>>> {uiLabelMap.OrderPriorityLow}</option>
>>>> +                    </select>
>>>> +                    <input type="submit" class="smallSubmit"
>>>> value="${uiLabelMap.FormFieldTitle_reserveInventory}"/>
>>>> +                  </td>
>>>> +                </tr>
>>>> +              </form>
>>>> +            </#if>
>>>>       </table>
>>>>   </div>
>>>> </div>
>>>> \ No newline at end of file
>>>>
>>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/
>>>> product/ inventory/InventoryReserveServices.xml
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>> inventory/InventoryReserveServices.xml (original)
>>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>> inventory/InventoryReserveServices.xml Thu Mar 19 13:58:08 2009
>>>> @@ -201,6 +201,7 @@
>>>>                           <set from-
>>>> field="parameters.reservedDatetime"
>>>> field="reserveOisgirMap.reservedDatetime"/>
>>>>                           <set from-field="promisedDatetime"
>>>> field="reserveOisgirMap.promisedDatetime"/>
>>>>                           <set from-
>>>> field="parameters.sequenceId"
>>>> field="reserveOisgirMap.sequenceId"/>
>>>> +                            <set from-
>>>> field="parameters.priority"  field="reserveOisgirMap.priority"/>
>>>>                           <call-service service-
>>>> name="reserveOrderItemInventory" in-map-name="reserveOisgirMap"/>
>>>>                           <clear-field field="reserveOisgirMap"/>
>>>>                       <else>
>>>> @@ -444,6 +445,7 @@
>>>>                       <set from-
>>>> field="parameters.reservedDatetime"
>>>> field="reserveOisgirMap.reservedDatetime"/>
>>>>                       <set from-field="parameters.deductAmount"
>>>> field="reserveOisgirMap.quantity" type="BigDecimal"/>
>>>>                       <set from-field="promisedDatetime"
>>>> field="reserveOisgirMap.promisedDatetime"/>
>>>> +                        <set from-field="parameters.priority"
>>>> field="reserveOisgirMap.priority"/>
>>>>                       <if-not-empty field="parameters.sequenceId">
>>>>                           <set
>>>> field="reserveOisgirMap.sequenceId"  from-
>>>> field="parameters.sequenceId"/>
>>>>                       </if-not-empty>
>>>> @@ -498,6 +500,7 @@
>>>>           <set-nonpk-fields map="parameters" value-
>>>> field="newOisgirEntity"/>
>>>>           <now-timestamp field="nowTimestamp"/>
>>>>           <set from-field="nowTimestamp"
>>>> field="newOisgirEntity.createdDatetime"/>
>>>> +            <set from-field="parameters.priority"
>>>> field="newOisgirEntity.priority"/>
>>>>           <if-empty field="newOisgirEntity.reservedDatetime">
>>>>               <set from-field="nowTimestamp"
>>>> field="newOisgirEntity.reservedDatetime"/>
>>>>           </if-empty>
>>>>
>>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/
>>>> product/ inventory/InventoryServices.xml
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>> inventory/InventoryServices.xml (original)
>>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/
>>>> inventory/InventoryServices.xml Thu Mar 19 13:58:08 2009
>>>> @@ -814,6 +814,7 @@
>>>>                   </condition-list>
>>>>               </condition-list>
>>>>           </condition-list>
>>>> +            <order-by field-name="priority"/>
>>>>           <order-by field-name="currentPromisedDate"/>
>>>>           <order-by field-name="reservedDatetime"/>
>>>>           <order-by field-name="sequenceId"/>
>>>> @@ -875,6 +876,9 @@
>>>>                   <log level="verbose" message="Adding $
>>>> {oisgir.orderId} to touchedOrderIdMap"/>
>>>>               </if-compare>
>>>>           </if-not-empty>
>>>> +            <entity-one entity-name="OrderHeader" value-
>>>> field="orderHeader">
>>>> +                <field-map field-name="orderId" from-
>>>> field="oisgir.orderId"/>
>>>> +            </entity-one>
>>>>
>>>>           <!-- require inventory is N because it had to be N to
>>>> begin with to have a negative ATP -->
>>>>           <clear-field field="resMap"/>
>>>> @@ -888,6 +892,7 @@
>>>>           <set field="resMap.shipGroupSeqId" from-
>>>> field="oisgir.shipGroupSeqId"/>
>>>>           <set field="resMap.sequenceId" from-
>>>> field="oisgir.sequenceId"/>
>>>>           <set field="resMap.facilityId" from-
>>>> field="parameters.facilityId"/>
>>>> +            <set field="resMap.priority" from-
>>>> field="orderHeader.priority"/>
>>>>           <log level="info" message="Re-reserving product [$
>>>> {resMap.productId}] for order item [${resMap.orderId}:$
>>>> {resMap.orderItemSeqId}] quantity [${resMap.quantity}]; facility
>>>> [$ {parameters.facilityId}]"/>
>>>>           <call-service service-
>>>> name="reserveProductInventoryByFacility" in-map-name="resMap"/>
>>>>       </iterate>
>>>> @@ -1249,5 +1254,43 @@
>>>>           </if-compare>
>>>>       </if-not-empty>
>>>>   </simple-method>
>>>> -
>>>> -</simple-methods>
>>>> +    <simple-method method-name="setOrderReservationPriority"
>>>> short- description="Sets priority of an order for Inventory
>>>> Reservation,  orders with HIGH priority would be served first.">
>>>> +        <set field="orderId" from-field="parameters.orderId"/>
>>>> +        <entity-one entity-name="OrderHeader" value-
>>>> field="orderHeader">
>>>> +            <field-map field-name="orderId" from-field="orderId"/>
>>>> +        </entity-one>
>>>> +        <set field="priority" from-field="parameters.priority"/>
>>>> +        <if-empty field="priority">
>>>> +            <entity-and list="oisgirs" entity-
>>>> name="OrderItemShipGrpInvRes">
>>>> +                <field-map field-name="orderId" from-
>>>> field="orderId"/>
>>>> +            </entity-and>
>>>> +            <iterate entry="oisgir" list="oisgirs">
>>>> +                <set field="oisgir.priority" default-value="2"/>
>>>> +                <store-value value-field="oisgir"/>
>>>> +            </iterate>
>>>> +            <set field="orderHeader.priority" default-value="2"/>
>>>> +            <store-value value-field="orderHeader"/>
>>>> +        <else>
>>>> +            <set field="orderHeader.priority" from-
>>>> field="priority"/>
>>>> +            <store-value value-field="orderHeader"/>
>>>> +            <entity-and list="oisgirs" entity-
>>>> name="OrderItemShipGrpInvRes">
>>>> +                <field-map field-name="orderId" from-
>>>> field="orderId"/>
>>>> +            </entity-and>
>>>> +            <iterate entry="oisgir" list="oisgirs">
>>>> +                <set field="oisgir.priority" from-
>>>> field="priority"/>
>>>> +                <store-value value-field="oisgir"/>
>>>> +                <clear-field field="oisgir"/>
>>>> +            </iterate>
>>>> +            <entity-condition entity-
>>>> name="OrderItemAndShipGrpInvResAndItem" list="oisgirais">
>>>> +                <condition-expr field-name="orderId"
>>>> operator="equals" from-field="orderId"/>
>>>> +            </entity-condition>
>>>> +            <iterate entry="oisgir" list="oisgirais">
>>>> +                <set
>>>> field="reassignInventoryReservationsCtx.productId" from-
>>>> field="oisgir.productId"/>
>>>> +                <set
>>>> field="reassignInventoryReservationsCtx.facilityId" from-
>>>> field="oisgir.facilityId"/>
>>>> +                <call-service service-
>>>> name="reassignInventoryReservations" in-map-
>>>> name="reassignInventoryReservationsCtx"/>
>>>> +                <clear-field
>>>> field="reassignInventoryReservationsCtx"/>
>>>> +            </iterate>
>>>> +        </else>
>>>> +        </if-empty>
>>>> +    </simple-method>
>>>> +</simple-methods>
>>>> \ No newline at end of file
>>>>
>>>> Modified: ofbiz/trunk/applications/product/servicedef/
>>>> services_facility.xml
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services_facility.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/product/servicedef/
>>>> services_facility.xml (original)
>>>> +++ ofbiz/trunk/applications/product/servicedef/
>>>> services_facility.xml Thu Mar 19 13:58:08 2009
>>>> @@ -262,6 +262,7 @@
>>>>       <attribute name="facilityId" type="String" mode="IN"
>>>> optional="false"/>
>>>>       <attribute name="fromDate" type="Timestamp" mode="IN"
>>>> optional="true"/>
>>>>       <attribute name="noLongerOnBackOrderIdSet" type="Set"
>>>> mode="OUT" optional="true"/>
>>>> +        <attribute name="priority" type="String" mode="IN"
>>>> optional="true"/>
>>>>   </service>
>>>>   <service name="balanceOrderItemsWithNegativeReservations"
>>>> engine="simple"
>>>>               location="component://product/script/org/ofbiz/
>>>> product/inventory/InventoryServices.xml"
>>>> invoke="balanceOrderItemsWithNegativeReservations">
>>>> @@ -319,6 +320,7 @@
>>>>       <attribute name="reserveOrderEnumId" type="String"
>>>> mode="IN"  optional="true"/>
>>>>       <attribute name="sequenceId" type="Long" mode="IN"
>>>> optional="true"/>
>>>>       <attribute name="quantityNotReserved" type="BigDecimal"
>>>> mode="OUT" optional="false"/>
>>>> +        <attribute name="priority" type="String" mode="IN"
>>>> optional="true"/>
>>>>   </service>
>>>>   <service name="reserveProductInventoryByContainer"
>>>> engine="simple"
>>>>               location="component://product/script/org/ofbiz/
>>>> product/inventory/InventoryReserveServices.xml"
>>>> invoke="reserveProductInventory" auth="true">
>>>> @@ -345,6 +347,7 @@
>>>>       <auto-attributes mode="IN" entity-
>>>> name="OrderItemShipGrpInvRes" include="nonpk" optional="true">
>>>>           <exclude field-name="createdDatetime"/>
>>>>       </auto-attributes>
>>>> +        <attribute name="priority" type="String" mode="IN"
>>>> optional="true"/>
>>>>       <override name="quantity" optional="false"/>
>>>>   </service>
>>>>   <service name="cancelOrderInventoryReservation" engine="simple"
>>>> @@ -873,4 +876,10 @@
>>>>       <permission-service service-
>>>> name="facilityGenericPermission"  main-action="DELETE"/>
>>>>       <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>   </service>
>>>> +    <service name="setOrderReservationPriority" engine="simple"
>>>> invoke="setOrderReservationPriority"
>>>> +                location="component://product/script/org/ofbiz/
>>>> product/inventory/InventoryServices.xml"  auth="true">
>>>> +        <description>set order priority</description>
>>>> +        <attribute name="priority" mode="IN" type="String"
>>>> optional="true"/>
>>>> +        <attribute name="orderId" mode="IN" type="String"
>>>> optional="false"/>
>>>> +    </service>
>>>> </services>
>>>>
>>>> Modified: ofbiz/trunk/specialpurpose/ecommerce/data/
>>>> DemoOrderTestData.xml
>>>> URL: 
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =
>>>> =
>>>> =
>>>> ===================================================================
>>>> --- ofbiz/trunk/specialpurpose/ecommerce/data/
>>>> DemoOrderTestData.xml  (original)
>>>> +++ ofbiz/trunk/specialpurpose/ecommerce/data/
>>>> DemoOrderTestData.xml  Thu Mar 19 13:58:08 2009
>>>> @@ -20,7 +20,7 @@
>>>>
>>>> <entity-engine-xml>
>>>>   <!--Demo data for a sales order -->
>>>> -    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"
>>>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"
>>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23
>>>> 16:49:27.392" visitId="10002" statusId="ORDER_APPROVED"
>>>> createdBy="admin" currencyUom="USD" webSiteId="OrderEntry"
>>>> productStoreId="9000" remainingSubTotal="12.45"
>>>> grandTotal="12.45" lastUpdatedStamp="2008-04-23 16:49:33.196"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"
>>>> createdStamp="2008-04-23 16:49:27.716"
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>> +    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"
>>>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"
>>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23
>>>> 16:49:27.392" priority="2" visitId="10002"
>>>> statusId="ORDER_APPROVED" createdBy="admin" currencyUom="USD"
>>>> webSiteId="OrderEntry" productStoreId="9000"
>>>> remainingSubTotal="12.45" grandTotal="12.45"
>>>> lastUpdatedStamp="2008-04-23 16:49:33.196"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"
>>>> createdStamp="2008-04-23 16:49:27.716"
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>   <OrderItem orderId="DEMO10090" orderItemSeqId="00001"
>>>> orderItemTypeId="PRODUCT_ORDER_ITEM" productId="GZ-2644"
>>>> prodCatalogId="DemoCatalog" isPromo="N" quantity="1.0"
>>>> selectedAmount="0.0" unitPrice="38.4" unitListPrice="48.0"
>>>> isModifiedPrice="N" itemDescription="Round Gizmo"
>>>> correspondingPoId="" statusId="ITEM_APPROVED"
>>>> lastUpdatedStamp="2008-04-23 16:49:33.514"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.484"
>>>> createdStamp="2008-04-23 16:49:28.128" createdTxStamp="2008-04-23
>>>> 16:49:27.349"/>
>>>>   <OrderItemPriceInfo orderItemPriceInfoId="9000"
>>>> orderId="DEMO10090" orderItemSeqId="00001"
>>>> productPriceRuleId="9000" productPriceActionSeqId="01"
>>>> modifyAmount="-9.600000000000001"
>>>> description="[PRODUCT_CATEGORY_IDIsPROMOTIONS] [list:
>>>> 48.0;avgCost: 48.0;margin:0.0] [type:PRICE_POL]"
>>>> lastUpdatedStamp="2008-04-23  16:49:28.682"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>> createdStamp="2008-04-23 16:49:28.682"
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>
>>>> @@ -36,7 +36,7 @@
>>>>
>>>>   <OrderItemShipGroup orderId="DEMO10090" shipGroupSeqId="00001"
>>>> shipmentMethodTypeId="NEXT_DAY" carrierPartyId="UPS"
>>>> carrierRoleTypeId="CARRIER" contactMechId="9015" maySplit="N"
>>>> giftMessage="" isGift="N" lastUpdatedStamp="2008-04-23
>>>> 16:49:28.412" lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>> createdStamp="2008-04-23 16:49:28.412" createdTxStamp="2008-04-23
>>>> 16:49:27.349"/>
>>>>   <OrderItemShipGroupAssoc orderId="DEMO10090"
>>>> orderItemSeqId="00001" shipGroupSeqId="00001" quantity="1.0"
>>>> lastUpdatedStamp="2008-04-23 16:49:28.454"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>> createdStamp="2008-04-23 16:49:28.454"
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>> -    <OrderItemShipGrpInvRes orderId="DEMO10090"
>>>> shipGroupSeqId="00001" orderItemSeqId="00001"
>>>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"
>>>> quantity="1.0" quantityNotAvailable="0.0"
>>>> reservedDatetime="2008-04-23 16:49:31.474"
>>>> createdDatetime="2008-04-23 16:49:31.474"
>>>> promisedDatetime="2008-05-08 16:49:27.392"
>>>> lastUpdatedStamp="2008-04-23 16:49:31.474"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23
>>>> 16:49:27.349"/>
>>>> +    <OrderItemShipGrpInvRes orderId="DEMO10090"
>>>> shipGroupSeqId="00001" orderItemSeqId="00001"
>>>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"
>>>> quantity="1.0" quantityNotAvailable="0.0"
>>>> reservedDatetime="2008-04-23 16:49:31.474"
>>>> createdDatetime="2008-04-23 16:49:31.474"
>>>> promisedDatetime="2008-05-08 16:49:27.392" priority="2"
>>>> lastUpdatedStamp="2008-04-23 16:49:31.474"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23
>>>> 16:49:27.349"/>
>>>>
>>>>   <OrderAdjustment orderAdjustmentId="9000"
>>>> orderAdjustmentTypeId="PROMOTION_ADJUSTMENT" orderId="DEMO10090"
>>>> orderItemSeqId="00001" shipGroupSeqId="_NA_" amount="-38.4"
>>>> productPromoId="9016" productPromoRuleId="01"
>>>> productPromoActionSeqId="01" createdDate="2008-04-23
>>>> 16:49:27.866"  createdByUserLogin="admin"
>>>> lastUpdatedStamp="2008-04-23 16:49:28.271"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>> createdStamp="2008-04-23 16:49:28.271" createdTxStamp="2008-04-23
>>>> 16:49:27.349"/>
>>>>   <OrderAdjustment orderAdjustmentId="9001"
>>>> orderAdjustmentTypeId="SHIPPING_CHARGES" orderId="DEMO10090"
>>>> orderItemSeqId="_NA_" shipGroupSeqId="00001" amount="12.45"
>>>> createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin"
>>>> lastUpdatedStamp="2008-04-23 16:49:28.431"
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"
>>>> createdStamp="2008-04-23 16:49:28.431"
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>
>>>
>>>
>>
>
>




Re: svn commit: r756001 - in /ofbiz/trunk: applications/order/config/ applications/order/entitydef/ applications/order/servicedef/ applications/order/webapp/ordermgr/WEB-INF/ applications/order/webapp/ordermgr/order/ applications/product/script/org/ofbiz/p

Posted by Vikas Mayur <vi...@hotwaxmedia.com>.
no problemo!

Uploaded the patch at https://issues.apache.org/jira/browse/OFBIZ-2245

Vikas

On Mar 20, 2009, at 1:19 AM, Jacques Le Roux wrote:

> Sorry Vikas,
>
> But as you see I'd have to wrap the lines :/
> And I have more interesting things to do :p
> Just add a patch to the related Jira issue please
>
> Jacques
>
> From: "Vikas Mayur" <vi...@hotwaxmedia.com>
>> Thanks Jacques!
>>
>> I have done few changes in rev. 756134 and to make them work,  
>> would  require to commit following changes. I would appreciate if  
>> you can  take a lead :) This is a small change, so not opening any  
>> new jira  issue.
>>
>> Index: framework/common/config/CommonUiLabels.xml
>> ===================================================================
>> --- framework/common/config/CommonUiLabels.xml (revision 756129)
>> +++ framework/common/config/CommonUiLabels.xml (working copy)
>> @@ -2877,6 +2877,9 @@
>>         <value xml:lang="zh">隐藏域</value>
>>         <value xml:lang="zh_CN">隐藏字段</value>
>>     </property>
>> +    <property key="CommonHigh">
>> +        <value xml:lang="en">High</value>
>> +    </property>
>>     <property key="CommonHome">
>>         <value xml:lang="en">Home</value>
>>         <value xml:lang="fr">Point de départ</value>
>> @@ -3445,6 +3448,9 @@
>>         <value xml:lang="en">http://whitepages.com/find_person_results.pl?fid=p&amp;ac=$ 
>>  {telecomNumber.areaCode?if_exists}&amp;s=&amp;p=$  
>> {telecomNumber.contactNumber?if_exists}&amp;pt=b&amp;x=40&amp;y=9</  
>> value>
>>         <value xml:lang="fr">http://www.pagesjaunes.fr/quidonc/aquiestcenumero.do 
>>  </value>
>>     </property>
>> +    <property key="CommonLow">
>> +        <value xml:lang="en">Low</value>
>> +    </property>
>>     <property key="CommonMain">
>>         <value xml:lang="ar">الرئيسية</ 
>> value>
>>         <value xml:lang="cs">Úvodní stránka</value>
>> @@ -3988,6 +3994,9 @@
>>         <value xml:lang="zh">Âê¶</value>
>>         <value xml:lang="zh_CN">不</value>
>>     </property>
>> +    <property key="CommonNormal">
>> +        <value xml:lang="en">Normal</value>
>> +    </property>
>>     <property key="CommonNoAccess">
>>         <value xml:lang="en">No Access to this function, contact   
>> your system administrator if you feel this is an error</value>
>>         <value xml:lang="es">Acceso no autorizado. Póngase en   
>> contacto con su administrador de sistemas si piensa que esto es un   
>> error.</value>
>>
>>
>> Thanks,
>> Vikas
>>
>>
>> On Mar 19, 2009, at 7:39 PM, Jacques Le Roux wrote:
>>
>>> +1 for CommonHigh, CommonLow etc.
>>>
>>> Jacques
>>>
>>> From: <mo...@apache.org>
>>>> Author: mor
>>>> Date: Thu Mar 19 13:58:08 2009
>>>> New Revision: 756001
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=756001&view=rev
>>>> Log:
>>>> Another new feature from Pranay and Co.
>>>> Now Order reservation can be changed manually by setting  
>>>> priority  (new field priority on OrderHeader and  
>>>> OrderItemShipGrpInvRes) for  the order. This way few orders can be
>>>> fulfilled while waiting for sufficient inventory to fulfill  
>>>> large  orders.
>>>>
>>>> Applied patch from OFBIZ-2224 (https://issues.apache.org/jira/browse/OFBIZ-2224 
>>>>  ).
>>>> Thanks Pratik Khera and Pranay Pandey.
>>>>
>>>> i18n Note: This commit has three new labels (IMO, they can be  
>>>> moved  to common component with names something like CommonHigh,  
>>>> CommonLow  etc.)
>>>>
>>>> Pratik, I have done minor changes in the service and they are
>>>> 1) Used <if-empty field=""> instead of <if> <condition> <if- 
>>>> empty>  as this makes the logic more clearer.
>>>> 2) Removed <condition-list> because their is only one expression  
>>>> in  the <entity-condition>
>>>> 3) Added <clear-field> in the iteration, to make sure that always  
>>>> a  new value will be set in the map that is passed to a service  
>>>> and  <store-value>
>>>>
>>>>
>>>> Modified:
>>>>  ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>>  ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>>  ofbiz/trunk/applications/order/servicedef/secas.xml
>>>>  ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/  
>>>> controller.xml
>>>>  ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
>>>>  ofbiz/trunk/applications/product/script/org/ofbiz/product/  
>>>> inventory/InventoryReserveServices.xml
>>>>  ofbiz/trunk/applications/product/script/org/ofbiz/product/  
>>>> inventory/InventoryServices.xml
>>>>  ofbiz/trunk/applications/product/servicedef/services_facility.xml
>>>>  ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
>>>>
>>>> Modified: ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderUiLabels.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/config/OrderUiLabels.xml   
>>>> (original)
>>>> +++ ofbiz/trunk/applications/order/config/OrderUiLabels.xml Thu  
>>>> Mar  19 13:58:08 2009
>>>> @@ -6271,6 +6271,15 @@
>>>>       <value xml:lang="zh">å
>>> ¬å¼€ä¾¿ç¬º (已打印)</value>
>>>>       <value xml:lang="zh_CN">å
>>> ¬å¼€æ³¨é‡Š (可打印)</value>
>>>>   </property>
>>>> +    <property key="OrderPriorityHigh">
>>>> +        <value xml:lang="en">High</value>
>>>> +    </property>
>>>> +    <property key="OrderPriorityLow">
>>>> +        <value xml:lang="en">Low</value>
>>>> +    </property>
>>>> +    <property key="OrderPriorityNormal">
>>>> +        <value xml:lang="en">Normal</value>
>>>> +    </property>
>>>>   <property key="OrderProcessOrder">
>>>>       <value xml:lang="de">Auftrag verarbeiten</value>
>>>>       <value xml:lang="en">Process Order</value>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/entitydef/entitymodel.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/entitydef/entitymodel.xml   
>>>> (original)
>>>> +++ ofbiz/trunk/applications/order/entitydef/entitymodel.xml Thu   
>>>> Mar 19 13:58:08 2009
>>>> @@ -337,6 +337,7 @@
>>>>     <field name="externalId" type="id"></field>
>>>>     <field name="salesChannelEnumId" type="id"></field>
>>>>     <field name="orderDate" type="date-time"></field>
>>>> +      <field name="priority" type="indicator"><description>Sets   
>>>> priority for Inventory Reservation</description></field>
>>>>     <field name="entryDate" type="date-time"></field>
>>>>     <field name="visitId" type="id"></field>
>>>>     <field name="statusId" type="id"></field>
>>>> @@ -888,6 +889,7 @@
>>>>     <field name="createdDatetime" type="date-time"></field>
>>>>     <field name="promisedDatetime" type="date-time"></field>
>>>>     <field name="currentPromisedDate" type="date-time"></field>
>>>> +      <field name="priority" type="indicator"><description>Sets   
>>>> priority for Inventory Reservation</description></field>
>>>>     <field name="sequenceId" type="numeric"></field>
>>>>     <field name="oldPickStartDate" col-name="PICK_START_DATE"   
>>>> type="date-time"></field> <!-- this is no longer used and is  
>>>> here  for migration or whatever if needed -->
>>>>     <prim-key field="orderId"/>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/servicedef/secas.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/secas.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/servicedef/secas.xml (original)
>>>> +++ ofbiz/trunk/applications/order/servicedef/secas.xml Thu Mar  
>>>> 19  13:58:08 2009
>>>> @@ -36,6 +36,9 @@
>>>>       <condition field-name="orderTypeId" operator="equals"   
>>>> value="PURCHASE_ORDER"/>
>>>>       <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>>   </eca>
>>>> +    <eca service="storeOrder" event="return">
>>>> +        <action service="setOrderReservationPriority"  
>>>> mode="sync"/>
>>>> +    </eca>
>>>>   <eca service="updateOrderItems" event="commit">
>>>>       <condition field-name="orderTypeId" operator="equals"   
>>>> value="PURCHASE_ORDER"/>
>>>>       <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/  
>>>> controller.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/  
>>>> controller.xml (original)
>>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/  
>>>> controller.xml Thu Mar 19 13:58:08 2009
>>>> @@ -1457,7 +1457,12 @@
>>>>       <response name="success" type="request-redirect"   
>>>> value="orderview"/>
>>>>       <response name="error" type="request-redirect"   
>>>> value="orderview"/>
>>>>   </request-map>
>>>> -
>>>> +    <request-map uri="setOrderReservationPriority">
>>>> +        <security https="true" auth="true"/>
>>>> +        <event type="service"  
>>>> invoke="setOrderReservationPriority"/>
>>>> +        <response name="success" type="view" value="orderview"/>
>>>> +        <response name="error" type="view" value="orderview"/>
>>>> +    </request-map>
>>>>   <!-- Lookup request mappings -->
>>>>   <request-map uri="LookupPerson"><security https="true"   
>>>> auth="true"/><response name="success" type="view"  
>>>> value="LookupPerson"/></request-map>
>>>>   <request-map uri="LookupPartyGroup"><security https="true"   
>>>> auth="true"/><response name="success" type="view"  
>>>> value="LookupPartyGroup"/></request-map>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/  
>>>> orderinfo.ftl
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/order/  
>>>> orderinfo.ftl (original)
>>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/order/  
>>>> orderinfo.ftl Thu Mar 19 13:58:08 2009
>>>> @@ -179,6 +179,24 @@
>>>>             </td>
>>>>           </tr>
>>>>           </#if>
>>>> +            <#if "SALES_ORDER" == orderHeader.orderTypeId>
>>>> +              <form action="setOrderReservationPriority"   
>>>> method="post" name="setOrderReservationPriority">
>>>> +                <input type = "hidden" name="orderId" value="$  
>>>> {orderId}"/>
>>>> +                <tr><td colspan="3"><hr/></td></tr>
>>>> +                <tr>
>>>> +                  <td align="right" valign="top" width="15%"   
>>>> class="label">&nbsp;${uiLabelMap.FormFieldTitle_priority}</td>
>>>> +                  <td width="5%">&nbsp;</td>
>>>> +                  <td valign="top" width="80%">
>>>> +                    <select name="priority">
>>>> +                      <option value="1">$  
>>>> {uiLabelMap.OrderPriorityHigh}</option>
>>>> +                      <option value="2"selected>$  
>>>> {uiLabelMap.OrderPriorityNormal}</option>
>>>> +                      <option value="3">$  
>>>> {uiLabelMap.OrderPriorityLow}</option>
>>>> +                    </select>
>>>> +                    <input type="submit" class="smallSubmit"   
>>>> value="${uiLabelMap.FormFieldTitle_reserveInventory}"/>
>>>> +                  </td>
>>>> +                </tr>
>>>> +              </form>
>>>> +            </#if>
>>>>       </table>
>>>>   </div>
>>>> </div>
>>>> \ No newline at end of file
>>>>
>>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/ 
>>>> product/ inventory/InventoryReserveServices.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/  
>>>> inventory/InventoryReserveServices.xml (original)
>>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/  
>>>> inventory/InventoryReserveServices.xml Thu Mar 19 13:58:08 2009
>>>> @@ -201,6 +201,7 @@
>>>>                           <set from-  
>>>> field="parameters.reservedDatetime"   
>>>> field="reserveOisgirMap.reservedDatetime"/>
>>>>                           <set from-field="promisedDatetime"   
>>>> field="reserveOisgirMap.promisedDatetime"/>
>>>>                           <set from- 
>>>> field="parameters.sequenceId"   
>>>> field="reserveOisgirMap.sequenceId"/>
>>>> +                            <set from- 
>>>> field="parameters.priority"  field="reserveOisgirMap.priority"/>
>>>>                           <call-service service-  
>>>> name="reserveOrderItemInventory" in-map-name="reserveOisgirMap"/>
>>>>                           <clear-field field="reserveOisgirMap"/>
>>>>                       <else>
>>>> @@ -444,6 +445,7 @@
>>>>                       <set from-  
>>>> field="parameters.reservedDatetime"   
>>>> field="reserveOisgirMap.reservedDatetime"/>
>>>>                       <set from-field="parameters.deductAmount"   
>>>> field="reserveOisgirMap.quantity" type="BigDecimal"/>
>>>>                       <set from-field="promisedDatetime"   
>>>> field="reserveOisgirMap.promisedDatetime"/>
>>>> +                        <set from-field="parameters.priority"   
>>>> field="reserveOisgirMap.priority"/>
>>>>                       <if-not-empty field="parameters.sequenceId">
>>>>                           <set  
>>>> field="reserveOisgirMap.sequenceId"  from- 
>>>> field="parameters.sequenceId"/>
>>>>                       </if-not-empty>
>>>> @@ -498,6 +500,7 @@
>>>>           <set-nonpk-fields map="parameters" value-  
>>>> field="newOisgirEntity"/>
>>>>           <now-timestamp field="nowTimestamp"/>
>>>>           <set from-field="nowTimestamp"   
>>>> field="newOisgirEntity.createdDatetime"/>
>>>> +            <set from-field="parameters.priority"   
>>>> field="newOisgirEntity.priority"/>
>>>>           <if-empty field="newOisgirEntity.reservedDatetime">
>>>>               <set from-field="nowTimestamp"   
>>>> field="newOisgirEntity.reservedDatetime"/>
>>>>           </if-empty>
>>>>
>>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/ 
>>>> product/ inventory/InventoryServices.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/  
>>>> inventory/InventoryServices.xml (original)
>>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/  
>>>> inventory/InventoryServices.xml Thu Mar 19 13:58:08 2009
>>>> @@ -814,6 +814,7 @@
>>>>                   </condition-list>
>>>>               </condition-list>
>>>>           </condition-list>
>>>> +            <order-by field-name="priority"/>
>>>>           <order-by field-name="currentPromisedDate"/>
>>>>           <order-by field-name="reservedDatetime"/>
>>>>           <order-by field-name="sequenceId"/>
>>>> @@ -875,6 +876,9 @@
>>>>                   <log level="verbose" message="Adding $  
>>>> {oisgir.orderId} to touchedOrderIdMap"/>
>>>>               </if-compare>
>>>>           </if-not-empty>
>>>> +            <entity-one entity-name="OrderHeader" value-  
>>>> field="orderHeader">
>>>> +                <field-map field-name="orderId" from-  
>>>> field="oisgir.orderId"/>
>>>> +            </entity-one>
>>>>
>>>>           <!-- require inventory is N because it had to be N to   
>>>> begin with to have a negative ATP -->
>>>>           <clear-field field="resMap"/>
>>>> @@ -888,6 +892,7 @@
>>>>           <set field="resMap.shipGroupSeqId" from-  
>>>> field="oisgir.shipGroupSeqId"/>
>>>>           <set field="resMap.sequenceId" from-  
>>>> field="oisgir.sequenceId"/>
>>>>           <set field="resMap.facilityId" from-  
>>>> field="parameters.facilityId"/>
>>>> +            <set field="resMap.priority" from-  
>>>> field="orderHeader.priority"/>
>>>>           <log level="info" message="Re-reserving product [$  
>>>> {resMap.productId}] for order item [${resMap.orderId}:$  
>>>> {resMap.orderItemSeqId}] quantity [${resMap.quantity}]; facility  
>>>> [$ {parameters.facilityId}]"/>
>>>>           <call-service service-  
>>>> name="reserveProductInventoryByFacility" in-map-name="resMap"/>
>>>>       </iterate>
>>>> @@ -1249,5 +1254,43 @@
>>>>           </if-compare>
>>>>       </if-not-empty>
>>>>   </simple-method>
>>>> -
>>>> -</simple-methods>
>>>> +    <simple-method method-name="setOrderReservationPriority"  
>>>> short- description="Sets priority of an order for Inventory  
>>>> Reservation,  orders with HIGH priority would be served first.">
>>>> +        <set field="orderId" from-field="parameters.orderId"/>
>>>> +        <entity-one entity-name="OrderHeader" value-  
>>>> field="orderHeader">
>>>> +            <field-map field-name="orderId" from-field="orderId"/>
>>>> +        </entity-one>
>>>> +        <set field="priority" from-field="parameters.priority"/>
>>>> +        <if-empty field="priority">
>>>> +            <entity-and list="oisgirs" entity-  
>>>> name="OrderItemShipGrpInvRes">
>>>> +                <field-map field-name="orderId" from-  
>>>> field="orderId"/>
>>>> +            </entity-and>
>>>> +            <iterate entry="oisgir" list="oisgirs">
>>>> +                <set field="oisgir.priority" default-value="2"/>
>>>> +                <store-value value-field="oisgir"/>
>>>> +            </iterate>
>>>> +            <set field="orderHeader.priority" default-value="2"/>
>>>> +            <store-value value-field="orderHeader"/>
>>>> +        <else>
>>>> +            <set field="orderHeader.priority" from-  
>>>> field="priority"/>
>>>> +            <store-value value-field="orderHeader"/>
>>>> +            <entity-and list="oisgirs" entity-  
>>>> name="OrderItemShipGrpInvRes">
>>>> +                <field-map field-name="orderId" from-  
>>>> field="orderId"/>
>>>> +            </entity-and>
>>>> +            <iterate entry="oisgir" list="oisgirs">
>>>> +                <set field="oisgir.priority" from- 
>>>> field="priority"/>
>>>> +                <store-value value-field="oisgir"/>
>>>> +                <clear-field field="oisgir"/>
>>>> +            </iterate>
>>>> +            <entity-condition entity-  
>>>> name="OrderItemAndShipGrpInvResAndItem" list="oisgirais">
>>>> +                <condition-expr field-name="orderId"   
>>>> operator="equals" from-field="orderId"/>
>>>> +            </entity-condition>
>>>> +            <iterate entry="oisgir" list="oisgirais">
>>>> +                <set   
>>>> field="reassignInventoryReservationsCtx.productId" from-  
>>>> field="oisgir.productId"/>
>>>> +                <set   
>>>> field="reassignInventoryReservationsCtx.facilityId" from-  
>>>> field="oisgir.facilityId"/>
>>>> +                <call-service service-  
>>>> name="reassignInventoryReservations" in-map-  
>>>> name="reassignInventoryReservationsCtx"/>
>>>> +                <clear-field   
>>>> field="reassignInventoryReservationsCtx"/>
>>>> +            </iterate>
>>>> +        </else>
>>>> +        </if-empty>
>>>> +    </simple-method>
>>>> +</simple-methods>
>>>> \ No newline at end of file
>>>>
>>>> Modified: ofbiz/trunk/applications/product/servicedef/  
>>>> services_facility.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services_facility.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/applications/product/servicedef/  
>>>> services_facility.xml (original)
>>>> +++ ofbiz/trunk/applications/product/servicedef/  
>>>> services_facility.xml Thu Mar 19 13:58:08 2009
>>>> @@ -262,6 +262,7 @@
>>>>       <attribute name="facilityId" type="String" mode="IN"   
>>>> optional="false"/>
>>>>       <attribute name="fromDate" type="Timestamp" mode="IN"   
>>>> optional="true"/>
>>>>       <attribute name="noLongerOnBackOrderIdSet" type="Set"   
>>>> mode="OUT" optional="true"/>
>>>> +        <attribute name="priority" type="String" mode="IN"    
>>>> optional="true"/>
>>>>   </service>
>>>>   <service name="balanceOrderItemsWithNegativeReservations"   
>>>> engine="simple"
>>>>               location="component://product/script/org/ofbiz/  
>>>> product/inventory/InventoryServices.xml"  
>>>> invoke="balanceOrderItemsWithNegativeReservations">
>>>> @@ -319,6 +320,7 @@
>>>>       <attribute name="reserveOrderEnumId" type="String"  
>>>> mode="IN"  optional="true"/>
>>>>       <attribute name="sequenceId" type="Long" mode="IN"   
>>>> optional="true"/>
>>>>       <attribute name="quantityNotReserved" type="BigDecimal"   
>>>> mode="OUT" optional="false"/>
>>>> +        <attribute name="priority" type="String" mode="IN"    
>>>> optional="true"/>
>>>>   </service>
>>>>   <service name="reserveProductInventoryByContainer"  
>>>> engine="simple"
>>>>               location="component://product/script/org/ofbiz/  
>>>> product/inventory/InventoryReserveServices.xml"  
>>>> invoke="reserveProductInventory" auth="true">
>>>> @@ -345,6 +347,7 @@
>>>>       <auto-attributes mode="IN" entity-  
>>>> name="OrderItemShipGrpInvRes" include="nonpk" optional="true">
>>>>           <exclude field-name="createdDatetime"/>
>>>>       </auto-attributes>
>>>> +        <attribute name="priority" type="String" mode="IN"    
>>>> optional="true"/>
>>>>       <override name="quantity" optional="false"/>
>>>>   </service>
>>>>   <service name="cancelOrderInventoryReservation" engine="simple"
>>>> @@ -873,4 +876,10 @@
>>>>       <permission-service service- 
>>>> name="facilityGenericPermission"  main-action="DELETE"/>
>>>>       <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>   </service>
>>>> +    <service name="setOrderReservationPriority" engine="simple"   
>>>> invoke="setOrderReservationPriority"
>>>> +                location="component://product/script/org/ofbiz/  
>>>> product/inventory/InventoryServices.xml"  auth="true">
>>>> +        <description>set order priority</description>
>>>> +        <attribute name="priority" mode="IN" type="String"   
>>>> optional="true"/>
>>>> +        <attribute name="orderId" mode="IN" type="String"   
>>>> optional="false"/>
>>>> +    </service>
>>>> </services>
>>>>
>>>> Modified: ofbiz/trunk/specialpurpose/ecommerce/data/  
>>>> DemoOrderTestData.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml?rev=756001&r1=756000&r2=756001&view=diff
>>>> = = = = = = = = =  
>>>> = 
>>>> = 
>>>> ===================================================================
>>>> --- ofbiz/trunk/specialpurpose/ecommerce/data/ 
>>>> DemoOrderTestData.xml  (original)
>>>> +++ ofbiz/trunk/specialpurpose/ecommerce/data/ 
>>>> DemoOrderTestData.xml  Thu Mar 19 13:58:08 2009
>>>> @@ -20,7 +20,7 @@
>>>>
>>>> <entity-engine-xml>
>>>>   <!--Demo data for a sales order -->
>>>> -    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"   
>>>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"  
>>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23   
>>>> 16:49:27.392" visitId="10002" statusId="ORDER_APPROVED"  
>>>> createdBy="admin" currencyUom="USD" webSiteId="OrderEntry"   
>>>> productStoreId="9000" remainingSubTotal="12.45"  
>>>> grandTotal="12.45" lastUpdatedStamp="2008-04-23 16:49:33.196"   
>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"   
>>>> createdStamp="2008-04-23 16:49:27.716"  
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>> +    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"   
>>>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"  
>>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23   
>>>> 16:49:27.392" priority="2" visitId="10002"   
>>>> statusId="ORDER_APPROVED" createdBy="admin" currencyUom="USD"   
>>>> webSiteId="OrderEntry" productStoreId="9000"   
>>>> remainingSubTotal="12.45" grandTotal="12.45"  
>>>> lastUpdatedStamp="2008-04-23 16:49:33.196"   
>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"   
>>>> createdStamp="2008-04-23 16:49:27.716"  
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>   <OrderItem orderId="DEMO10090" orderItemSeqId="00001"   
>>>> orderItemTypeId="PRODUCT_ORDER_ITEM" productId="GZ-2644"  
>>>> prodCatalogId="DemoCatalog" isPromo="N" quantity="1.0"   
>>>> selectedAmount="0.0" unitPrice="38.4" unitListPrice="48.0"  
>>>> isModifiedPrice="N" itemDescription="Round Gizmo"   
>>>> correspondingPoId="" statusId="ITEM_APPROVED"   
>>>> lastUpdatedStamp="2008-04-23 16:49:33.514"   
>>>> lastUpdatedTxStamp="2008-04-23 16:49:33.484"   
>>>> createdStamp="2008-04-23 16:49:28.128" createdTxStamp="2008-04-23  
>>>> 16:49:27.349"/>
>>>>   <OrderItemPriceInfo orderItemPriceInfoId="9000"   
>>>> orderId="DEMO10090" orderItemSeqId="00001"   
>>>> productPriceRuleId="9000" productPriceActionSeqId="01"   
>>>> modifyAmount="-9.600000000000001"   
>>>> description="[PRODUCT_CATEGORY_IDIsPROMOTIONS] [list: 
>>>> 48.0;avgCost: 48.0;margin:0.0] [type:PRICE_POL]"  
>>>> lastUpdatedStamp="2008-04-23  16:49:28.682"  
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"   
>>>> createdStamp="2008-04-23 16:49:28.682"  
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>
>>>> @@ -36,7 +36,7 @@
>>>>
>>>>   <OrderItemShipGroup orderId="DEMO10090" shipGroupSeqId="00001"   
>>>> shipmentMethodTypeId="NEXT_DAY" carrierPartyId="UPS"  
>>>> carrierRoleTypeId="CARRIER" contactMechId="9015" maySplit="N"   
>>>> giftMessage="" isGift="N" lastUpdatedStamp="2008-04-23  
>>>> 16:49:28.412" lastUpdatedTxStamp="2008-04-23 16:49:27.349"   
>>>> createdStamp="2008-04-23 16:49:28.412" createdTxStamp="2008-04-23  
>>>> 16:49:27.349"/>
>>>>   <OrderItemShipGroupAssoc orderId="DEMO10090"   
>>>> orderItemSeqId="00001" shipGroupSeqId="00001" quantity="1.0"  
>>>> lastUpdatedStamp="2008-04-23 16:49:28.454"   
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"   
>>>> createdStamp="2008-04-23 16:49:28.454"  
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>> -    <OrderItemShipGrpInvRes orderId="DEMO10090"   
>>>> shipGroupSeqId="00001" orderItemSeqId="00001"   
>>>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"   
>>>> quantity="1.0" quantityNotAvailable="0.0"   
>>>> reservedDatetime="2008-04-23 16:49:31.474"  
>>>> createdDatetime="2008-04-23 16:49:31.474"   
>>>> promisedDatetime="2008-05-08 16:49:27.392"   
>>>> lastUpdatedStamp="2008-04-23 16:49:31.474"   
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"   
>>>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23  
>>>> 16:49:27.349"/>
>>>> +    <OrderItemShipGrpInvRes orderId="DEMO10090"   
>>>> shipGroupSeqId="00001" orderItemSeqId="00001"   
>>>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"   
>>>> quantity="1.0" quantityNotAvailable="0.0"   
>>>> reservedDatetime="2008-04-23 16:49:31.474"  
>>>> createdDatetime="2008-04-23 16:49:31.474"   
>>>> promisedDatetime="2008-05-08 16:49:27.392" priority="2"   
>>>> lastUpdatedStamp="2008-04-23 16:49:31.474"   
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"   
>>>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23  
>>>> 16:49:27.349"/>
>>>>
>>>>   <OrderAdjustment orderAdjustmentId="9000"   
>>>> orderAdjustmentTypeId="PROMOTION_ADJUSTMENT" orderId="DEMO10090"  
>>>> orderItemSeqId="00001" shipGroupSeqId="_NA_" amount="-38.4"   
>>>> productPromoId="9016" productPromoRuleId="01"  
>>>> productPromoActionSeqId="01" createdDate="2008-04-23  
>>>> 16:49:27.866"  createdByUserLogin="admin"  
>>>> lastUpdatedStamp="2008-04-23 16:49:28.271"  
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"   
>>>> createdStamp="2008-04-23 16:49:28.271" createdTxStamp="2008-04-23  
>>>> 16:49:27.349"/>
>>>>   <OrderAdjustment orderAdjustmentId="9001"   
>>>> orderAdjustmentTypeId="SHIPPING_CHARGES" orderId="DEMO10090"  
>>>> orderItemSeqId="_NA_" shipGroupSeqId="00001" amount="12.45"   
>>>> createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin"  
>>>> lastUpdatedStamp="2008-04-23 16:49:28.431"   
>>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"   
>>>> createdStamp="2008-04-23 16:49:28.431"  
>>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>>
>>>
>>>
>>
>
>


Re: svn commit: r756001 - in /ofbiz/trunk: applications/order/config/ applications/order/entitydef/ applications/order/servicedef/ applications/order/webapp/ordermgr/WEB-INF/ applications/order/webapp/ordermgr/order/ applications/product/script/org/ofbiz/p

Posted by Jacques Le Roux <ja...@les7arts.com>.
Sorry Vikas,

But as you see I'd have to wrap the lines :/
And I have more interesting things to do :p
Just add a patch to the related Jira issue please

Jacques

From: "Vikas Mayur" <vi...@hotwaxmedia.com>
> Thanks Jacques!
>
> I have done few changes in rev. 756134 and to make them work, would  require to commit following changes. I would appreciate if 
> you can  take a lead :) This is a small change, so not opening any new jira  issue.
>
> Index: framework/common/config/CommonUiLabels.xml
> ===================================================================
> --- framework/common/config/CommonUiLabels.xml (revision 756129)
> +++ framework/common/config/CommonUiLabels.xml (working copy)
> @@ -2877,6 +2877,9 @@
>          <value xml:lang="zh">隐藏域</value>
>          <value xml:lang="zh_CN">隐藏字段</value>
>      </property>
> +    <property key="CommonHigh">
> +        <value xml:lang="en">High</value>
> +    </property>
>      <property key="CommonHome">
>          <value xml:lang="en">Home</value>
>          <value xml:lang="fr">Point de départ</value>
> @@ -3445,6 +3448,9 @@
>          <value xml:lang="en">http://whitepages.com/find_person_results.pl?fid=p&amp;ac=$ 
> {telecomNumber.areaCode?if_exists}&amp;s=&amp;p=$ {telecomNumber.contactNumber?if_exists}&amp;pt=b&amp;x=40&amp;y=9</ value>
>          <value xml:lang="fr">http://www.pagesjaunes.fr/quidonc/aquiestcenumero.do </value>
>      </property>
> +    <property key="CommonLow">
> +        <value xml:lang="en">Low</value>
> +    </property>
>      <property key="CommonMain">
>          <value xml:lang="ar">الرئيسية</value>
>          <value xml:lang="cs">Úvodní stránka</value>
> @@ -3988,6 +3994,9 @@
>          <value xml:lang="zh">Âê¶</value>
>          <value xml:lang="zh_CN">不</value>
>      </property>
> +    <property key="CommonNormal">
> +        <value xml:lang="en">Normal</value>
> +    </property>
>      <property key="CommonNoAccess">
>          <value xml:lang="en">No Access to this function, contact  your system administrator if you feel this is an error</value>
>          <value xml:lang="es">Acceso no autorizado. Póngase en  contacto con su administrador de sistemas si piensa que esto es 
> un  error.</value>
>
>
> Thanks,
> Vikas
>
>
> On Mar 19, 2009, at 7:39 PM, Jacques Le Roux wrote:
>
>> +1 for CommonHigh, CommonLow etc.
>>
>> Jacques
>>
>> From: <mo...@apache.org>
>>> Author: mor
>>> Date: Thu Mar 19 13:58:08 2009
>>> New Revision: 756001
>>>
>>> URL: http://svn.apache.org/viewvc?rev=756001&view=rev
>>> Log:
>>> Another new feature from Pranay and Co.
>>> Now Order reservation can be changed manually by setting priority  (new field priority on OrderHeader and 
>>> OrderItemShipGrpInvRes) for  the order. This way few orders can be
>>> fulfilled while waiting for sufficient inventory to fulfill large  orders.
>>>
>>> Applied patch from OFBIZ-2224 (https://issues.apache.org/jira/browse/OFBIZ-2224 ).
>>> Thanks Pratik Khera and Pranay Pandey.
>>>
>>> i18n Note: This commit has three new labels (IMO, they can be moved  to common component with names something like CommonHigh, 
>>> CommonLow  etc.)
>>>
>>> Pratik, I have done minor changes in the service and they are
>>> 1) Used <if-empty field=""> instead of <if> <condition> <if-empty>  as this makes the logic more clearer.
>>> 2) Removed <condition-list> because their is only one expression in  the <entity-condition>
>>> 3) Added <clear-field> in the iteration, to make sure that always a  new value will be set in the map that is passed to a 
>>> service and  <store-value>
>>>
>>>
>>> Modified:
>>>   ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>>   ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>>   ofbiz/trunk/applications/order/servicedef/secas.xml
>>>   ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ controller.xml
>>>   ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
>>>   ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryReserveServices.xml
>>>   ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryServices.xml
>>>   ofbiz/trunk/applications/product/servicedef/services_facility.xml
>>>   ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
>>>
>>> Modified: ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderUiLabels.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/order/config/OrderUiLabels.xml  (original)
>>> +++ ofbiz/trunk/applications/order/config/OrderUiLabels.xml Thu Mar  19 13:58:08 2009
>>> @@ -6271,6 +6271,15 @@
>>>        <value xml:lang="zh">å
>> ¬å¼€ä¾¿ç¬º (已打印)</value>
>>>        <value xml:lang="zh_CN">å
>> ¬å¼€æ³¨é‡Š (可打印)</value>
>>>    </property>
>>> +    <property key="OrderPriorityHigh">
>>> +        <value xml:lang="en">High</value>
>>> +    </property>
>>> +    <property key="OrderPriorityLow">
>>> +        <value xml:lang="en">Low</value>
>>> +    </property>
>>> +    <property key="OrderPriorityNormal">
>>> +        <value xml:lang="en">Normal</value>
>>> +    </property>
>>>    <property key="OrderProcessOrder">
>>>        <value xml:lang="de">Auftrag verarbeiten</value>
>>>        <value xml:lang="en">Process Order</value>
>>>
>>> Modified: ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/entitydef/entitymodel.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/order/entitydef/entitymodel.xml  (original)
>>> +++ ofbiz/trunk/applications/order/entitydef/entitymodel.xml Thu  Mar 19 13:58:08 2009
>>> @@ -337,6 +337,7 @@
>>>      <field name="externalId" type="id"></field>
>>>      <field name="salesChannelEnumId" type="id"></field>
>>>      <field name="orderDate" type="date-time"></field>
>>> +      <field name="priority" type="indicator"><description>Sets  priority for Inventory Reservation</description></field>
>>>      <field name="entryDate" type="date-time"></field>
>>>      <field name="visitId" type="id"></field>
>>>      <field name="statusId" type="id"></field>
>>> @@ -888,6 +889,7 @@
>>>      <field name="createdDatetime" type="date-time"></field>
>>>      <field name="promisedDatetime" type="date-time"></field>
>>>      <field name="currentPromisedDate" type="date-time"></field>
>>> +      <field name="priority" type="indicator"><description>Sets  priority for Inventory Reservation</description></field>
>>>      <field name="sequenceId" type="numeric"></field>
>>>      <field name="oldPickStartDate" col-name="PICK_START_DATE"  type="date-time"></field> <!-- this is no longer used and is 
>>> here  for migration or whatever if needed -->
>>>      <prim-key field="orderId"/>
>>>
>>> Modified: ofbiz/trunk/applications/order/servicedef/secas.xml
>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/secas.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/order/servicedef/secas.xml (original)
>>> +++ ofbiz/trunk/applications/order/servicedef/secas.xml Thu Mar 19  13:58:08 2009
>>> @@ -36,6 +36,9 @@
>>>        <condition field-name="orderTypeId" operator="equals"  value="PURCHASE_ORDER"/>
>>>        <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>    </eca>
>>> +    <eca service="storeOrder" event="return">
>>> +        <action service="setOrderReservationPriority" mode="sync"/>
>>> +    </eca>
>>>    <eca service="updateOrderItems" event="commit">
>>>        <condition field-name="orderTypeId" operator="equals"  value="PURCHASE_ORDER"/>
>>>        <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>>
>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ controller.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ controller.xml (original)
>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ controller.xml Thu Mar 19 13:58:08 2009
>>> @@ -1457,7 +1457,12 @@
>>>        <response name="success" type="request-redirect"  value="orderview"/>
>>>        <response name="error" type="request-redirect"  value="orderview"/>
>>>    </request-map>
>>> -
>>> +    <request-map uri="setOrderReservationPriority">
>>> +        <security https="true" auth="true"/>
>>> +        <event type="service" invoke="setOrderReservationPriority"/>
>>> +        <response name="success" type="view" value="orderview"/>
>>> +        <response name="error" type="view" value="orderview"/>
>>> +    </request-map>
>>>    <!-- Lookup request mappings -->
>>>    <request-map uri="LookupPerson"><security https="true"  auth="true"/><response name="success" type="view" 
>>> value="LookupPerson"/></request-map>
>>>    <request-map uri="LookupPartyGroup"><security https="true"  auth="true"/><response name="success" type="view" 
>>> value="LookupPartyGroup"/></request-map>
>>>
>>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/ orderinfo.ftl
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/order/webapp/ordermgr/order/ orderinfo.ftl (original)
>>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/order/ orderinfo.ftl Thu Mar 19 13:58:08 2009
>>> @@ -179,6 +179,24 @@
>>>              </td>
>>>            </tr>
>>>            </#if>
>>> +            <#if "SALES_ORDER" == orderHeader.orderTypeId>
>>> +              <form action="setOrderReservationPriority"  method="post" name="setOrderReservationPriority">
>>> +                <input type = "hidden" name="orderId" value="$ {orderId}"/>
>>> +                <tr><td colspan="3"><hr/></td></tr>
>>> +                <tr>
>>> +                  <td align="right" valign="top" width="15%"  class="label">&nbsp;${uiLabelMap.FormFieldTitle_priority}</td>
>>> +                  <td width="5%">&nbsp;</td>
>>> +                  <td valign="top" width="80%">
>>> +                    <select name="priority">
>>> +                      <option value="1">$ {uiLabelMap.OrderPriorityHigh}</option>
>>> +                      <option value="2"selected>$ {uiLabelMap.OrderPriorityNormal}</option>
>>> +                      <option value="3">$ {uiLabelMap.OrderPriorityLow}</option>
>>> +                    </select>
>>> +                    <input type="submit" class="smallSubmit"  value="${uiLabelMap.FormFieldTitle_reserveInventory}"/>
>>> +                  </td>
>>> +                </tr>
>>> +              </form>
>>> +            </#if>
>>>        </table>
>>>    </div>
>>> </div>
>>> \ No newline at end of file
>>>
>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryReserveServices.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryReserveServices.xml (original)
>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryReserveServices.xml Thu Mar 19 13:58:08 2009
>>> @@ -201,6 +201,7 @@
>>>                            <set from- field="parameters.reservedDatetime"  field="reserveOisgirMap.reservedDatetime"/>
>>>                            <set from-field="promisedDatetime"  field="reserveOisgirMap.promisedDatetime"/>
>>>                            <set from-field="parameters.sequenceId"  field="reserveOisgirMap.sequenceId"/>
>>> +                            <set from-field="parameters.priority"  field="reserveOisgirMap.priority"/>
>>>                            <call-service service- name="reserveOrderItemInventory" in-map-name="reserveOisgirMap"/>
>>>                            <clear-field field="reserveOisgirMap"/>
>>>                        <else>
>>> @@ -444,6 +445,7 @@
>>>                        <set from- field="parameters.reservedDatetime"  field="reserveOisgirMap.reservedDatetime"/>
>>>                        <set from-field="parameters.deductAmount"  field="reserveOisgirMap.quantity" type="BigDecimal"/>
>>>                        <set from-field="promisedDatetime"  field="reserveOisgirMap.promisedDatetime"/>
>>> +                        <set from-field="parameters.priority"  field="reserveOisgirMap.priority"/>
>>>                        <if-not-empty field="parameters.sequenceId">
>>>                            <set field="reserveOisgirMap.sequenceId"  from-field="parameters.sequenceId"/>
>>>                        </if-not-empty>
>>> @@ -498,6 +500,7 @@
>>>            <set-nonpk-fields map="parameters" value- field="newOisgirEntity"/>
>>>            <now-timestamp field="nowTimestamp"/>
>>>            <set from-field="nowTimestamp"  field="newOisgirEntity.createdDatetime"/>
>>> +            <set from-field="parameters.priority"  field="newOisgirEntity.priority"/>
>>>            <if-empty field="newOisgirEntity.reservedDatetime">
>>>                <set from-field="nowTimestamp"  field="newOisgirEntity.reservedDatetime"/>
>>>            </if-empty>
>>>
>>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryServices.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryServices.xml (original)
>>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/ inventory/InventoryServices.xml Thu Mar 19 13:58:08 2009
>>> @@ -814,6 +814,7 @@
>>>                    </condition-list>
>>>                </condition-list>
>>>            </condition-list>
>>> +            <order-by field-name="priority"/>
>>>            <order-by field-name="currentPromisedDate"/>
>>>            <order-by field-name="reservedDatetime"/>
>>>            <order-by field-name="sequenceId"/>
>>> @@ -875,6 +876,9 @@
>>>                    <log level="verbose" message="Adding $ {oisgir.orderId} to touchedOrderIdMap"/>
>>>                </if-compare>
>>>            </if-not-empty>
>>> +            <entity-one entity-name="OrderHeader" value- field="orderHeader">
>>> +                <field-map field-name="orderId" from- field="oisgir.orderId"/>
>>> +            </entity-one>
>>>
>>>            <!-- require inventory is N because it had to be N to  begin with to have a negative ATP -->
>>>            <clear-field field="resMap"/>
>>> @@ -888,6 +892,7 @@
>>>            <set field="resMap.shipGroupSeqId" from- field="oisgir.shipGroupSeqId"/>
>>>            <set field="resMap.sequenceId" from- field="oisgir.sequenceId"/>
>>>            <set field="resMap.facilityId" from- field="parameters.facilityId"/>
>>> +            <set field="resMap.priority" from- field="orderHeader.priority"/>
>>>            <log level="info" message="Re-reserving product [$ {resMap.productId}] for order item [${resMap.orderId}:$ 
>>> {resMap.orderItemSeqId}] quantity [${resMap.quantity}]; facility [$ {parameters.facilityId}]"/>
>>>            <call-service service- name="reserveProductInventoryByFacility" in-map-name="resMap"/>
>>>        </iterate>
>>> @@ -1249,5 +1254,43 @@
>>>            </if-compare>
>>>        </if-not-empty>
>>>    </simple-method>
>>> -
>>> -</simple-methods>
>>> +    <simple-method method-name="setOrderReservationPriority" short- description="Sets priority of an order for Inventory 
>>> Reservation,  orders with HIGH priority would be served first.">
>>> +        <set field="orderId" from-field="parameters.orderId"/>
>>> +        <entity-one entity-name="OrderHeader" value- field="orderHeader">
>>> +            <field-map field-name="orderId" from-field="orderId"/>
>>> +        </entity-one>
>>> +        <set field="priority" from-field="parameters.priority"/>
>>> +        <if-empty field="priority">
>>> +            <entity-and list="oisgirs" entity- name="OrderItemShipGrpInvRes">
>>> +                <field-map field-name="orderId" from- field="orderId"/>
>>> +            </entity-and>
>>> +            <iterate entry="oisgir" list="oisgirs">
>>> +                <set field="oisgir.priority" default-value="2"/>
>>> +                <store-value value-field="oisgir"/>
>>> +            </iterate>
>>> +            <set field="orderHeader.priority" default-value="2"/>
>>> +            <store-value value-field="orderHeader"/>
>>> +        <else>
>>> +            <set field="orderHeader.priority" from- field="priority"/>
>>> +            <store-value value-field="orderHeader"/>
>>> +            <entity-and list="oisgirs" entity- name="OrderItemShipGrpInvRes">
>>> +                <field-map field-name="orderId" from- field="orderId"/>
>>> +            </entity-and>
>>> +            <iterate entry="oisgir" list="oisgirs">
>>> +                <set field="oisgir.priority" from-field="priority"/>
>>> +                <store-value value-field="oisgir"/>
>>> +                <clear-field field="oisgir"/>
>>> +            </iterate>
>>> +            <entity-condition entity- name="OrderItemAndShipGrpInvResAndItem" list="oisgirais">
>>> +                <condition-expr field-name="orderId"  operator="equals" from-field="orderId"/>
>>> +            </entity-condition>
>>> +            <iterate entry="oisgir" list="oisgirais">
>>> +                <set  field="reassignInventoryReservationsCtx.productId" from- field="oisgir.productId"/>
>>> +                <set  field="reassignInventoryReservationsCtx.facilityId" from- field="oisgir.facilityId"/>
>>> +                <call-service service- name="reassignInventoryReservations" in-map- name="reassignInventoryReservationsCtx"/>
>>> +                <clear-field  field="reassignInventoryReservationsCtx"/>
>>> +            </iterate>
>>> +        </else>
>>> +        </if-empty>
>>> +    </simple-method>
>>> +</simple-methods>
>>> \ No newline at end of file
>>>
>>> Modified: ofbiz/trunk/applications/product/servicedef/ services_facility.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services_facility.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/applications/product/servicedef/ services_facility.xml (original)
>>> +++ ofbiz/trunk/applications/product/servicedef/ services_facility.xml Thu Mar 19 13:58:08 2009
>>> @@ -262,6 +262,7 @@
>>>        <attribute name="facilityId" type="String" mode="IN"  optional="false"/>
>>>        <attribute name="fromDate" type="Timestamp" mode="IN"  optional="true"/>
>>>        <attribute name="noLongerOnBackOrderIdSet" type="Set"  mode="OUT" optional="true"/>
>>> +        <attribute name="priority" type="String" mode="IN"   optional="true"/>
>>>    </service>
>>>    <service name="balanceOrderItemsWithNegativeReservations"  engine="simple"
>>>                location="component://product/script/org/ofbiz/ product/inventory/InventoryServices.xml" 
>>> invoke="balanceOrderItemsWithNegativeReservations">
>>> @@ -319,6 +320,7 @@
>>>        <attribute name="reserveOrderEnumId" type="String" mode="IN"  optional="true"/>
>>>        <attribute name="sequenceId" type="Long" mode="IN"  optional="true"/>
>>>        <attribute name="quantityNotReserved" type="BigDecimal"  mode="OUT" optional="false"/>
>>> +        <attribute name="priority" type="String" mode="IN"   optional="true"/>
>>>    </service>
>>>    <service name="reserveProductInventoryByContainer" engine="simple"
>>>                location="component://product/script/org/ofbiz/ product/inventory/InventoryReserveServices.xml" 
>>> invoke="reserveProductInventory" auth="true">
>>> @@ -345,6 +347,7 @@
>>>        <auto-attributes mode="IN" entity- name="OrderItemShipGrpInvRes" include="nonpk" optional="true">
>>>            <exclude field-name="createdDatetime"/>
>>>        </auto-attributes>
>>> +        <attribute name="priority" type="String" mode="IN"   optional="true"/>
>>>        <override name="quantity" optional="false"/>
>>>    </service>
>>>    <service name="cancelOrderInventoryReservation" engine="simple"
>>> @@ -873,4 +876,10 @@
>>>        <permission-service service-name="facilityGenericPermission"  main-action="DELETE"/>
>>>        <auto-attributes include="pk" mode="IN" optional="false"/>
>>>    </service>
>>> +    <service name="setOrderReservationPriority" engine="simple"  invoke="setOrderReservationPriority"
>>> +                location="component://product/script/org/ofbiz/ product/inventory/InventoryServices.xml"  auth="true">
>>> +        <description>set order priority</description>
>>> +        <attribute name="priority" mode="IN" type="String"  optional="true"/>
>>> +        <attribute name="orderId" mode="IN" type="String"  optional="false"/>
>>> +    </service>
>>> </services>
>>>
>>> Modified: ofbiz/trunk/specialpurpose/ecommerce/data/ DemoOrderTestData.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml?rev=756001&r1=756000&r2=756001&view=diff
>>> = = = = = = = = = =====================================================================
>>> --- ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml  (original)
>>> +++ ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml  Thu Mar 19 13:58:08 2009
>>> @@ -20,7 +20,7 @@
>>>
>>> <entity-engine-xml>
>>>    <!--Demo data for a sales order -->
>>> -    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"  orderName="" salesChannelEnumId="WEB_SALES_CHANNEL" 
>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23  16:49:27.392" visitId="10002" statusId="ORDER_APPROVED" 
>>> createdBy="admin" currencyUom="USD" webSiteId="OrderEntry"  productStoreId="9000" remainingSubTotal="12.45" grandTotal="12.45" 
>>> lastUpdatedStamp="2008-04-23 16:49:33.196"  lastUpdatedTxStamp="2008-04-23 16:49:33.118"  createdStamp="2008-04-23 16:49:27.716" 
>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>> +    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"  orderName="" salesChannelEnumId="WEB_SALES_CHANNEL" 
>>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23  16:49:27.392" priority="2" visitId="10002"  statusId="ORDER_APPROVED" 
>>> createdBy="admin" currencyUom="USD"  webSiteId="OrderEntry" productStoreId="9000"  remainingSubTotal="12.45" grandTotal="12.45" 
>>> lastUpdatedStamp="2008-04-23 16:49:33.196"  lastUpdatedTxStamp="2008-04-23 16:49:33.118"  createdStamp="2008-04-23 16:49:27.716" 
>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>    <OrderItem orderId="DEMO10090" orderItemSeqId="00001"  orderItemTypeId="PRODUCT_ORDER_ITEM" productId="GZ-2644" 
>>> prodCatalogId="DemoCatalog" isPromo="N" quantity="1.0"  selectedAmount="0.0" unitPrice="38.4" unitListPrice="48.0" 
>>> isModifiedPrice="N" itemDescription="Round Gizmo"  correspondingPoId="" statusId="ITEM_APPROVED"  lastUpdatedStamp="2008-04-23 
>>> 16:49:33.514"  lastUpdatedTxStamp="2008-04-23 16:49:33.484"  createdStamp="2008-04-23 16:49:28.128" createdTxStamp="2008-04-23 
>>> 16:49:27.349"/>
>>>    <OrderItemPriceInfo orderItemPriceInfoId="9000"  orderId="DEMO10090" orderItemSeqId="00001"  productPriceRuleId="9000" 
>>> productPriceActionSeqId="01"  modifyAmount="-9.600000000000001"  description="[PRODUCT_CATEGORY_IDIsPROMOTIONS] 
>>> [list:48.0;avgCost: 48.0;margin:0.0] [type:PRICE_POL]" lastUpdatedStamp="2008-04-23  16:49:28.682" 
>>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"  createdStamp="2008-04-23 16:49:28.682" createdTxStamp="2008-04-23  16:49:27.349"/>
>>>
>>> @@ -36,7 +36,7 @@
>>>
>>>    <OrderItemShipGroup orderId="DEMO10090" shipGroupSeqId="00001"  shipmentMethodTypeId="NEXT_DAY" carrierPartyId="UPS" 
>>> carrierRoleTypeId="CARRIER" contactMechId="9015" maySplit="N"  giftMessage="" isGift="N" lastUpdatedStamp="2008-04-23 
>>> 16:49:28.412" lastUpdatedTxStamp="2008-04-23 16:49:27.349"  createdStamp="2008-04-23 16:49:28.412" createdTxStamp="2008-04-23 
>>> 16:49:27.349"/>
>>>    <OrderItemShipGroupAssoc orderId="DEMO10090"  orderItemSeqId="00001" shipGroupSeqId="00001" quantity="1.0" 
>>> lastUpdatedStamp="2008-04-23 16:49:28.454"  lastUpdatedTxStamp="2008-04-23 16:49:27.349"  createdStamp="2008-04-23 16:49:28.454" 
>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>> -    <OrderItemShipGrpInvRes orderId="DEMO10090"  shipGroupSeqId="00001" orderItemSeqId="00001"  inventoryItemId="9001" 
>>> reserveOrderEnumId="INVRO_FIFO_REC"  quantity="1.0" quantityNotAvailable="0.0"  reservedDatetime="2008-04-23 16:49:31.474" 
>>> createdDatetime="2008-04-23 16:49:31.474"  promisedDatetime="2008-05-08 16:49:27.392"  lastUpdatedStamp="2008-04-23 
>>> 16:49:31.474"  lastUpdatedTxStamp="2008-04-23 16:49:27.349"  createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23 
>>> 16:49:27.349"/>
>>> +    <OrderItemShipGrpInvRes orderId="DEMO10090"  shipGroupSeqId="00001" orderItemSeqId="00001"  inventoryItemId="9001" 
>>> reserveOrderEnumId="INVRO_FIFO_REC"  quantity="1.0" quantityNotAvailable="0.0"  reservedDatetime="2008-04-23 16:49:31.474" 
>>> createdDatetime="2008-04-23 16:49:31.474"  promisedDatetime="2008-05-08 16:49:27.392" priority="2"  lastUpdatedStamp="2008-04-23 
>>> 16:49:31.474"  lastUpdatedTxStamp="2008-04-23 16:49:27.349"  createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23 
>>> 16:49:27.349"/>
>>>
>>>    <OrderAdjustment orderAdjustmentId="9000"  orderAdjustmentTypeId="PROMOTION_ADJUSTMENT" orderId="DEMO10090" 
>>> orderItemSeqId="00001" shipGroupSeqId="_NA_" amount="-38.4"  productPromoId="9016" productPromoRuleId="01" 
>>> productPromoActionSeqId="01" createdDate="2008-04-23 16:49:27.866"  createdByUserLogin="admin" lastUpdatedStamp="2008-04-23 
>>> 16:49:28.271" lastUpdatedTxStamp="2008-04-23 16:49:27.349"  createdStamp="2008-04-23 16:49:28.271" createdTxStamp="2008-04-23 
>>> 16:49:27.349"/>
>>>    <OrderAdjustment orderAdjustmentId="9001"  orderAdjustmentTypeId="SHIPPING_CHARGES" orderId="DEMO10090" 
>>> orderItemSeqId="_NA_" shipGroupSeqId="00001" amount="12.45"  createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin" 
>>> lastUpdatedStamp="2008-04-23 16:49:28.431"  lastUpdatedTxStamp="2008-04-23 16:49:27.349"  createdStamp="2008-04-23 16:49:28.431" 
>>> createdTxStamp="2008-04-23  16:49:27.349"/>
>>>
>>
>>
>
> 



Re: svn commit: r756001 - in /ofbiz/trunk: applications/order/config/ applications/order/entitydef/ applications/order/servicedef/ applications/order/webapp/ordermgr/WEB-INF/ applications/order/webapp/ordermgr/order/ applications/product/script/org/ofbiz/p

Posted by Vikas Mayur <vi...@hotwaxmedia.com>.
Thanks Jacques!

I have done few changes in rev. 756134 and to make them work, would  
require to commit following changes. I would appreciate if you can  
take a lead :) This is a small change, so not opening any new jira  
issue.

Index: framework/common/config/CommonUiLabels.xml
===================================================================
--- framework/common/config/CommonUiLabels.xml	(revision 756129)
+++ framework/common/config/CommonUiLabels.xml	(working copy)
@@ -2877,6 +2877,9 @@
          <value xml:lang="zh">隐藏域</value>
          <value xml:lang="zh_CN">隐藏字段</value>
      </property>
+    <property key="CommonHigh">
+        <value xml:lang="en">High</value>
+    </property>
      <property key="CommonHome">
          <value xml:lang="en">Home</value>
          <value xml:lang="fr">Point de départ</value>
@@ -3445,6 +3448,9 @@
          <value xml:lang="en">http://whitepages.com/find_person_results.pl?fid=p&amp;ac=$ 
{telecomNumber.areaCode?if_exists}&amp;s=&amp;p=$ 
{telecomNumber.contactNumber?if_exists}&amp;pt=b&amp;x=40&amp;y=9</ 
value>
          <value xml:lang="fr">http://www.pagesjaunes.fr/quidonc/aquiestcenumero.do 
</value>
      </property>
+    <property key="CommonLow">
+        <value xml:lang="en">Low</value>
+    </property>
      <property key="CommonMain">
          <value xml:lang="ar">الرئيسية</value>
          <value xml:lang="cs">Úvodní stránka</value>
@@ -3988,6 +3994,9 @@
          <value xml:lang="zh">Âê¶</value>
          <value xml:lang="zh_CN">不</value>
      </property>
+    <property key="CommonNormal">
+        <value xml:lang="en">Normal</value>
+    </property>
      <property key="CommonNoAccess">
          <value xml:lang="en">No Access to this function, contact  
your system administrator if you feel this is an error</value>
          <value xml:lang="es">Acceso no autorizado. Póngase en  
contacto con su administrador de sistemas si piensa que esto es un  
error.</value>


Thanks,
Vikas


On Mar 19, 2009, at 7:39 PM, Jacques Le Roux wrote:

> +1 for CommonHigh, CommonLow etc.
>
> Jacques
>
> From: <mo...@apache.org>
>> Author: mor
>> Date: Thu Mar 19 13:58:08 2009
>> New Revision: 756001
>>
>> URL: http://svn.apache.org/viewvc?rev=756001&view=rev
>> Log:
>> Another new feature from Pranay and Co.
>> Now Order reservation can be changed manually by setting priority  
>> (new field priority on OrderHeader and OrderItemShipGrpInvRes) for  
>> the order. This way few orders can be
>> fulfilled while waiting for sufficient inventory to fulfill large  
>> orders.
>>
>> Applied patch from OFBIZ-2224 (https://issues.apache.org/jira/browse/OFBIZ-2224 
>> ).
>> Thanks Pratik Khera and Pranay Pandey.
>>
>> i18n Note: This commit has three new labels (IMO, they can be moved  
>> to common component with names something like CommonHigh, CommonLow  
>> etc.)
>>
>> Pratik, I have done minor changes in the service and they are
>> 1) Used <if-empty field=""> instead of <if> <condition> <if-empty>  
>> as this makes the logic more clearer.
>> 2) Removed <condition-list> because their is only one expression in  
>> the <entity-condition>
>> 3) Added <clear-field> in the iteration, to make sure that always a  
>> new value will be set in the map that is passed to a service and  
>> <store-value>
>>
>>
>> Modified:
>>   ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>>   ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>>   ofbiz/trunk/applications/order/servicedef/secas.xml
>>   ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ 
>> controller.xml
>>   ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
>>   ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryReserveServices.xml
>>   ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryServices.xml
>>   ofbiz/trunk/applications/product/servicedef/services_facility.xml
>>   ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
>>
>> Modified: ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderUiLabels.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/order/config/OrderUiLabels.xml  
>> (original)
>> +++ ofbiz/trunk/applications/order/config/OrderUiLabels.xml Thu Mar  
>> 19 13:58:08 2009
>> @@ -6271,6 +6271,15 @@
>>        <value xml:lang="zh">å
> ¬å¼€ä¾¿ç¬º (已打印)</value>
>>        <value xml:lang="zh_CN">å
> ¬å¼€æ³¨é‡Š (可打印)</value>
>>    </property>
>> +    <property key="OrderPriorityHigh">
>> +        <value xml:lang="en">High</value>
>> +    </property>
>> +    <property key="OrderPriorityLow">
>> +        <value xml:lang="en">Low</value>
>> +    </property>
>> +    <property key="OrderPriorityNormal">
>> +        <value xml:lang="en">Normal</value>
>> +    </property>
>>    <property key="OrderProcessOrder">
>>        <value xml:lang="de">Auftrag verarbeiten</value>
>>        <value xml:lang="en">Process Order</value>
>>
>> Modified: ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/entitydef/entitymodel.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/order/entitydef/entitymodel.xml  
>> (original)
>> +++ ofbiz/trunk/applications/order/entitydef/entitymodel.xml Thu  
>> Mar 19 13:58:08 2009
>> @@ -337,6 +337,7 @@
>>      <field name="externalId" type="id"></field>
>>      <field name="salesChannelEnumId" type="id"></field>
>>      <field name="orderDate" type="date-time"></field>
>> +      <field name="priority" type="indicator"><description>Sets  
>> priority for Inventory Reservation</description></field>
>>      <field name="entryDate" type="date-time"></field>
>>      <field name="visitId" type="id"></field>
>>      <field name="statusId" type="id"></field>
>> @@ -888,6 +889,7 @@
>>      <field name="createdDatetime" type="date-time"></field>
>>      <field name="promisedDatetime" type="date-time"></field>
>>      <field name="currentPromisedDate" type="date-time"></field>
>> +      <field name="priority" type="indicator"><description>Sets  
>> priority for Inventory Reservation</description></field>
>>      <field name="sequenceId" type="numeric"></field>
>>      <field name="oldPickStartDate" col-name="PICK_START_DATE"  
>> type="date-time"></field> <!-- this is no longer used and is here  
>> for migration or whatever if needed -->
>>      <prim-key field="orderId"/>
>>
>> Modified: ofbiz/trunk/applications/order/servicedef/secas.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/secas.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/order/servicedef/secas.xml (original)
>> +++ ofbiz/trunk/applications/order/servicedef/secas.xml Thu Mar 19  
>> 13:58:08 2009
>> @@ -36,6 +36,9 @@
>>        <condition field-name="orderTypeId" operator="equals"  
>> value="PURCHASE_ORDER"/>
>>        <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>    </eca>
>> +    <eca service="storeOrder" event="return">
>> +        <action service="setOrderReservationPriority" mode="sync"/>
>> +    </eca>
>>    <eca service="updateOrderItems" event="commit">
>>        <condition field-name="orderTypeId" operator="equals"  
>> value="PURCHASE_ORDER"/>
>>        <action service="setUnitPriceAsLastPrice" mode="sync"/>
>>
>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ 
>> controller.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ 
>> controller.xml (original)
>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/ 
>> controller.xml Thu Mar 19 13:58:08 2009
>> @@ -1457,7 +1457,12 @@
>>        <response name="success" type="request-redirect"  
>> value="orderview"/>
>>        <response name="error" type="request-redirect"  
>> value="orderview"/>
>>    </request-map>
>> -
>> +    <request-map uri="setOrderReservationPriority">
>> +        <security https="true" auth="true"/>
>> +        <event type="service" invoke="setOrderReservationPriority"/>
>> +        <response name="success" type="view" value="orderview"/>
>> +        <response name="error" type="view" value="orderview"/>
>> +    </request-map>
>>    <!-- Lookup request mappings -->
>>    <request-map uri="LookupPerson"><security https="true"  
>> auth="true"/><response name="success" type="view"  
>> value="LookupPerson"/></request-map>
>>    <request-map uri="LookupPartyGroup"><security https="true"  
>> auth="true"/><response name="success" type="view"  
>> value="LookupPartyGroup"/></request-map>
>>
>> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/ 
>> orderinfo.ftl
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/order/webapp/ordermgr/order/ 
>> orderinfo.ftl (original)
>> +++ ofbiz/trunk/applications/order/webapp/ordermgr/order/ 
>> orderinfo.ftl Thu Mar 19 13:58:08 2009
>> @@ -179,6 +179,24 @@
>>              </td>
>>            </tr>
>>            </#if>
>> +            <#if "SALES_ORDER" == orderHeader.orderTypeId>
>> +              <form action="setOrderReservationPriority"  
>> method="post" name="setOrderReservationPriority">
>> +                <input type = "hidden" name="orderId" value="$ 
>> {orderId}"/>
>> +                <tr><td colspan="3"><hr/></td></tr>
>> +                <tr>
>> +                  <td align="right" valign="top" width="15%"  
>> class="label">&nbsp;${uiLabelMap.FormFieldTitle_priority}</td>
>> +                  <td width="5%">&nbsp;</td>
>> +                  <td valign="top" width="80%">
>> +                    <select name="priority">
>> +                      <option value="1">$ 
>> {uiLabelMap.OrderPriorityHigh}</option>
>> +                      <option value="2"selected>$ 
>> {uiLabelMap.OrderPriorityNormal}</option>
>> +                      <option value="3">$ 
>> {uiLabelMap.OrderPriorityLow}</option>
>> +                    </select>
>> +                    <input type="submit" class="smallSubmit"  
>> value="${uiLabelMap.FormFieldTitle_reserveInventory}"/>
>> +                  </td>
>> +                </tr>
>> +              </form>
>> +            </#if>
>>        </table>
>>    </div>
>> </div>
>> \ No newline at end of file
>>
>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryReserveServices.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryReserveServices.xml (original)
>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryReserveServices.xml Thu Mar 19 13:58:08 2009
>> @@ -201,6 +201,7 @@
>>                            <set from- 
>> field="parameters.reservedDatetime"  
>> field="reserveOisgirMap.reservedDatetime"/>
>>                            <set from-field="promisedDatetime"  
>> field="reserveOisgirMap.promisedDatetime"/>
>>                            <set from-field="parameters.sequenceId"  
>> field="reserveOisgirMap.sequenceId"/>
>> +                            <set from-field="parameters.priority"  
>> field="reserveOisgirMap.priority"/>
>>                            <call-service service- 
>> name="reserveOrderItemInventory" in-map-name="reserveOisgirMap"/>
>>                            <clear-field field="reserveOisgirMap"/>
>>                        <else>
>> @@ -444,6 +445,7 @@
>>                        <set from- 
>> field="parameters.reservedDatetime"  
>> field="reserveOisgirMap.reservedDatetime"/>
>>                        <set from-field="parameters.deductAmount"  
>> field="reserveOisgirMap.quantity" type="BigDecimal"/>
>>                        <set from-field="promisedDatetime"  
>> field="reserveOisgirMap.promisedDatetime"/>
>> +                        <set from-field="parameters.priority"  
>> field="reserveOisgirMap.priority"/>
>>                        <if-not-empty field="parameters.sequenceId">
>>                            <set field="reserveOisgirMap.sequenceId"  
>> from-field="parameters.sequenceId"/>
>>                        </if-not-empty>
>> @@ -498,6 +500,7 @@
>>            <set-nonpk-fields map="parameters" value- 
>> field="newOisgirEntity"/>
>>            <now-timestamp field="nowTimestamp"/>
>>            <set from-field="nowTimestamp"  
>> field="newOisgirEntity.createdDatetime"/>
>> +            <set from-field="parameters.priority"  
>> field="newOisgirEntity.priority"/>
>>            <if-empty field="newOisgirEntity.reservedDatetime">
>>                <set from-field="nowTimestamp"  
>> field="newOisgirEntity.reservedDatetime"/>
>>            </if-empty>
>>
>> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryServices.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryServices.xml (original)
>> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/ 
>> inventory/InventoryServices.xml Thu Mar 19 13:58:08 2009
>> @@ -814,6 +814,7 @@
>>                    </condition-list>
>>                </condition-list>
>>            </condition-list>
>> +            <order-by field-name="priority"/>
>>            <order-by field-name="currentPromisedDate"/>
>>            <order-by field-name="reservedDatetime"/>
>>            <order-by field-name="sequenceId"/>
>> @@ -875,6 +876,9 @@
>>                    <log level="verbose" message="Adding $ 
>> {oisgir.orderId} to touchedOrderIdMap"/>
>>                </if-compare>
>>            </if-not-empty>
>> +            <entity-one entity-name="OrderHeader" value- 
>> field="orderHeader">
>> +                <field-map field-name="orderId" from- 
>> field="oisgir.orderId"/>
>> +            </entity-one>
>>
>>            <!-- require inventory is N because it had to be N to  
>> begin with to have a negative ATP -->
>>            <clear-field field="resMap"/>
>> @@ -888,6 +892,7 @@
>>            <set field="resMap.shipGroupSeqId" from- 
>> field="oisgir.shipGroupSeqId"/>
>>            <set field="resMap.sequenceId" from- 
>> field="oisgir.sequenceId"/>
>>            <set field="resMap.facilityId" from- 
>> field="parameters.facilityId"/>
>> +            <set field="resMap.priority" from- 
>> field="orderHeader.priority"/>
>>            <log level="info" message="Re-reserving product [$ 
>> {resMap.productId}] for order item [${resMap.orderId}:$ 
>> {resMap.orderItemSeqId}] quantity [${resMap.quantity}]; facility [$ 
>> {parameters.facilityId}]"/>
>>            <call-service service- 
>> name="reserveProductInventoryByFacility" in-map-name="resMap"/>
>>        </iterate>
>> @@ -1249,5 +1254,43 @@
>>            </if-compare>
>>        </if-not-empty>
>>    </simple-method>
>> -
>> -</simple-methods>
>> +    <simple-method method-name="setOrderReservationPriority" short- 
>> description="Sets priority of an order for Inventory Reservation,  
>> orders with HIGH priority would be served first.">
>> +        <set field="orderId" from-field="parameters.orderId"/>
>> +        <entity-one entity-name="OrderHeader" value- 
>> field="orderHeader">
>> +            <field-map field-name="orderId" from-field="orderId"/>
>> +        </entity-one>
>> +        <set field="priority" from-field="parameters.priority"/>
>> +        <if-empty field="priority">
>> +            <entity-and list="oisgirs" entity- 
>> name="OrderItemShipGrpInvRes">
>> +                <field-map field-name="orderId" from- 
>> field="orderId"/>
>> +            </entity-and>
>> +            <iterate entry="oisgir" list="oisgirs">
>> +                <set field="oisgir.priority" default-value="2"/>
>> +                <store-value value-field="oisgir"/>
>> +            </iterate>
>> +            <set field="orderHeader.priority" default-value="2"/>
>> +            <store-value value-field="orderHeader"/>
>> +        <else>
>> +            <set field="orderHeader.priority" from- 
>> field="priority"/>
>> +            <store-value value-field="orderHeader"/>
>> +            <entity-and list="oisgirs" entity- 
>> name="OrderItemShipGrpInvRes">
>> +                <field-map field-name="orderId" from- 
>> field="orderId"/>
>> +            </entity-and>
>> +            <iterate entry="oisgir" list="oisgirs">
>> +                <set field="oisgir.priority" from-field="priority"/>
>> +                <store-value value-field="oisgir"/>
>> +                <clear-field field="oisgir"/>
>> +            </iterate>
>> +            <entity-condition entity- 
>> name="OrderItemAndShipGrpInvResAndItem" list="oisgirais">
>> +                <condition-expr field-name="orderId"  
>> operator="equals" from-field="orderId"/>
>> +            </entity-condition>
>> +            <iterate entry="oisgir" list="oisgirais">
>> +                <set  
>> field="reassignInventoryReservationsCtx.productId" from- 
>> field="oisgir.productId"/>
>> +                <set  
>> field="reassignInventoryReservationsCtx.facilityId" from- 
>> field="oisgir.facilityId"/>
>> +                <call-service service- 
>> name="reassignInventoryReservations" in-map- 
>> name="reassignInventoryReservationsCtx"/>
>> +                <clear-field  
>> field="reassignInventoryReservationsCtx"/>
>> +            </iterate>
>> +        </else>
>> +        </if-empty>
>> +    </simple-method>
>> +</simple-methods>
>> \ No newline at end of file
>>
>> Modified: ofbiz/trunk/applications/product/servicedef/ 
>> services_facility.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services_facility.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/applications/product/servicedef/ 
>> services_facility.xml (original)
>> +++ ofbiz/trunk/applications/product/servicedef/ 
>> services_facility.xml Thu Mar 19 13:58:08 2009
>> @@ -262,6 +262,7 @@
>>        <attribute name="facilityId" type="String" mode="IN"  
>> optional="false"/>
>>        <attribute name="fromDate" type="Timestamp" mode="IN"  
>> optional="true"/>
>>        <attribute name="noLongerOnBackOrderIdSet" type="Set"  
>> mode="OUT" optional="true"/>
>> +        <attribute name="priority" type="String" mode="IN"   
>> optional="true"/>
>>    </service>
>>    <service name="balanceOrderItemsWithNegativeReservations"  
>> engine="simple"
>>                location="component://product/script/org/ofbiz/ 
>> product/inventory/InventoryServices.xml"  
>> invoke="balanceOrderItemsWithNegativeReservations">
>> @@ -319,6 +320,7 @@
>>        <attribute name="reserveOrderEnumId" type="String" mode="IN"  
>> optional="true"/>
>>        <attribute name="sequenceId" type="Long" mode="IN"  
>> optional="true"/>
>>        <attribute name="quantityNotReserved" type="BigDecimal"  
>> mode="OUT" optional="false"/>
>> +        <attribute name="priority" type="String" mode="IN"   
>> optional="true"/>
>>    </service>
>>    <service name="reserveProductInventoryByContainer" engine="simple"
>>                location="component://product/script/org/ofbiz/ 
>> product/inventory/InventoryReserveServices.xml"  
>> invoke="reserveProductInventory" auth="true">
>> @@ -345,6 +347,7 @@
>>        <auto-attributes mode="IN" entity- 
>> name="OrderItemShipGrpInvRes" include="nonpk" optional="true">
>>            <exclude field-name="createdDatetime"/>
>>        </auto-attributes>
>> +        <attribute name="priority" type="String" mode="IN"   
>> optional="true"/>
>>        <override name="quantity" optional="false"/>
>>    </service>
>>    <service name="cancelOrderInventoryReservation" engine="simple"
>> @@ -873,4 +876,10 @@
>>        <permission-service service-name="facilityGenericPermission"  
>> main-action="DELETE"/>
>>        <auto-attributes include="pk" mode="IN" optional="false"/>
>>    </service>
>> +    <service name="setOrderReservationPriority" engine="simple"  
>> invoke="setOrderReservationPriority"
>> +                location="component://product/script/org/ofbiz/ 
>> product/inventory/InventoryServices.xml"  auth="true">
>> +        <description>set order priority</description>
>> +        <attribute name="priority" mode="IN" type="String"  
>> optional="true"/>
>> +        <attribute name="orderId" mode="IN" type="String"  
>> optional="false"/>
>> +    </service>
>> </services>
>>
>> Modified: ofbiz/trunk/specialpurpose/ecommerce/data/ 
>> DemoOrderTestData.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml?rev=756001&r1=756000&r2=756001&view=diff
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> = 
>> =====================================================================
>> --- ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml  
>> (original)
>> +++ ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml  
>> Thu Mar 19 13:58:08 2009
>> @@ -20,7 +20,7 @@
>>
>> <entity-engine-xml>
>>    <!--Demo data for a sales order -->
>> -    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"  
>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"  
>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23  
>> 16:49:27.392" visitId="10002" statusId="ORDER_APPROVED"  
>> createdBy="admin" currencyUom="USD" webSiteId="OrderEntry"  
>> productStoreId="9000" remainingSubTotal="12.45" grandTotal="12.45"  
>> lastUpdatedStamp="2008-04-23 16:49:33.196"  
>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"  
>> createdStamp="2008-04-23 16:49:27.716" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>> +    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER"  
>> orderName="" salesChannelEnumId="WEB_SALES_CHANNEL"  
>> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23  
>> 16:49:27.392" priority="2" visitId="10002"  
>> statusId="ORDER_APPROVED" createdBy="admin" currencyUom="USD"  
>> webSiteId="OrderEntry" productStoreId="9000"  
>> remainingSubTotal="12.45" grandTotal="12.45"  
>> lastUpdatedStamp="2008-04-23 16:49:33.196"  
>> lastUpdatedTxStamp="2008-04-23 16:49:33.118"  
>> createdStamp="2008-04-23 16:49:27.716" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>>    <OrderItem orderId="DEMO10090" orderItemSeqId="00001"  
>> orderItemTypeId="PRODUCT_ORDER_ITEM" productId="GZ-2644"  
>> prodCatalogId="DemoCatalog" isPromo="N" quantity="1.0"  
>> selectedAmount="0.0" unitPrice="38.4" unitListPrice="48.0"  
>> isModifiedPrice="N" itemDescription="Round Gizmo"  
>> correspondingPoId="" statusId="ITEM_APPROVED"  
>> lastUpdatedStamp="2008-04-23 16:49:33.514"  
>> lastUpdatedTxStamp="2008-04-23 16:49:33.484"  
>> createdStamp="2008-04-23 16:49:28.128" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>>    <OrderItemPriceInfo orderItemPriceInfoId="9000"  
>> orderId="DEMO10090" orderItemSeqId="00001"  
>> productPriceRuleId="9000" productPriceActionSeqId="01"  
>> modifyAmount="-9.600000000000001"  
>> description="[PRODUCT_CATEGORY_IDIsPROMOTIONS] [list:48.0;avgCost: 
>> 48.0;margin:0.0] [type:PRICE_POL]" lastUpdatedStamp="2008-04-23  
>> 16:49:28.682" lastUpdatedTxStamp="2008-04-23 16:49:27.349"  
>> createdStamp="2008-04-23 16:49:28.682" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>>
>> @@ -36,7 +36,7 @@
>>
>>    <OrderItemShipGroup orderId="DEMO10090" shipGroupSeqId="00001"  
>> shipmentMethodTypeId="NEXT_DAY" carrierPartyId="UPS"  
>> carrierRoleTypeId="CARRIER" contactMechId="9015" maySplit="N"  
>> giftMessage="" isGift="N" lastUpdatedStamp="2008-04-23  
>> 16:49:28.412" lastUpdatedTxStamp="2008-04-23 16:49:27.349"  
>> createdStamp="2008-04-23 16:49:28.412" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>>    <OrderItemShipGroupAssoc orderId="DEMO10090"  
>> orderItemSeqId="00001" shipGroupSeqId="00001" quantity="1.0"  
>> lastUpdatedStamp="2008-04-23 16:49:28.454"  
>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"  
>> createdStamp="2008-04-23 16:49:28.454" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>> -    <OrderItemShipGrpInvRes orderId="DEMO10090"  
>> shipGroupSeqId="00001" orderItemSeqId="00001"  
>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"  
>> quantity="1.0" quantityNotAvailable="0.0"  
>> reservedDatetime="2008-04-23 16:49:31.474"  
>> createdDatetime="2008-04-23 16:49:31.474"  
>> promisedDatetime="2008-05-08 16:49:27.392"  
>> lastUpdatedStamp="2008-04-23 16:49:31.474"  
>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"  
>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>> +    <OrderItemShipGrpInvRes orderId="DEMO10090"  
>> shipGroupSeqId="00001" orderItemSeqId="00001"  
>> inventoryItemId="9001" reserveOrderEnumId="INVRO_FIFO_REC"  
>> quantity="1.0" quantityNotAvailable="0.0"  
>> reservedDatetime="2008-04-23 16:49:31.474"  
>> createdDatetime="2008-04-23 16:49:31.474"  
>> promisedDatetime="2008-05-08 16:49:27.392" priority="2"  
>> lastUpdatedStamp="2008-04-23 16:49:31.474"  
>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"  
>> createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>>
>>    <OrderAdjustment orderAdjustmentId="9000"  
>> orderAdjustmentTypeId="PROMOTION_ADJUSTMENT" orderId="DEMO10090"  
>> orderItemSeqId="00001" shipGroupSeqId="_NA_" amount="-38.4"  
>> productPromoId="9016" productPromoRuleId="01"  
>> productPromoActionSeqId="01" createdDate="2008-04-23 16:49:27.866"  
>> createdByUserLogin="admin" lastUpdatedStamp="2008-04-23  
>> 16:49:28.271" lastUpdatedTxStamp="2008-04-23 16:49:27.349"  
>> createdStamp="2008-04-23 16:49:28.271" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>>    <OrderAdjustment orderAdjustmentId="9001"  
>> orderAdjustmentTypeId="SHIPPING_CHARGES" orderId="DEMO10090"  
>> orderItemSeqId="_NA_" shipGroupSeqId="00001" amount="12.45"  
>> createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin"  
>> lastUpdatedStamp="2008-04-23 16:49:28.431"  
>> lastUpdatedTxStamp="2008-04-23 16:49:27.349"  
>> createdStamp="2008-04-23 16:49:28.431" createdTxStamp="2008-04-23  
>> 16:49:27.349"/>
>>
>
>


Re: svn commit: r756001 - in /ofbiz/trunk: applications/order/config/ applications/order/entitydef/ applications/order/servicedef/ applications/order/webapp/ordermgr/WEB-INF/ applications/order/webapp/ordermgr/order/ applications/product/script/org/ofbiz/p

Posted by Jacques Le Roux <ja...@les7arts.com>.
+1 for CommonHigh, CommonLow etc.

Jacques

From: <mo...@apache.org>
> Author: mor
> Date: Thu Mar 19 13:58:08 2009
> New Revision: 756001
>
> URL: http://svn.apache.org/viewvc?rev=756001&view=rev
> Log:
> Another new feature from Pranay and Co.
> Now Order reservation can be changed manually by setting priority (new field priority on OrderHeader and OrderItemShipGrpInvRes) 
> for the order. This way few orders can be
> fulfilled while waiting for sufficient inventory to fulfill large orders.
>
> Applied patch from OFBIZ-2224 (https://issues.apache.org/jira/browse/OFBIZ-2224).
> Thanks Pratik Khera and Pranay Pandey.
>
> i18n Note: This commit has three new labels (IMO, they can be moved to common component with names something like CommonHigh, 
> CommonLow etc.)
>
> Pratik, I have done minor changes in the service and they are
> 1) Used <if-empty field=""> instead of <if> <condition> <if-empty> as this makes the logic more clearer.
> 2) Removed <condition-list> because their is only one expression in the <entity-condition>
> 3) Added <clear-field> in the iteration, to make sure that always a new value will be set in the map that is passed to a service 
> and <store-value>
>
>
> Modified:
>    ofbiz/trunk/applications/order/config/OrderUiLabels.xml
>    ofbiz/trunk/applications/order/entitydef/entitymodel.xml
>    ofbiz/trunk/applications/order/servicedef/secas.xml
>    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml
>    ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
>    ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml
>    ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml
>    ofbiz/trunk/applications/product/servicedef/services_facility.xml
>    ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
>
> Modified: ofbiz/trunk/applications/order/config/OrderUiLabels.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/config/OrderUiLabels.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/config/OrderUiLabels.xml (original)
> +++ ofbiz/trunk/applications/order/config/OrderUiLabels.xml Thu Mar 19 13:58:08 2009
> @@ -6271,6 +6271,15 @@
>         <value xml:lang="zh">公开便笺 (已打印)</value>
>         <value xml:lang="zh_CN">公开注释 (可打印)</value>
>     </property>
> +    <property key="OrderPriorityHigh">
> +        <value xml:lang="en">High</value>
> +    </property>
> +    <property key="OrderPriorityLow">
> +        <value xml:lang="en">Low</value>
> +    </property>
> +    <property key="OrderPriorityNormal">
> +        <value xml:lang="en">Normal</value>
> +    </property>
>     <property key="OrderProcessOrder">
>         <value xml:lang="de">Auftrag verarbeiten</value>
>         <value xml:lang="en">Process Order</value>
>
> Modified: ofbiz/trunk/applications/order/entitydef/entitymodel.xml
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/entitydef/entitymodel.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/entitydef/entitymodel.xml (original)
> +++ ofbiz/trunk/applications/order/entitydef/entitymodel.xml Thu Mar 19 13:58:08 2009
> @@ -337,6 +337,7 @@
>       <field name="externalId" type="id"></field>
>       <field name="salesChannelEnumId" type="id"></field>
>       <field name="orderDate" type="date-time"></field>
> +      <field name="priority" type="indicator"><description>Sets priority for Inventory Reservation</description></field>
>       <field name="entryDate" type="date-time"></field>
>       <field name="visitId" type="id"></field>
>       <field name="statusId" type="id"></field>
> @@ -888,6 +889,7 @@
>       <field name="createdDatetime" type="date-time"></field>
>       <field name="promisedDatetime" type="date-time"></field>
>       <field name="currentPromisedDate" type="date-time"></field>
> +      <field name="priority" type="indicator"><description>Sets priority for Inventory Reservation</description></field>
>       <field name="sequenceId" type="numeric"></field>
>       <field name="oldPickStartDate" col-name="PICK_START_DATE" type="date-time"></field> <!-- this is no longer used and is here 
> for migration or whatever if needed -->
>       <prim-key field="orderId"/>
>
> Modified: ofbiz/trunk/applications/order/servicedef/secas.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/secas.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/servicedef/secas.xml (original)
> +++ ofbiz/trunk/applications/order/servicedef/secas.xml Thu Mar 19 13:58:08 2009
> @@ -36,6 +36,9 @@
>         <condition field-name="orderTypeId" operator="equals" value="PURCHASE_ORDER"/>
>         <action service="setUnitPriceAsLastPrice" mode="sync"/>
>     </eca>
> +    <eca service="storeOrder" event="return">
> +        <action service="setOrderReservationPriority" mode="sync"/>
> +    </eca>
>     <eca service="updateOrderItems" event="commit">
>         <condition field-name="orderTypeId" operator="equals" value="PURCHASE_ORDER"/>
>         <action service="setUnitPriceAsLastPrice" mode="sync"/>
>
> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml (original)
> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/controller.xml Thu Mar 19 13:58:08 2009
> @@ -1457,7 +1457,12 @@
>         <response name="success" type="request-redirect" value="orderview"/>
>         <response name="error" type="request-redirect" value="orderview"/>
>     </request-map>
> -
> +    <request-map uri="setOrderReservationPriority">
> +        <security https="true" auth="true"/>
> +        <event type="service" invoke="setOrderReservationPriority"/>
> +        <response name="success" type="view" value="orderview"/>
> +        <response name="error" type="view" value="orderview"/>
> +    </request-map>
>     <!-- Lookup request mappings -->
>     <request-map uri="LookupPerson"><security https="true" auth="true"/><response name="success" type="view" 
> value="LookupPerson"/></request-map>
>     <request-map uri="LookupPartyGroup"><security https="true" auth="true"/><response name="success" type="view" 
> value="LookupPartyGroup"/></request-map>
>
> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl (original)
> +++ ofbiz/trunk/applications/order/webapp/ordermgr/order/orderinfo.ftl Thu Mar 19 13:58:08 2009
> @@ -179,6 +179,24 @@
>               </td>
>             </tr>
>             </#if>
> +            <#if "SALES_ORDER" == orderHeader.orderTypeId>
> +              <form action="setOrderReservationPriority" method="post" name="setOrderReservationPriority">
> +                <input type = "hidden" name="orderId" value="${orderId}"/>
> +                <tr><td colspan="3"><hr/></td></tr>
> +                <tr>
> +                  <td align="right" valign="top" width="15%" class="label">&nbsp;${uiLabelMap.FormFieldTitle_priority}</td>
> +                  <td width="5%">&nbsp;</td>
> +                  <td valign="top" width="80%">
> +                    <select name="priority">
> +                      <option value="1">${uiLabelMap.OrderPriorityHigh}</option>
> +                      <option value="2"selected>${uiLabelMap.OrderPriorityNormal}</option>
> +                      <option value="3">${uiLabelMap.OrderPriorityLow}</option>
> +                    </select>
> +                    <input type="submit" class="smallSubmit" value="${uiLabelMap.FormFieldTitle_reserveInventory}"/>
> +                  </td>
> +                </tr>
> +              </form>
> +            </#if>
>         </table>
>     </div>
> </div>
> \ No newline at end of file
>
> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml (original)
> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml Thu Mar 19 13:58:08 2009
> @@ -201,6 +201,7 @@
>                             <set from-field="parameters.reservedDatetime" field="reserveOisgirMap.reservedDatetime"/>
>                             <set from-field="promisedDatetime" field="reserveOisgirMap.promisedDatetime"/>
>                             <set from-field="parameters.sequenceId" field="reserveOisgirMap.sequenceId"/>
> +                            <set from-field="parameters.priority" field="reserveOisgirMap.priority"/>
>                             <call-service service-name="reserveOrderItemInventory" in-map-name="reserveOisgirMap"/>
>                             <clear-field field="reserveOisgirMap"/>
>                         <else>
> @@ -444,6 +445,7 @@
>                         <set from-field="parameters.reservedDatetime" field="reserveOisgirMap.reservedDatetime"/>
>                         <set from-field="parameters.deductAmount" field="reserveOisgirMap.quantity" type="BigDecimal"/>
>                         <set from-field="promisedDatetime" field="reserveOisgirMap.promisedDatetime"/>
> +                        <set from-field="parameters.priority" field="reserveOisgirMap.priority"/>
>                         <if-not-empty field="parameters.sequenceId">
>                             <set field="reserveOisgirMap.sequenceId" from-field="parameters.sequenceId"/>
>                         </if-not-empty>
> @@ -498,6 +500,7 @@
>             <set-nonpk-fields map="parameters" value-field="newOisgirEntity"/>
>             <now-timestamp field="nowTimestamp"/>
>             <set from-field="nowTimestamp" field="newOisgirEntity.createdDatetime"/>
> +            <set from-field="parameters.priority" field="newOisgirEntity.priority"/>
>             <if-empty field="newOisgirEntity.reservedDatetime">
>                 <set from-field="nowTimestamp" field="newOisgirEntity.reservedDatetime"/>
>             </if-empty>
>
> Modified: ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml (original)
> +++ ofbiz/trunk/applications/product/script/org/ofbiz/product/inventory/InventoryServices.xml Thu Mar 19 13:58:08 2009
> @@ -814,6 +814,7 @@
>                     </condition-list>
>                 </condition-list>
>             </condition-list>
> +            <order-by field-name="priority"/>
>             <order-by field-name="currentPromisedDate"/>
>             <order-by field-name="reservedDatetime"/>
>             <order-by field-name="sequenceId"/>
> @@ -875,6 +876,9 @@
>                     <log level="verbose" message="Adding ${oisgir.orderId} to touchedOrderIdMap"/>
>                 </if-compare>
>             </if-not-empty>
> +            <entity-one entity-name="OrderHeader" value-field="orderHeader">
> +                <field-map field-name="orderId" from-field="oisgir.orderId"/>
> +            </entity-one>
>
>             <!-- require inventory is N because it had to be N to begin with to have a negative ATP -->
>             <clear-field field="resMap"/>
> @@ -888,6 +892,7 @@
>             <set field="resMap.shipGroupSeqId" from-field="oisgir.shipGroupSeqId"/>
>             <set field="resMap.sequenceId" from-field="oisgir.sequenceId"/>
>             <set field="resMap.facilityId" from-field="parameters.facilityId"/>
> +            <set field="resMap.priority" from-field="orderHeader.priority"/>
>             <log level="info" message="Re-reserving product [${resMap.productId}] for order item 
> [${resMap.orderId}:${resMap.orderItemSeqId}] quantity [${resMap.quantity}]; facility [${parameters.facilityId}]"/>
>             <call-service service-name="reserveProductInventoryByFacility" in-map-name="resMap"/>
>         </iterate>
> @@ -1249,5 +1254,43 @@
>             </if-compare>
>         </if-not-empty>
>     </simple-method>
> -
> -</simple-methods>
> +    <simple-method method-name="setOrderReservationPriority" short-description="Sets priority of an order for Inventory 
> Reservation, orders with HIGH priority would be served first.">
> +        <set field="orderId" from-field="parameters.orderId"/>
> +        <entity-one entity-name="OrderHeader" value-field="orderHeader">
> +            <field-map field-name="orderId" from-field="orderId"/>
> +        </entity-one>
> +        <set field="priority" from-field="parameters.priority"/>
> +        <if-empty field="priority">
> +            <entity-and list="oisgirs" entity-name="OrderItemShipGrpInvRes">
> +                <field-map field-name="orderId" from-field="orderId"/>
> +            </entity-and>
> +            <iterate entry="oisgir" list="oisgirs">
> +                <set field="oisgir.priority" default-value="2"/>
> +                <store-value value-field="oisgir"/>
> +            </iterate>
> +            <set field="orderHeader.priority" default-value="2"/>
> +            <store-value value-field="orderHeader"/>
> +        <else>
> +            <set field="orderHeader.priority" from-field="priority"/>
> +            <store-value value-field="orderHeader"/>
> +            <entity-and list="oisgirs" entity-name="OrderItemShipGrpInvRes">
> +                <field-map field-name="orderId" from-field="orderId"/>
> +            </entity-and>
> +            <iterate entry="oisgir" list="oisgirs">
> +                <set field="oisgir.priority" from-field="priority"/>
> +                <store-value value-field="oisgir"/>
> +                <clear-field field="oisgir"/>
> +            </iterate>
> +            <entity-condition entity-name="OrderItemAndShipGrpInvResAndItem" list="oisgirais">
> +                <condition-expr field-name="orderId" operator="equals" from-field="orderId"/>
> +            </entity-condition>
> +            <iterate entry="oisgir" list="oisgirais">
> +                <set field="reassignInventoryReservationsCtx.productId" from-field="oisgir.productId"/>
> +                <set field="reassignInventoryReservationsCtx.facilityId" from-field="oisgir.facilityId"/>
> +                <call-service service-name="reassignInventoryReservations" in-map-name="reassignInventoryReservationsCtx"/>
> +                <clear-field field="reassignInventoryReservationsCtx"/>
> +            </iterate>
> +        </else>
> +        </if-empty>
> +    </simple-method>
> +</simple-methods>
> \ No newline at end of file
>
> Modified: ofbiz/trunk/applications/product/servicedef/services_facility.xml
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/servicedef/services_facility.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/product/servicedef/services_facility.xml (original)
> +++ ofbiz/trunk/applications/product/servicedef/services_facility.xml Thu Mar 19 13:58:08 2009
> @@ -262,6 +262,7 @@
>         <attribute name="facilityId" type="String" mode="IN" optional="false"/>
>         <attribute name="fromDate" type="Timestamp" mode="IN" optional="true"/>
>         <attribute name="noLongerOnBackOrderIdSet" type="Set" mode="OUT" optional="true"/>
> +        <attribute name="priority" type="String" mode="IN"  optional="true"/>
>     </service>
>     <service name="balanceOrderItemsWithNegativeReservations" engine="simple"
>                 location="component://product/script/org/ofbiz/product/inventory/InventoryServices.xml" 
> invoke="balanceOrderItemsWithNegativeReservations">
> @@ -319,6 +320,7 @@
>         <attribute name="reserveOrderEnumId" type="String" mode="IN" optional="true"/>
>         <attribute name="sequenceId" type="Long" mode="IN" optional="true"/>
>         <attribute name="quantityNotReserved" type="BigDecimal" mode="OUT" optional="false"/>
> +        <attribute name="priority" type="String" mode="IN"  optional="true"/>
>     </service>
>     <service name="reserveProductInventoryByContainer" engine="simple"
>                 location="component://product/script/org/ofbiz/product/inventory/InventoryReserveServices.xml" 
> invoke="reserveProductInventory" auth="true">
> @@ -345,6 +347,7 @@
>         <auto-attributes mode="IN" entity-name="OrderItemShipGrpInvRes" include="nonpk" optional="true">
>             <exclude field-name="createdDatetime"/>
>         </auto-attributes>
> +        <attribute name="priority" type="String" mode="IN"  optional="true"/>
>         <override name="quantity" optional="false"/>
>     </service>
>     <service name="cancelOrderInventoryReservation" engine="simple"
> @@ -873,4 +876,10 @@
>         <permission-service service-name="facilityGenericPermission" main-action="DELETE"/>
>         <auto-attributes include="pk" mode="IN" optional="false"/>
>     </service>
> +    <service name="setOrderReservationPriority" engine="simple" invoke="setOrderReservationPriority"
> +                location="component://product/script/org/ofbiz/product/inventory/InventoryServices.xml"  auth="true">
> +        <description>set order priority</description>
> +        <attribute name="priority" mode="IN" type="String" optional="true"/>
> +        <attribute name="orderId" mode="IN" type="String" optional="false"/>
> +    </service>
> </services>
>
> Modified: ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml?rev=756001&r1=756000&r2=756001&view=diff
> ==============================================================================
> --- ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml (original)
> +++ ofbiz/trunk/specialpurpose/ecommerce/data/DemoOrderTestData.xml Thu Mar 19 13:58:08 2009
> @@ -20,7 +20,7 @@
>
> <entity-engine-xml>
>     <!--Demo data for a sales order -->
> -    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER" orderName="" salesChannelEnumId="WEB_SALES_CHANNEL" 
> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23 16:49:27.392" visitId="10002" statusId="ORDER_APPROVED" 
> createdBy="admin" currencyUom="USD" webSiteId="OrderEntry" productStoreId="9000" remainingSubTotal="12.45" grandTotal="12.45" 
> lastUpdatedStamp="2008-04-23 16:49:33.196" lastUpdatedTxStamp="2008-04-23 16:49:33.118" createdStamp="2008-04-23 16:49:27.716" 
> createdTxStamp="2008-04-23 16:49:27.349"/>
> +    <OrderHeader orderId="DEMO10090" orderTypeId="SALES_ORDER" orderName="" salesChannelEnumId="WEB_SALES_CHANNEL" 
> orderDate="2008-04-23 16:49:27.392" entryDate="2008-04-23 16:49:27.392" priority="2" visitId="10002" statusId="ORDER_APPROVED" 
> createdBy="admin" currencyUom="USD" webSiteId="OrderEntry" productStoreId="9000" remainingSubTotal="12.45" grandTotal="12.45" 
> lastUpdatedStamp="2008-04-23 16:49:33.196" lastUpdatedTxStamp="2008-04-23 16:49:33.118" createdStamp="2008-04-23 16:49:27.716" 
> createdTxStamp="2008-04-23 16:49:27.349"/>
>     <OrderItem orderId="DEMO10090" orderItemSeqId="00001" orderItemTypeId="PRODUCT_ORDER_ITEM" productId="GZ-2644" 
> prodCatalogId="DemoCatalog" isPromo="N" quantity="1.0" selectedAmount="0.0" unitPrice="38.4" unitListPrice="48.0" 
> isModifiedPrice="N" itemDescription="Round Gizmo" correspondingPoId="" statusId="ITEM_APPROVED" lastUpdatedStamp="2008-04-23 
> 16:49:33.514" lastUpdatedTxStamp="2008-04-23 16:49:33.484" createdStamp="2008-04-23 16:49:28.128" createdTxStamp="2008-04-23 
> 16:49:27.349"/>
>     <OrderItemPriceInfo orderItemPriceInfoId="9000" orderId="DEMO10090" orderItemSeqId="00001" productPriceRuleId="9000" 
> productPriceActionSeqId="01" modifyAmount="-9.600000000000001" description="[PRODUCT_CATEGORY_IDIsPROMOTIONS] 
> [list:48.0;avgCost:48.0;margin:0.0] [type:PRICE_POL]" lastUpdatedStamp="2008-04-23 16:49:28.682" lastUpdatedTxStamp="2008-04-23 
> 16:49:27.349" createdStamp="2008-04-23 16:49:28.682" createdTxStamp="2008-04-23 16:49:27.349"/>
>
> @@ -36,7 +36,7 @@
>
>     <OrderItemShipGroup orderId="DEMO10090" shipGroupSeqId="00001" shipmentMethodTypeId="NEXT_DAY" carrierPartyId="UPS" 
> carrierRoleTypeId="CARRIER" contactMechId="9015" maySplit="N" giftMessage="" isGift="N" lastUpdatedStamp="2008-04-23 16:49:28.412" 
> lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.412" createdTxStamp="2008-04-23 16:49:27.349"/>
>     <OrderItemShipGroupAssoc orderId="DEMO10090" orderItemSeqId="00001" shipGroupSeqId="00001" quantity="1.0" 
> lastUpdatedStamp="2008-04-23 16:49:28.454" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.454" 
> createdTxStamp="2008-04-23 16:49:27.349"/>
> -    <OrderItemShipGrpInvRes orderId="DEMO10090" shipGroupSeqId="00001" orderItemSeqId="00001" inventoryItemId="9001" 
> reserveOrderEnumId="INVRO_FIFO_REC" quantity="1.0" quantityNotAvailable="0.0" reservedDatetime="2008-04-23 16:49:31.474" 
> createdDatetime="2008-04-23 16:49:31.474" promisedDatetime="2008-05-08 16:49:27.392" lastUpdatedStamp="2008-04-23 16:49:31.474" 
> lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23 16:49:27.349"/>
> +    <OrderItemShipGrpInvRes orderId="DEMO10090" shipGroupSeqId="00001" orderItemSeqId="00001" inventoryItemId="9001" 
> reserveOrderEnumId="INVRO_FIFO_REC" quantity="1.0" quantityNotAvailable="0.0" reservedDatetime="2008-04-23 16:49:31.474" 
> createdDatetime="2008-04-23 16:49:31.474" promisedDatetime="2008-05-08 16:49:27.392" priority="2" lastUpdatedStamp="2008-04-23 
> 16:49:31.474" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:31.474" createdTxStamp="2008-04-23 
> 16:49:27.349"/>
>
>     <OrderAdjustment orderAdjustmentId="9000" orderAdjustmentTypeId="PROMOTION_ADJUSTMENT" orderId="DEMO10090" 
> orderItemSeqId="00001" shipGroupSeqId="_NA_" amount="-38.4" productPromoId="9016" productPromoRuleId="01" 
> productPromoActionSeqId="01" createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin" lastUpdatedStamp="2008-04-23 
> 16:49:28.271" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.271" createdTxStamp="2008-04-23 
> 16:49:27.349"/>
>     <OrderAdjustment orderAdjustmentId="9001" orderAdjustmentTypeId="SHIPPING_CHARGES" orderId="DEMO10090" orderItemSeqId="_NA_" 
> shipGroupSeqId="00001" amount="12.45" createdDate="2008-04-23 16:49:27.866" createdByUserLogin="admin" 
> lastUpdatedStamp="2008-04-23 16:49:28.431" lastUpdatedTxStamp="2008-04-23 16:49:27.349" createdStamp="2008-04-23 16:49:28.431" 
> createdTxStamp="2008-04-23 16:49:27.349"/>
>
>