You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by nm...@apache.org on 2016/01/24 20:06:33 UTC
svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Author: nmalin
Date: Sun Jan 24 19:06:32 2016
New Revision: 1726542
URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
Log:
I converted following services from simple to entity-auto :
createOrderNotificationLog
createOrderItemBilling
createOrderAdjustment
updateOrderAdjustment
createOrderAdjustmentBilling
createOrderShipment
updateOrderShipment
deleteOrderShipment
createCommunicationEventOrder
removeCommunicationEventOrder
createOrderItemShipGroup
createOrderContactMech
removeOrderContactMech
createOrderTerm
removeOrderTerm
createOrderRequirementCommitment
And from java to entity-auto for createOrderPaymentPreference
Related issue OFBIZ-6854.
Removed:
ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
Modified:
ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
ofbiz/trunk/applications/order/servicedef/services.xml
ofbiz/trunk/applications/order/servicedef/services_requirement.xml
ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml (original)
+++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml Sun Jan 24 19:06:32 2016
@@ -88,56 +88,8 @@ under the License.
<field-to-result field="orderInfo.totalOrders" result-name="totalOrders"/>
</if-not-empty>
</simple-method>
- <simple-method method-name="createOrderShipment" short-description="Create OrderShipment">
- <set value="Create OrderShipment" field="operationName"/>
- <check-permission permission="ORDERMGR" action="_CREATE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderShipment"/>
- </check-permission>
- <check-errors/>
-
- <make-value value-field="newEntity" entity-name="OrderShipment"/>
- <set-pk-fields map="parameters" value-field="newEntity"/>
- <set-nonpk-fields map="parameters" value-field="newEntity"/>
-
- <create-value value-field="newEntity"/>
- </simple-method>
- <simple-method method-name="updateOrderShipment" short-description="Update OrderShipment">
- <set value="Update OrderShipment" field="operationName"/>
- <check-permission permission="ORDERMGR" action="_UPDATE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
- </check-permission>
- <check-errors/>
-
- <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
- <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
- <store-value value-field="lookedUpValue"/>
- </simple-method>
- <simple-method method-name="deleteOrderShipment" short-description="Delete OrderShipment">
- <set value="Delete OrderShipment" field="operationName"/>
- <check-permission permission="ORDERMGR" action="_DELETE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
- </check-permission>
- <check-errors/>
-
- <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
- <remove-value value-field="lookedUpValue"/>
- </simple-method>
<!-- order requirement methods -->
- <simple-method method-name="createOrderRequirementCommitment" short-description="Create OrderRequirementCommitment">
- <!-- The permission checking is commented out to make this service work also when triggered from ecommerce orders -->
- <!--
- <check-permission permission="ORDERMGR" action="_CREATE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCheckCreateOrderRequirementCommitment"/>
- </check-permission>
- <check-errors/>
- -->
- <make-value value-field="newEntity" entity-name="OrderRequirementCommitment"/>
- <set-pk-fields map="parameters" value-field="newEntity"/>
- <set-nonpk-fields map="parameters" value-field="newEntity"/>
- <create-value value-field="newEntity"/>
- </simple-method>
-
<simple-method method-name="createRequirementAndCommitment" short-description="create a requirement and commitment for it">
<set value="PRODUCT_REQUIREMENT" field="inputMap.requirementTypeId"/>
@@ -759,20 +711,6 @@ under the License.
</simple-method>
<!-- OrderItemShipGroup -->
- <simple-method method-name="createOrderItemShipGroup" short-description="Create OrderItemShipGroup">
- <check-permission permission="ORDERMGR" action="_CREATE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
- </check-permission>
- <check-errors/>
- <make-value value-field="newEntity" entity-name="OrderItemShipGroup"/>
- <set-pk-fields map="parameters" value-field="newEntity"/>
- <set-nonpk-fields map="parameters" value-field="newEntity"/>
- <if-empty field="newEntity.shipGroupSeqId">
- <make-next-seq-id value-field="newEntity" seq-field-name="shipGroupSeqId"/> <!-- this finds the next sub-sequence ID -->
- <field-to-result field="newEntity.shipGroupSeqId" result-name="shipGroupSeqId"/>
- </if-empty>
- <create-value value-field="newEntity"/>
- </simple-method>
<simple-method method-name="updateOrderItemShipGroup" short-description="Update OrderItemShipGroup">
<check-permission permission="ORDERMGR" action="_UPDATE">
<fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
@@ -832,16 +770,6 @@ under the License.
<field-to-result field="orderItemShipGroupInvRes.promisedDatetime" result-name="estimatedShipDate"/>
</simple-method>
- <simple-method method-name="createOrderContactMech" short-description="Create OrderContactMech">
- <check-permission permission="ORDERMGR" action="_CREATE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderContactMech"/>
- </check-permission>
- <check-errors/>
- <make-value value-field="newEntity" entity-name="OrderContactMech"/>
- <set-pk-fields map="parameters" value-field="newEntity"/>
- <create-value value-field="newEntity"/>
- </simple-method>
-
<simple-method method-name="removeOrderContactMech" short-description="Remove OrderContactMech">
<check-permission permission="ORDERMGR" action="_DELETE">
<fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
@@ -861,39 +789,6 @@ under the License.
<store-value value-field="orderHeaderNote"/>
</simple-method>
- <simple-method method-name="createOrderTerm" short-description="Create an OrderTerm">
- <check-permission permission="ORDERMGR" action="_CREATE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderTerm"/>
- </check-permission>
- <check-errors/>
-
- <make-value value-field="newEntity" entity-name="OrderTerm"/>
- <set-pk-fields map="parameters" value-field="newEntity"/>
- <set value="_NA_" set-if-empty="true" field="newEntity.orderItemSeqId"/>
- <set-nonpk-fields map="parameters" value-field="newEntity"/>
-
- <create-value value-field="newEntity"/>
- </simple-method>
- <simple-method method-name="updateOrderTerm" short-description="Update OrderTerm">
- <check-permission permission="ORDERMGR" action="_DELETE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderTerm"/>
- </check-permission>
- <check-errors/>
-
- <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
- <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
- <store-value value-field="lookedUpValue"/>
- </simple-method>
- <simple-method method-name="removeOrderTerm" short-description="Remove OrderTerm">
- <check-permission permission="ORDERMGR" action="_DELETE">
- <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderTerm"/>
- </check-permission>
- <check-errors/>
-
- <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
- <remove-value value-field="lookedUpValue"/>
- </simple-method>
-
<simple-method method-name="addPaymentMethodToOrder" short-description="Create an PaymentMethodToOrder">
<check-permission permission="ORDERMGR" action="_CREATE">
<fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
@@ -1373,7 +1268,7 @@ under the License.
</iterate>
</iterate>
</simple-method>
-
+
<simple-method method-name="updateShippingMethodAndCharges" short-description="Updates shipping method and shipping charges from Order View page when Shipment is in picked status and items of Order are packed">
<!-- splitting shipmentMethodAndAmount request parameter value that contains "*" symbol
into "shipmentMethod" and "newAmount".
@@ -1424,6 +1319,7 @@ under the License.
</else>
</if-compare-field>
</simple-method>
+
<simple-method method-name="productAvailabalityByFacility" short-description="Calculate ATP and Qoh According For each facility">
<set field="facilityMap.ownerPartyId" from-field="parameters.ownerPartyId"/>
<find-by-and map="facilityMap" list="facilityList" entity-name="Facility"/>
@@ -1442,7 +1338,7 @@ under the License.
</iterate>
<field-to-result field="availabalityList"/>
</simple-method>
-
+
<simple-method method-name="createOrderPaymentApplication" short-description="Create Order Payment Application">
<entity-one value-field="paymentMap" entity-name="Payment"/>
<set field="createCtx.amountApplied" from-field="paymentMap.amount" type="BigDecimal"/>
@@ -1459,7 +1355,7 @@ under the License.
<call-service service-name="createPaymentApplication" in-map-name="createCtx"/>
</if-not-empty>
</simple-method>
-
+
<simple-method method-name="MoveItemBetweenShipGroups" short-description="Move order items between ship groups">
<entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
<field-map field-name="orderId" from-field="parameters.orderId"/>
@@ -1483,7 +1379,7 @@ under the License.
<set field="map.shipGroupSeqId" from-field="parameters.toGroupIndex"/>
<set field="map.quantity" value="${orderItemShipGroupAssoc.quantity + parameters.quantity}" type="BigDecimal"/>
<call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
-
+
<entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
<field-map field-name="orderId" from-field="parameters.orderId"/>
<field-map field-name="orderItemSeqId" from-field="parameters.orderItemSeqId"/>
@@ -1495,7 +1391,7 @@ under the License.
</add-error>
</if-empty>
<check-errors/>
-
+
<clear-field field="map"/>
<set field="map.orderId" from-field="parameters.orderId"/>
<set field="map.orderItemSeqId" from-field="parameters.orderItemSeqId"/>
@@ -1503,5 +1399,5 @@ under the License.
<set field="map.quantity" value="${orderItemShipGroupAssoc.quantity - parameters.quantity}" type="BigDecimal"/>
<call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
</simple-method>
-
+
</simple-methods>
Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml (original)
+++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml Sun Jan 24 19:06:32 2016
@@ -44,33 +44,12 @@ under the License.
</else>
</if-compare>
</simple-method>
-
- <simple-method method-name="createOrderAdjustment" short-description="Create an OrderAdjustment">
-
- <make-value entity-name="OrderAdjustment" value-field="newEntity"/>
- <set-nonpk-fields map="parameters" value-field="newEntity"/>
-
- <sequenced-id sequence-name="OrderAdjustment" field="newEntity.orderAdjustmentId"/>
- <field-to-result field="newEntity.orderAdjustmentId" result-name="orderAdjustmentId"/>
-
- <now-timestamp field="newEntity.createdDate"/>
- <set from-field="userLogin.userLoginId" field="newEntity.createdByUserLogin"/>
-
- <create-value value-field="newEntity"/>
- </simple-method>
- <simple-method method-name="updateOrderAdjustment" short-description="Update an OrderAdjustment">
- <check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></check-permission>
- <check-errors/>
- <entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
- <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
- <store-value value-field="lookedUpValue"/>
- </simple-method>
<simple-method method-name="deleteOrderAdjustment" short-description="Delete an OrderAdjustment">
<check-permission permission="ORDERMGR" action="_DELETE"><fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></check-permission>
<check-errors/>
<entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
<remove-value value-field="lookedUpValue"/>
-
+
<if-not-empty field="parameters.productPromoCodeId">
<entity-one entity-name="OrderProductPromoCode" value-field="lookedUpValue"/>
<if-not-empty field="lookedUpValue">
@@ -79,30 +58,6 @@ under the License.
</if-not-empty>
</simple-method>
- <simple-method method-name="createOrderAdjustmentBilling" short-description="Create an OrderAdjustmentBilling">
- <make-value entity-name="OrderAdjustmentBilling" value-field="newEntity"/>
- <set-nonpk-fields map="parameters" value-field="newEntity"/>
- <set-pk-fields map="parameters" value-field="newEntity"/>
- <create-value value-field="newEntity"/>
- </simple-method>
-
- <simple-method method-name="createOrderItemBilling" short-description="Create an OrderItemBilling">
- <make-value entity-name="OrderItemBilling" value-field="newEntity"/>
- <set-nonpk-fields map="parameters" value-field="newEntity"/>
- <set-pk-fields map="parameters" value-field="newEntity"/>
- <create-value value-field="newEntity"/>
- </simple-method>
-
- <simple-method method-name="createNotificationLog" short-description="Log an order notification" login-required="false">
- <make-value entity-name="OrderNotification" value-field="orderNotification"/>
- <sequenced-id sequence-name="OrderNotification" field="orderNotification.orderNotificationId"/>
- <set from-field="parameters.orderId" field="orderNotification.orderId"/>
- <set from-field="parameters.emailType" field="orderNotification.emailType"/>
- <set from-field="parameters.comments" field="orderNotification.comments"/>
- <now-timestamp field="orderNotification.notificationDate"/>
- <create-value value-field="orderNotification"/>
- </simple-method>
-
<simple-method method-name="updateOrderStatusFromReceipt" short-description="Update Order Status From ShipmentReceipt">
<check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderStatusFromReceipt"/></check-permission>
<check-errors/>
Modified: ofbiz/trunk/applications/order/servicedef/services.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/servicedef/services.xml (original)
+++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan 24 19:06:32 2016
@@ -76,8 +76,7 @@ under the License.
<attribute name="assignedRoleTypeId" type="String" mode="IN" optional="true"/>
</service>
- <service name="createOrderNotificationLog" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createNotificationLog">
+ <service name="createOrderNotificationLog" engine="entity-auto" default-entity-name="OrderNotification" invoke="create">
<description>Logs when a notification was sent</description>
<attribute name="orderId" type="String" mode="IN" optional="false"/>
<attribute name="emailType" type="String" mode="IN" optional="false"/>
@@ -164,17 +163,10 @@ under the License.
<attribute name="orderId" type="String" mode="OUT" optional="false"/>
</service>
- <service name="createOrderItemBilling" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderItemBilling">
+ <service name="createOrderItemBilling" default-entity-name="OrderItemBilling" engine="entity-auto" invoke="create">
<description>Create a new order item billing record</description>
- <attribute name="orderId" type="String" mode="IN" optional="false"/>
- <attribute name="orderItemSeqId" type="String" mode="IN" optional="false"/>
- <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
- <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
- <attribute name="quantity" type="BigDecimal" mode="IN" optional="true"/>
- <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
- <attribute name="itemIssuanceId" type="String" mode="IN" optional="true"/>
- <attribute name="shipmentReceiptId" type="String" mode="IN" optional="true"/>
+ <auto-attributes mode="IN" include="pk"/>
+ <auto-attributes mode="IN" include="nonpk" optional="true"/>
</service>
<service name="orderAdjustmentPermissionCheck" engine="simple"
@@ -183,8 +175,7 @@ under the License.
<implements service="permissionInterface"/>
</service>
- <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustment">
+ <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="create">
<description>Creates a new order adjustment record</description>
<permission-service service-name="orderAdjustmentPermissionCheck" main-action="CREATE"/>
<auto-attributes mode="OUT" include="pk" optional="false"/>
@@ -192,9 +183,9 @@ under the License.
<override name="orderAdjustmentTypeId" optional="false"/>
<override name="orderId" optional="false"/>
</service>
- <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="updateOrderAdjustment">
+ <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="update">
<description>Update an order adjustment record</description>
+ <permission-service service-name="orderAdjustmentPermissionCheck" main-action="UPDATE"/>
<auto-attributes mode="IN" include="pk" optional="false"/>
<auto-attributes mode="IN" include="nonpk" optional="true"/>
<override name="orderId" optional="false"/>
@@ -207,13 +198,10 @@ under the License.
<attribute name="productPromoCodeId" type="String" mode="IN" optional="true"/>
</service>
- <service name="createOrderAdjustmentBilling" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustmentBilling">
+ <service name="createOrderAdjustmentBilling" default-entity-name="OrderAdjustmentBilling" engine="entity-auto" invoke="create">
<description>Create a new order adjustment billing record</description>
- <attribute name="orderAdjustmentId" type="String" mode="IN" optional="false"/>
- <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
- <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
- <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
+ <auto-attributes mode="IN" include="pk"/>
+ <auto-attributes mode="IN" include="nonpk" optional="true"/>
</service>
<service name="createPaymentFromPreference" engine="java"
@@ -435,15 +423,10 @@ under the License.
<attribute name="roleTypeId" type="String" mode="IN"/>
</service>
- <service name="createOrderPaymentPreference" engine="java"
- location="org.ofbiz.order.order.OrderServices" invoke="createPaymentPreference">
+ <service name="createOrderPaymentPreference" default-entity-name="OrderPaymentPreference" engine="entity-auto" invoke="create">
<description>Creates an order payment preference</description>
- <attribute name="orderId" type="String" mode="IN"/>
- <attribute name="paymentMethodTypeId" type="String" mode="IN"/>
- <attribute name="paymentMethodId" type="String" mode="IN" optional="true"/>
- <attribute name="statusId" type="String" mode="IN" optional="true"/>
- <attribute name="maxAmount" type="BigDecimal" mode="IN" optional="true"/>
- <attribute name="orderPaymentPreferenceId" type="String" mode="OUT"/>
+ <auto-attributes include="nonpk" mode="IN" optional="true"/>
+ <auto-attributes include="pk" mode="OUT"/>
</service>
<service name="createOrderNote" engine="java"
@@ -568,20 +551,17 @@ under the License.
</service>
<!-- OrderShipment Services -->
- <service name="createOrderShipment" default-entity-name="OrderShipment" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderShipment" auth="true">
+ <service name="createOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="create" auth="true">
<description>Create OrderShipment</description>
<auto-attributes include="pk" mode="IN" optional="false"/>
<auto-attributes include="nonpk" mode="IN" optional="true"/>
</service>
- <service name="updateOrderShipment" default-entity-name="OrderShipment" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderShipment" auth="true">
+ <service name="updateOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="update" auth="true">
<description>Update OrderShipment</description>
<auto-attributes include="pk" mode="IN" optional="false"/>
<auto-attributes include="nonpk" mode="IN" optional="true"/>
</service>
- <service name="deleteOrderShipment" default-entity-name="OrderShipment" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="deleteOrderShipment" auth="true">
+ <service name="deleteOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="delete" auth="true">
<description>Delete OrderShipment</description>
<auto-attributes include="pk" mode="IN" optional="false"/>
</service>
@@ -667,21 +647,27 @@ under the License.
</service>
<!-- CommunicationEvents related to orders -->
- <service name="createCommunicationEventOrder" engine="simple"
- location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="createCommunicationEventOrder" auth="true">
+ <service name="createCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="create" auth="true">
<description>Create a Communication Event Order</description>
- <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="CREATE"/>
+ </required-permissions>
+ <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
</service>
- <service name="removeCommunicationEventOrder" engine="simple"
- location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="removeCommunicationEventOrder" auth="true">
+ <service name="removeCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="delete" auth="true">
<description>Remove a Communication Event Order</description>
- <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="DELETE"/>
+ </required-permissions>
+ <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
</service>
<!-- Order Shipping and Contacts -->
- <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderItemShipGroup" auth="true">
+ <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="entity-auto" invoke="create" auth="true">
<description>Creates a new OrderItemShipGroup.</description>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="CREATE"/>
+ </required-permissions>
<auto-attributes mode="IN" include="nonpk" optional="true"/>
<attribute name="orderId" type="String" mode="IN" optional="false"/>
<attribute name="shipGroupSeqId" type="String" mode="INOUT" optional="true"/>
@@ -696,42 +682,52 @@ under the License.
<attribute name="contactMechPurposeTypeId" type="String" mode="IN" optional="true"/>
</service>
- <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderContactMech" auth="true">
+ <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="create" auth="true">
<description>Create Order Contact Mech</description>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="CREATE"/>
+ </required-permissions>
<auto-attributes include="pk" mode="IN" optional="false"/>
</service>
- <service name="updateOrderContactMech" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true" default-entity-name="OrderContactMech">
+ <service name="updateOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
+ location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true">
<description>Update Order Contact Mech</description>
<auto-attributes include="pk" mode="IN" optional="false"/>
<attribute name="oldContactMechId" type="String" mode="IN" optional="true"/>
</service>
- <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderContactMech" auth="true">
+ <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="delete" auth="true">
<description>Remove Order Contact Mech</description>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="DELETE"/>
+ </required-permissions>
<auto-attributes include="pk" mode="IN" optional="false"/>
</service>
<!-- OrderTerms -->
- <service name="createOrderTerm" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderTerm" auth="true">
+ <service name="createOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="create" auth="true">
<description>Create an Order Term</description>
- <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
- <auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
- <override name="orderItemSeqId" optional="true"/>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="CREATE"/>
+ </required-permissions>
+ <auto-attributes include="pk" mode="IN" optional="false"/>
+ <auto-attributes include="nonpk" mode="IN" optional="true"/>
+ <override name="orderItemSeqId" optional="true" default-value="_NA_"/>
</service>
- <service name="updateOrderTerm" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderTerm" auth="true">
+ <service name="updateOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="update" auth="true">
<description>Update an Order Term</description>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="UPDATE"/>
+ </required-permissions>
<auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
<auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
</service>
- <service name="removeOrderTerm" engine="simple"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderTerm" auth="true">
+ <service name="removeOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="delete" auth="true">
<description>Remove an Order Term</description>
+ <required-permissions join-type="AND">
+ <check-permission permission="ORDERMGR" action="DELETE"/>
+ </required-permissions>
<auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
</service>
Modified: ofbiz/trunk/applications/order/servicedef/services_requirement.xml
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_requirement.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/servicedef/services_requirement.xml (original)
+++ ofbiz/trunk/applications/order/servicedef/services_requirement.xml Sun Jan 24 19:06:32 2016
@@ -105,8 +105,7 @@ under the License.
<attribute mode="OUT" name="quantityTotal" type="BigDecimal" optional="true"/>
<attribute mode="OUT" name="amountTotal" type="BigDecimal" optional="true"/>
</service>
- <service name="createOrderRequirementCommitment" engine="simple" auth="true"
- location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderRequirementCommitment">
+ <service name="createOrderRequirementCommitment" engine="entity-auto" auth="true" default-entity-name="OrderRequirementCommitment" invoke="create">
<auto-attributes entity-name="OrderRequirementCommitment" include="pk" mode="IN" optional="false"/>
<auto-attributes entity-name="OrderRequirementCommitment" include="nonpk" mode="IN" optional="true"/>
</service>
Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=1726542&r1=1726541&r2=1726542&view=diff
==============================================================================
--- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java (original)
+++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java Sun Jan 24 19:06:32 2016
@@ -2824,52 +2824,6 @@ public class OrderServices {
return ServiceUtil.returnSuccess();
}
- /** Service to create an order payment preference */
- public static Map<String, Object> createPaymentPreference(DispatchContext ctx, Map<String, ? extends Object> context) {
- Map<String, Object> result = new HashMap<String, Object>();
- Delegator delegator = ctx.getDelegator();
- String orderId = (String) context.get("orderId");
- String statusId = (String) context.get("statusId");
- String paymentMethodTypeId = (String) context.get("paymentMethodTypeId");
- String paymentMethodId = (String) context.get("paymentMethodId");
- BigDecimal maxAmount = (BigDecimal) context.get("maxAmount");
- GenericValue userLogin = (GenericValue) context.get("userLogin");
- Locale locale = (Locale) context.get("locale");
-
- String prefId = null;
-
- try {
- prefId = delegator.getNextSeqId("OrderPaymentPreference");
- } catch (IllegalArgumentException e) {
- return ServiceUtil.returnError(UtilProperties.getMessage(resource_error,
- "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFailure", locale));
- }
-
- Map<String, Object> fields = UtilMisc.<String, Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId, "paymentMethodTypeId",
- paymentMethodTypeId, "paymentMethodId", paymentMethodId, "maxAmount", maxAmount);
-
- if (statusId != null) {
- fields.put("statusId", statusId);
- }
-
- try {
- GenericValue v = delegator.makeValue("OrderPaymentPreference", fields);
- v.set("createdDate", UtilDateTime.nowTimestamp());
- if (userLogin != null) {
- v.set("createdByUserLogin", userLogin.getString("userLoginId"));
- }
- delegator.create(v);
- } catch (GenericEntityException e) {
- result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_ERROR);
- result.put(ModelService.ERROR_MESSAGE, UtilProperties.getMessage(resource,
- "OrderOrderPaymentPreferencesCannotBeCreated", UtilMisc.toMap("errorString", e.getMessage()), locale));
- return ServiceUtil.returnFailure();
- }
- result.put("orderPaymentPreferenceId", prefId);
- result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_SUCCESS);
- return result;
- }
-
/** Service to get order header information as standard results. */
public static Map<String, Object> getOrderHeaderInformation(DispatchContext dctx, Map<String, ? extends Object> context) {
Delegator delegator = dctx.getDelegator();
Re: svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Posted by Nicolas Malin <ni...@nereide.fr>.
Thanks so lot both to correct my mistake :)
Nicolas
On 15/05/2018 08:25, Deepak Dixit wrote:
> Thanks Suraj for reporting,
>
> This has been fixed at r#1831608 and backported to 17.12 and 16.11 as well.
>
>
>
> Thanks & Regards
> --
> Deepak Dixit
> www.hotwax.co
>
> On Tue, May 15, 2018 at 11:43 AM, Suraj Khurana <
> suraj.khurana@hotwaxsystems.com> wrote:
>
>> Hi Nicolas,
>>
>> Permission action is not handled properly in this commit while converting
>> services to entity-auto. It should be _CREATE, _DELETE instead of CREATE,
>> DELETE.
>>
>> I have created a JIRA for the same here
>> <https://issues.apache.org/jira/browse/OFBIZ-10405> and uploaded a patch
>> for review.
>> <https://issues.apache.org/jira/browse/OFBIZ-10405>
>>
>> --
>> Thanks and Regards,
>> *Suraj Khurana* | Omni-channel OMS Technical Expert
>> HotWax Commerce by HotWax Systems
>> Plot no. 80, Scheme no. 78, Vijay Nagar, Indore, M.P. India 452010
>>
>> On Fri, Feb 12, 2016 at 2:45 PM, Jacques Le Roux <
>> jacques.le.roux@les7arts.com> wrote:
>>
>>> Le 12/02/2016 08:23, Nicolas Malin a écrit :
>>>
>>>> Hi Jacques
>>>>
>>>> Le 11/02/2016 23:10, Jacques Le Roux a écrit :
>>>>
>>>>> Le 11/02/2016 22:27, Jacques Le Roux a écrit :
>>>>>
>>>>>> Hi Nicolas, All,
>>>>>>
>>>>>> I globally like the changes, I just have a question.
>>>>>>
>>>>>> With these changes the OrderSecurityError* properties which were used
>>>>>> here are now useless, should we not remove them?
>>>>>> Despite the lost, I think it's reasonable because these errors have
>>>>>> very few chances to appear, and if they appear it's not so hard to
>> find
>>>>>> your way through the log.
>>>>>>
>>>>> Yes I planned to remove all unused label before declare the conversion
>>>> end.
>>>>
>>>>> Oops, I did not see the elephant in the room: they are permission
>>>>> issues. So I guess the message to the user will be less clear. Not sure
>>>>> it's a real issue but we might consider it...
>>>>>
>>>> By default I return to more generic label (
>> http://ofbiz.135035.n4.nabble
>>>> .com/entity-auto-improvement-Act-2-td4655973.html) but if you think
>> that
>>>> it's big lost, I can try to improve the generic order permission
>> service to
>>>> resolve them
>>>>
>>> No, it's OK with me, let's go
>>>
>>> Jacques
>>>
>>>
>>>
>>>> Nicolas
>>>>
>>>>> Jacques
>>>>>
>>>>>
>>>>>> Jacques
>>>>>>
>>>>>>
>>>>>> Le 24/01/2016 20:06, nmalin@apache.org a écrit :
>>>>>>
>>>>>>> Author: nmalin
>>>>>>> Date: Sun Jan 24 19:06:32 2016
>>>>>>> New Revision: 1726542
>>>>>>>
>>>>>>> URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
>>>>>>> Log:
>>>>>>>
>>>>>>>
>>>>>>> I converted following services from simple to entity-auto :
>>>>>>>
>>>>>>> createOrderNotificationLog
>>>>>>> createOrderItemBilling
>>>>>>> createOrderAdjustment
>>>>>>> updateOrderAdjustment
>>>>>>> createOrderAdjustmentBilling
>>>>>>> createOrderShipment
>>>>>>> updateOrderShipment
>>>>>>> deleteOrderShipment
>>>>>>> createCommunicationEventOrder
>>>>>>> removeCommunicationEventOrder
>>>>>>> createOrderItemShipGroup
>>>>>>> createOrderContactMech
>>>>>>> removeOrderContactMech
>>>>>>> createOrderTerm
>>>>>>> removeOrderTerm
>>>>>>> createOrderRequirementCommitment
>>>>>>>
>>>>>>> And from java to entity-auto for createOrderPaymentPreference
>>>>>>> Related issue OFBIZ-6854.
>>>>>>>
>>>>>>> Removed:
>>>>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
>>>>>>> Modified:
>>>>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
>>>>>>> OrderServices.xml
>>>>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
>>>>>>> OrderSimpleMethods.xml
>>>>>>> ofbiz/trunk/applications/order/servicedef/services.xml
>>>>>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>>>>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>>>>>> OrderServices.java
>>>>>>>
>>>>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/
>> order/
>>>>>>> OrderServices.xml
>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>>>> script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&
>>>>>>> r1=1726541&r2=1726542&view=diff
>>>>>>> ============================================================
>>>>>>> ==================
>>>>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/
>> order/OrderServices.xml
>>>>>>> (original)
>>>>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/
>> order/OrderServices.xml
>>>>>>> Sun Jan 24 19:06:32 2016
>>>>>>> @@ -88,56 +88,8 @@ under the License.
>>>>>>> <field-to-result field="orderInfo.totalOrders"
>>>>>>> result-name="totalOrders"/>
>>>>>>> </if-not-empty>
>>>>>>> </simple-method>
>>>>>>> - <simple-method method-name="createOrderShipment"
>>>>>>> short-description="Create OrderShipment">
>>>>>>> - <set value="Create OrderShipment" field="operationName"/>
>>>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunCreateOrderShipment"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> -
>>>>>>> - <make-value value-field="newEntity"
>>>>>>> entity-name="OrderShipment"/>
>>>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>>>> -
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> - <simple-method method-name="updateOrderShipment"
>>>>>>> short-description="Update OrderShipment">
>>>>>>> - <set value="Update OrderShipment" field="operationName"/>
>>>>>>> - <check-permission permission="ORDERMGR" action="_UPDATE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> -
>>>>>>> - <entity-one entity-name="OrderShipment"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <set-nonpk-fields map="parameters"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <store-value value-field="lookedUpValue"/>
>>>>>>> - </simple-method>
>>>>>>> - <simple-method method-name="deleteOrderShipment"
>>>>>>> short-description="Delete OrderShipment">
>>>>>>> - <set value="Delete OrderShipment" field="operationName"/>
>>>>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> -
>>>>>>> - <entity-one entity-name="OrderShipment"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <remove-value value-field="lookedUpValue"/>
>>>>>>> - </simple-method>
>>>>>>> <!-- order requirement methods -->
>>>>>>> - <simple-method method-name="createOrderRequirementCommitment"
>>>>>>> short-description="Create OrderRequirementCommitment">
>>>>>>> - <!-- The permission checking is commented out to make this
>>>>>>> service work also when triggered from ecommerce orders -->
>>>>>>> - <!--
>>>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunCheckCreateOrderRequirement
>>>>>>> Commitment"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> - -->
>>>>>>> - <make-value value-field="newEntity"
>>>>>>> entity-name="OrderRequirementCommitment"/>
>>>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> -
>>>>>>> <simple-method method-name="createRequirementAndCommitment"
>>>>>>> short-description="create a requirement and commitment for it">
>>>>>>> <set value="PRODUCT_REQUIREMENT"
>>>>>>> field="inputMap.requirementTypeId"/>
>>>>>>> @@ -759,20 +711,6 @@ under the License.
>>>>>>> </simple-method>
>>>>>>> <!-- OrderItemShipGroup -->
>>>>>>> - <simple-method method-name="createOrderItemShipGroup"
>>>>>>> short-description="Create OrderItemShipGroup">
>>>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> - <make-value value-field="newEntity"
>>>>>>> entity-name="OrderItemShipGroup"/>
>>>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <if-empty field="newEntity.shipGroupSeqId">
>>>>>>> - <make-next-seq-id value-field="newEntity"
>>>>>>> seq-field-name="shipGroupSeqId"/> <!-- this finds the next
>>>>>>> sub-sequence ID -->
>>>>>>> - <field-to-result field="newEntity.shipGroupSeqId"
>>>>>>> result-name="shipGroupSeqId"/>
>>>>>>> - </if-empty>
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> <simple-method method-name="updateOrderItemShipGroup"
>>>>>>> short-description="Update OrderItemShipGroup">
>>>>>>> <check-permission permission="ORDERMGR" action="_UPDATE">
>>>>>>> <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
>>>>>>> @@ -832,16 +770,6 @@ under the License.
>>>>>>> <field-to-result field="orderItemShipGroupInvRes.
>> promisedDatetime"
>>>>>>> result-name="estimatedShipDate"/>
>>>>>>> </simple-method>
>>>>>>> - <simple-method method-name="createOrderContactMech"
>>>>>>> short-description="Create OrderContactMech">
>>>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunCreateOrderContactMech"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> - <make-value value-field="newEntity"
>>>>>>> entity-name="OrderContactMech"/>
>>>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> -
>>>>>>> <simple-method method-name="removeOrderContactMech"
>>>>>>> short-description="Remove OrderContactMech">
>>>>>>> <check-permission permission="ORDERMGR" action="_DELETE">
>>>>>>> <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
>>>>>>> @@ -861,39 +789,6 @@ under the License.
>>>>>>> <store-value value-field="orderHeaderNote"/>
>>>>>>> </simple-method>
>>>>>>> - <simple-method method-name="createOrderTerm"
>>>>>>> short-description="Create an OrderTerm">
>>>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunCreateOrderTerm"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> -
>>>>>>> - <make-value value-field="newEntity"
>> entity-name="OrderTerm"/>
>>>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <set value="_NA_" set-if-empty="true"
>>>>>>> field="newEntity.orderItemSeqId"/>
>>>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>>>> -
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> - <simple-method method-name="updateOrderTerm"
>>>>>>> short-description="Update OrderTerm">
>>>>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunUpdateOrderTerm"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> -
>>>>>>> - <entity-one entity-name="OrderTerm"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <set-nonpk-fields map="parameters"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <store-value value-field="lookedUpValue"/>
>>>>>>> - </simple-method>
>>>>>>> - <simple-method method-name="removeOrderTerm"
>>>>>>> short-description="Remove OrderTerm">
>>>>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunRemoveOrderTerm"/>
>>>>>>> - </check-permission>
>>>>>>> - <check-errors/>
>>>>>>> -
>>>>>>> - <entity-one entity-name="OrderTerm"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <remove-value value-field="lookedUpValue"/>
>>>>>>> - </simple-method>
>>>>>>> -
>>>>>>> <simple-method method-name="addPaymentMethodToOrder"
>>>>>>> short-description="Create an PaymentMethodToOrder">
>>>>>>> <check-permission permission="ORDERMGR" action="_CREATE">
>>>>>>> <fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
>>>>>>> @@ -1373,7 +1268,7 @@ under the License.
>>>>>>> </iterate>
>>>>>>> </iterate>
>>>>>>> </simple-method>
>>>>>>> -
>>>>>>> +
>>>>>>> <simple-method method-name="updateShippingMethodAndCharges"
>>>>>>> short-description="Updates shipping method and shipping charges from
>> Order
>>>>>>> View page when Shipment is in picked status and items of Order are
>> packed">
>>>>>>> <!-- splitting shipmentMethodAndAmount request parameter
>>>>>>> value that contains "*" symbol
>>>>>>> into "shipmentMethod" and "newAmount".
>>>>>>> @@ -1424,6 +1319,7 @@ under the License.
>>>>>>> </else>
>>>>>>> </if-compare-field>
>>>>>>> </simple-method>
>>>>>>> +
>>>>>>> <simple-method method-name="productAvailabalityByFacility"
>>>>>>> short-description="Calculate ATP and Qoh According For each
>> facility">
>>>>>>> <set field="facilityMap.ownerPartyId"
>>>>>>> from-field="parameters.ownerPartyId"/>
>>>>>>> <find-by-and map="facilityMap" list="facilityList"
>>>>>>> entity-name="Facility"/>
>>>>>>> @@ -1442,7 +1338,7 @@ under the License.
>>>>>>> </iterate>
>>>>>>> <field-to-result field="availabalityList"/>
>>>>>>> </simple-method>
>>>>>>> -
>>>>>>> +
>>>>>>> <simple-method method-name="createOrderPaymentApplication"
>>>>>>> short-description="Create Order Payment Application">
>>>>>>> <entity-one value-field="paymentMap"
>> entity-name="Payment"/>
>>>>>>> <set field="createCtx.amountApplied"
>>>>>>> from-field="paymentMap.amount" type="BigDecimal"/>
>>>>>>> @@ -1459,7 +1355,7 @@ under the License.
>>>>>>> <call-service service-name="createPaymentApplication"
>>>>>>> in-map-name="createCtx"/>
>>>>>>> </if-not-empty>
>>>>>>> </simple-method>
>>>>>>> -
>>>>>>> +
>>>>>>> <simple-method method-name="MoveItemBetweenShipGroups"
>>>>>>> short-description="Move order items between ship groups">
>>>>>>> <entity-one entity-name="OrderItemShipGroupAssoc"
>>>>>>> value-field="orderItemShipGroupAssoc">
>>>>>>> <field-map field-name="orderId"
>>>>>>> from-field="parameters.orderId"/>
>>>>>>> @@ -1483,7 +1379,7 @@ under the License.
>>>>>>> <set field="map.shipGroupSeqId"
>>>>>>> from-field="parameters.toGroupIndex"/>
>>>>>>> <set field="map.quantity" value="${
>> orderItemShipGroupAssoc.quantity
>>>>>>> + parameters.quantity}" type="BigDecimal"/>
>>>>>>> <call-service service-name="updateOrderItemShipGroupAssoc"
>>>>>>> in-map-name="map"/>
>>>>>>> -
>>>>>>> +
>>>>>>> <entity-one entity-name="OrderItemShipGroupAssoc"
>>>>>>> value-field="orderItemShipGroupAssoc">
>>>>>>> <field-map field-name="orderId"
>>>>>>> from-field="parameters.orderId"/>
>>>>>>> <field-map field-name="orderItemSeqId"
>>>>>>> from-field="parameters.orderItemSeqId"/>
>>>>>>> @@ -1495,7 +1391,7 @@ under the License.
>>>>>>> </add-error>
>>>>>>> </if-empty>
>>>>>>> <check-errors/>
>>>>>>> -
>>>>>>> +
>>>>>>> <clear-field field="map"/>
>>>>>>> <set field="map.orderId" from-field="parameters.orderId"/>
>>>>>>> <set field="map.orderItemSeqId"
>>>>>>> from-field="parameters.orderItemSeqId"/>
>>>>>>> @@ -1503,5 +1399,5 @@ under the License.
>>>>>>> <set field="map.quantity" value="${
>> orderItemShipGroupAssoc.quantity
>>>>>>> - parameters.quantity}" type="BigDecimal"/>
>>>>>>> <call-service service-name="updateOrderItemShipGroupAssoc"
>>>>>>> in-map-name="map"/>
>>>>>>> </simple-method>
>>>>>>> -
>>>>>>> +
>>>>>>> </simple-methods>
>>>>>>>
>>>>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/
>> order/
>>>>>>> OrderSimpleMethods.xml
>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>>>> script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=
>>>>>>> 1726542&r1=1726541&r2=1726542&view=diff
>>>>>>> ============================================================
>>>>>>> ==================
>>>>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/
>> order/OrderSimpleMethods.xml
>>>>>>> (original)
>>>>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/
>> order/OrderSimpleMethods.xml
>>>>>>> Sun Jan 24 19:06:32 2016
>>>>>>> @@ -44,33 +44,12 @@ under the License.
>>>>>>> </else>
>>>>>>> </if-compare>
>>>>>>> </simple-method>
>>>>>>> -
>>>>>>> - <simple-method method-name="createOrderAdjustment"
>>>>>>> short-description="Create an OrderAdjustment">
>>>>>>> -
>>>>>>> - <make-value entity-name="OrderAdjustment"
>>>>>>> value-field="newEntity"/>
>>>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>>>> -
>>>>>>> - <sequenced-id sequence-name="OrderAdjustment"
>>>>>>> field="newEntity.orderAdjustmentId"/>
>>>>>>> - <field-to-result field="newEntity.orderAdjustmentId"
>>>>>>> result-name="orderAdjustmentId"/>
>>>>>>> -
>>>>>>> - <now-timestamp field="newEntity.createdDate"/>
>>>>>>> - <set from-field="userLogin.userLoginId"
>>>>>>> field="newEntity.createdByUserLogin"/>
>>>>>>> -
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> - <simple-method method-name="updateOrderAdjustment"
>>>>>>> short-description="Update an OrderAdjustment">
>>>>>>> - <check-permission permission="ORDERMGR"
>>>>>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></
>>>>>>> check-permission>
>>>>>>> - <check-errors/>
>>>>>>> - <entity-one entity-name="OrderAdjustment"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <set-nonpk-fields map="parameters"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> - <store-value value-field="lookedUpValue"/>
>>>>>>> - </simple-method>
>>>>>>> <simple-method method-name="deleteOrderAdjustment"
>>>>>>> short-description="Delete an OrderAdjustment">
>>>>>>> <check-permission permission="ORDERMGR"
>>>>>>> action="_DELETE"><fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></
>>>>>>> check-permission>
>>>>>>> <check-errors/>
>>>>>>> <entity-one entity-name="OrderAdjustment"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> <remove-value value-field="lookedUpValue"/>
>>>>>>> -
>>>>>>> +
>>>>>>> <if-not-empty field="parameters.productPromoCodeId">
>>>>>>> <entity-one entity-name="OrderProductPromoCode"
>>>>>>> value-field="lookedUpValue"/>
>>>>>>> <if-not-empty field="lookedUpValue">
>>>>>>> @@ -79,30 +58,6 @@ under the License.
>>>>>>> </if-not-empty>
>>>>>>> </simple-method>
>>>>>>> - <simple-method method-name="createOrderAdjustmentBilling"
>>>>>>> short-description="Create an OrderAdjustmentBilling">
>>>>>>> - <make-value entity-name="OrderAdjustmentBilling"
>>>>>>> value-field="newEntity"/>
>>>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> -
>>>>>>> - <simple-method method-name="createOrderItemBilling"
>>>>>>> short-description="Create an OrderItemBilling">
>>>>>>> - <make-value entity-name="OrderItemBilling"
>>>>>>> value-field="newEntity"/>
>>>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>>>> - <create-value value-field="newEntity"/>
>>>>>>> - </simple-method>
>>>>>>> -
>>>>>>> - <simple-method method-name="createNotificationLog"
>>>>>>> short-description="Log an order notification" login-required="false">
>>>>>>> - <make-value entity-name="OrderNotification"
>>>>>>> value-field="orderNotification"/>
>>>>>>> - <sequenced-id sequence-name="OrderNotification"
>>>>>>> field="orderNotification.orderNotificationId"/>
>>>>>>> - <set from-field="parameters.orderId"
>>>>>>> field="orderNotification.orderId"/>
>>>>>>> - <set from-field="parameters.emailType"
>>>>>>> field="orderNotification.emailType"/>
>>>>>>> - <set from-field="parameters.comments"
>>>>>>> field="orderNotification.comments"/>
>>>>>>> - <now-timestamp field="orderNotification.notificationDate"/>
>>>>>>> - <create-value value-field="orderNotification"/>
>>>>>>> - </simple-method>
>>>>>>> -
>>>>>>> <simple-method method-name="updateOrderStatusFromReceipt"
>>>>>>> short-description="Update Order Status From ShipmentReceipt">
>>>>>>> <check-permission permission="ORDERMGR"
>>>>>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>>>>>> property="OrderSecurityErrorToRunUpdateOrderStatusFromReceip
>>>>>>> t"/></check-permission>
>>>>>>> <check-errors/>
>>>>>>>
>>>>>>> Modified: ofbiz/trunk/applications/order/servicedef/services.xml
>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>>>> servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>>>>>> ============================================================
>>>>>>> ==================
>>>>>>> --- ofbiz/trunk/applications/order/servicedef/services.xml
>> (original)
>>>>>>> +++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan
>> 24
>>>>>>> 19:06:32 2016
>>>>>>> @@ -76,8 +76,7 @@ under the License.
>>>>>>> <attribute name="assignedRoleTypeId" type="String"
>> mode="IN"
>>>>>>> optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="createOrderNotificationLog" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderSimpleMethods.xml"
>>>>>>> invoke="createNotificationLog">
>>>>>>> + <service name="createOrderNotificationLog" engine="entity-auto"
>>>>>>> default-entity-name="OrderNotification" invoke="create">
>>>>>>> <description>Logs when a notification was
>> sent</description>
>>>>>>> <attribute name="orderId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> <attribute name="emailType" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> @@ -164,17 +163,10 @@ under the License.
>>>>>>> <attribute name="orderId" type="String" mode="OUT"
>>>>>>> optional="false"/>
>>>>>>> </service>
>>>>>>> - <service name="createOrderItemBilling" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderSimpleMethods.xml"
>>>>>>> invoke="createOrderItemBilling">
>>>>>>> + <service name="createOrderItemBilling"
>>>>>>> default-entity-name="OrderItemBilling" engine="entity-auto"
>>>>>>> invoke="create">
>>>>>>> <description>Create a new order item billing
>>>>>>> record</description>
>>>>>>> - <attribute name="orderId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> - <attribute name="orderItemSeqId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> - <attribute name="invoiceId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> - <attribute name="quantity" type="BigDecimal" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> - <attribute name="amount" type="BigDecimal" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> - <attribute name="itemIssuanceId" type="String" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> - <attribute name="shipmentReceiptId" type="String" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> + <auto-attributes mode="IN" include="pk"/>
>>>>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>>>>> </service>
>>>>>>> <service name="orderAdjustmentPermissionCheck"
>> engine="simple"
>>>>>>> @@ -183,8 +175,7 @@ under the License.
>>>>>>> <implements service="permissionInterface"/>
>>>>>>> </service>
>>>>>>> - <service name="createOrderAdjustment"
>>>>>>> default-entity-name="OrderAdjustment" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderSimpleMethods.xml"
>>>>>>> invoke="createOrderAdjustment">
>>>>>>> + <service name="createOrderAdjustment"
>>>>>>> default-entity-name="OrderAdjustment" engine="entity-auto"
>>>>>>> invoke="create">
>>>>>>> <description>Creates a new order adjustment
>>>>>>> record</description>
>>>>>>> <permission-service service-name="
>> orderAdjustmentPermissionCheck"
>>>>>>> main-action="CREATE"/>
>>>>>>> <auto-attributes mode="OUT" include="pk" optional="false"/>
>>>>>>> @@ -192,9 +183,9 @@ under the License.
>>>>>>> <override name="orderAdjustmentTypeId" optional="false"/>
>>>>>>> <override name="orderId" optional="false"/>
>>>>>>> </service>
>>>>>>> - <service name="updateOrderAdjustment"
>>>>>>> default-entity-name="OrderAdjustment" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderSimpleMethods.xml"
>>>>>>> invoke="updateOrderAdjustment">
>>>>>>> + <service name="updateOrderAdjustment"
>>>>>>> default-entity-name="OrderAdjustment" engine="entity-auto"
>>>>>>> invoke="update">
>>>>>>> <description>Update an order adjustment
>> record</description>
>>>>>>> + <permission-service service-name="
>> orderAdjustmentPermissionCheck"
>>>>>>> main-action="UPDATE"/>
>>>>>>> <auto-attributes mode="IN" include="pk" optional="false"/>
>>>>>>> <auto-attributes mode="IN" include="nonpk"
>> optional="true"/>
>>>>>>> <override name="orderId" optional="false"/>
>>>>>>> @@ -207,13 +198,10 @@ under the License.
>>>>>>> <attribute name="productPromoCodeId" type="String"
>> mode="IN"
>>>>>>> optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="createOrderAdjustmentBilling" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderSimpleMethods.xml"
>>>>>>> invoke="createOrderAdjustmentBilling">
>>>>>>> + <service name="createOrderAdjustmentBilling"
>>>>>>> default-entity-name="OrderAdjustmentBilling" engine="entity-auto"
>>>>>>> invoke="create">
>>>>>>> <description>Create a new order adjustment billing
>>>>>>> record</description>
>>>>>>> - <attribute name="orderAdjustmentId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> - <attribute name="invoiceId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> - <attribute name="amount" type="BigDecimal" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> + <auto-attributes mode="IN" include="pk"/>
>>>>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>>>>> </service>
>>>>>>> <service name="createPaymentFromPreference" engine="java"
>>>>>>> @@ -435,15 +423,10 @@ under the License.
>>>>>>> <attribute name="roleTypeId" type="String" mode="IN"/>
>>>>>>> </service>
>>>>>>> - <service name="createOrderPaymentPreference" engine="java"
>>>>>>> - location="org.ofbiz.order.order.OrderServices"
>>>>>>> invoke="createPaymentPreference">
>>>>>>> + <service name="createOrderPaymentPreference"
>>>>>>> default-entity-name="OrderPaymentPreference" engine="entity-auto"
>>>>>>> invoke="create">
>>>>>>> <description>Creates an order payment
>>>>>>> preference</description>
>>>>>>> - <attribute name="orderId" type="String" mode="IN"/>
>>>>>>> - <attribute name="paymentMethodTypeId" type="String"
>>>>>>> mode="IN"/>
>>>>>>> - <attribute name="paymentMethodId" type="String" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> - <attribute name="statusId" type="String" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> - <attribute name="maxAmount" type="BigDecimal" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> - <attribute name="orderPaymentPreferenceId" type="String"
>>>>>>> mode="OUT"/>
>>>>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>>>>> + <auto-attributes include="pk" mode="OUT"/>
>>>>>>> </service>
>>>>>>> <service name="createOrderNote" engine="java"
>>>>>>> @@ -568,20 +551,17 @@ under the License.
>>>>>>> </service>
>>>>>>> <!-- OrderShipment Services -->
>>>>>>> - <service name="createOrderShipment" default-entity-name="
>> OrderShipment"
>>>>>>> engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="createOrderShipment" auth="true">
>>>>>>> + <service name="createOrderShipment" engine="entity-auto"
>>>>>>> default-entity-name="OrderShipment" invoke="create" auth="true">
>>>>>>> <description>Create OrderShipment</description>
>>>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>>> <auto-attributes include="nonpk" mode="IN"
>> optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="updateOrderShipment" default-entity-name="
>> OrderShipment"
>>>>>>> engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="updateOrderShipment" auth="true">
>>>>>>> + <service name="updateOrderShipment" engine="entity-auto"
>>>>>>> default-entity-name="OrderShipment" invoke="update" auth="true">
>>>>>>> <description>Update OrderShipment</description>
>>>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>>> <auto-attributes include="nonpk" mode="IN"
>> optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="deleteOrderShipment" default-entity-name="
>> OrderShipment"
>>>>>>> engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="deleteOrderShipment" auth="true">
>>>>>>> + <service name="deleteOrderShipment" engine="entity-auto"
>>>>>>> default-entity-name="OrderShipment" invoke="delete" auth="true">
>>>>>>> <description>Delete OrderShipment</description>
>>>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>>> </service>
>>>>>>> @@ -667,21 +647,27 @@ under the License.
>>>>>>> </service>
>>>>>>> <!-- CommunicationEvents related to orders -->
>>>>>>> - <service name="createCommunicationEventOrder" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/communica
>>>>>>> tion/CommunicationEventServices.xml" invoke="
>> createCommunicationEventOrder"
>>>>>>> auth="true">
>>>>>>> + <service name="createCommunicationEventOrder"
>>>>>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
>>>>>>> invoke="create" auth="true">
>>>>>>> <description>Create a Communication Event
>> Order</description>
>>>>>>> - <auto-attributes entity-name="CommunicationEventOrder"
>>>>>>> include="pk" mode="IN" optional="false"/>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="CREATE"/>
>>>>>>> + </required-permissions>
>>>>>>> + <auto-attributes entity-name="CommunicationEventOrder"
>>>>>>> include="pk" mode="IN"/>
>>>>>>> </service>
>>>>>>> - <service name="removeCommunicationEventOrder" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/communica
>>>>>>> tion/CommunicationEventServices.xml" invoke="
>> removeCommunicationEventOrder"
>>>>>>> auth="true">
>>>>>>> + <service name="removeCommunicationEventOrder"
>>>>>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
>>>>>>> invoke="delete" auth="true">
>>>>>>> <description>Remove a Communication Event
>> Order</description>
>>>>>>> - <auto-attributes entity-name="CommunicationEventOrder"
>>>>>>> include="pk" mode="IN" optional="false"/>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="DELETE"/>
>>>>>>> + </required-permissions>
>>>>>>> + <auto-attributes entity-name="CommunicationEventOrder"
>>>>>>> include="pk" mode="IN"/>
>>>>>>> </service>
>>>>>>> <!-- Order Shipping and Contacts -->
>>>>>>> - <service name="createOrderItemShipGroup"
>>>>>>> default-entity-name="OrderItemShipGroup" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="createOrderItemShipGroup" auth="true">
>>>>>>> + <service name="createOrderItemShipGroup"
>>>>>>> default-entity-name="OrderItemShipGroup" engine="entity-auto"
>>>>>>> invoke="create" auth="true">
>>>>>>> <description>Creates a new OrderItemShipGroup.</
>> description>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="CREATE"/>
>>>>>>> + </required-permissions>
>>>>>>> <auto-attributes mode="IN" include="nonpk"
>> optional="true"/>
>>>>>>> <attribute name="orderId" type="String" mode="IN"
>>>>>>> optional="false"/>
>>>>>>> <attribute name="shipGroupSeqId" type="String" mode="INOUT"
>>>>>>> optional="true"/>
>>>>>>> @@ -696,42 +682,52 @@ under the License.
>>>>>>> <attribute name="contactMechPurposeTypeId" type="String"
>>>>>>> mode="IN" optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="createOrderContactMech"
>>>>>>> default-entity-name="OrderContactMech" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="createOrderContactMech" auth="true">
>>>>>>> + <service name="createOrderContactMech"
>>>>>>> default-entity-name="OrderContactMech" engine="entity-auto"
>>>>>>> invoke="create" auth="true">
>>>>>>> <description>Create Order Contact Mech</description>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="CREATE"/>
>>>>>>> + </required-permissions>
>>>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>>> </service>
>>>>>>> - <service name="updateOrderContactMech" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="updateOrderContactMech" auth="true"
>>>>>>> default-entity-name="OrderContactMech">
>>>>>>> + <service name="updateOrderContactMech"
>>>>>>> default-entity-name="OrderContactMech" engine="simple"
>>>>>>> + location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="updateOrderContactMech" auth="true">
>>>>>>> <description>Update Order Contact Mech</description>
>>>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>>> <attribute name="oldContactMechId" type="String" mode="IN"
>>>>>>> optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="removeOrderContactMech"
>>>>>>> default-entity-name="OrderContactMech" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="removeOrderContactMech" auth="true">
>>>>>>> + <service name="removeOrderContactMech"
>>>>>>> default-entity-name="OrderContactMech" engine="entity-auto"
>>>>>>> invoke="delete" auth="true">
>>>>>>> <description>Remove Order Contact Mech</description>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="DELETE"/>
>>>>>>> + </required-permissions>
>>>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>>> </service>
>>>>>>> <!-- OrderTerms -->
>>>>>>> - <service name="createOrderTerm" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="createOrderTerm" auth="true">
>>>>>>> + <service name="createOrderTerm" default-entity-name="OrderTerm"
>>>>>>> engine="entity-auto" invoke="create" auth="true">
>>>>>>> <description>Create an Order Term</description>
>>>>>>> - <auto-attributes entity-name="OrderTerm" include="pk"
>>>>>>> mode="IN" optional="false"/>
>>>>>>> - <auto-attributes entity-name="OrderTerm" include="nonpk"
>>>>>>> mode="IN" optional="true"/>
>>>>>>> - <override name="orderItemSeqId" optional="true"/>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="CREATE"/>
>>>>>>> + </required-permissions>
>>>>>>> + <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>>>>> + <override name="orderItemSeqId" optional="true"
>>>>>>> default-value="_NA_"/>
>>>>>>> </service>
>>>>>>> - <service name="updateOrderTerm" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="updateOrderTerm" auth="true">
>>>>>>> + <service name="updateOrderTerm" default-entity-name="OrderTerm"
>>>>>>> engine="entity-auto" invoke="update" auth="true">
>>>>>>> <description>Update an Order Term</description>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="UPDATE"/>
>>>>>>> + </required-permissions>
>>>>>>> <auto-attributes entity-name="OrderTerm" include="pk"
>>>>>>> mode="IN" optional="false"/>
>>>>>>> <auto-attributes entity-name="OrderTerm" include="nonpk"
>>>>>>> mode="IN" optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="removeOrderTerm" engine="simple"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="removeOrderTerm" auth="true">
>>>>>>> + <service name="removeOrderTerm" default-entity-name="OrderTerm"
>>>>>>> engine="entity-auto" invoke="delete" auth="true">
>>>>>>> <description>Remove an Order Term</description>
>>>>>>> + <required-permissions join-type="AND">
>>>>>>> + <check-permission permission="ORDERMGR"
>> action="DELETE"/>
>>>>>>> + </required-permissions>
>>>>>>> <auto-attributes entity-name="OrderTerm" include="pk"
>>>>>>> mode="IN" optional="false"/>
>>>>>>> </service>
>>>>>>>
>>>>>>> Modified: ofbiz/trunk/applications/order/servicedef/services_
>> requireme
>>>>>>> nt.xml
>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>>>> servicedef/services_requirement.xml?rev=1726542&r1=1726541&
>>>>>>> r2=1726542&view=diff
>>>>>>> ============================================================
>>>>>>> ==================
>>>>>>> --- ofbiz/trunk/applications/order/servicedef/services_
>> requirement.xml
>>>>>>> (original)
>>>>>>> +++ ofbiz/trunk/applications/order/servicedef/services_
>> requirement.xml
>>>>>>> Sun Jan 24 19:06:32 2016
>>>>>>> @@ -105,8 +105,7 @@ under the License.
>>>>>>> <attribute mode="OUT" name="quantityTotal"
>> type="BigDecimal"
>>>>>>> optional="true"/>
>>>>>>> <attribute mode="OUT" name="amountTotal" type="BigDecimal"
>>>>>>> optional="true"/>
>>>>>>> </service>
>>>>>>> - <service name="createOrderRequirementCommitment"
>> engine="simple"
>>>>>>> auth="true"
>>>>>>> - location="component://order/script/org/ofbiz/order/order/
>> OrderServices.xml"
>>>>>>> invoke="createOrderRequirementCommitment">
>>>>>>> + <service name="createOrderRequirementCommitment"
>>>>>>> engine="entity-auto" auth="true" default-entity-name="
>> OrderRequirementCommitment"
>>>>>>> invoke="create">
>>>>>>> <auto-attributes entity-name="OrderRequirementCommitment"
>>>>>>> include="pk" mode="IN" optional="false"/>
>>>>>>> <auto-attributes entity-name="OrderRequirementCommitment"
>>>>>>> include="nonpk" mode="IN" optional="true"/>
>>>>>>> </service>
>>>>>>>
>>>>>>> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>>>>>> OrderServices.java
>>>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>>>> src/org/ofbiz/order/order/OrderServices.java?rev=1726542
>>>>>>> &r1=1726541&r2=1726542&view=diff
>>>>>>> ============================================================
>>>>>>> ==================
>>>>>>> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/
>> order/OrderServices.java
>>>>>>> (original)
>>>>>>> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/
>> order/OrderServices.java
>>>>>>> Sun Jan 24 19:06:32 2016
>>>>>>> @@ -2824,52 +2824,6 @@ public class OrderServices {
>>>>>>> return ServiceUtil.returnSuccess();
>>>>>>> }
>>>>>>> - /** Service to create an order payment preference */
>>>>>>> - public static Map<String, Object> createPaymentPreference(
>> DispatchContext
>>>>>>> ctx, Map<String, ? extends Object> context) {
>>>>>>> - Map<String, Object> result = new HashMap<String, Object>();
>>>>>>> - Delegator delegator = ctx.getDelegator();
>>>>>>> - String orderId = (String) context.get("orderId");
>>>>>>> - String statusId = (String) context.get("statusId");
>>>>>>> - String paymentMethodTypeId = (String)
>>>>>>> context.get("paymentMethodTypeId");
>>>>>>> - String paymentMethodId = (String)
>>>>>>> context.get("paymentMethodId");
>>>>>>> - BigDecimal maxAmount = (BigDecimal)
>> context.get("maxAmount");
>>>>>>> - GenericValue userLogin = (GenericValue)
>>>>>>> context.get("userLogin");
>>>>>>> - Locale locale = (Locale) context.get("locale");
>>>>>>> -
>>>>>>> - String prefId = null;
>>>>>>> -
>>>>>>> - try {
>>>>>>> - prefId = delegator.getNextSeqId("OrderP
>>>>>>> aymentPreference");
>>>>>>> - } catch (IllegalArgumentException e) {
>>>>>>> - return ServiceUtil.returnError(UtilPr
>>>>>>> operties.getMessage(resource_error,
>>>>>>> - "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFa
>> ilure",
>>>>>>> locale));
>>>>>>> - }
>>>>>>> -
>>>>>>> - Map<String, Object> fields = UtilMisc.<String,
>>>>>>> Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId,
>>>>>>> "paymentMethodTypeId",
>>>>>>> - paymentMethodTypeId, "paymentMethodId",
>>>>>>> paymentMethodId, "maxAmount", maxAmount);
>>>>>>> -
>>>>>>> - if (statusId != null) {
>>>>>>> - fields.put("statusId", statusId);
>>>>>>> - }
>>>>>>> -
>>>>>>> - try {
>>>>>>> - GenericValue v = delegator.makeValue("
>> OrderPaymentPreference",
>>>>>>> fields);
>>>>>>> - v.set("createdDate", UtilDateTime.nowTimestamp());
>>>>>>> - if (userLogin != null) {
>>>>>>> - v.set("createdByUserLogin",
>>>>>>> userLogin.getString("userLoginId"));
>>>>>>> - }
>>>>>>> - delegator.create(v);
>>>>>>> - } catch (GenericEntityException e) {
>>>>>>> - result.put(ModelService.RESPONSE_MESSAGE,
>>>>>>> ModelService.RESPOND_ERROR);
>>>>>>> - result.put(ModelService.ERROR_MESSAGE,
>>>>>>> UtilProperties.getMessage(resource,
>>>>>>> - "OrderOrderPaymentPreferencesCannotBeCreated",
>>>>>>> UtilMisc.toMap("errorString", e.getMessage()), locale));
>>>>>>> - return ServiceUtil.returnFailure();
>>>>>>> - }
>>>>>>> - result.put("orderPaymentPreferenceId", prefId);
>>>>>>> - result.put(ModelService.RESPONSE_MESSAGE,
>>>>>>> ModelService.RESPOND_SUCCESS);
>>>>>>> - return result;
>>>>>>> - }
>>>>>>> -
>>>>>>> /** Service to get order header information as standard
>> results.
>>>>>>> */
>>>>>>> public static Map<String, Object> getOrderHeaderInformation(
>> DispatchContext
>>>>>>> dctx, Map<String, ? extends Object> context) {
>>>>>>> Delegator delegator = dctx.getDelegator();
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
Re: svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Posted by Deepak Dixit <de...@hotwaxsystems.com>.
Thanks Suraj for reporting,
This has been fixed at r#1831608 and backported to 17.12 and 16.11 as well.
Thanks & Regards
--
Deepak Dixit
www.hotwax.co
On Tue, May 15, 2018 at 11:43 AM, Suraj Khurana <
suraj.khurana@hotwaxsystems.com> wrote:
> Hi Nicolas,
>
> Permission action is not handled properly in this commit while converting
> services to entity-auto. It should be _CREATE, _DELETE instead of CREATE,
> DELETE.
>
> I have created a JIRA for the same here
> <https://issues.apache.org/jira/browse/OFBIZ-10405> and uploaded a patch
> for review.
> <https://issues.apache.org/jira/browse/OFBIZ-10405>
>
> --
> Thanks and Regards,
> *Suraj Khurana* | Omni-channel OMS Technical Expert
> HotWax Commerce by HotWax Systems
> Plot no. 80, Scheme no. 78, Vijay Nagar, Indore, M.P. India 452010
>
> On Fri, Feb 12, 2016 at 2:45 PM, Jacques Le Roux <
> jacques.le.roux@les7arts.com> wrote:
>
> > Le 12/02/2016 08:23, Nicolas Malin a écrit :
> >
> >>
> >> Hi Jacques
> >>
> >> Le 11/02/2016 23:10, Jacques Le Roux a écrit :
> >>
> >>> Le 11/02/2016 22:27, Jacques Le Roux a écrit :
> >>>
> >>>> Hi Nicolas, All,
> >>>>
> >>>> I globally like the changes, I just have a question.
> >>>>
> >>>> With these changes the OrderSecurityError* properties which were used
> >>>> here are now useless, should we not remove them?
> >>>> Despite the lost, I think it's reasonable because these errors have
> >>>> very few chances to appear, and if they appear it's not so hard to
> find
> >>>> your way through the log.
> >>>>
> >>> Yes I planned to remove all unused label before declare the conversion
> >> end.
> >>
> >>>
> >>> Oops, I did not see the elephant in the room: they are permission
> >>> issues. So I guess the message to the user will be less clear. Not sure
> >>> it's a real issue but we might consider it...
> >>>
> >> By default I return to more generic label (
> http://ofbiz.135035.n4.nabble
> >> .com/entity-auto-improvement-Act-2-td4655973.html) but if you think
> that
> >> it's big lost, I can try to improve the generic order permission
> service to
> >> resolve them
> >>
> >
> > No, it's OK with me, let's go
> >
> > Jacques
> >
> >
> >
> >> Nicolas
> >>
> >>>
> >>> Jacques
> >>>
> >>>
> >>>> Jacques
> >>>>
> >>>>
> >>>> Le 24/01/2016 20:06, nmalin@apache.org a écrit :
> >>>>
> >>>>> Author: nmalin
> >>>>> Date: Sun Jan 24 19:06:32 2016
> >>>>> New Revision: 1726542
> >>>>>
> >>>>> URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
> >>>>> Log:
> >>>>>
> >>>>>
> >>>>> I converted following services from simple to entity-auto :
> >>>>>
> >>>>> createOrderNotificationLog
> >>>>> createOrderItemBilling
> >>>>> createOrderAdjustment
> >>>>> updateOrderAdjustment
> >>>>> createOrderAdjustmentBilling
> >>>>> createOrderShipment
> >>>>> updateOrderShipment
> >>>>> deleteOrderShipment
> >>>>> createCommunicationEventOrder
> >>>>> removeCommunicationEventOrder
> >>>>> createOrderItemShipGroup
> >>>>> createOrderContactMech
> >>>>> removeOrderContactMech
> >>>>> createOrderTerm
> >>>>> removeOrderTerm
> >>>>> createOrderRequirementCommitment
> >>>>>
> >>>>> And from java to entity-auto for createOrderPaymentPreference
> >>>>> Related issue OFBIZ-6854.
> >>>>>
> >>>>> Removed:
> >>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
> >>>>> Modified:
> >>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
> >>>>> OrderServices.xml
> >>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
> >>>>> OrderSimpleMethods.xml
> >>>>> ofbiz/trunk/applications/order/servicedef/services.xml
> >>>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
> >>>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> >>>>> OrderServices.java
> >>>>>
> >>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/
> order/
> >>>>> OrderServices.xml
> >>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
> >>>>> script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&
> >>>>> r1=1726541&r2=1726542&view=diff
> >>>>> ============================================================
> >>>>> ==================
> >>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/
> order/OrderServices.xml
> >>>>> (original)
> >>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/
> order/OrderServices.xml
> >>>>> Sun Jan 24 19:06:32 2016
> >>>>> @@ -88,56 +88,8 @@ under the License.
> >>>>> <field-to-result field="orderInfo.totalOrders"
> >>>>> result-name="totalOrders"/>
> >>>>> </if-not-empty>
> >>>>> </simple-method>
> >>>>> - <simple-method method-name="createOrderShipment"
> >>>>> short-description="Create OrderShipment">
> >>>>> - <set value="Create OrderShipment" field="operationName"/>
> >>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunCreateOrderShipment"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> -
> >>>>> - <make-value value-field="newEntity"
> >>>>> entity-name="OrderShipment"/>
> >>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> >>>>> -
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> - <simple-method method-name="updateOrderShipment"
> >>>>> short-description="Update OrderShipment">
> >>>>> - <set value="Update OrderShipment" field="operationName"/>
> >>>>> - <check-permission permission="ORDERMGR" action="_UPDATE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> -
> >>>>> - <entity-one entity-name="OrderShipment"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <set-nonpk-fields map="parameters"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <store-value value-field="lookedUpValue"/>
> >>>>> - </simple-method>
> >>>>> - <simple-method method-name="deleteOrderShipment"
> >>>>> short-description="Delete OrderShipment">
> >>>>> - <set value="Delete OrderShipment" field="operationName"/>
> >>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> -
> >>>>> - <entity-one entity-name="OrderShipment"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <remove-value value-field="lookedUpValue"/>
> >>>>> - </simple-method>
> >>>>> <!-- order requirement methods -->
> >>>>> - <simple-method method-name="createOrderRequirementCommitment"
> >>>>> short-description="Create OrderRequirementCommitment">
> >>>>> - <!-- The permission checking is commented out to make this
> >>>>> service work also when triggered from ecommerce orders -->
> >>>>> - <!--
> >>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunCheckCreateOrderRequirement
> >>>>> Commitment"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> - -->
> >>>>> - <make-value value-field="newEntity"
> >>>>> entity-name="OrderRequirementCommitment"/>
> >>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> -
> >>>>> <simple-method method-name="createRequirementAndCommitment"
> >>>>> short-description="create a requirement and commitment for it">
> >>>>> <set value="PRODUCT_REQUIREMENT"
> >>>>> field="inputMap.requirementTypeId"/>
> >>>>> @@ -759,20 +711,6 @@ under the License.
> >>>>> </simple-method>
> >>>>> <!-- OrderItemShipGroup -->
> >>>>> - <simple-method method-name="createOrderItemShipGroup"
> >>>>> short-description="Create OrderItemShipGroup">
> >>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> - <make-value value-field="newEntity"
> >>>>> entity-name="OrderItemShipGroup"/>
> >>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <if-empty field="newEntity.shipGroupSeqId">
> >>>>> - <make-next-seq-id value-field="newEntity"
> >>>>> seq-field-name="shipGroupSeqId"/> <!-- this finds the next
> >>>>> sub-sequence ID -->
> >>>>> - <field-to-result field="newEntity.shipGroupSeqId"
> >>>>> result-name="shipGroupSeqId"/>
> >>>>> - </if-empty>
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> <simple-method method-name="updateOrderItemShipGroup"
> >>>>> short-description="Update OrderItemShipGroup">
> >>>>> <check-permission permission="ORDERMGR" action="_UPDATE">
> >>>>> <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
> >>>>> @@ -832,16 +770,6 @@ under the License.
> >>>>> <field-to-result field="orderItemShipGroupInvRes.
> promisedDatetime"
> >>>>> result-name="estimatedShipDate"/>
> >>>>> </simple-method>
> >>>>> - <simple-method method-name="createOrderContactMech"
> >>>>> short-description="Create OrderContactMech">
> >>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunCreateOrderContactMech"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> - <make-value value-field="newEntity"
> >>>>> entity-name="OrderContactMech"/>
> >>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> -
> >>>>> <simple-method method-name="removeOrderContactMech"
> >>>>> short-description="Remove OrderContactMech">
> >>>>> <check-permission permission="ORDERMGR" action="_DELETE">
> >>>>> <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
> >>>>> @@ -861,39 +789,6 @@ under the License.
> >>>>> <store-value value-field="orderHeaderNote"/>
> >>>>> </simple-method>
> >>>>> - <simple-method method-name="createOrderTerm"
> >>>>> short-description="Create an OrderTerm">
> >>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunCreateOrderTerm"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> -
> >>>>> - <make-value value-field="newEntity"
> entity-name="OrderTerm"/>
> >>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <set value="_NA_" set-if-empty="true"
> >>>>> field="newEntity.orderItemSeqId"/>
> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> >>>>> -
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> - <simple-method method-name="updateOrderTerm"
> >>>>> short-description="Update OrderTerm">
> >>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunUpdateOrderTerm"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> -
> >>>>> - <entity-one entity-name="OrderTerm"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <set-nonpk-fields map="parameters"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <store-value value-field="lookedUpValue"/>
> >>>>> - </simple-method>
> >>>>> - <simple-method method-name="removeOrderTerm"
> >>>>> short-description="Remove OrderTerm">
> >>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
> >>>>> - <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunRemoveOrderTerm"/>
> >>>>> - </check-permission>
> >>>>> - <check-errors/>
> >>>>> -
> >>>>> - <entity-one entity-name="OrderTerm"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <remove-value value-field="lookedUpValue"/>
> >>>>> - </simple-method>
> >>>>> -
> >>>>> <simple-method method-name="addPaymentMethodToOrder"
> >>>>> short-description="Create an PaymentMethodToOrder">
> >>>>> <check-permission permission="ORDERMGR" action="_CREATE">
> >>>>> <fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
> >>>>> @@ -1373,7 +1268,7 @@ under the License.
> >>>>> </iterate>
> >>>>> </iterate>
> >>>>> </simple-method>
> >>>>> -
> >>>>> +
> >>>>> <simple-method method-name="updateShippingMethodAndCharges"
> >>>>> short-description="Updates shipping method and shipping charges from
> Order
> >>>>> View page when Shipment is in picked status and items of Order are
> packed">
> >>>>> <!-- splitting shipmentMethodAndAmount request parameter
> >>>>> value that contains "*" symbol
> >>>>> into "shipmentMethod" and "newAmount".
> >>>>> @@ -1424,6 +1319,7 @@ under the License.
> >>>>> </else>
> >>>>> </if-compare-field>
> >>>>> </simple-method>
> >>>>> +
> >>>>> <simple-method method-name="productAvailabalityByFacility"
> >>>>> short-description="Calculate ATP and Qoh According For each
> facility">
> >>>>> <set field="facilityMap.ownerPartyId"
> >>>>> from-field="parameters.ownerPartyId"/>
> >>>>> <find-by-and map="facilityMap" list="facilityList"
> >>>>> entity-name="Facility"/>
> >>>>> @@ -1442,7 +1338,7 @@ under the License.
> >>>>> </iterate>
> >>>>> <field-to-result field="availabalityList"/>
> >>>>> </simple-method>
> >>>>> -
> >>>>> +
> >>>>> <simple-method method-name="createOrderPaymentApplication"
> >>>>> short-description="Create Order Payment Application">
> >>>>> <entity-one value-field="paymentMap"
> entity-name="Payment"/>
> >>>>> <set field="createCtx.amountApplied"
> >>>>> from-field="paymentMap.amount" type="BigDecimal"/>
> >>>>> @@ -1459,7 +1355,7 @@ under the License.
> >>>>> <call-service service-name="createPaymentApplication"
> >>>>> in-map-name="createCtx"/>
> >>>>> </if-not-empty>
> >>>>> </simple-method>
> >>>>> -
> >>>>> +
> >>>>> <simple-method method-name="MoveItemBetweenShipGroups"
> >>>>> short-description="Move order items between ship groups">
> >>>>> <entity-one entity-name="OrderItemShipGroupAssoc"
> >>>>> value-field="orderItemShipGroupAssoc">
> >>>>> <field-map field-name="orderId"
> >>>>> from-field="parameters.orderId"/>
> >>>>> @@ -1483,7 +1379,7 @@ under the License.
> >>>>> <set field="map.shipGroupSeqId"
> >>>>> from-field="parameters.toGroupIndex"/>
> >>>>> <set field="map.quantity" value="${
> orderItemShipGroupAssoc.quantity
> >>>>> + parameters.quantity}" type="BigDecimal"/>
> >>>>> <call-service service-name="updateOrderItemShipGroupAssoc"
> >>>>> in-map-name="map"/>
> >>>>> -
> >>>>> +
> >>>>> <entity-one entity-name="OrderItemShipGroupAssoc"
> >>>>> value-field="orderItemShipGroupAssoc">
> >>>>> <field-map field-name="orderId"
> >>>>> from-field="parameters.orderId"/>
> >>>>> <field-map field-name="orderItemSeqId"
> >>>>> from-field="parameters.orderItemSeqId"/>
> >>>>> @@ -1495,7 +1391,7 @@ under the License.
> >>>>> </add-error>
> >>>>> </if-empty>
> >>>>> <check-errors/>
> >>>>> -
> >>>>> +
> >>>>> <clear-field field="map"/>
> >>>>> <set field="map.orderId" from-field="parameters.orderId"/>
> >>>>> <set field="map.orderItemSeqId"
> >>>>> from-field="parameters.orderItemSeqId"/>
> >>>>> @@ -1503,5 +1399,5 @@ under the License.
> >>>>> <set field="map.quantity" value="${
> orderItemShipGroupAssoc.quantity
> >>>>> - parameters.quantity}" type="BigDecimal"/>
> >>>>> <call-service service-name="updateOrderItemShipGroupAssoc"
> >>>>> in-map-name="map"/>
> >>>>> </simple-method>
> >>>>> -
> >>>>> +
> >>>>> </simple-methods>
> >>>>>
> >>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/
> order/
> >>>>> OrderSimpleMethods.xml
> >>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
> >>>>> script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=
> >>>>> 1726542&r1=1726541&r2=1726542&view=diff
> >>>>> ============================================================
> >>>>> ==================
> >>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/
> order/OrderSimpleMethods.xml
> >>>>> (original)
> >>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/
> order/OrderSimpleMethods.xml
> >>>>> Sun Jan 24 19:06:32 2016
> >>>>> @@ -44,33 +44,12 @@ under the License.
> >>>>> </else>
> >>>>> </if-compare>
> >>>>> </simple-method>
> >>>>> -
> >>>>> - <simple-method method-name="createOrderAdjustment"
> >>>>> short-description="Create an OrderAdjustment">
> >>>>> -
> >>>>> - <make-value entity-name="OrderAdjustment"
> >>>>> value-field="newEntity"/>
> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> >>>>> -
> >>>>> - <sequenced-id sequence-name="OrderAdjustment"
> >>>>> field="newEntity.orderAdjustmentId"/>
> >>>>> - <field-to-result field="newEntity.orderAdjustmentId"
> >>>>> result-name="orderAdjustmentId"/>
> >>>>> -
> >>>>> - <now-timestamp field="newEntity.createdDate"/>
> >>>>> - <set from-field="userLogin.userLoginId"
> >>>>> field="newEntity.createdByUserLogin"/>
> >>>>> -
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> - <simple-method method-name="updateOrderAdjustment"
> >>>>> short-description="Update an OrderAdjustment">
> >>>>> - <check-permission permission="ORDERMGR"
> >>>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></
> >>>>> check-permission>
> >>>>> - <check-errors/>
> >>>>> - <entity-one entity-name="OrderAdjustment"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <set-nonpk-fields map="parameters"
> >>>>> value-field="lookedUpValue"/>
> >>>>> - <store-value value-field="lookedUpValue"/>
> >>>>> - </simple-method>
> >>>>> <simple-method method-name="deleteOrderAdjustment"
> >>>>> short-description="Delete an OrderAdjustment">
> >>>>> <check-permission permission="ORDERMGR"
> >>>>> action="_DELETE"><fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></
> >>>>> check-permission>
> >>>>> <check-errors/>
> >>>>> <entity-one entity-name="OrderAdjustment"
> >>>>> value-field="lookedUpValue"/>
> >>>>> <remove-value value-field="lookedUpValue"/>
> >>>>> -
> >>>>> +
> >>>>> <if-not-empty field="parameters.productPromoCodeId">
> >>>>> <entity-one entity-name="OrderProductPromoCode"
> >>>>> value-field="lookedUpValue"/>
> >>>>> <if-not-empty field="lookedUpValue">
> >>>>> @@ -79,30 +58,6 @@ under the License.
> >>>>> </if-not-empty>
> >>>>> </simple-method>
> >>>>> - <simple-method method-name="createOrderAdjustmentBilling"
> >>>>> short-description="Create an OrderAdjustmentBilling">
> >>>>> - <make-value entity-name="OrderAdjustmentBilling"
> >>>>> value-field="newEntity"/>
> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> -
> >>>>> - <simple-method method-name="createOrderItemBilling"
> >>>>> short-description="Create an OrderItemBilling">
> >>>>> - <make-value entity-name="OrderItemBilling"
> >>>>> value-field="newEntity"/>
> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
> >>>>> - <create-value value-field="newEntity"/>
> >>>>> - </simple-method>
> >>>>> -
> >>>>> - <simple-method method-name="createNotificationLog"
> >>>>> short-description="Log an order notification" login-required="false">
> >>>>> - <make-value entity-name="OrderNotification"
> >>>>> value-field="orderNotification"/>
> >>>>> - <sequenced-id sequence-name="OrderNotification"
> >>>>> field="orderNotification.orderNotificationId"/>
> >>>>> - <set from-field="parameters.orderId"
> >>>>> field="orderNotification.orderId"/>
> >>>>> - <set from-field="parameters.emailType"
> >>>>> field="orderNotification.emailType"/>
> >>>>> - <set from-field="parameters.comments"
> >>>>> field="orderNotification.comments"/>
> >>>>> - <now-timestamp field="orderNotification.notificationDate"/>
> >>>>> - <create-value value-field="orderNotification"/>
> >>>>> - </simple-method>
> >>>>> -
> >>>>> <simple-method method-name="updateOrderStatusFromReceipt"
> >>>>> short-description="Update Order Status From ShipmentReceipt">
> >>>>> <check-permission permission="ORDERMGR"
> >>>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
> >>>>> property="OrderSecurityErrorToRunUpdateOrderStatusFromReceip
> >>>>> t"/></check-permission>
> >>>>> <check-errors/>
> >>>>>
> >>>>> Modified: ofbiz/trunk/applications/order/servicedef/services.xml
> >>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
> >>>>> servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
> >>>>> ============================================================
> >>>>> ==================
> >>>>> --- ofbiz/trunk/applications/order/servicedef/services.xml
> (original)
> >>>>> +++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan
> 24
> >>>>> 19:06:32 2016
> >>>>> @@ -76,8 +76,7 @@ under the License.
> >>>>> <attribute name="assignedRoleTypeId" type="String"
> mode="IN"
> >>>>> optional="true"/>
> >>>>> </service>
> >>>>> - <service name="createOrderNotificationLog" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderSimpleMethods.xml"
> >>>>> invoke="createNotificationLog">
> >>>>> + <service name="createOrderNotificationLog" engine="entity-auto"
> >>>>> default-entity-name="OrderNotification" invoke="create">
> >>>>> <description>Logs when a notification was
> sent</description>
> >>>>> <attribute name="orderId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> <attribute name="emailType" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> @@ -164,17 +163,10 @@ under the License.
> >>>>> <attribute name="orderId" type="String" mode="OUT"
> >>>>> optional="false"/>
> >>>>> </service>
> >>>>> - <service name="createOrderItemBilling" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderSimpleMethods.xml"
> >>>>> invoke="createOrderItemBilling">
> >>>>> + <service name="createOrderItemBilling"
> >>>>> default-entity-name="OrderItemBilling" engine="entity-auto"
> >>>>> invoke="create">
> >>>>> <description>Create a new order item billing
> >>>>> record</description>
> >>>>> - <attribute name="orderId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> - <attribute name="orderItemSeqId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> - <attribute name="invoiceId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> - <attribute name="quantity" type="BigDecimal" mode="IN"
> >>>>> optional="true"/>
> >>>>> - <attribute name="amount" type="BigDecimal" mode="IN"
> >>>>> optional="true"/>
> >>>>> - <attribute name="itemIssuanceId" type="String" mode="IN"
> >>>>> optional="true"/>
> >>>>> - <attribute name="shipmentReceiptId" type="String" mode="IN"
> >>>>> optional="true"/>
> >>>>> + <auto-attributes mode="IN" include="pk"/>
> >>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
> >>>>> </service>
> >>>>> <service name="orderAdjustmentPermissionCheck"
> engine="simple"
> >>>>> @@ -183,8 +175,7 @@ under the License.
> >>>>> <implements service="permissionInterface"/>
> >>>>> </service>
> >>>>> - <service name="createOrderAdjustment"
> >>>>> default-entity-name="OrderAdjustment" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderSimpleMethods.xml"
> >>>>> invoke="createOrderAdjustment">
> >>>>> + <service name="createOrderAdjustment"
> >>>>> default-entity-name="OrderAdjustment" engine="entity-auto"
> >>>>> invoke="create">
> >>>>> <description>Creates a new order adjustment
> >>>>> record</description>
> >>>>> <permission-service service-name="
> orderAdjustmentPermissionCheck"
> >>>>> main-action="CREATE"/>
> >>>>> <auto-attributes mode="OUT" include="pk" optional="false"/>
> >>>>> @@ -192,9 +183,9 @@ under the License.
> >>>>> <override name="orderAdjustmentTypeId" optional="false"/>
> >>>>> <override name="orderId" optional="false"/>
> >>>>> </service>
> >>>>> - <service name="updateOrderAdjustment"
> >>>>> default-entity-name="OrderAdjustment" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderSimpleMethods.xml"
> >>>>> invoke="updateOrderAdjustment">
> >>>>> + <service name="updateOrderAdjustment"
> >>>>> default-entity-name="OrderAdjustment" engine="entity-auto"
> >>>>> invoke="update">
> >>>>> <description>Update an order adjustment
> record</description>
> >>>>> + <permission-service service-name="
> orderAdjustmentPermissionCheck"
> >>>>> main-action="UPDATE"/>
> >>>>> <auto-attributes mode="IN" include="pk" optional="false"/>
> >>>>> <auto-attributes mode="IN" include="nonpk"
> optional="true"/>
> >>>>> <override name="orderId" optional="false"/>
> >>>>> @@ -207,13 +198,10 @@ under the License.
> >>>>> <attribute name="productPromoCodeId" type="String"
> mode="IN"
> >>>>> optional="true"/>
> >>>>> </service>
> >>>>> - <service name="createOrderAdjustmentBilling" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderSimpleMethods.xml"
> >>>>> invoke="createOrderAdjustmentBilling">
> >>>>> + <service name="createOrderAdjustmentBilling"
> >>>>> default-entity-name="OrderAdjustmentBilling" engine="entity-auto"
> >>>>> invoke="create">
> >>>>> <description>Create a new order adjustment billing
> >>>>> record</description>
> >>>>> - <attribute name="orderAdjustmentId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> - <attribute name="invoiceId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> - <attribute name="amount" type="BigDecimal" mode="IN"
> >>>>> optional="true"/>
> >>>>> + <auto-attributes mode="IN" include="pk"/>
> >>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
> >>>>> </service>
> >>>>> <service name="createPaymentFromPreference" engine="java"
> >>>>> @@ -435,15 +423,10 @@ under the License.
> >>>>> <attribute name="roleTypeId" type="String" mode="IN"/>
> >>>>> </service>
> >>>>> - <service name="createOrderPaymentPreference" engine="java"
> >>>>> - location="org.ofbiz.order.order.OrderServices"
> >>>>> invoke="createPaymentPreference">
> >>>>> + <service name="createOrderPaymentPreference"
> >>>>> default-entity-name="OrderPaymentPreference" engine="entity-auto"
> >>>>> invoke="create">
> >>>>> <description>Creates an order payment
> >>>>> preference</description>
> >>>>> - <attribute name="orderId" type="String" mode="IN"/>
> >>>>> - <attribute name="paymentMethodTypeId" type="String"
> >>>>> mode="IN"/>
> >>>>> - <attribute name="paymentMethodId" type="String" mode="IN"
> >>>>> optional="true"/>
> >>>>> - <attribute name="statusId" type="String" mode="IN"
> >>>>> optional="true"/>
> >>>>> - <attribute name="maxAmount" type="BigDecimal" mode="IN"
> >>>>> optional="true"/>
> >>>>> - <attribute name="orderPaymentPreferenceId" type="String"
> >>>>> mode="OUT"/>
> >>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
> >>>>> + <auto-attributes include="pk" mode="OUT"/>
> >>>>> </service>
> >>>>> <service name="createOrderNote" engine="java"
> >>>>> @@ -568,20 +551,17 @@ under the License.
> >>>>> </service>
> >>>>> <!-- OrderShipment Services -->
> >>>>> - <service name="createOrderShipment" default-entity-name="
> OrderShipment"
> >>>>> engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="createOrderShipment" auth="true">
> >>>>> + <service name="createOrderShipment" engine="entity-auto"
> >>>>> default-entity-name="OrderShipment" invoke="create" auth="true">
> >>>>> <description>Create OrderShipment</description>
> >>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
> >>>>> <auto-attributes include="nonpk" mode="IN"
> optional="true"/>
> >>>>> </service>
> >>>>> - <service name="updateOrderShipment" default-entity-name="
> OrderShipment"
> >>>>> engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="updateOrderShipment" auth="true">
> >>>>> + <service name="updateOrderShipment" engine="entity-auto"
> >>>>> default-entity-name="OrderShipment" invoke="update" auth="true">
> >>>>> <description>Update OrderShipment</description>
> >>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
> >>>>> <auto-attributes include="nonpk" mode="IN"
> optional="true"/>
> >>>>> </service>
> >>>>> - <service name="deleteOrderShipment" default-entity-name="
> OrderShipment"
> >>>>> engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="deleteOrderShipment" auth="true">
> >>>>> + <service name="deleteOrderShipment" engine="entity-auto"
> >>>>> default-entity-name="OrderShipment" invoke="delete" auth="true">
> >>>>> <description>Delete OrderShipment</description>
> >>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
> >>>>> </service>
> >>>>> @@ -667,21 +647,27 @@ under the License.
> >>>>> </service>
> >>>>> <!-- CommunicationEvents related to orders -->
> >>>>> - <service name="createCommunicationEventOrder" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/communica
> >>>>> tion/CommunicationEventServices.xml" invoke="
> createCommunicationEventOrder"
> >>>>> auth="true">
> >>>>> + <service name="createCommunicationEventOrder"
> >>>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
> >>>>> invoke="create" auth="true">
> >>>>> <description>Create a Communication Event
> Order</description>
> >>>>> - <auto-attributes entity-name="CommunicationEventOrder"
> >>>>> include="pk" mode="IN" optional="false"/>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="CREATE"/>
> >>>>> + </required-permissions>
> >>>>> + <auto-attributes entity-name="CommunicationEventOrder"
> >>>>> include="pk" mode="IN"/>
> >>>>> </service>
> >>>>> - <service name="removeCommunicationEventOrder" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/communica
> >>>>> tion/CommunicationEventServices.xml" invoke="
> removeCommunicationEventOrder"
> >>>>> auth="true">
> >>>>> + <service name="removeCommunicationEventOrder"
> >>>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
> >>>>> invoke="delete" auth="true">
> >>>>> <description>Remove a Communication Event
> Order</description>
> >>>>> - <auto-attributes entity-name="CommunicationEventOrder"
> >>>>> include="pk" mode="IN" optional="false"/>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="DELETE"/>
> >>>>> + </required-permissions>
> >>>>> + <auto-attributes entity-name="CommunicationEventOrder"
> >>>>> include="pk" mode="IN"/>
> >>>>> </service>
> >>>>> <!-- Order Shipping and Contacts -->
> >>>>> - <service name="createOrderItemShipGroup"
> >>>>> default-entity-name="OrderItemShipGroup" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="createOrderItemShipGroup" auth="true">
> >>>>> + <service name="createOrderItemShipGroup"
> >>>>> default-entity-name="OrderItemShipGroup" engine="entity-auto"
> >>>>> invoke="create" auth="true">
> >>>>> <description>Creates a new OrderItemShipGroup.</
> description>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="CREATE"/>
> >>>>> + </required-permissions>
> >>>>> <auto-attributes mode="IN" include="nonpk"
> optional="true"/>
> >>>>> <attribute name="orderId" type="String" mode="IN"
> >>>>> optional="false"/>
> >>>>> <attribute name="shipGroupSeqId" type="String" mode="INOUT"
> >>>>> optional="true"/>
> >>>>> @@ -696,42 +682,52 @@ under the License.
> >>>>> <attribute name="contactMechPurposeTypeId" type="String"
> >>>>> mode="IN" optional="true"/>
> >>>>> </service>
> >>>>> - <service name="createOrderContactMech"
> >>>>> default-entity-name="OrderContactMech" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="createOrderContactMech" auth="true">
> >>>>> + <service name="createOrderContactMech"
> >>>>> default-entity-name="OrderContactMech" engine="entity-auto"
> >>>>> invoke="create" auth="true">
> >>>>> <description>Create Order Contact Mech</description>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="CREATE"/>
> >>>>> + </required-permissions>
> >>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
> >>>>> </service>
> >>>>> - <service name="updateOrderContactMech" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="updateOrderContactMech" auth="true"
> >>>>> default-entity-name="OrderContactMech">
> >>>>> + <service name="updateOrderContactMech"
> >>>>> default-entity-name="OrderContactMech" engine="simple"
> >>>>> + location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="updateOrderContactMech" auth="true">
> >>>>> <description>Update Order Contact Mech</description>
> >>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
> >>>>> <attribute name="oldContactMechId" type="String" mode="IN"
> >>>>> optional="true"/>
> >>>>> </service>
> >>>>> - <service name="removeOrderContactMech"
> >>>>> default-entity-name="OrderContactMech" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="removeOrderContactMech" auth="true">
> >>>>> + <service name="removeOrderContactMech"
> >>>>> default-entity-name="OrderContactMech" engine="entity-auto"
> >>>>> invoke="delete" auth="true">
> >>>>> <description>Remove Order Contact Mech</description>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="DELETE"/>
> >>>>> + </required-permissions>
> >>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
> >>>>> </service>
> >>>>> <!-- OrderTerms -->
> >>>>> - <service name="createOrderTerm" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="createOrderTerm" auth="true">
> >>>>> + <service name="createOrderTerm" default-entity-name="OrderTerm"
> >>>>> engine="entity-auto" invoke="create" auth="true">
> >>>>> <description>Create an Order Term</description>
> >>>>> - <auto-attributes entity-name="OrderTerm" include="pk"
> >>>>> mode="IN" optional="false"/>
> >>>>> - <auto-attributes entity-name="OrderTerm" include="nonpk"
> >>>>> mode="IN" optional="true"/>
> >>>>> - <override name="orderItemSeqId" optional="true"/>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="CREATE"/>
> >>>>> + </required-permissions>
> >>>>> + <auto-attributes include="pk" mode="IN" optional="false"/>
> >>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
> >>>>> + <override name="orderItemSeqId" optional="true"
> >>>>> default-value="_NA_"/>
> >>>>> </service>
> >>>>> - <service name="updateOrderTerm" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="updateOrderTerm" auth="true">
> >>>>> + <service name="updateOrderTerm" default-entity-name="OrderTerm"
> >>>>> engine="entity-auto" invoke="update" auth="true">
> >>>>> <description>Update an Order Term</description>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="UPDATE"/>
> >>>>> + </required-permissions>
> >>>>> <auto-attributes entity-name="OrderTerm" include="pk"
> >>>>> mode="IN" optional="false"/>
> >>>>> <auto-attributes entity-name="OrderTerm" include="nonpk"
> >>>>> mode="IN" optional="true"/>
> >>>>> </service>
> >>>>> - <service name="removeOrderTerm" engine="simple"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="removeOrderTerm" auth="true">
> >>>>> + <service name="removeOrderTerm" default-entity-name="OrderTerm"
> >>>>> engine="entity-auto" invoke="delete" auth="true">
> >>>>> <description>Remove an Order Term</description>
> >>>>> + <required-permissions join-type="AND">
> >>>>> + <check-permission permission="ORDERMGR"
> action="DELETE"/>
> >>>>> + </required-permissions>
> >>>>> <auto-attributes entity-name="OrderTerm" include="pk"
> >>>>> mode="IN" optional="false"/>
> >>>>> </service>
> >>>>>
> >>>>> Modified: ofbiz/trunk/applications/order/servicedef/services_
> requireme
> >>>>> nt.xml
> >>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
> >>>>> servicedef/services_requirement.xml?rev=1726542&r1=1726541&
> >>>>> r2=1726542&view=diff
> >>>>> ============================================================
> >>>>> ==================
> >>>>> --- ofbiz/trunk/applications/order/servicedef/services_
> requirement.xml
> >>>>> (original)
> >>>>> +++ ofbiz/trunk/applications/order/servicedef/services_
> requirement.xml
> >>>>> Sun Jan 24 19:06:32 2016
> >>>>> @@ -105,8 +105,7 @@ under the License.
> >>>>> <attribute mode="OUT" name="quantityTotal"
> type="BigDecimal"
> >>>>> optional="true"/>
> >>>>> <attribute mode="OUT" name="amountTotal" type="BigDecimal"
> >>>>> optional="true"/>
> >>>>> </service>
> >>>>> - <service name="createOrderRequirementCommitment"
> engine="simple"
> >>>>> auth="true"
> >>>>> - location="component://order/script/org/ofbiz/order/order/
> OrderServices.xml"
> >>>>> invoke="createOrderRequirementCommitment">
> >>>>> + <service name="createOrderRequirementCommitment"
> >>>>> engine="entity-auto" auth="true" default-entity-name="
> OrderRequirementCommitment"
> >>>>> invoke="create">
> >>>>> <auto-attributes entity-name="OrderRequirementCommitment"
> >>>>> include="pk" mode="IN" optional="false"/>
> >>>>> <auto-attributes entity-name="OrderRequirementCommitment"
> >>>>> include="nonpk" mode="IN" optional="true"/>
> >>>>> </service>
> >>>>>
> >>>>> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
> >>>>> OrderServices.java
> >>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
> >>>>> src/org/ofbiz/order/order/OrderServices.java?rev=1726542
> >>>>> &r1=1726541&r2=1726542&view=diff
> >>>>> ============================================================
> >>>>> ==================
> >>>>> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/
> order/OrderServices.java
> >>>>> (original)
> >>>>> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/
> order/OrderServices.java
> >>>>> Sun Jan 24 19:06:32 2016
> >>>>> @@ -2824,52 +2824,6 @@ public class OrderServices {
> >>>>> return ServiceUtil.returnSuccess();
> >>>>> }
> >>>>> - /** Service to create an order payment preference */
> >>>>> - public static Map<String, Object> createPaymentPreference(
> DispatchContext
> >>>>> ctx, Map<String, ? extends Object> context) {
> >>>>> - Map<String, Object> result = new HashMap<String, Object>();
> >>>>> - Delegator delegator = ctx.getDelegator();
> >>>>> - String orderId = (String) context.get("orderId");
> >>>>> - String statusId = (String) context.get("statusId");
> >>>>> - String paymentMethodTypeId = (String)
> >>>>> context.get("paymentMethodTypeId");
> >>>>> - String paymentMethodId = (String)
> >>>>> context.get("paymentMethodId");
> >>>>> - BigDecimal maxAmount = (BigDecimal)
> context.get("maxAmount");
> >>>>> - GenericValue userLogin = (GenericValue)
> >>>>> context.get("userLogin");
> >>>>> - Locale locale = (Locale) context.get("locale");
> >>>>> -
> >>>>> - String prefId = null;
> >>>>> -
> >>>>> - try {
> >>>>> - prefId = delegator.getNextSeqId("OrderP
> >>>>> aymentPreference");
> >>>>> - } catch (IllegalArgumentException e) {
> >>>>> - return ServiceUtil.returnError(UtilPr
> >>>>> operties.getMessage(resource_error,
> >>>>> - "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFa
> ilure",
> >>>>> locale));
> >>>>> - }
> >>>>> -
> >>>>> - Map<String, Object> fields = UtilMisc.<String,
> >>>>> Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId,
> >>>>> "paymentMethodTypeId",
> >>>>> - paymentMethodTypeId, "paymentMethodId",
> >>>>> paymentMethodId, "maxAmount", maxAmount);
> >>>>> -
> >>>>> - if (statusId != null) {
> >>>>> - fields.put("statusId", statusId);
> >>>>> - }
> >>>>> -
> >>>>> - try {
> >>>>> - GenericValue v = delegator.makeValue("
> OrderPaymentPreference",
> >>>>> fields);
> >>>>> - v.set("createdDate", UtilDateTime.nowTimestamp());
> >>>>> - if (userLogin != null) {
> >>>>> - v.set("createdByUserLogin",
> >>>>> userLogin.getString("userLoginId"));
> >>>>> - }
> >>>>> - delegator.create(v);
> >>>>> - } catch (GenericEntityException e) {
> >>>>> - result.put(ModelService.RESPONSE_MESSAGE,
> >>>>> ModelService.RESPOND_ERROR);
> >>>>> - result.put(ModelService.ERROR_MESSAGE,
> >>>>> UtilProperties.getMessage(resource,
> >>>>> - "OrderOrderPaymentPreferencesCannotBeCreated",
> >>>>> UtilMisc.toMap("errorString", e.getMessage()), locale));
> >>>>> - return ServiceUtil.returnFailure();
> >>>>> - }
> >>>>> - result.put("orderPaymentPreferenceId", prefId);
> >>>>> - result.put(ModelService.RESPONSE_MESSAGE,
> >>>>> ModelService.RESPOND_SUCCESS);
> >>>>> - return result;
> >>>>> - }
> >>>>> -
> >>>>> /** Service to get order header information as standard
> results.
> >>>>> */
> >>>>> public static Map<String, Object> getOrderHeaderInformation(
> DispatchContext
> >>>>> dctx, Map<String, ? extends Object> context) {
> >>>>> Delegator delegator = dctx.getDelegator();
> >>>>>
> >>>>>
> >>>>>
> >>>>>
> >>>>
> >>>>
> >>>
> >>>
> >>>
> >>
> >>
> >>
> >
>
Re: svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Posted by Suraj Khurana <su...@hotwaxsystems.com>.
Hi Nicolas,
Permission action is not handled properly in this commit while converting
services to entity-auto. It should be _CREATE, _DELETE instead of CREATE,
DELETE.
I have created a JIRA for the same here
<https://issues.apache.org/jira/browse/OFBIZ-10405> and uploaded a patch
for review.
<https://issues.apache.org/jira/browse/OFBIZ-10405>
--
Thanks and Regards,
*Suraj Khurana* | Omni-channel OMS Technical Expert
HotWax Commerce by HotWax Systems
Plot no. 80, Scheme no. 78, Vijay Nagar, Indore, M.P. India 452010
On Fri, Feb 12, 2016 at 2:45 PM, Jacques Le Roux <
jacques.le.roux@les7arts.com> wrote:
> Le 12/02/2016 08:23, Nicolas Malin a écrit :
>
>>
>> Hi Jacques
>>
>> Le 11/02/2016 23:10, Jacques Le Roux a écrit :
>>
>>> Le 11/02/2016 22:27, Jacques Le Roux a écrit :
>>>
>>>> Hi Nicolas, All,
>>>>
>>>> I globally like the changes, I just have a question.
>>>>
>>>> With these changes the OrderSecurityError* properties which were used
>>>> here are now useless, should we not remove them?
>>>> Despite the lost, I think it's reasonable because these errors have
>>>> very few chances to appear, and if they appear it's not so hard to find
>>>> your way through the log.
>>>>
>>> Yes I planned to remove all unused label before declare the conversion
>> end.
>>
>>>
>>> Oops, I did not see the elephant in the room: they are permission
>>> issues. So I guess the message to the user will be less clear. Not sure
>>> it's a real issue but we might consider it...
>>>
>> By default I return to more generic label (http://ofbiz.135035.n4.nabble
>> .com/entity-auto-improvement-Act-2-td4655973.html) but if you think that
>> it's big lost, I can try to improve the generic order permission service to
>> resolve them
>>
>
> No, it's OK with me, let's go
>
> Jacques
>
>
>
>> Nicolas
>>
>>>
>>> Jacques
>>>
>>>
>>>> Jacques
>>>>
>>>>
>>>> Le 24/01/2016 20:06, nmalin@apache.org a écrit :
>>>>
>>>>> Author: nmalin
>>>>> Date: Sun Jan 24 19:06:32 2016
>>>>> New Revision: 1726542
>>>>>
>>>>> URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
>>>>> Log:
>>>>>
>>>>>
>>>>> I converted following services from simple to entity-auto :
>>>>>
>>>>> createOrderNotificationLog
>>>>> createOrderItemBilling
>>>>> createOrderAdjustment
>>>>> updateOrderAdjustment
>>>>> createOrderAdjustmentBilling
>>>>> createOrderShipment
>>>>> updateOrderShipment
>>>>> deleteOrderShipment
>>>>> createCommunicationEventOrder
>>>>> removeCommunicationEventOrder
>>>>> createOrderItemShipGroup
>>>>> createOrderContactMech
>>>>> removeOrderContactMech
>>>>> createOrderTerm
>>>>> removeOrderTerm
>>>>> createOrderRequirementCommitment
>>>>>
>>>>> And from java to entity-auto for createOrderPaymentPreference
>>>>> Related issue OFBIZ-6854.
>>>>>
>>>>> Removed:
>>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
>>>>> Modified:
>>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
>>>>> OrderServices.xml
>>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
>>>>> OrderSimpleMethods.xml
>>>>> ofbiz/trunk/applications/order/servicedef/services.xml
>>>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>>>> OrderServices.java
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
>>>>> OrderServices.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>> script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&
>>>>> r1=1726541&r2=1726542&view=diff
>>>>> ============================================================
>>>>> ==================
>>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>>>> (original)
>>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>>>> Sun Jan 24 19:06:32 2016
>>>>> @@ -88,56 +88,8 @@ under the License.
>>>>> <field-to-result field="orderInfo.totalOrders"
>>>>> result-name="totalOrders"/>
>>>>> </if-not-empty>
>>>>> </simple-method>
>>>>> - <simple-method method-name="createOrderShipment"
>>>>> short-description="Create OrderShipment">
>>>>> - <set value="Create OrderShipment" field="operationName"/>
>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunCreateOrderShipment"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> -
>>>>> - <make-value value-field="newEntity"
>>>>> entity-name="OrderShipment"/>
>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>> -
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> - <simple-method method-name="updateOrderShipment"
>>>>> short-description="Update OrderShipment">
>>>>> - <set value="Update OrderShipment" field="operationName"/>
>>>>> - <check-permission permission="ORDERMGR" action="_UPDATE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> -
>>>>> - <entity-one entity-name="OrderShipment"
>>>>> value-field="lookedUpValue"/>
>>>>> - <set-nonpk-fields map="parameters"
>>>>> value-field="lookedUpValue"/>
>>>>> - <store-value value-field="lookedUpValue"/>
>>>>> - </simple-method>
>>>>> - <simple-method method-name="deleteOrderShipment"
>>>>> short-description="Delete OrderShipment">
>>>>> - <set value="Delete OrderShipment" field="operationName"/>
>>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> -
>>>>> - <entity-one entity-name="OrderShipment"
>>>>> value-field="lookedUpValue"/>
>>>>> - <remove-value value-field="lookedUpValue"/>
>>>>> - </simple-method>
>>>>> <!-- order requirement methods -->
>>>>> - <simple-method method-name="createOrderRequirementCommitment"
>>>>> short-description="Create OrderRequirementCommitment">
>>>>> - <!-- The permission checking is commented out to make this
>>>>> service work also when triggered from ecommerce orders -->
>>>>> - <!--
>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunCheckCreateOrderRequirement
>>>>> Commitment"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> - -->
>>>>> - <make-value value-field="newEntity"
>>>>> entity-name="OrderRequirementCommitment"/>
>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> -
>>>>> <simple-method method-name="createRequirementAndCommitment"
>>>>> short-description="create a requirement and commitment for it">
>>>>> <set value="PRODUCT_REQUIREMENT"
>>>>> field="inputMap.requirementTypeId"/>
>>>>> @@ -759,20 +711,6 @@ under the License.
>>>>> </simple-method>
>>>>> <!-- OrderItemShipGroup -->
>>>>> - <simple-method method-name="createOrderItemShipGroup"
>>>>> short-description="Create OrderItemShipGroup">
>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> - <make-value value-field="newEntity"
>>>>> entity-name="OrderItemShipGroup"/>
>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>> - <if-empty field="newEntity.shipGroupSeqId">
>>>>> - <make-next-seq-id value-field="newEntity"
>>>>> seq-field-name="shipGroupSeqId"/> <!-- this finds the next
>>>>> sub-sequence ID -->
>>>>> - <field-to-result field="newEntity.shipGroupSeqId"
>>>>> result-name="shipGroupSeqId"/>
>>>>> - </if-empty>
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> <simple-method method-name="updateOrderItemShipGroup"
>>>>> short-description="Update OrderItemShipGroup">
>>>>> <check-permission permission="ORDERMGR" action="_UPDATE">
>>>>> <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
>>>>> @@ -832,16 +770,6 @@ under the License.
>>>>> <field-to-result field="orderItemShipGroupInvRes.promisedDatetime"
>>>>> result-name="estimatedShipDate"/>
>>>>> </simple-method>
>>>>> - <simple-method method-name="createOrderContactMech"
>>>>> short-description="Create OrderContactMech">
>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunCreateOrderContactMech"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> - <make-value value-field="newEntity"
>>>>> entity-name="OrderContactMech"/>
>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> -
>>>>> <simple-method method-name="removeOrderContactMech"
>>>>> short-description="Remove OrderContactMech">
>>>>> <check-permission permission="ORDERMGR" action="_DELETE">
>>>>> <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
>>>>> @@ -861,39 +789,6 @@ under the License.
>>>>> <store-value value-field="orderHeaderNote"/>
>>>>> </simple-method>
>>>>> - <simple-method method-name="createOrderTerm"
>>>>> short-description="Create an OrderTerm">
>>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunCreateOrderTerm"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> -
>>>>> - <make-value value-field="newEntity" entity-name="OrderTerm"/>
>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>> - <set value="_NA_" set-if-empty="true"
>>>>> field="newEntity.orderItemSeqId"/>
>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>> -
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> - <simple-method method-name="updateOrderTerm"
>>>>> short-description="Update OrderTerm">
>>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunUpdateOrderTerm"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> -
>>>>> - <entity-one entity-name="OrderTerm"
>>>>> value-field="lookedUpValue"/>
>>>>> - <set-nonpk-fields map="parameters"
>>>>> value-field="lookedUpValue"/>
>>>>> - <store-value value-field="lookedUpValue"/>
>>>>> - </simple-method>
>>>>> - <simple-method method-name="removeOrderTerm"
>>>>> short-description="Remove OrderTerm">
>>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>>> - <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunRemoveOrderTerm"/>
>>>>> - </check-permission>
>>>>> - <check-errors/>
>>>>> -
>>>>> - <entity-one entity-name="OrderTerm"
>>>>> value-field="lookedUpValue"/>
>>>>> - <remove-value value-field="lookedUpValue"/>
>>>>> - </simple-method>
>>>>> -
>>>>> <simple-method method-name="addPaymentMethodToOrder"
>>>>> short-description="Create an PaymentMethodToOrder">
>>>>> <check-permission permission="ORDERMGR" action="_CREATE">
>>>>> <fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
>>>>> @@ -1373,7 +1268,7 @@ under the License.
>>>>> </iterate>
>>>>> </iterate>
>>>>> </simple-method>
>>>>> -
>>>>> +
>>>>> <simple-method method-name="updateShippingMethodAndCharges"
>>>>> short-description="Updates shipping method and shipping charges from Order
>>>>> View page when Shipment is in picked status and items of Order are packed">
>>>>> <!-- splitting shipmentMethodAndAmount request parameter
>>>>> value that contains "*" symbol
>>>>> into "shipmentMethod" and "newAmount".
>>>>> @@ -1424,6 +1319,7 @@ under the License.
>>>>> </else>
>>>>> </if-compare-field>
>>>>> </simple-method>
>>>>> +
>>>>> <simple-method method-name="productAvailabalityByFacility"
>>>>> short-description="Calculate ATP and Qoh According For each facility">
>>>>> <set field="facilityMap.ownerPartyId"
>>>>> from-field="parameters.ownerPartyId"/>
>>>>> <find-by-and map="facilityMap" list="facilityList"
>>>>> entity-name="Facility"/>
>>>>> @@ -1442,7 +1338,7 @@ under the License.
>>>>> </iterate>
>>>>> <field-to-result field="availabalityList"/>
>>>>> </simple-method>
>>>>> -
>>>>> +
>>>>> <simple-method method-name="createOrderPaymentApplication"
>>>>> short-description="Create Order Payment Application">
>>>>> <entity-one value-field="paymentMap" entity-name="Payment"/>
>>>>> <set field="createCtx.amountApplied"
>>>>> from-field="paymentMap.amount" type="BigDecimal"/>
>>>>> @@ -1459,7 +1355,7 @@ under the License.
>>>>> <call-service service-name="createPaymentApplication"
>>>>> in-map-name="createCtx"/>
>>>>> </if-not-empty>
>>>>> </simple-method>
>>>>> -
>>>>> +
>>>>> <simple-method method-name="MoveItemBetweenShipGroups"
>>>>> short-description="Move order items between ship groups">
>>>>> <entity-one entity-name="OrderItemShipGroupAssoc"
>>>>> value-field="orderItemShipGroupAssoc">
>>>>> <field-map field-name="orderId"
>>>>> from-field="parameters.orderId"/>
>>>>> @@ -1483,7 +1379,7 @@ under the License.
>>>>> <set field="map.shipGroupSeqId"
>>>>> from-field="parameters.toGroupIndex"/>
>>>>> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity
>>>>> + parameters.quantity}" type="BigDecimal"/>
>>>>> <call-service service-name="updateOrderItemShipGroupAssoc"
>>>>> in-map-name="map"/>
>>>>> -
>>>>> +
>>>>> <entity-one entity-name="OrderItemShipGroupAssoc"
>>>>> value-field="orderItemShipGroupAssoc">
>>>>> <field-map field-name="orderId"
>>>>> from-field="parameters.orderId"/>
>>>>> <field-map field-name="orderItemSeqId"
>>>>> from-field="parameters.orderItemSeqId"/>
>>>>> @@ -1495,7 +1391,7 @@ under the License.
>>>>> </add-error>
>>>>> </if-empty>
>>>>> <check-errors/>
>>>>> -
>>>>> +
>>>>> <clear-field field="map"/>
>>>>> <set field="map.orderId" from-field="parameters.orderId"/>
>>>>> <set field="map.orderItemSeqId"
>>>>> from-field="parameters.orderItemSeqId"/>
>>>>> @@ -1503,5 +1399,5 @@ under the License.
>>>>> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity
>>>>> - parameters.quantity}" type="BigDecimal"/>
>>>>> <call-service service-name="updateOrderItemShipGroupAssoc"
>>>>> in-map-name="map"/>
>>>>> </simple-method>
>>>>> -
>>>>> +
>>>>> </simple-methods>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/
>>>>> OrderSimpleMethods.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>> script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=
>>>>> 1726542&r1=1726541&r2=1726542&view=diff
>>>>> ============================================================
>>>>> ==================
>>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>>>> (original)
>>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>>>> Sun Jan 24 19:06:32 2016
>>>>> @@ -44,33 +44,12 @@ under the License.
>>>>> </else>
>>>>> </if-compare>
>>>>> </simple-method>
>>>>> -
>>>>> - <simple-method method-name="createOrderAdjustment"
>>>>> short-description="Create an OrderAdjustment">
>>>>> -
>>>>> - <make-value entity-name="OrderAdjustment"
>>>>> value-field="newEntity"/>
>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>> -
>>>>> - <sequenced-id sequence-name="OrderAdjustment"
>>>>> field="newEntity.orderAdjustmentId"/>
>>>>> - <field-to-result field="newEntity.orderAdjustmentId"
>>>>> result-name="orderAdjustmentId"/>
>>>>> -
>>>>> - <now-timestamp field="newEntity.createdDate"/>
>>>>> - <set from-field="userLogin.userLoginId"
>>>>> field="newEntity.createdByUserLogin"/>
>>>>> -
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> - <simple-method method-name="updateOrderAdjustment"
>>>>> short-description="Update an OrderAdjustment">
>>>>> - <check-permission permission="ORDERMGR"
>>>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></
>>>>> check-permission>
>>>>> - <check-errors/>
>>>>> - <entity-one entity-name="OrderAdjustment"
>>>>> value-field="lookedUpValue"/>
>>>>> - <set-nonpk-fields map="parameters"
>>>>> value-field="lookedUpValue"/>
>>>>> - <store-value value-field="lookedUpValue"/>
>>>>> - </simple-method>
>>>>> <simple-method method-name="deleteOrderAdjustment"
>>>>> short-description="Delete an OrderAdjustment">
>>>>> <check-permission permission="ORDERMGR"
>>>>> action="_DELETE"><fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></
>>>>> check-permission>
>>>>> <check-errors/>
>>>>> <entity-one entity-name="OrderAdjustment"
>>>>> value-field="lookedUpValue"/>
>>>>> <remove-value value-field="lookedUpValue"/>
>>>>> -
>>>>> +
>>>>> <if-not-empty field="parameters.productPromoCodeId">
>>>>> <entity-one entity-name="OrderProductPromoCode"
>>>>> value-field="lookedUpValue"/>
>>>>> <if-not-empty field="lookedUpValue">
>>>>> @@ -79,30 +58,6 @@ under the License.
>>>>> </if-not-empty>
>>>>> </simple-method>
>>>>> - <simple-method method-name="createOrderAdjustmentBilling"
>>>>> short-description="Create an OrderAdjustmentBilling">
>>>>> - <make-value entity-name="OrderAdjustmentBilling"
>>>>> value-field="newEntity"/>
>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> -
>>>>> - <simple-method method-name="createOrderItemBilling"
>>>>> short-description="Create an OrderItemBilling">
>>>>> - <make-value entity-name="OrderItemBilling"
>>>>> value-field="newEntity"/>
>>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>>> - <create-value value-field="newEntity"/>
>>>>> - </simple-method>
>>>>> -
>>>>> - <simple-method method-name="createNotificationLog"
>>>>> short-description="Log an order notification" login-required="false">
>>>>> - <make-value entity-name="OrderNotification"
>>>>> value-field="orderNotification"/>
>>>>> - <sequenced-id sequence-name="OrderNotification"
>>>>> field="orderNotification.orderNotificationId"/>
>>>>> - <set from-field="parameters.orderId"
>>>>> field="orderNotification.orderId"/>
>>>>> - <set from-field="parameters.emailType"
>>>>> field="orderNotification.emailType"/>
>>>>> - <set from-field="parameters.comments"
>>>>> field="orderNotification.comments"/>
>>>>> - <now-timestamp field="orderNotification.notificationDate"/>
>>>>> - <create-value value-field="orderNotification"/>
>>>>> - </simple-method>
>>>>> -
>>>>> <simple-method method-name="updateOrderStatusFromReceipt"
>>>>> short-description="Update Order Status From ShipmentReceipt">
>>>>> <check-permission permission="ORDERMGR"
>>>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>>>> property="OrderSecurityErrorToRunUpdateOrderStatusFromReceip
>>>>> t"/></check-permission>
>>>>> <check-errors/>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/servicedef/services.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>> servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>>>> ============================================================
>>>>> ==================
>>>>> --- ofbiz/trunk/applications/order/servicedef/services.xml (original)
>>>>> +++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan 24
>>>>> 19:06:32 2016
>>>>> @@ -76,8 +76,7 @@ under the License.
>>>>> <attribute name="assignedRoleTypeId" type="String" mode="IN"
>>>>> optional="true"/>
>>>>> </service>
>>>>> - <service name="createOrderNotificationLog" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>>>> invoke="createNotificationLog">
>>>>> + <service name="createOrderNotificationLog" engine="entity-auto"
>>>>> default-entity-name="OrderNotification" invoke="create">
>>>>> <description>Logs when a notification was sent</description>
>>>>> <attribute name="orderId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> <attribute name="emailType" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> @@ -164,17 +163,10 @@ under the License.
>>>>> <attribute name="orderId" type="String" mode="OUT"
>>>>> optional="false"/>
>>>>> </service>
>>>>> - <service name="createOrderItemBilling" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>>>> invoke="createOrderItemBilling">
>>>>> + <service name="createOrderItemBilling"
>>>>> default-entity-name="OrderItemBilling" engine="entity-auto"
>>>>> invoke="create">
>>>>> <description>Create a new order item billing
>>>>> record</description>
>>>>> - <attribute name="orderId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> - <attribute name="orderItemSeqId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> - <attribute name="invoiceId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> - <attribute name="quantity" type="BigDecimal" mode="IN"
>>>>> optional="true"/>
>>>>> - <attribute name="amount" type="BigDecimal" mode="IN"
>>>>> optional="true"/>
>>>>> - <attribute name="itemIssuanceId" type="String" mode="IN"
>>>>> optional="true"/>
>>>>> - <attribute name="shipmentReceiptId" type="String" mode="IN"
>>>>> optional="true"/>
>>>>> + <auto-attributes mode="IN" include="pk"/>
>>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>>> </service>
>>>>> <service name="orderAdjustmentPermissionCheck" engine="simple"
>>>>> @@ -183,8 +175,7 @@ under the License.
>>>>> <implements service="permissionInterface"/>
>>>>> </service>
>>>>> - <service name="createOrderAdjustment"
>>>>> default-entity-name="OrderAdjustment" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>>>> invoke="createOrderAdjustment">
>>>>> + <service name="createOrderAdjustment"
>>>>> default-entity-name="OrderAdjustment" engine="entity-auto"
>>>>> invoke="create">
>>>>> <description>Creates a new order adjustment
>>>>> record</description>
>>>>> <permission-service service-name="orderAdjustmentPermissionCheck"
>>>>> main-action="CREATE"/>
>>>>> <auto-attributes mode="OUT" include="pk" optional="false"/>
>>>>> @@ -192,9 +183,9 @@ under the License.
>>>>> <override name="orderAdjustmentTypeId" optional="false"/>
>>>>> <override name="orderId" optional="false"/>
>>>>> </service>
>>>>> - <service name="updateOrderAdjustment"
>>>>> default-entity-name="OrderAdjustment" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>>>> invoke="updateOrderAdjustment">
>>>>> + <service name="updateOrderAdjustment"
>>>>> default-entity-name="OrderAdjustment" engine="entity-auto"
>>>>> invoke="update">
>>>>> <description>Update an order adjustment record</description>
>>>>> + <permission-service service-name="orderAdjustmentPermissionCheck"
>>>>> main-action="UPDATE"/>
>>>>> <auto-attributes mode="IN" include="pk" optional="false"/>
>>>>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>>> <override name="orderId" optional="false"/>
>>>>> @@ -207,13 +198,10 @@ under the License.
>>>>> <attribute name="productPromoCodeId" type="String" mode="IN"
>>>>> optional="true"/>
>>>>> </service>
>>>>> - <service name="createOrderAdjustmentBilling" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>>>> invoke="createOrderAdjustmentBilling">
>>>>> + <service name="createOrderAdjustmentBilling"
>>>>> default-entity-name="OrderAdjustmentBilling" engine="entity-auto"
>>>>> invoke="create">
>>>>> <description>Create a new order adjustment billing
>>>>> record</description>
>>>>> - <attribute name="orderAdjustmentId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> - <attribute name="invoiceId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> - <attribute name="amount" type="BigDecimal" mode="IN"
>>>>> optional="true"/>
>>>>> + <auto-attributes mode="IN" include="pk"/>
>>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>>> </service>
>>>>> <service name="createPaymentFromPreference" engine="java"
>>>>> @@ -435,15 +423,10 @@ under the License.
>>>>> <attribute name="roleTypeId" type="String" mode="IN"/>
>>>>> </service>
>>>>> - <service name="createOrderPaymentPreference" engine="java"
>>>>> - location="org.ofbiz.order.order.OrderServices"
>>>>> invoke="createPaymentPreference">
>>>>> + <service name="createOrderPaymentPreference"
>>>>> default-entity-name="OrderPaymentPreference" engine="entity-auto"
>>>>> invoke="create">
>>>>> <description>Creates an order payment
>>>>> preference</description>
>>>>> - <attribute name="orderId" type="String" mode="IN"/>
>>>>> - <attribute name="paymentMethodTypeId" type="String"
>>>>> mode="IN"/>
>>>>> - <attribute name="paymentMethodId" type="String" mode="IN"
>>>>> optional="true"/>
>>>>> - <attribute name="statusId" type="String" mode="IN"
>>>>> optional="true"/>
>>>>> - <attribute name="maxAmount" type="BigDecimal" mode="IN"
>>>>> optional="true"/>
>>>>> - <attribute name="orderPaymentPreferenceId" type="String"
>>>>> mode="OUT"/>
>>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>>> + <auto-attributes include="pk" mode="OUT"/>
>>>>> </service>
>>>>> <service name="createOrderNote" engine="java"
>>>>> @@ -568,20 +551,17 @@ under the License.
>>>>> </service>
>>>>> <!-- OrderShipment Services -->
>>>>> - <service name="createOrderShipment" default-entity-name="OrderShipment"
>>>>> engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="createOrderShipment" auth="true">
>>>>> + <service name="createOrderShipment" engine="entity-auto"
>>>>> default-entity-name="OrderShipment" invoke="create" auth="true">
>>>>> <description>Create OrderShipment</description>
>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>>> </service>
>>>>> - <service name="updateOrderShipment" default-entity-name="OrderShipment"
>>>>> engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="updateOrderShipment" auth="true">
>>>>> + <service name="updateOrderShipment" engine="entity-auto"
>>>>> default-entity-name="OrderShipment" invoke="update" auth="true">
>>>>> <description>Update OrderShipment</description>
>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>>> </service>
>>>>> - <service name="deleteOrderShipment" default-entity-name="OrderShipment"
>>>>> engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="deleteOrderShipment" auth="true">
>>>>> + <service name="deleteOrderShipment" engine="entity-auto"
>>>>> default-entity-name="OrderShipment" invoke="delete" auth="true">
>>>>> <description>Delete OrderShipment</description>
>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>> </service>
>>>>> @@ -667,21 +647,27 @@ under the License.
>>>>> </service>
>>>>> <!-- CommunicationEvents related to orders -->
>>>>> - <service name="createCommunicationEventOrder" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/communica
>>>>> tion/CommunicationEventServices.xml" invoke="createCommunicationEventOrder"
>>>>> auth="true">
>>>>> + <service name="createCommunicationEventOrder"
>>>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
>>>>> invoke="create" auth="true">
>>>>> <description>Create a Communication Event Order</description>
>>>>> - <auto-attributes entity-name="CommunicationEventOrder"
>>>>> include="pk" mode="IN" optional="false"/>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>>> + </required-permissions>
>>>>> + <auto-attributes entity-name="CommunicationEventOrder"
>>>>> include="pk" mode="IN"/>
>>>>> </service>
>>>>> - <service name="removeCommunicationEventOrder" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/communica
>>>>> tion/CommunicationEventServices.xml" invoke="removeCommunicationEventOrder"
>>>>> auth="true">
>>>>> + <service name="removeCommunicationEventOrder"
>>>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
>>>>> invoke="delete" auth="true">
>>>>> <description>Remove a Communication Event Order</description>
>>>>> - <auto-attributes entity-name="CommunicationEventOrder"
>>>>> include="pk" mode="IN" optional="false"/>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>>>> + </required-permissions>
>>>>> + <auto-attributes entity-name="CommunicationEventOrder"
>>>>> include="pk" mode="IN"/>
>>>>> </service>
>>>>> <!-- Order Shipping and Contacts -->
>>>>> - <service name="createOrderItemShipGroup"
>>>>> default-entity-name="OrderItemShipGroup" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="createOrderItemShipGroup" auth="true">
>>>>> + <service name="createOrderItemShipGroup"
>>>>> default-entity-name="OrderItemShipGroup" engine="entity-auto"
>>>>> invoke="create" auth="true">
>>>>> <description>Creates a new OrderItemShipGroup.</description>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>>> + </required-permissions>
>>>>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>>> <attribute name="orderId" type="String" mode="IN"
>>>>> optional="false"/>
>>>>> <attribute name="shipGroupSeqId" type="String" mode="INOUT"
>>>>> optional="true"/>
>>>>> @@ -696,42 +682,52 @@ under the License.
>>>>> <attribute name="contactMechPurposeTypeId" type="String"
>>>>> mode="IN" optional="true"/>
>>>>> </service>
>>>>> - <service name="createOrderContactMech"
>>>>> default-entity-name="OrderContactMech" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="createOrderContactMech" auth="true">
>>>>> + <service name="createOrderContactMech"
>>>>> default-entity-name="OrderContactMech" engine="entity-auto"
>>>>> invoke="create" auth="true">
>>>>> <description>Create Order Contact Mech</description>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>>> + </required-permissions>
>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>> </service>
>>>>> - <service name="updateOrderContactMech" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="updateOrderContactMech" auth="true"
>>>>> default-entity-name="OrderContactMech">
>>>>> + <service name="updateOrderContactMech"
>>>>> default-entity-name="OrderContactMech" engine="simple"
>>>>> + location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="updateOrderContactMech" auth="true">
>>>>> <description>Update Order Contact Mech</description>
>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>> <attribute name="oldContactMechId" type="String" mode="IN"
>>>>> optional="true"/>
>>>>> </service>
>>>>> - <service name="removeOrderContactMech"
>>>>> default-entity-name="OrderContactMech" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="removeOrderContactMech" auth="true">
>>>>> + <service name="removeOrderContactMech"
>>>>> default-entity-name="OrderContactMech" engine="entity-auto"
>>>>> invoke="delete" auth="true">
>>>>> <description>Remove Order Contact Mech</description>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>>>> + </required-permissions>
>>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>> </service>
>>>>> <!-- OrderTerms -->
>>>>> - <service name="createOrderTerm" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="createOrderTerm" auth="true">
>>>>> + <service name="createOrderTerm" default-entity-name="OrderTerm"
>>>>> engine="entity-auto" invoke="create" auth="true">
>>>>> <description>Create an Order Term</description>
>>>>> - <auto-attributes entity-name="OrderTerm" include="pk"
>>>>> mode="IN" optional="false"/>
>>>>> - <auto-attributes entity-name="OrderTerm" include="nonpk"
>>>>> mode="IN" optional="true"/>
>>>>> - <override name="orderItemSeqId" optional="true"/>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>>> + </required-permissions>
>>>>> + <auto-attributes include="pk" mode="IN" optional="false"/>
>>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>>> + <override name="orderItemSeqId" optional="true"
>>>>> default-value="_NA_"/>
>>>>> </service>
>>>>> - <service name="updateOrderTerm" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="updateOrderTerm" auth="true">
>>>>> + <service name="updateOrderTerm" default-entity-name="OrderTerm"
>>>>> engine="entity-auto" invoke="update" auth="true">
>>>>> <description>Update an Order Term</description>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="UPDATE"/>
>>>>> + </required-permissions>
>>>>> <auto-attributes entity-name="OrderTerm" include="pk"
>>>>> mode="IN" optional="false"/>
>>>>> <auto-attributes entity-name="OrderTerm" include="nonpk"
>>>>> mode="IN" optional="true"/>
>>>>> </service>
>>>>> - <service name="removeOrderTerm" engine="simple"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="removeOrderTerm" auth="true">
>>>>> + <service name="removeOrderTerm" default-entity-name="OrderTerm"
>>>>> engine="entity-auto" invoke="delete" auth="true">
>>>>> <description>Remove an Order Term</description>
>>>>> + <required-permissions join-type="AND">
>>>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>>>> + </required-permissions>
>>>>> <auto-attributes entity-name="OrderTerm" include="pk"
>>>>> mode="IN" optional="false"/>
>>>>> </service>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/servicedef/services_requireme
>>>>> nt.xml
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>> servicedef/services_requirement.xml?rev=1726542&r1=1726541&
>>>>> r2=1726542&view=diff
>>>>> ============================================================
>>>>> ==================
>>>>> --- ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>>>> (original)
>>>>> +++ ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>>>> Sun Jan 24 19:06:32 2016
>>>>> @@ -105,8 +105,7 @@ under the License.
>>>>> <attribute mode="OUT" name="quantityTotal" type="BigDecimal"
>>>>> optional="true"/>
>>>>> <attribute mode="OUT" name="amountTotal" type="BigDecimal"
>>>>> optional="true"/>
>>>>> </service>
>>>>> - <service name="createOrderRequirementCommitment" engine="simple"
>>>>> auth="true"
>>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>>>> invoke="createOrderRequirementCommitment">
>>>>> + <service name="createOrderRequirementCommitment"
>>>>> engine="entity-auto" auth="true" default-entity-name="OrderRequirementCommitment"
>>>>> invoke="create">
>>>>> <auto-attributes entity-name="OrderRequirementCommitment"
>>>>> include="pk" mode="IN" optional="false"/>
>>>>> <auto-attributes entity-name="OrderRequirementCommitment"
>>>>> include="nonpk" mode="IN" optional="true"/>
>>>>> </service>
>>>>>
>>>>> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/
>>>>> OrderServices.java
>>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/
>>>>> src/org/ofbiz/order/order/OrderServices.java?rev=1726542
>>>>> &r1=1726541&r2=1726542&view=diff
>>>>> ============================================================
>>>>> ==================
>>>>> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>>>> (original)
>>>>> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>>>> Sun Jan 24 19:06:32 2016
>>>>> @@ -2824,52 +2824,6 @@ public class OrderServices {
>>>>> return ServiceUtil.returnSuccess();
>>>>> }
>>>>> - /** Service to create an order payment preference */
>>>>> - public static Map<String, Object> createPaymentPreference(DispatchContext
>>>>> ctx, Map<String, ? extends Object> context) {
>>>>> - Map<String, Object> result = new HashMap<String, Object>();
>>>>> - Delegator delegator = ctx.getDelegator();
>>>>> - String orderId = (String) context.get("orderId");
>>>>> - String statusId = (String) context.get("statusId");
>>>>> - String paymentMethodTypeId = (String)
>>>>> context.get("paymentMethodTypeId");
>>>>> - String paymentMethodId = (String)
>>>>> context.get("paymentMethodId");
>>>>> - BigDecimal maxAmount = (BigDecimal) context.get("maxAmount");
>>>>> - GenericValue userLogin = (GenericValue)
>>>>> context.get("userLogin");
>>>>> - Locale locale = (Locale) context.get("locale");
>>>>> -
>>>>> - String prefId = null;
>>>>> -
>>>>> - try {
>>>>> - prefId = delegator.getNextSeqId("OrderP
>>>>> aymentPreference");
>>>>> - } catch (IllegalArgumentException e) {
>>>>> - return ServiceUtil.returnError(UtilPr
>>>>> operties.getMessage(resource_error,
>>>>> - "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFailure",
>>>>> locale));
>>>>> - }
>>>>> -
>>>>> - Map<String, Object> fields = UtilMisc.<String,
>>>>> Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId,
>>>>> "paymentMethodTypeId",
>>>>> - paymentMethodTypeId, "paymentMethodId",
>>>>> paymentMethodId, "maxAmount", maxAmount);
>>>>> -
>>>>> - if (statusId != null) {
>>>>> - fields.put("statusId", statusId);
>>>>> - }
>>>>> -
>>>>> - try {
>>>>> - GenericValue v = delegator.makeValue("OrderPaymentPreference",
>>>>> fields);
>>>>> - v.set("createdDate", UtilDateTime.nowTimestamp());
>>>>> - if (userLogin != null) {
>>>>> - v.set("createdByUserLogin",
>>>>> userLogin.getString("userLoginId"));
>>>>> - }
>>>>> - delegator.create(v);
>>>>> - } catch (GenericEntityException e) {
>>>>> - result.put(ModelService.RESPONSE_MESSAGE,
>>>>> ModelService.RESPOND_ERROR);
>>>>> - result.put(ModelService.ERROR_MESSAGE,
>>>>> UtilProperties.getMessage(resource,
>>>>> - "OrderOrderPaymentPreferencesCannotBeCreated",
>>>>> UtilMisc.toMap("errorString", e.getMessage()), locale));
>>>>> - return ServiceUtil.returnFailure();
>>>>> - }
>>>>> - result.put("orderPaymentPreferenceId", prefId);
>>>>> - result.put(ModelService.RESPONSE_MESSAGE,
>>>>> ModelService.RESPOND_SUCCESS);
>>>>> - return result;
>>>>> - }
>>>>> -
>>>>> /** Service to get order header information as standard results.
>>>>> */
>>>>> public static Map<String, Object> getOrderHeaderInformation(DispatchContext
>>>>> dctx, Map<String, ? extends Object> context) {
>>>>> Delegator delegator = dctx.getDelegator();
>>>>>
>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>
>>>
>>>
>>
>>
>>
>
Re: svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Posted by Jacques Le Roux <ja...@les7arts.com>.
Le 12/02/2016 08:23, Nicolas Malin a écrit :
>
> Hi Jacques
>
> Le 11/02/2016 23:10, Jacques Le Roux a écrit :
>> Le 11/02/2016 22:27, Jacques Le Roux a écrit :
>>> Hi Nicolas, All,
>>>
>>> I globally like the changes, I just have a question.
>>>
>>> With these changes the OrderSecurityError* properties which were used here are now useless, should we not remove them?
>>> Despite the lost, I think it's reasonable because these errors have very few chances to appear, and if they appear it's not so hard to find your
>>> way through the log.
> Yes I planned to remove all unused label before declare the conversion end.
>>
>> Oops, I did not see the elephant in the room: they are permission issues. So I guess the message to the user will be less clear. Not sure it's a
>> real issue but we might consider it...
> By default I return to more generic label (http://ofbiz.135035.n4.nabble.com/entity-auto-improvement-Act-2-td4655973.html) but if you think that
> it's big lost, I can try to improve the generic order permission service to resolve them
No, it's OK with me, let's go
Jacques
>
> Nicolas
>>
>> Jacques
>>
>>>
>>> Jacques
>>>
>>>
>>> Le 24/01/2016 20:06, nmalin@apache.org a écrit :
>>>> Author: nmalin
>>>> Date: Sun Jan 24 19:06:32 2016
>>>> New Revision: 1726542
>>>>
>>>> URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
>>>> Log:
>>>>
>>>>
>>>> I converted following services from simple to entity-auto :
>>>>
>>>> createOrderNotificationLog
>>>> createOrderItemBilling
>>>> createOrderAdjustment
>>>> updateOrderAdjustment
>>>> createOrderAdjustmentBilling
>>>> createOrderShipment
>>>> updateOrderShipment
>>>> deleteOrderShipment
>>>> createCommunicationEventOrder
>>>> removeCommunicationEventOrder
>>>> createOrderItemShipGroup
>>>> createOrderContactMech
>>>> removeOrderContactMech
>>>> createOrderTerm
>>>> removeOrderTerm
>>>> createOrderRequirementCommitment
>>>>
>>>> And from java to entity-auto for createOrderPaymentPreference
>>>> Related issue OFBIZ-6854.
>>>>
>>>> Removed:
>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
>>>> Modified:
>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>>> ofbiz/trunk/applications/order/servicedef/services.xml
>>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>>>
>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>>> URL:
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>>> ==============================================================================
>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml (original)
>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml Sun Jan 24 19:06:32 2016
>>>> @@ -88,56 +88,8 @@ under the License.
>>>> <field-to-result field="orderInfo.totalOrders" result-name="totalOrders"/>
>>>> </if-not-empty>
>>>> </simple-method>
>>>> - <simple-method method-name="createOrderShipment" short-description="Create OrderShipment">
>>>> - <set value="Create OrderShipment" field="operationName"/>
>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderShipment"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> -
>>>> - <make-value value-field="newEntity" entity-name="OrderShipment"/>
>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>> -
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> - <simple-method method-name="updateOrderShipment" short-description="Update OrderShipment">
>>>> - <set value="Update OrderShipment" field="operationName"/>
>>>> - <check-permission permission="ORDERMGR" action="_UPDATE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> -
>>>> - <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
>>>> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
>>>> - <store-value value-field="lookedUpValue"/>
>>>> - </simple-method>
>>>> - <simple-method method-name="deleteOrderShipment" short-description="Delete OrderShipment">
>>>> - <set value="Delete OrderShipment" field="operationName"/>
>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> -
>>>> - <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
>>>> - <remove-value value-field="lookedUpValue"/>
>>>> - </simple-method>
>>>> <!-- order requirement methods -->
>>>> - <simple-method method-name="createOrderRequirementCommitment" short-description="Create OrderRequirementCommitment">
>>>> - <!-- The permission checking is commented out to make this service work also when triggered from ecommerce orders -->
>>>> - <!--
>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCheckCreateOrderRequirementCommitment"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> - -->
>>>> - <make-value value-field="newEntity" entity-name="OrderRequirementCommitment"/>
>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> -
>>>> <simple-method method-name="createRequirementAndCommitment" short-description="create a requirement and commitment for it">
>>>> <set value="PRODUCT_REQUIREMENT" field="inputMap.requirementTypeId"/>
>>>> @@ -759,20 +711,6 @@ under the License.
>>>> </simple-method>
>>>> <!-- OrderItemShipGroup -->
>>>> - <simple-method method-name="createOrderItemShipGroup" short-description="Create OrderItemShipGroup">
>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> - <make-value value-field="newEntity" entity-name="OrderItemShipGroup"/>
>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>> - <if-empty field="newEntity.shipGroupSeqId">
>>>> - <make-next-seq-id value-field="newEntity" seq-field-name="shipGroupSeqId"/> <!-- this finds the next sub-sequence ID -->
>>>> - <field-to-result field="newEntity.shipGroupSeqId" result-name="shipGroupSeqId"/>
>>>> - </if-empty>
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> <simple-method method-name="updateOrderItemShipGroup" short-description="Update OrderItemShipGroup">
>>>> <check-permission permission="ORDERMGR" action="_UPDATE">
>>>> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
>>>> @@ -832,16 +770,6 @@ under the License.
>>>> <field-to-result field="orderItemShipGroupInvRes.promisedDatetime" result-name="estimatedShipDate"/>
>>>> </simple-method>
>>>> - <simple-method method-name="createOrderContactMech" short-description="Create OrderContactMech">
>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderContactMech"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> - <make-value value-field="newEntity" entity-name="OrderContactMech"/>
>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> -
>>>> <simple-method method-name="removeOrderContactMech" short-description="Remove OrderContactMech">
>>>> <check-permission permission="ORDERMGR" action="_DELETE">
>>>> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
>>>> @@ -861,39 +789,6 @@ under the License.
>>>> <store-value value-field="orderHeaderNote"/>
>>>> </simple-method>
>>>> - <simple-method method-name="createOrderTerm" short-description="Create an OrderTerm">
>>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderTerm"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> -
>>>> - <make-value value-field="newEntity" entity-name="OrderTerm"/>
>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>> - <set value="_NA_" set-if-empty="true" field="newEntity.orderItemSeqId"/>
>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>> -
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> - <simple-method method-name="updateOrderTerm" short-description="Update OrderTerm">
>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderTerm"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> -
>>>> - <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
>>>> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
>>>> - <store-value value-field="lookedUpValue"/>
>>>> - </simple-method>
>>>> - <simple-method method-name="removeOrderTerm" short-description="Remove OrderTerm">
>>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderTerm"/>
>>>> - </check-permission>
>>>> - <check-errors/>
>>>> -
>>>> - <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
>>>> - <remove-value value-field="lookedUpValue"/>
>>>> - </simple-method>
>>>> -
>>>> <simple-method method-name="addPaymentMethodToOrder" short-description="Create an PaymentMethodToOrder">
>>>> <check-permission permission="ORDERMGR" action="_CREATE">
>>>> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
>>>> @@ -1373,7 +1268,7 @@ under the License.
>>>> </iterate>
>>>> </iterate>
>>>> </simple-method>
>>>> -
>>>> +
>>>> <simple-method method-name="updateShippingMethodAndCharges" short-description="Updates shipping method and shipping charges from Order View
>>>> page when Shipment is in picked status and items of Order are packed">
>>>> <!-- splitting shipmentMethodAndAmount request parameter value that contains "*" symbol
>>>> into "shipmentMethod" and "newAmount".
>>>> @@ -1424,6 +1319,7 @@ under the License.
>>>> </else>
>>>> </if-compare-field>
>>>> </simple-method>
>>>> +
>>>> <simple-method method-name="productAvailabalityByFacility" short-description="Calculate ATP and Qoh According For each facility">
>>>> <set field="facilityMap.ownerPartyId" from-field="parameters.ownerPartyId"/>
>>>> <find-by-and map="facilityMap" list="facilityList" entity-name="Facility"/>
>>>> @@ -1442,7 +1338,7 @@ under the License.
>>>> </iterate>
>>>> <field-to-result field="availabalityList"/>
>>>> </simple-method>
>>>> -
>>>> +
>>>> <simple-method method-name="createOrderPaymentApplication" short-description="Create Order Payment Application">
>>>> <entity-one value-field="paymentMap" entity-name="Payment"/>
>>>> <set field="createCtx.amountApplied" from-field="paymentMap.amount" type="BigDecimal"/>
>>>> @@ -1459,7 +1355,7 @@ under the License.
>>>> <call-service service-name="createPaymentApplication" in-map-name="createCtx"/>
>>>> </if-not-empty>
>>>> </simple-method>
>>>> -
>>>> +
>>>> <simple-method method-name="MoveItemBetweenShipGroups" short-description="Move order items between ship groups">
>>>> <entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
>>>> <field-map field-name="orderId" from-field="parameters.orderId"/>
>>>> @@ -1483,7 +1379,7 @@ under the License.
>>>> <set field="map.shipGroupSeqId" from-field="parameters.toGroupIndex"/>
>>>> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity + parameters.quantity}" type="BigDecimal"/>
>>>> <call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
>>>> -
>>>> +
>>>> <entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
>>>> <field-map field-name="orderId" from-field="parameters.orderId"/>
>>>> <field-map field-name="orderItemSeqId" from-field="parameters.orderItemSeqId"/>
>>>> @@ -1495,7 +1391,7 @@ under the License.
>>>> </add-error>
>>>> </if-empty>
>>>> <check-errors/>
>>>> -
>>>> +
>>>> <clear-field field="map"/>
>>>> <set field="map.orderId" from-field="parameters.orderId"/>
>>>> <set field="map.orderItemSeqId" from-field="parameters.orderItemSeqId"/>
>>>> @@ -1503,5 +1399,5 @@ under the License.
>>>> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity - parameters.quantity}" type="BigDecimal"/>
>>>> <call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
>>>> </simple-method>
>>>> -
>>>> +
>>>> </simple-methods>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>>> URL:
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>>> ==============================================================================
>>>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml (original)
>>>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml Sun Jan 24 19:06:32 2016
>>>> @@ -44,33 +44,12 @@ under the License.
>>>> </else>
>>>> </if-compare>
>>>> </simple-method>
>>>> -
>>>> - <simple-method method-name="createOrderAdjustment" short-description="Create an OrderAdjustment">
>>>> -
>>>> - <make-value entity-name="OrderAdjustment" value-field="newEntity"/>
>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>> -
>>>> - <sequenced-id sequence-name="OrderAdjustment" field="newEntity.orderAdjustmentId"/>
>>>> - <field-to-result field="newEntity.orderAdjustmentId" result-name="orderAdjustmentId"/>
>>>> -
>>>> - <now-timestamp field="newEntity.createdDate"/>
>>>> - <set from-field="userLogin.userLoginId" field="newEntity.createdByUserLogin"/>
>>>> -
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> - <simple-method method-name="updateOrderAdjustment" short-description="Update an OrderAdjustment">
>>>> - <check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>>> property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></check-permission>
>>>> - <check-errors/>
>>>> - <entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
>>>> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
>>>> - <store-value value-field="lookedUpValue"/>
>>>> - </simple-method>
>>>> <simple-method method-name="deleteOrderAdjustment" short-description="Delete an OrderAdjustment">
>>>> <check-permission permission="ORDERMGR" action="_DELETE"><fail-property resource="OrderErrorUiLabels"
>>>> property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></check-permission>
>>>> <check-errors/>
>>>> <entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
>>>> <remove-value value-field="lookedUpValue"/>
>>>> -
>>>> +
>>>> <if-not-empty field="parameters.productPromoCodeId">
>>>> <entity-one entity-name="OrderProductPromoCode" value-field="lookedUpValue"/>
>>>> <if-not-empty field="lookedUpValue">
>>>> @@ -79,30 +58,6 @@ under the License.
>>>> </if-not-empty>
>>>> </simple-method>
>>>> - <simple-method method-name="createOrderAdjustmentBilling" short-description="Create an OrderAdjustmentBilling">
>>>> - <make-value entity-name="OrderAdjustmentBilling" value-field="newEntity"/>
>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> -
>>>> - <simple-method method-name="createOrderItemBilling" short-description="Create an OrderItemBilling">
>>>> - <make-value entity-name="OrderItemBilling" value-field="newEntity"/>
>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>>> - <create-value value-field="newEntity"/>
>>>> - </simple-method>
>>>> -
>>>> - <simple-method method-name="createNotificationLog" short-description="Log an order notification" login-required="false">
>>>> - <make-value entity-name="OrderNotification" value-field="orderNotification"/>
>>>> - <sequenced-id sequence-name="OrderNotification" field="orderNotification.orderNotificationId"/>
>>>> - <set from-field="parameters.orderId" field="orderNotification.orderId"/>
>>>> - <set from-field="parameters.emailType" field="orderNotification.emailType"/>
>>>> - <set from-field="parameters.comments" field="orderNotification.comments"/>
>>>> - <now-timestamp field="orderNotification.notificationDate"/>
>>>> - <create-value value-field="orderNotification"/>
>>>> - </simple-method>
>>>> -
>>>> <simple-method method-name="updateOrderStatusFromReceipt" short-description="Update Order Status From ShipmentReceipt">
>>>> <check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>>> property="OrderSecurityErrorToRunUpdateOrderStatusFromReceipt"/></check-permission>
>>>> <check-errors/>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/servicedef/services.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>>> ==============================================================================
>>>> --- ofbiz/trunk/applications/order/servicedef/services.xml (original)
>>>> +++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan 24 19:06:32 2016
>>>> @@ -76,8 +76,7 @@ under the License.
>>>> <attribute name="assignedRoleTypeId" type="String" mode="IN" optional="true"/>
>>>> </service>
>>>> - <service name="createOrderNotificationLog" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createNotificationLog">
>>>> + <service name="createOrderNotificationLog" engine="entity-auto" default-entity-name="OrderNotification" invoke="create">
>>>> <description>Logs when a notification was sent</description>
>>>> <attribute name="orderId" type="String" mode="IN" optional="false"/>
>>>> <attribute name="emailType" type="String" mode="IN" optional="false"/>
>>>> @@ -164,17 +163,10 @@ under the License.
>>>> <attribute name="orderId" type="String" mode="OUT" optional="false"/>
>>>> </service>
>>>> - <service name="createOrderItemBilling" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderItemBilling">
>>>> + <service name="createOrderItemBilling" default-entity-name="OrderItemBilling" engine="entity-auto" invoke="create">
>>>> <description>Create a new order item billing record</description>
>>>> - <attribute name="orderId" type="String" mode="IN" optional="false"/>
>>>> - <attribute name="orderItemSeqId" type="String" mode="IN" optional="false"/>
>>>> - <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
>>>> - <attribute name="quantity" type="BigDecimal" mode="IN" optional="true"/>
>>>> - <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
>>>> - <attribute name="itemIssuanceId" type="String" mode="IN" optional="true"/>
>>>> - <attribute name="shipmentReceiptId" type="String" mode="IN" optional="true"/>
>>>> + <auto-attributes mode="IN" include="pk"/>
>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>> </service>
>>>> <service name="orderAdjustmentPermissionCheck" engine="simple"
>>>> @@ -183,8 +175,7 @@ under the License.
>>>> <implements service="permissionInterface"/>
>>>> </service>
>>>> - <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustment">
>>>> + <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="create">
>>>> <description>Creates a new order adjustment record</description>
>>>> <permission-service service-name="orderAdjustmentPermissionCheck" main-action="CREATE"/>
>>>> <auto-attributes mode="OUT" include="pk" optional="false"/>
>>>> @@ -192,9 +183,9 @@ under the License.
>>>> <override name="orderAdjustmentTypeId" optional="false"/>
>>>> <override name="orderId" optional="false"/>
>>>> </service>
>>>> - <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="updateOrderAdjustment">
>>>> + <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="update">
>>>> <description>Update an order adjustment record</description>
>>>> + <permission-service service-name="orderAdjustmentPermissionCheck" main-action="UPDATE"/>
>>>> <auto-attributes mode="IN" include="pk" optional="false"/>
>>>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>> <override name="orderId" optional="false"/>
>>>> @@ -207,13 +198,10 @@ under the License.
>>>> <attribute name="productPromoCodeId" type="String" mode="IN" optional="true"/>
>>>> </service>
>>>> - <service name="createOrderAdjustmentBilling" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustmentBilling">
>>>> + <service name="createOrderAdjustmentBilling" default-entity-name="OrderAdjustmentBilling" engine="entity-auto" invoke="create">
>>>> <description>Create a new order adjustment billing record</description>
>>>> - <attribute name="orderAdjustmentId" type="String" mode="IN" optional="false"/>
>>>> - <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
>>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
>>>> - <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
>>>> + <auto-attributes mode="IN" include="pk"/>
>>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>> </service>
>>>> <service name="createPaymentFromPreference" engine="java"
>>>> @@ -435,15 +423,10 @@ under the License.
>>>> <attribute name="roleTypeId" type="String" mode="IN"/>
>>>> </service>
>>>> - <service name="createOrderPaymentPreference" engine="java"
>>>> - location="org.ofbiz.order.order.OrderServices" invoke="createPaymentPreference">
>>>> + <service name="createOrderPaymentPreference" default-entity-name="OrderPaymentPreference" engine="entity-auto" invoke="create">
>>>> <description>Creates an order payment preference</description>
>>>> - <attribute name="orderId" type="String" mode="IN"/>
>>>> - <attribute name="paymentMethodTypeId" type="String" mode="IN"/>
>>>> - <attribute name="paymentMethodId" type="String" mode="IN" optional="true"/>
>>>> - <attribute name="statusId" type="String" mode="IN" optional="true"/>
>>>> - <attribute name="maxAmount" type="BigDecimal" mode="IN" optional="true"/>
>>>> - <attribute name="orderPaymentPreferenceId" type="String" mode="OUT"/>
>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>> + <auto-attributes include="pk" mode="OUT"/>
>>>> </service>
>>>> <service name="createOrderNote" engine="java"
>>>> @@ -568,20 +551,17 @@ under the License.
>>>> </service>
>>>> <!-- OrderShipment Services -->
>>>> - <service name="createOrderShipment" default-entity-name="OrderShipment" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderShipment" auth="true">
>>>> + <service name="createOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="create" auth="true">
>>>> <description>Create OrderShipment</description>
>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>> </service>
>>>> - <service name="updateOrderShipment" default-entity-name="OrderShipment" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderShipment" auth="true">
>>>> + <service name="updateOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="update" auth="true">
>>>> <description>Update OrderShipment</description>
>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>> </service>
>>>> - <service name="deleteOrderShipment" default-entity-name="OrderShipment" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="deleteOrderShipment" auth="true">
>>>> + <service name="deleteOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="delete" auth="true">
>>>> <description>Delete OrderShipment</description>
>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>> </service>
>>>> @@ -667,21 +647,27 @@ under the License.
>>>> </service>
>>>> <!-- CommunicationEvents related to orders -->
>>>> - <service name="createCommunicationEventOrder" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="createCommunicationEventOrder"
>>>> auth="true">
>>>> + <service name="createCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="create" auth="true">
>>>> <description>Create a Communication Event Order</description>
>>>> - <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>> + </required-permissions>
>>>> + <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
>>>> </service>
>>>> - <service name="removeCommunicationEventOrder" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="removeCommunicationEventOrder"
>>>> auth="true">
>>>> + <service name="removeCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="delete" auth="true">
>>>> <description>Remove a Communication Event Order</description>
>>>> - <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>>> + </required-permissions>
>>>> + <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
>>>> </service>
>>>> <!-- Order Shipping and Contacts -->
>>>> - <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderItemShipGroup" auth="true">
>>>> + <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="entity-auto" invoke="create" auth="true">
>>>> <description>Creates a new OrderItemShipGroup.</description>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>> + </required-permissions>
>>>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>>> <attribute name="orderId" type="String" mode="IN" optional="false"/>
>>>> <attribute name="shipGroupSeqId" type="String" mode="INOUT" optional="true"/>
>>>> @@ -696,42 +682,52 @@ under the License.
>>>> <attribute name="contactMechPurposeTypeId" type="String" mode="IN" optional="true"/>
>>>> </service>
>>>> - <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderContactMech" auth="true">
>>>> + <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="create" auth="true">
>>>> <description>Create Order Contact Mech</description>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>> + </required-permissions>
>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>> </service>
>>>> - <service name="updateOrderContactMech" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true"
>>>> default-entity-name="OrderContactMech">
>>>> + <service name="updateOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
>>>> + location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true">
>>>> <description>Update Order Contact Mech</description>
>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>> <attribute name="oldContactMechId" type="String" mode="IN" optional="true"/>
>>>> </service>
>>>> - <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderContactMech" auth="true">
>>>> + <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="delete" auth="true">
>>>> <description>Remove Order Contact Mech</description>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>>> + </required-permissions>
>>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>>> </service>
>>>> <!-- OrderTerms -->
>>>> - <service name="createOrderTerm" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderTerm" auth="true">
>>>> + <service name="createOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="create" auth="true">
>>>> <description>Create an Order Term</description>
>>>> - <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
>>>> - <auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
>>>> - <override name="orderItemSeqId" optional="true"/>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>>> + </required-permissions>
>>>> + <auto-attributes include="pk" mode="IN" optional="false"/>
>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>>> + <override name="orderItemSeqId" optional="true" default-value="_NA_"/>
>>>> </service>
>>>> - <service name="updateOrderTerm" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderTerm" auth="true">
>>>> + <service name="updateOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="update" auth="true">
>>>> <description>Update an Order Term</description>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="UPDATE"/>
>>>> + </required-permissions>
>>>> <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
>>>> <auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
>>>> </service>
>>>> - <service name="removeOrderTerm" engine="simple"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderTerm" auth="true">
>>>> + <service name="removeOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="delete" auth="true">
>>>> <description>Remove an Order Term</description>
>>>> + <required-permissions join-type="AND">
>>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>>> + </required-permissions>
>>>> <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
>>>> </service>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_requirement.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>>> ==============================================================================
>>>> --- ofbiz/trunk/applications/order/servicedef/services_requirement.xml (original)
>>>> +++ ofbiz/trunk/applications/order/servicedef/services_requirement.xml Sun Jan 24 19:06:32 2016
>>>> @@ -105,8 +105,7 @@ under the License.
>>>> <attribute mode="OUT" name="quantityTotal" type="BigDecimal" optional="true"/>
>>>> <attribute mode="OUT" name="amountTotal" type="BigDecimal" optional="true"/>
>>>> </service>
>>>> - <service name="createOrderRequirementCommitment" engine="simple" auth="true"
>>>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderRequirementCommitment">
>>>> + <service name="createOrderRequirementCommitment" engine="entity-auto" auth="true" default-entity-name="OrderRequirementCommitment"
>>>> invoke="create">
>>>> <auto-attributes entity-name="OrderRequirementCommitment" include="pk" mode="IN" optional="false"/>
>>>> <auto-attributes entity-name="OrderRequirementCommitment" include="nonpk" mode="IN" optional="true"/>
>>>> </service>
>>>>
>>>> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>>> URL:
>>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=1726542&r1=1726541&r2=1726542&view=diff
>>>> ==============================================================================
>>>> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java (original)
>>>> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java Sun Jan 24 19:06:32 2016
>>>> @@ -2824,52 +2824,6 @@ public class OrderServices {
>>>> return ServiceUtil.returnSuccess();
>>>> }
>>>> - /** Service to create an order payment preference */
>>>> - public static Map<String, Object> createPaymentPreference(DispatchContext ctx, Map<String, ? extends Object> context) {
>>>> - Map<String, Object> result = new HashMap<String, Object>();
>>>> - Delegator delegator = ctx.getDelegator();
>>>> - String orderId = (String) context.get("orderId");
>>>> - String statusId = (String) context.get("statusId");
>>>> - String paymentMethodTypeId = (String) context.get("paymentMethodTypeId");
>>>> - String paymentMethodId = (String) context.get("paymentMethodId");
>>>> - BigDecimal maxAmount = (BigDecimal) context.get("maxAmount");
>>>> - GenericValue userLogin = (GenericValue) context.get("userLogin");
>>>> - Locale locale = (Locale) context.get("locale");
>>>> -
>>>> - String prefId = null;
>>>> -
>>>> - try {
>>>> - prefId = delegator.getNextSeqId("OrderPaymentPreference");
>>>> - } catch (IllegalArgumentException e) {
>>>> - return ServiceUtil.returnError(UtilProperties.getMessage(resource_error,
>>>> - "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFailure", locale));
>>>> - }
>>>> -
>>>> - Map<String, Object> fields = UtilMisc.<String, Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId, "paymentMethodTypeId",
>>>> - paymentMethodTypeId, "paymentMethodId", paymentMethodId, "maxAmount", maxAmount);
>>>> -
>>>> - if (statusId != null) {
>>>> - fields.put("statusId", statusId);
>>>> - }
>>>> -
>>>> - try {
>>>> - GenericValue v = delegator.makeValue("OrderPaymentPreference", fields);
>>>> - v.set("createdDate", UtilDateTime.nowTimestamp());
>>>> - if (userLogin != null) {
>>>> - v.set("createdByUserLogin", userLogin.getString("userLoginId"));
>>>> - }
>>>> - delegator.create(v);
>>>> - } catch (GenericEntityException e) {
>>>> - result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_ERROR);
>>>> - result.put(ModelService.ERROR_MESSAGE, UtilProperties.getMessage(resource,
>>>> - "OrderOrderPaymentPreferencesCannotBeCreated", UtilMisc.toMap("errorString", e.getMessage()), locale));
>>>> - return ServiceUtil.returnFailure();
>>>> - }
>>>> - result.put("orderPaymentPreferenceId", prefId);
>>>> - result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_SUCCESS);
>>>> - return result;
>>>> - }
>>>> -
>>>> /** Service to get order header information as standard results. */
>>>> public static Map<String, Object> getOrderHeaderInformation(DispatchContext dctx, Map<String, ? extends Object> context) {
>>>> Delegator delegator = dctx.getDelegator();
>>>>
>>>>
>>>>
>>>
>>>
>>
>>
>>
>
>
>
Re: svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Posted by Nicolas Malin <ni...@nereide.fr>.
Hi Jacques
Le 11/02/2016 23:10, Jacques Le Roux a écrit :
> Le 11/02/2016 22:27, Jacques Le Roux a écrit :
>> Hi Nicolas, All,
>>
>> I globally like the changes, I just have a question.
>>
>> With these changes the OrderSecurityError* properties which were used
>> here are now useless, should we not remove them?
>> Despite the lost, I think it's reasonable because these errors have
>> very few chances to appear, and if they appear it's not so hard to
>> find your way through the log.
Yes I planned to remove all unused label before declare the conversion end.
>
> Oops, I did not see the elephant in the room: they are permission
> issues. So I guess the message to the user will be less clear. Not
> sure it's a real issue but we might consider it...
By default I return to more generic label
(http://ofbiz.135035.n4.nabble.com/entity-auto-improvement-Act-2-td4655973.html)
but if you think that it's big lost, I can try to improve the generic
order permission service to resolve them
Nicolas
>
> Jacques
>
>>
>> Jacques
>>
>>
>> Le 24/01/2016 20:06, nmalin@apache.org a écrit :
>>> Author: nmalin
>>> Date: Sun Jan 24 19:06:32 2016
>>> New Revision: 1726542
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
>>> Log:
>>>
>>>
>>> I converted following services from simple to entity-auto :
>>>
>>> createOrderNotificationLog
>>> createOrderItemBilling
>>> createOrderAdjustment
>>> updateOrderAdjustment
>>> createOrderAdjustmentBilling
>>> createOrderShipment
>>> updateOrderShipment
>>> deleteOrderShipment
>>> createCommunicationEventOrder
>>> removeCommunicationEventOrder
>>> createOrderItemShipGroup
>>> createOrderContactMech
>>> removeOrderContactMech
>>> createOrderTerm
>>> removeOrderTerm
>>> createOrderRequirementCommitment
>>>
>>> And from java to entity-auto for createOrderPaymentPreference
>>> Related issue OFBIZ-6854.
>>>
>>> Removed:
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
>>> Modified:
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>>
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>>
>>> ofbiz/trunk/applications/order/servicedef/services.xml
>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>>
>>>
>>> Modified:
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>> URL:
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>> ==============================================================================
>>>
>>> ---
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>> (original)
>>> +++
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>>> Sun Jan 24 19:06:32 2016
>>> @@ -88,56 +88,8 @@ under the License.
>>> <field-to-result field="orderInfo.totalOrders"
>>> result-name="totalOrders"/>
>>> </if-not-empty>
>>> </simple-method>
>>> - <simple-method method-name="createOrderShipment"
>>> short-description="Create OrderShipment">
>>> - <set value="Create OrderShipment" field="operationName"/>
>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunCreateOrderShipment"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> -
>>> - <make-value value-field="newEntity"
>>> entity-name="OrderShipment"/>
>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>> -
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> - <simple-method method-name="updateOrderShipment"
>>> short-description="Update OrderShipment">
>>> - <set value="Update OrderShipment" field="operationName"/>
>>> - <check-permission permission="ORDERMGR" action="_UPDATE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> -
>>> - <entity-one entity-name="OrderShipment"
>>> value-field="lookedUpValue"/>
>>> - <set-nonpk-fields map="parameters"
>>> value-field="lookedUpValue"/>
>>> - <store-value value-field="lookedUpValue"/>
>>> - </simple-method>
>>> - <simple-method method-name="deleteOrderShipment"
>>> short-description="Delete OrderShipment">
>>> - <set value="Delete OrderShipment" field="operationName"/>
>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> -
>>> - <entity-one entity-name="OrderShipment"
>>> value-field="lookedUpValue"/>
>>> - <remove-value value-field="lookedUpValue"/>
>>> - </simple-method>
>>> <!-- order requirement methods -->
>>> - <simple-method method-name="createOrderRequirementCommitment"
>>> short-description="Create OrderRequirementCommitment">
>>> - <!-- The permission checking is commented out to make this
>>> service work also when triggered from ecommerce orders -->
>>> - <!--
>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunCheckCreateOrderRequirementCommitment"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> - -->
>>> - <make-value value-field="newEntity"
>>> entity-name="OrderRequirementCommitment"/>
>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> -
>>> <simple-method method-name="createRequirementAndCommitment"
>>> short-description="create a requirement and commitment for it">
>>> <set value="PRODUCT_REQUIREMENT"
>>> field="inputMap.requirementTypeId"/>
>>> @@ -759,20 +711,6 @@ under the License.
>>> </simple-method>
>>> <!-- OrderItemShipGroup -->
>>> - <simple-method method-name="createOrderItemShipGroup"
>>> short-description="Create OrderItemShipGroup">
>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> - <make-value value-field="newEntity"
>>> entity-name="OrderItemShipGroup"/>
>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>> - <if-empty field="newEntity.shipGroupSeqId">
>>> - <make-next-seq-id value-field="newEntity"
>>> seq-field-name="shipGroupSeqId"/> <!-- this finds the next
>>> sub-sequence ID -->
>>> - <field-to-result field="newEntity.shipGroupSeqId"
>>> result-name="shipGroupSeqId"/>
>>> - </if-empty>
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> <simple-method method-name="updateOrderItemShipGroup"
>>> short-description="Update OrderItemShipGroup">
>>> <check-permission permission="ORDERMGR" action="_UPDATE">
>>> <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
>>> @@ -832,16 +770,6 @@ under the License.
>>> <field-to-result
>>> field="orderItemShipGroupInvRes.promisedDatetime"
>>> result-name="estimatedShipDate"/>
>>> </simple-method>
>>> - <simple-method method-name="createOrderContactMech"
>>> short-description="Create OrderContactMech">
>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunCreateOrderContactMech"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> - <make-value value-field="newEntity"
>>> entity-name="OrderContactMech"/>
>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> -
>>> <simple-method method-name="removeOrderContactMech"
>>> short-description="Remove OrderContactMech">
>>> <check-permission permission="ORDERMGR" action="_DELETE">
>>> <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
>>> @@ -861,39 +789,6 @@ under the License.
>>> <store-value value-field="orderHeaderNote"/>
>>> </simple-method>
>>> - <simple-method method-name="createOrderTerm"
>>> short-description="Create an OrderTerm">
>>> - <check-permission permission="ORDERMGR" action="_CREATE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunCreateOrderTerm"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> -
>>> - <make-value value-field="newEntity" entity-name="OrderTerm"/>
>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>> - <set value="_NA_" set-if-empty="true"
>>> field="newEntity.orderItemSeqId"/>
>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>> -
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> - <simple-method method-name="updateOrderTerm"
>>> short-description="Update OrderTerm">
>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunUpdateOrderTerm"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> -
>>> - <entity-one entity-name="OrderTerm"
>>> value-field="lookedUpValue"/>
>>> - <set-nonpk-fields map="parameters"
>>> value-field="lookedUpValue"/>
>>> - <store-value value-field="lookedUpValue"/>
>>> - </simple-method>
>>> - <simple-method method-name="removeOrderTerm"
>>> short-description="Remove OrderTerm">
>>> - <check-permission permission="ORDERMGR" action="_DELETE">
>>> - <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunRemoveOrderTerm"/>
>>> - </check-permission>
>>> - <check-errors/>
>>> -
>>> - <entity-one entity-name="OrderTerm"
>>> value-field="lookedUpValue"/>
>>> - <remove-value value-field="lookedUpValue"/>
>>> - </simple-method>
>>> -
>>> <simple-method method-name="addPaymentMethodToOrder"
>>> short-description="Create an PaymentMethodToOrder">
>>> <check-permission permission="ORDERMGR" action="_CREATE">
>>> <fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
>>> @@ -1373,7 +1268,7 @@ under the License.
>>> </iterate>
>>> </iterate>
>>> </simple-method>
>>> -
>>> +
>>> <simple-method method-name="updateShippingMethodAndCharges"
>>> short-description="Updates shipping method and shipping charges from
>>> Order View page when Shipment is in picked status and items of Order
>>> are packed">
>>> <!-- splitting shipmentMethodAndAmount request parameter
>>> value that contains "*" symbol
>>> into "shipmentMethod" and "newAmount".
>>> @@ -1424,6 +1319,7 @@ under the License.
>>> </else>
>>> </if-compare-field>
>>> </simple-method>
>>> +
>>> <simple-method method-name="productAvailabalityByFacility"
>>> short-description="Calculate ATP and Qoh According For each facility">
>>> <set field="facilityMap.ownerPartyId"
>>> from-field="parameters.ownerPartyId"/>
>>> <find-by-and map="facilityMap" list="facilityList"
>>> entity-name="Facility"/>
>>> @@ -1442,7 +1338,7 @@ under the License.
>>> </iterate>
>>> <field-to-result field="availabalityList"/>
>>> </simple-method>
>>> -
>>> +
>>> <simple-method method-name="createOrderPaymentApplication"
>>> short-description="Create Order Payment Application">
>>> <entity-one value-field="paymentMap" entity-name="Payment"/>
>>> <set field="createCtx.amountApplied"
>>> from-field="paymentMap.amount" type="BigDecimal"/>
>>> @@ -1459,7 +1355,7 @@ under the License.
>>> <call-service service-name="createPaymentApplication"
>>> in-map-name="createCtx"/>
>>> </if-not-empty>
>>> </simple-method>
>>> -
>>> +
>>> <simple-method method-name="MoveItemBetweenShipGroups"
>>> short-description="Move order items between ship groups">
>>> <entity-one entity-name="OrderItemShipGroupAssoc"
>>> value-field="orderItemShipGroupAssoc">
>>> <field-map field-name="orderId"
>>> from-field="parameters.orderId"/>
>>> @@ -1483,7 +1379,7 @@ under the License.
>>> <set field="map.shipGroupSeqId"
>>> from-field="parameters.toGroupIndex"/>
>>> <set field="map.quantity"
>>> value="${orderItemShipGroupAssoc.quantity + parameters.quantity}"
>>> type="BigDecimal"/>
>>> <call-service service-name="updateOrderItemShipGroupAssoc"
>>> in-map-name="map"/>
>>> -
>>> +
>>> <entity-one entity-name="OrderItemShipGroupAssoc"
>>> value-field="orderItemShipGroupAssoc">
>>> <field-map field-name="orderId"
>>> from-field="parameters.orderId"/>
>>> <field-map field-name="orderItemSeqId"
>>> from-field="parameters.orderItemSeqId"/>
>>> @@ -1495,7 +1391,7 @@ under the License.
>>> </add-error>
>>> </if-empty>
>>> <check-errors/>
>>> -
>>> +
>>> <clear-field field="map"/>
>>> <set field="map.orderId" from-field="parameters.orderId"/>
>>> <set field="map.orderItemSeqId"
>>> from-field="parameters.orderItemSeqId"/>
>>> @@ -1503,5 +1399,5 @@ under the License.
>>> <set field="map.quantity"
>>> value="${orderItemShipGroupAssoc.quantity - parameters.quantity}"
>>> type="BigDecimal"/>
>>> <call-service service-name="updateOrderItemShipGroupAssoc"
>>> in-map-name="map"/>
>>> </simple-method>
>>> -
>>> +
>>> </simple-methods>
>>>
>>> Modified:
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>> URL:
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>> ==============================================================================
>>>
>>> ---
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>> (original)
>>> +++
>>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>>> Sun Jan 24 19:06:32 2016
>>> @@ -44,33 +44,12 @@ under the License.
>>> </else>
>>> </if-compare>
>>> </simple-method>
>>> -
>>> - <simple-method method-name="createOrderAdjustment"
>>> short-description="Create an OrderAdjustment">
>>> -
>>> - <make-value entity-name="OrderAdjustment"
>>> value-field="newEntity"/>
>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>> -
>>> - <sequenced-id sequence-name="OrderAdjustment"
>>> field="newEntity.orderAdjustmentId"/>
>>> - <field-to-result field="newEntity.orderAdjustmentId"
>>> result-name="orderAdjustmentId"/>
>>> -
>>> - <now-timestamp field="newEntity.createdDate"/>
>>> - <set from-field="userLogin.userLoginId"
>>> field="newEntity.createdByUserLogin"/>
>>> -
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> - <simple-method method-name="updateOrderAdjustment"
>>> short-description="Update an OrderAdjustment">
>>> - <check-permission permission="ORDERMGR"
>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></check-permission>
>>> - <check-errors/>
>>> - <entity-one entity-name="OrderAdjustment"
>>> value-field="lookedUpValue"/>
>>> - <set-nonpk-fields map="parameters"
>>> value-field="lookedUpValue"/>
>>> - <store-value value-field="lookedUpValue"/>
>>> - </simple-method>
>>> <simple-method method-name="deleteOrderAdjustment"
>>> short-description="Delete an OrderAdjustment">
>>> <check-permission permission="ORDERMGR"
>>> action="_DELETE"><fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></check-permission>
>>> <check-errors/>
>>> <entity-one entity-name="OrderAdjustment"
>>> value-field="lookedUpValue"/>
>>> <remove-value value-field="lookedUpValue"/>
>>> -
>>> +
>>> <if-not-empty field="parameters.productPromoCodeId">
>>> <entity-one entity-name="OrderProductPromoCode"
>>> value-field="lookedUpValue"/>
>>> <if-not-empty field="lookedUpValue">
>>> @@ -79,30 +58,6 @@ under the License.
>>> </if-not-empty>
>>> </simple-method>
>>> - <simple-method method-name="createOrderAdjustmentBilling"
>>> short-description="Create an OrderAdjustmentBilling">
>>> - <make-value entity-name="OrderAdjustmentBilling"
>>> value-field="newEntity"/>
>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> -
>>> - <simple-method method-name="createOrderItemBilling"
>>> short-description="Create an OrderItemBilling">
>>> - <make-value entity-name="OrderItemBilling"
>>> value-field="newEntity"/>
>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>>> - <create-value value-field="newEntity"/>
>>> - </simple-method>
>>> -
>>> - <simple-method method-name="createNotificationLog"
>>> short-description="Log an order notification" login-required="false">
>>> - <make-value entity-name="OrderNotification"
>>> value-field="orderNotification"/>
>>> - <sequenced-id sequence-name="OrderNotification"
>>> field="orderNotification.orderNotificationId"/>
>>> - <set from-field="parameters.orderId"
>>> field="orderNotification.orderId"/>
>>> - <set from-field="parameters.emailType"
>>> field="orderNotification.emailType"/>
>>> - <set from-field="parameters.comments"
>>> field="orderNotification.comments"/>
>>> - <now-timestamp field="orderNotification.notificationDate"/>
>>> - <create-value value-field="orderNotification"/>
>>> - </simple-method>
>>> -
>>> <simple-method method-name="updateOrderStatusFromReceipt"
>>> short-description="Update Order Status From ShipmentReceipt">
>>> <check-permission permission="ORDERMGR"
>>> action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>>> property="OrderSecurityErrorToRunUpdateOrderStatusFromReceipt"/></check-permission>
>>> <check-errors/>
>>>
>>> Modified: ofbiz/trunk/applications/order/servicedef/services.xml
>>> URL:
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>> ==============================================================================
>>>
>>> --- ofbiz/trunk/applications/order/servicedef/services.xml (original)
>>> +++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan
>>> 24 19:06:32 2016
>>> @@ -76,8 +76,7 @@ under the License.
>>> <attribute name="assignedRoleTypeId" type="String"
>>> mode="IN" optional="true"/>
>>> </service>
>>> - <service name="createOrderNotificationLog" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>> invoke="createNotificationLog">
>>> + <service name="createOrderNotificationLog" engine="entity-auto"
>>> default-entity-name="OrderNotification" invoke="create">
>>> <description>Logs when a notification was sent</description>
>>> <attribute name="orderId" type="String" mode="IN"
>>> optional="false"/>
>>> <attribute name="emailType" type="String" mode="IN"
>>> optional="false"/>
>>> @@ -164,17 +163,10 @@ under the License.
>>> <attribute name="orderId" type="String" mode="OUT"
>>> optional="false"/>
>>> </service>
>>> - <service name="createOrderItemBilling" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>> invoke="createOrderItemBilling">
>>> + <service name="createOrderItemBilling"
>>> default-entity-name="OrderItemBilling" engine="entity-auto"
>>> invoke="create">
>>> <description>Create a new order item billing
>>> record</description>
>>> - <attribute name="orderId" type="String" mode="IN"
>>> optional="false"/>
>>> - <attribute name="orderItemSeqId" type="String" mode="IN"
>>> optional="false"/>
>>> - <attribute name="invoiceId" type="String" mode="IN"
>>> optional="false"/>
>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
>>> optional="false"/>
>>> - <attribute name="quantity" type="BigDecimal" mode="IN"
>>> optional="true"/>
>>> - <attribute name="amount" type="BigDecimal" mode="IN"
>>> optional="true"/>
>>> - <attribute name="itemIssuanceId" type="String" mode="IN"
>>> optional="true"/>
>>> - <attribute name="shipmentReceiptId" type="String" mode="IN"
>>> optional="true"/>
>>> + <auto-attributes mode="IN" include="pk"/>
>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>> </service>
>>> <service name="orderAdjustmentPermissionCheck" engine="simple"
>>> @@ -183,8 +175,7 @@ under the License.
>>> <implements service="permissionInterface"/>
>>> </service>
>>> - <service name="createOrderAdjustment"
>>> default-entity-name="OrderAdjustment" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>> invoke="createOrderAdjustment">
>>> + <service name="createOrderAdjustment"
>>> default-entity-name="OrderAdjustment" engine="entity-auto"
>>> invoke="create">
>>> <description>Creates a new order adjustment
>>> record</description>
>>> <permission-service
>>> service-name="orderAdjustmentPermissionCheck" main-action="CREATE"/>
>>> <auto-attributes mode="OUT" include="pk" optional="false"/>
>>> @@ -192,9 +183,9 @@ under the License.
>>> <override name="orderAdjustmentTypeId" optional="false"/>
>>> <override name="orderId" optional="false"/>
>>> </service>
>>> - <service name="updateOrderAdjustment"
>>> default-entity-name="OrderAdjustment" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>> invoke="updateOrderAdjustment">
>>> + <service name="updateOrderAdjustment"
>>> default-entity-name="OrderAdjustment" engine="entity-auto"
>>> invoke="update">
>>> <description>Update an order adjustment record</description>
>>> + <permission-service
>>> service-name="orderAdjustmentPermissionCheck" main-action="UPDATE"/>
>>> <auto-attributes mode="IN" include="pk" optional="false"/>
>>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>> <override name="orderId" optional="false"/>
>>> @@ -207,13 +198,10 @@ under the License.
>>> <attribute name="productPromoCodeId" type="String"
>>> mode="IN" optional="true"/>
>>> </service>
>>> - <service name="createOrderAdjustmentBilling" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml"
>>> invoke="createOrderAdjustmentBilling">
>>> + <service name="createOrderAdjustmentBilling"
>>> default-entity-name="OrderAdjustmentBilling" engine="entity-auto"
>>> invoke="create">
>>> <description>Create a new order adjustment billing
>>> record</description>
>>> - <attribute name="orderAdjustmentId" type="String" mode="IN"
>>> optional="false"/>
>>> - <attribute name="invoiceId" type="String" mode="IN"
>>> optional="false"/>
>>> - <attribute name="invoiceItemSeqId" type="String" mode="IN"
>>> optional="false"/>
>>> - <attribute name="amount" type="BigDecimal" mode="IN"
>>> optional="true"/>
>>> + <auto-attributes mode="IN" include="pk"/>
>>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>> </service>
>>> <service name="createPaymentFromPreference" engine="java"
>>> @@ -435,15 +423,10 @@ under the License.
>>> <attribute name="roleTypeId" type="String" mode="IN"/>
>>> </service>
>>> - <service name="createOrderPaymentPreference" engine="java"
>>> - location="org.ofbiz.order.order.OrderServices"
>>> invoke="createPaymentPreference">
>>> + <service name="createOrderPaymentPreference"
>>> default-entity-name="OrderPaymentPreference" engine="entity-auto"
>>> invoke="create">
>>> <description>Creates an order payment
>>> preference</description>
>>> - <attribute name="orderId" type="String" mode="IN"/>
>>> - <attribute name="paymentMethodTypeId" type="String"
>>> mode="IN"/>
>>> - <attribute name="paymentMethodId" type="String" mode="IN"
>>> optional="true"/>
>>> - <attribute name="statusId" type="String" mode="IN"
>>> optional="true"/>
>>> - <attribute name="maxAmount" type="BigDecimal" mode="IN"
>>> optional="true"/>
>>> - <attribute name="orderPaymentPreferenceId" type="String"
>>> mode="OUT"/>
>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>> + <auto-attributes include="pk" mode="OUT"/>
>>> </service>
>>> <service name="createOrderNote" engine="java"
>>> @@ -568,20 +551,17 @@ under the License.
>>> </service>
>>> <!-- OrderShipment Services -->
>>> - <service name="createOrderShipment"
>>> default-entity-name="OrderShipment" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="createOrderShipment" auth="true">
>>> + <service name="createOrderShipment" engine="entity-auto"
>>> default-entity-name="OrderShipment" invoke="create" auth="true">
>>> <description>Create OrderShipment</description>
>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>> </service>
>>> - <service name="updateOrderShipment"
>>> default-entity-name="OrderShipment" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="updateOrderShipment" auth="true">
>>> + <service name="updateOrderShipment" engine="entity-auto"
>>> default-entity-name="OrderShipment" invoke="update" auth="true">
>>> <description>Update OrderShipment</description>
>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>> </service>
>>> - <service name="deleteOrderShipment"
>>> default-entity-name="OrderShipment" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="deleteOrderShipment" auth="true">
>>> + <service name="deleteOrderShipment" engine="entity-auto"
>>> default-entity-name="OrderShipment" invoke="delete" auth="true">
>>> <description>Delete OrderShipment</description>
>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>> </service>
>>> @@ -667,21 +647,27 @@ under the License.
>>> </service>
>>> <!-- CommunicationEvents related to orders -->
>>> - <service name="createCommunicationEventOrder" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml"
>>> invoke="createCommunicationEventOrder" auth="true">
>>> + <service name="createCommunicationEventOrder"
>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
>>> invoke="create" auth="true">
>>> <description>Create a Communication Event Order</description>
>>> - <auto-attributes entity-name="CommunicationEventOrder"
>>> include="pk" mode="IN" optional="false"/>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>> + </required-permissions>
>>> + <auto-attributes entity-name="CommunicationEventOrder"
>>> include="pk" mode="IN"/>
>>> </service>
>>> - <service name="removeCommunicationEventOrder" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml"
>>> invoke="removeCommunicationEventOrder" auth="true">
>>> + <service name="removeCommunicationEventOrder"
>>> default-entity-name="CommunicationEventOrder" engine="entity-auto"
>>> invoke="delete" auth="true">
>>> <description>Remove a Communication Event Order</description>
>>> - <auto-attributes entity-name="CommunicationEventOrder"
>>> include="pk" mode="IN" optional="false"/>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>> + </required-permissions>
>>> + <auto-attributes entity-name="CommunicationEventOrder"
>>> include="pk" mode="IN"/>
>>> </service>
>>> <!-- Order Shipping and Contacts -->
>>> - <service name="createOrderItemShipGroup"
>>> default-entity-name="OrderItemShipGroup" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="createOrderItemShipGroup" auth="true">
>>> + <service name="createOrderItemShipGroup"
>>> default-entity-name="OrderItemShipGroup" engine="entity-auto"
>>> invoke="create" auth="true">
>>> <description>Creates a new OrderItemShipGroup.</description>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>> + </required-permissions>
>>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>>> <attribute name="orderId" type="String" mode="IN"
>>> optional="false"/>
>>> <attribute name="shipGroupSeqId" type="String"
>>> mode="INOUT" optional="true"/>
>>> @@ -696,42 +682,52 @@ under the License.
>>> <attribute name="contactMechPurposeTypeId" type="String"
>>> mode="IN" optional="true"/>
>>> </service>
>>> - <service name="createOrderContactMech"
>>> default-entity-name="OrderContactMech" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="createOrderContactMech" auth="true">
>>> + <service name="createOrderContactMech"
>>> default-entity-name="OrderContactMech" engine="entity-auto"
>>> invoke="create" auth="true">
>>> <description>Create Order Contact Mech</description>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>> + </required-permissions>
>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>> </service>
>>> - <service name="updateOrderContactMech" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="updateOrderContactMech" auth="true"
>>> default-entity-name="OrderContactMech">
>>> + <service name="updateOrderContactMech"
>>> default-entity-name="OrderContactMech" engine="simple"
>>> +
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="updateOrderContactMech" auth="true">
>>> <description>Update Order Contact Mech</description>
>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>> <attribute name="oldContactMechId" type="String" mode="IN"
>>> optional="true"/>
>>> </service>
>>> - <service name="removeOrderContactMech"
>>> default-entity-name="OrderContactMech" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="removeOrderContactMech" auth="true">
>>> + <service name="removeOrderContactMech"
>>> default-entity-name="OrderContactMech" engine="entity-auto"
>>> invoke="delete" auth="true">
>>> <description>Remove Order Contact Mech</description>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>> + </required-permissions>
>>> <auto-attributes include="pk" mode="IN" optional="false"/>
>>> </service>
>>> <!-- OrderTerms -->
>>> - <service name="createOrderTerm" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="createOrderTerm" auth="true">
>>> + <service name="createOrderTerm" default-entity-name="OrderTerm"
>>> engine="entity-auto" invoke="create" auth="true">
>>> <description>Create an Order Term</description>
>>> - <auto-attributes entity-name="OrderTerm" include="pk"
>>> mode="IN" optional="false"/>
>>> - <auto-attributes entity-name="OrderTerm" include="nonpk"
>>> mode="IN" optional="true"/>
>>> - <override name="orderItemSeqId" optional="true"/>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>>> + </required-permissions>
>>> + <auto-attributes include="pk" mode="IN" optional="false"/>
>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>>> + <override name="orderItemSeqId" optional="true"
>>> default-value="_NA_"/>
>>> </service>
>>> - <service name="updateOrderTerm" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="updateOrderTerm" auth="true">
>>> + <service name="updateOrderTerm" default-entity-name="OrderTerm"
>>> engine="entity-auto" invoke="update" auth="true">
>>> <description>Update an Order Term</description>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="UPDATE"/>
>>> + </required-permissions>
>>> <auto-attributes entity-name="OrderTerm" include="pk"
>>> mode="IN" optional="false"/>
>>> <auto-attributes entity-name="OrderTerm" include="nonpk"
>>> mode="IN" optional="true"/>
>>> </service>
>>> - <service name="removeOrderTerm" engine="simple"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="removeOrderTerm" auth="true">
>>> + <service name="removeOrderTerm" default-entity-name="OrderTerm"
>>> engine="entity-auto" invoke="delete" auth="true">
>>> <description>Remove an Order Term</description>
>>> + <required-permissions join-type="AND">
>>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>>> + </required-permissions>
>>> <auto-attributes entity-name="OrderTerm" include="pk"
>>> mode="IN" optional="false"/>
>>> </service>
>>>
>>> Modified:
>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>> URL:
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_requirement.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>>> ==============================================================================
>>>
>>> ---
>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>> (original)
>>> +++
>>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>>> Sun Jan 24 19:06:32 2016
>>> @@ -105,8 +105,7 @@ under the License.
>>> <attribute mode="OUT" name="quantityTotal"
>>> type="BigDecimal" optional="true"/>
>>> <attribute mode="OUT" name="amountTotal" type="BigDecimal"
>>> optional="true"/>
>>> </service>
>>> - <service name="createOrderRequirementCommitment"
>>> engine="simple" auth="true"
>>> -
>>> location="component://order/script/org/ofbiz/order/order/OrderServices.xml"
>>> invoke="createOrderRequirementCommitment">
>>> + <service name="createOrderRequirementCommitment"
>>> engine="entity-auto" auth="true"
>>> default-entity-name="OrderRequirementCommitment" invoke="create">
>>> <auto-attributes entity-name="OrderRequirementCommitment"
>>> include="pk" mode="IN" optional="false"/>
>>> <auto-attributes entity-name="OrderRequirementCommitment"
>>> include="nonpk" mode="IN" optional="true"/>
>>> </service>
>>>
>>> Modified:
>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>> URL:
>>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=1726542&r1=1726541&r2=1726542&view=diff
>>> ==============================================================================
>>>
>>> ---
>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>> (original)
>>> +++
>>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>> Sun Jan 24 19:06:32 2016
>>> @@ -2824,52 +2824,6 @@ public class OrderServices {
>>> return ServiceUtil.returnSuccess();
>>> }
>>> - /** Service to create an order payment preference */
>>> - public static Map<String, Object>
>>> createPaymentPreference(DispatchContext ctx, Map<String, ? extends
>>> Object> context) {
>>> - Map<String, Object> result = new HashMap<String, Object>();
>>> - Delegator delegator = ctx.getDelegator();
>>> - String orderId = (String) context.get("orderId");
>>> - String statusId = (String) context.get("statusId");
>>> - String paymentMethodTypeId = (String)
>>> context.get("paymentMethodTypeId");
>>> - String paymentMethodId = (String)
>>> context.get("paymentMethodId");
>>> - BigDecimal maxAmount = (BigDecimal) context.get("maxAmount");
>>> - GenericValue userLogin = (GenericValue)
>>> context.get("userLogin");
>>> - Locale locale = (Locale) context.get("locale");
>>> -
>>> - String prefId = null;
>>> -
>>> - try {
>>> - prefId = delegator.getNextSeqId("OrderPaymentPreference");
>>> - } catch (IllegalArgumentException e) {
>>> - return
>>> ServiceUtil.returnError(UtilProperties.getMessage(resource_error,
>>> -
>>> "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFailure",
>>> locale));
>>> - }
>>> -
>>> - Map<String, Object> fields = UtilMisc.<String,
>>> Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId,
>>> "paymentMethodTypeId",
>>> - paymentMethodTypeId, "paymentMethodId",
>>> paymentMethodId, "maxAmount", maxAmount);
>>> -
>>> - if (statusId != null) {
>>> - fields.put("statusId", statusId);
>>> - }
>>> -
>>> - try {
>>> - GenericValue v =
>>> delegator.makeValue("OrderPaymentPreference", fields);
>>> - v.set("createdDate", UtilDateTime.nowTimestamp());
>>> - if (userLogin != null) {
>>> - v.set("createdByUserLogin",
>>> userLogin.getString("userLoginId"));
>>> - }
>>> - delegator.create(v);
>>> - } catch (GenericEntityException e) {
>>> - result.put(ModelService.RESPONSE_MESSAGE,
>>> ModelService.RESPOND_ERROR);
>>> - result.put(ModelService.ERROR_MESSAGE,
>>> UtilProperties.getMessage(resource,
>>> - "OrderOrderPaymentPreferencesCannotBeCreated",
>>> UtilMisc.toMap("errorString", e.getMessage()), locale));
>>> - return ServiceUtil.returnFailure();
>>> - }
>>> - result.put("orderPaymentPreferenceId", prefId);
>>> - result.put(ModelService.RESPONSE_MESSAGE,
>>> ModelService.RESPOND_SUCCESS);
>>> - return result;
>>> - }
>>> -
>>> /** Service to get order header information as standard
>>> results. */
>>> public static Map<String, Object>
>>> getOrderHeaderInformation(DispatchContext dctx, Map<String, ?
>>> extends Object> context) {
>>> Delegator delegator = dctx.getDelegator();
>>>
>>>
>>>
>>
>>
>
>
>
Re: svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Posted by Jacques Le Roux <ja...@les7arts.com>.
Le 11/02/2016 22:27, Jacques Le Roux a écrit :
> Hi Nicolas, All,
>
> I globally like the changes, I just have a question.
>
> With these changes the OrderSecurityError* properties which were used here are now useless, should we not remove them?
> Despite the lost, I think it's reasonable because these errors have very few chances to appear, and if they appear it's not so hard to find your way
> through the log.
Oops, I did not see the elephant in the room: they are permission issues. So I guess the message to the user will be less clear. Not sure it's a real
issue but we might consider it...
Jacques
>
> Jacques
>
>
> Le 24/01/2016 20:06, nmalin@apache.org a écrit :
>> Author: nmalin
>> Date: Sun Jan 24 19:06:32 2016
>> New Revision: 1726542
>>
>> URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
>> Log:
>>
>>
>> I converted following services from simple to entity-auto :
>>
>> createOrderNotificationLog
>> createOrderItemBilling
>> createOrderAdjustment
>> updateOrderAdjustment
>> createOrderAdjustmentBilling
>> createOrderShipment
>> updateOrderShipment
>> deleteOrderShipment
>> createCommunicationEventOrder
>> removeCommunicationEventOrder
>> createOrderItemShipGroup
>> createOrderContactMech
>> removeOrderContactMech
>> createOrderTerm
>> removeOrderTerm
>> createOrderRequirementCommitment
>>
>> And from java to entity-auto for createOrderPaymentPreference
>> Related issue OFBIZ-6854.
>>
>> Removed:
>> ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
>> Modified:
>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>> ofbiz/trunk/applications/order/servicedef/services.xml
>> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>>
>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
>> URL:
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml (original)
>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml Sun Jan 24 19:06:32 2016
>> @@ -88,56 +88,8 @@ under the License.
>> <field-to-result field="orderInfo.totalOrders" result-name="totalOrders"/>
>> </if-not-empty>
>> </simple-method>
>> - <simple-method method-name="createOrderShipment" short-description="Create OrderShipment">
>> - <set value="Create OrderShipment" field="operationName"/>
>> - <check-permission permission="ORDERMGR" action="_CREATE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderShipment"/>
>> - </check-permission>
>> - <check-errors/>
>> -
>> - <make-value value-field="newEntity" entity-name="OrderShipment"/>
>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>> -
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> - <simple-method method-name="updateOrderShipment" short-description="Update OrderShipment">
>> - <set value="Update OrderShipment" field="operationName"/>
>> - <check-permission permission="ORDERMGR" action="_UPDATE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>> - </check-permission>
>> - <check-errors/>
>> -
>> - <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
>> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
>> - <store-value value-field="lookedUpValue"/>
>> - </simple-method>
>> - <simple-method method-name="deleteOrderShipment" short-description="Delete OrderShipment">
>> - <set value="Delete OrderShipment" field="operationName"/>
>> - <check-permission permission="ORDERMGR" action="_DELETE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
>> - </check-permission>
>> - <check-errors/>
>> -
>> - <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
>> - <remove-value value-field="lookedUpValue"/>
>> - </simple-method>
>> <!-- order requirement methods -->
>> - <simple-method method-name="createOrderRequirementCommitment" short-description="Create OrderRequirementCommitment">
>> - <!-- The permission checking is commented out to make this service work also when triggered from ecommerce orders -->
>> - <!--
>> - <check-permission permission="ORDERMGR" action="_CREATE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCheckCreateOrderRequirementCommitment"/>
>> - </check-permission>
>> - <check-errors/>
>> - -->
>> - <make-value value-field="newEntity" entity-name="OrderRequirementCommitment"/>
>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> -
>> <simple-method method-name="createRequirementAndCommitment" short-description="create a requirement and commitment for it">
>> <set value="PRODUCT_REQUIREMENT" field="inputMap.requirementTypeId"/>
>> @@ -759,20 +711,6 @@ under the License.
>> </simple-method>
>> <!-- OrderItemShipGroup -->
>> - <simple-method method-name="createOrderItemShipGroup" short-description="Create OrderItemShipGroup">
>> - <check-permission permission="ORDERMGR" action="_CREATE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
>> - </check-permission>
>> - <check-errors/>
>> - <make-value value-field="newEntity" entity-name="OrderItemShipGroup"/>
>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>> - <if-empty field="newEntity.shipGroupSeqId">
>> - <make-next-seq-id value-field="newEntity" seq-field-name="shipGroupSeqId"/> <!-- this finds the next sub-sequence ID -->
>> - <field-to-result field="newEntity.shipGroupSeqId" result-name="shipGroupSeqId"/>
>> - </if-empty>
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> <simple-method method-name="updateOrderItemShipGroup" short-description="Update OrderItemShipGroup">
>> <check-permission permission="ORDERMGR" action="_UPDATE">
>> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
>> @@ -832,16 +770,6 @@ under the License.
>> <field-to-result field="orderItemShipGroupInvRes.promisedDatetime" result-name="estimatedShipDate"/>
>> </simple-method>
>> - <simple-method method-name="createOrderContactMech" short-description="Create OrderContactMech">
>> - <check-permission permission="ORDERMGR" action="_CREATE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderContactMech"/>
>> - </check-permission>
>> - <check-errors/>
>> - <make-value value-field="newEntity" entity-name="OrderContactMech"/>
>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> -
>> <simple-method method-name="removeOrderContactMech" short-description="Remove OrderContactMech">
>> <check-permission permission="ORDERMGR" action="_DELETE">
>> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
>> @@ -861,39 +789,6 @@ under the License.
>> <store-value value-field="orderHeaderNote"/>
>> </simple-method>
>> - <simple-method method-name="createOrderTerm" short-description="Create an OrderTerm">
>> - <check-permission permission="ORDERMGR" action="_CREATE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderTerm"/>
>> - </check-permission>
>> - <check-errors/>
>> -
>> - <make-value value-field="newEntity" entity-name="OrderTerm"/>
>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>> - <set value="_NA_" set-if-empty="true" field="newEntity.orderItemSeqId"/>
>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>> -
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> - <simple-method method-name="updateOrderTerm" short-description="Update OrderTerm">
>> - <check-permission permission="ORDERMGR" action="_DELETE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderTerm"/>
>> - </check-permission>
>> - <check-errors/>
>> -
>> - <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
>> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
>> - <store-value value-field="lookedUpValue"/>
>> - </simple-method>
>> - <simple-method method-name="removeOrderTerm" short-description="Remove OrderTerm">
>> - <check-permission permission="ORDERMGR" action="_DELETE">
>> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderTerm"/>
>> - </check-permission>
>> - <check-errors/>
>> -
>> - <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
>> - <remove-value value-field="lookedUpValue"/>
>> - </simple-method>
>> -
>> <simple-method method-name="addPaymentMethodToOrder" short-description="Create an PaymentMethodToOrder">
>> <check-permission permission="ORDERMGR" action="_CREATE">
>> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
>> @@ -1373,7 +1268,7 @@ under the License.
>> </iterate>
>> </iterate>
>> </simple-method>
>> -
>> +
>> <simple-method method-name="updateShippingMethodAndCharges" short-description="Updates shipping method and shipping charges from Order View
>> page when Shipment is in picked status and items of Order are packed">
>> <!-- splitting shipmentMethodAndAmount request parameter value that contains "*" symbol
>> into "shipmentMethod" and "newAmount".
>> @@ -1424,6 +1319,7 @@ under the License.
>> </else>
>> </if-compare-field>
>> </simple-method>
>> +
>> <simple-method method-name="productAvailabalityByFacility" short-description="Calculate ATP and Qoh According For each facility">
>> <set field="facilityMap.ownerPartyId" from-field="parameters.ownerPartyId"/>
>> <find-by-and map="facilityMap" list="facilityList" entity-name="Facility"/>
>> @@ -1442,7 +1338,7 @@ under the License.
>> </iterate>
>> <field-to-result field="availabalityList"/>
>> </simple-method>
>> -
>> +
>> <simple-method method-name="createOrderPaymentApplication" short-description="Create Order Payment Application">
>> <entity-one value-field="paymentMap" entity-name="Payment"/>
>> <set field="createCtx.amountApplied" from-field="paymentMap.amount" type="BigDecimal"/>
>> @@ -1459,7 +1355,7 @@ under the License.
>> <call-service service-name="createPaymentApplication" in-map-name="createCtx"/>
>> </if-not-empty>
>> </simple-method>
>> -
>> +
>> <simple-method method-name="MoveItemBetweenShipGroups" short-description="Move order items between ship groups">
>> <entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
>> <field-map field-name="orderId" from-field="parameters.orderId"/>
>> @@ -1483,7 +1379,7 @@ under the License.
>> <set field="map.shipGroupSeqId" from-field="parameters.toGroupIndex"/>
>> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity + parameters.quantity}" type="BigDecimal"/>
>> <call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
>> -
>> +
>> <entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
>> <field-map field-name="orderId" from-field="parameters.orderId"/>
>> <field-map field-name="orderItemSeqId" from-field="parameters.orderItemSeqId"/>
>> @@ -1495,7 +1391,7 @@ under the License.
>> </add-error>
>> </if-empty>
>> <check-errors/>
>> -
>> +
>> <clear-field field="map"/>
>> <set field="map.orderId" from-field="parameters.orderId"/>
>> <set field="map.orderItemSeqId" from-field="parameters.orderItemSeqId"/>
>> @@ -1503,5 +1399,5 @@ under the License.
>> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity - parameters.quantity}" type="BigDecimal"/>
>> <call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
>> </simple-method>
>> -
>> +
>> </simple-methods>
>>
>> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
>> URL:
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml (original)
>> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml Sun Jan 24 19:06:32 2016
>> @@ -44,33 +44,12 @@ under the License.
>> </else>
>> </if-compare>
>> </simple-method>
>> -
>> - <simple-method method-name="createOrderAdjustment" short-description="Create an OrderAdjustment">
>> -
>> - <make-value entity-name="OrderAdjustment" value-field="newEntity"/>
>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>> -
>> - <sequenced-id sequence-name="OrderAdjustment" field="newEntity.orderAdjustmentId"/>
>> - <field-to-result field="newEntity.orderAdjustmentId" result-name="orderAdjustmentId"/>
>> -
>> - <now-timestamp field="newEntity.createdDate"/>
>> - <set from-field="userLogin.userLoginId" field="newEntity.createdByUserLogin"/>
>> -
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> - <simple-method method-name="updateOrderAdjustment" short-description="Update an OrderAdjustment">
>> - <check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>> property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></check-permission>
>> - <check-errors/>
>> - <entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
>> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
>> - <store-value value-field="lookedUpValue"/>
>> - </simple-method>
>> <simple-method method-name="deleteOrderAdjustment" short-description="Delete an OrderAdjustment">
>> <check-permission permission="ORDERMGR" action="_DELETE"><fail-property resource="OrderErrorUiLabels"
>> property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></check-permission>
>> <check-errors/>
>> <entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
>> <remove-value value-field="lookedUpValue"/>
>> -
>> +
>> <if-not-empty field="parameters.productPromoCodeId">
>> <entity-one entity-name="OrderProductPromoCode" value-field="lookedUpValue"/>
>> <if-not-empty field="lookedUpValue">
>> @@ -79,30 +58,6 @@ under the License.
>> </if-not-empty>
>> </simple-method>
>> - <simple-method method-name="createOrderAdjustmentBilling" short-description="Create an OrderAdjustmentBilling">
>> - <make-value entity-name="OrderAdjustmentBilling" value-field="newEntity"/>
>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> -
>> - <simple-method method-name="createOrderItemBilling" short-description="Create an OrderItemBilling">
>> - <make-value entity-name="OrderItemBilling" value-field="newEntity"/>
>> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
>> - <set-pk-fields map="parameters" value-field="newEntity"/>
>> - <create-value value-field="newEntity"/>
>> - </simple-method>
>> -
>> - <simple-method method-name="createNotificationLog" short-description="Log an order notification" login-required="false">
>> - <make-value entity-name="OrderNotification" value-field="orderNotification"/>
>> - <sequenced-id sequence-name="OrderNotification" field="orderNotification.orderNotificationId"/>
>> - <set from-field="parameters.orderId" field="orderNotification.orderId"/>
>> - <set from-field="parameters.emailType" field="orderNotification.emailType"/>
>> - <set from-field="parameters.comments" field="orderNotification.comments"/>
>> - <now-timestamp field="orderNotification.notificationDate"/>
>> - <create-value value-field="orderNotification"/>
>> - </simple-method>
>> -
>> <simple-method method-name="updateOrderStatusFromReceipt" short-description="Update Order Status From ShipmentReceipt">
>> <check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels"
>> property="OrderSecurityErrorToRunUpdateOrderStatusFromReceipt"/></check-permission>
>> <check-errors/>
>>
>> Modified: ofbiz/trunk/applications/order/servicedef/services.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/order/servicedef/services.xml (original)
>> +++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan 24 19:06:32 2016
>> @@ -76,8 +76,7 @@ under the License.
>> <attribute name="assignedRoleTypeId" type="String" mode="IN" optional="true"/>
>> </service>
>> - <service name="createOrderNotificationLog" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createNotificationLog">
>> + <service name="createOrderNotificationLog" engine="entity-auto" default-entity-name="OrderNotification" invoke="create">
>> <description>Logs when a notification was sent</description>
>> <attribute name="orderId" type="String" mode="IN" optional="false"/>
>> <attribute name="emailType" type="String" mode="IN" optional="false"/>
>> @@ -164,17 +163,10 @@ under the License.
>> <attribute name="orderId" type="String" mode="OUT" optional="false"/>
>> </service>
>> - <service name="createOrderItemBilling" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderItemBilling">
>> + <service name="createOrderItemBilling" default-entity-name="OrderItemBilling" engine="entity-auto" invoke="create">
>> <description>Create a new order item billing record</description>
>> - <attribute name="orderId" type="String" mode="IN" optional="false"/>
>> - <attribute name="orderItemSeqId" type="String" mode="IN" optional="false"/>
>> - <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
>> - <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
>> - <attribute name="quantity" type="BigDecimal" mode="IN" optional="true"/>
>> - <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
>> - <attribute name="itemIssuanceId" type="String" mode="IN" optional="true"/>
>> - <attribute name="shipmentReceiptId" type="String" mode="IN" optional="true"/>
>> + <auto-attributes mode="IN" include="pk"/>
>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>> </service>
>> <service name="orderAdjustmentPermissionCheck" engine="simple"
>> @@ -183,8 +175,7 @@ under the License.
>> <implements service="permissionInterface"/>
>> </service>
>> - <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustment">
>> + <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="create">
>> <description>Creates a new order adjustment record</description>
>> <permission-service service-name="orderAdjustmentPermissionCheck" main-action="CREATE"/>
>> <auto-attributes mode="OUT" include="pk" optional="false"/>
>> @@ -192,9 +183,9 @@ under the License.
>> <override name="orderAdjustmentTypeId" optional="false"/>
>> <override name="orderId" optional="false"/>
>> </service>
>> - <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="updateOrderAdjustment">
>> + <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="update">
>> <description>Update an order adjustment record</description>
>> + <permission-service service-name="orderAdjustmentPermissionCheck" main-action="UPDATE"/>
>> <auto-attributes mode="IN" include="pk" optional="false"/>
>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>> <override name="orderId" optional="false"/>
>> @@ -207,13 +198,10 @@ under the License.
>> <attribute name="productPromoCodeId" type="String" mode="IN" optional="true"/>
>> </service>
>> - <service name="createOrderAdjustmentBilling" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustmentBilling">
>> + <service name="createOrderAdjustmentBilling" default-entity-name="OrderAdjustmentBilling" engine="entity-auto" invoke="create">
>> <description>Create a new order adjustment billing record</description>
>> - <attribute name="orderAdjustmentId" type="String" mode="IN" optional="false"/>
>> - <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
>> - <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
>> - <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
>> + <auto-attributes mode="IN" include="pk"/>
>> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
>> </service>
>> <service name="createPaymentFromPreference" engine="java"
>> @@ -435,15 +423,10 @@ under the License.
>> <attribute name="roleTypeId" type="String" mode="IN"/>
>> </service>
>> - <service name="createOrderPaymentPreference" engine="java"
>> - location="org.ofbiz.order.order.OrderServices" invoke="createPaymentPreference">
>> + <service name="createOrderPaymentPreference" default-entity-name="OrderPaymentPreference" engine="entity-auto" invoke="create">
>> <description>Creates an order payment preference</description>
>> - <attribute name="orderId" type="String" mode="IN"/>
>> - <attribute name="paymentMethodTypeId" type="String" mode="IN"/>
>> - <attribute name="paymentMethodId" type="String" mode="IN" optional="true"/>
>> - <attribute name="statusId" type="String" mode="IN" optional="true"/>
>> - <attribute name="maxAmount" type="BigDecimal" mode="IN" optional="true"/>
>> - <attribute name="orderPaymentPreferenceId" type="String" mode="OUT"/>
>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>> + <auto-attributes include="pk" mode="OUT"/>
>> </service>
>> <service name="createOrderNote" engine="java"
>> @@ -568,20 +551,17 @@ under the License.
>> </service>
>> <!-- OrderShipment Services -->
>> - <service name="createOrderShipment" default-entity-name="OrderShipment" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderShipment" auth="true">
>> + <service name="createOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="create" auth="true">
>> <description>Create OrderShipment</description>
>> <auto-attributes include="pk" mode="IN" optional="false"/>
>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>> </service>
>> - <service name="updateOrderShipment" default-entity-name="OrderShipment" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderShipment" auth="true">
>> + <service name="updateOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="update" auth="true">
>> <description>Update OrderShipment</description>
>> <auto-attributes include="pk" mode="IN" optional="false"/>
>> <auto-attributes include="nonpk" mode="IN" optional="true"/>
>> </service>
>> - <service name="deleteOrderShipment" default-entity-name="OrderShipment" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="deleteOrderShipment" auth="true">
>> + <service name="deleteOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="delete" auth="true">
>> <description>Delete OrderShipment</description>
>> <auto-attributes include="pk" mode="IN" optional="false"/>
>> </service>
>> @@ -667,21 +647,27 @@ under the License.
>> </service>
>> <!-- CommunicationEvents related to orders -->
>> - <service name="createCommunicationEventOrder" engine="simple"
>> - location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="createCommunicationEventOrder" auth="true">
>> + <service name="createCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="create" auth="true">
>> <description>Create a Communication Event Order</description>
>> - <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>> + </required-permissions>
>> + <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
>> </service>
>> - <service name="removeCommunicationEventOrder" engine="simple"
>> - location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="removeCommunicationEventOrder" auth="true">
>> + <service name="removeCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="delete" auth="true">
>> <description>Remove a Communication Event Order</description>
>> - <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>> + </required-permissions>
>> + <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
>> </service>
>> <!-- Order Shipping and Contacts -->
>> - <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderItemShipGroup" auth="true">
>> + <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="entity-auto" invoke="create" auth="true">
>> <description>Creates a new OrderItemShipGroup.</description>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>> + </required-permissions>
>> <auto-attributes mode="IN" include="nonpk" optional="true"/>
>> <attribute name="orderId" type="String" mode="IN" optional="false"/>
>> <attribute name="shipGroupSeqId" type="String" mode="INOUT" optional="true"/>
>> @@ -696,42 +682,52 @@ under the License.
>> <attribute name="contactMechPurposeTypeId" type="String" mode="IN" optional="true"/>
>> </service>
>> - <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderContactMech" auth="true">
>> + <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="create" auth="true">
>> <description>Create Order Contact Mech</description>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>> + </required-permissions>
>> <auto-attributes include="pk" mode="IN" optional="false"/>
>> </service>
>> - <service name="updateOrderContactMech" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true"
>> default-entity-name="OrderContactMech">
>> + <service name="updateOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
>> + location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true">
>> <description>Update Order Contact Mech</description>
>> <auto-attributes include="pk" mode="IN" optional="false"/>
>> <attribute name="oldContactMechId" type="String" mode="IN" optional="true"/>
>> </service>
>> - <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderContactMech" auth="true">
>> + <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="delete" auth="true">
>> <description>Remove Order Contact Mech</description>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>> + </required-permissions>
>> <auto-attributes include="pk" mode="IN" optional="false"/>
>> </service>
>> <!-- OrderTerms -->
>> - <service name="createOrderTerm" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderTerm" auth="true">
>> + <service name="createOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="create" auth="true">
>> <description>Create an Order Term</description>
>> - <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
>> - <auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
>> - <override name="orderItemSeqId" optional="true"/>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="CREATE"/>
>> + </required-permissions>
>> + <auto-attributes include="pk" mode="IN" optional="false"/>
>> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
>> + <override name="orderItemSeqId" optional="true" default-value="_NA_"/>
>> </service>
>> - <service name="updateOrderTerm" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderTerm" auth="true">
>> + <service name="updateOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="update" auth="true">
>> <description>Update an Order Term</description>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="UPDATE"/>
>> + </required-permissions>
>> <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
>> <auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
>> </service>
>> - <service name="removeOrderTerm" engine="simple"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderTerm" auth="true">
>> + <service name="removeOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="delete" auth="true">
>> <description>Remove an Order Term</description>
>> + <required-permissions join-type="AND">
>> + <check-permission permission="ORDERMGR" action="DELETE"/>
>> + </required-permissions>
>> <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
>> </service>
>>
>> Modified: ofbiz/trunk/applications/order/servicedef/services_requirement.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_requirement.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/order/servicedef/services_requirement.xml (original)
>> +++ ofbiz/trunk/applications/order/servicedef/services_requirement.xml Sun Jan 24 19:06:32 2016
>> @@ -105,8 +105,7 @@ under the License.
>> <attribute mode="OUT" name="quantityTotal" type="BigDecimal" optional="true"/>
>> <attribute mode="OUT" name="amountTotal" type="BigDecimal" optional="true"/>
>> </service>
>> - <service name="createOrderRequirementCommitment" engine="simple" auth="true"
>> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderRequirementCommitment">
>> + <service name="createOrderRequirementCommitment" engine="entity-auto" auth="true" default-entity-name="OrderRequirementCommitment"
>> invoke="create">
>> <auto-attributes entity-name="OrderRequirementCommitment" include="pk" mode="IN" optional="false"/>
>> <auto-attributes entity-name="OrderRequirementCommitment" include="nonpk" mode="IN" optional="true"/>
>> </service>
>>
>> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>> URL:
>> http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=1726542&r1=1726541&r2=1726542&view=diff
>> ==============================================================================
>> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java (original)
>> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java Sun Jan 24 19:06:32 2016
>> @@ -2824,52 +2824,6 @@ public class OrderServices {
>> return ServiceUtil.returnSuccess();
>> }
>> - /** Service to create an order payment preference */
>> - public static Map<String, Object> createPaymentPreference(DispatchContext ctx, Map<String, ? extends Object> context) {
>> - Map<String, Object> result = new HashMap<String, Object>();
>> - Delegator delegator = ctx.getDelegator();
>> - String orderId = (String) context.get("orderId");
>> - String statusId = (String) context.get("statusId");
>> - String paymentMethodTypeId = (String) context.get("paymentMethodTypeId");
>> - String paymentMethodId = (String) context.get("paymentMethodId");
>> - BigDecimal maxAmount = (BigDecimal) context.get("maxAmount");
>> - GenericValue userLogin = (GenericValue) context.get("userLogin");
>> - Locale locale = (Locale) context.get("locale");
>> -
>> - String prefId = null;
>> -
>> - try {
>> - prefId = delegator.getNextSeqId("OrderPaymentPreference");
>> - } catch (IllegalArgumentException e) {
>> - return ServiceUtil.returnError(UtilProperties.getMessage(resource_error,
>> - "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFailure", locale));
>> - }
>> -
>> - Map<String, Object> fields = UtilMisc.<String, Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId, "paymentMethodTypeId",
>> - paymentMethodTypeId, "paymentMethodId", paymentMethodId, "maxAmount", maxAmount);
>> -
>> - if (statusId != null) {
>> - fields.put("statusId", statusId);
>> - }
>> -
>> - try {
>> - GenericValue v = delegator.makeValue("OrderPaymentPreference", fields);
>> - v.set("createdDate", UtilDateTime.nowTimestamp());
>> - if (userLogin != null) {
>> - v.set("createdByUserLogin", userLogin.getString("userLoginId"));
>> - }
>> - delegator.create(v);
>> - } catch (GenericEntityException e) {
>> - result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_ERROR);
>> - result.put(ModelService.ERROR_MESSAGE, UtilProperties.getMessage(resource,
>> - "OrderOrderPaymentPreferencesCannotBeCreated", UtilMisc.toMap("errorString", e.getMessage()), locale));
>> - return ServiceUtil.returnFailure();
>> - }
>> - result.put("orderPaymentPreferenceId", prefId);
>> - result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_SUCCESS);
>> - return result;
>> - }
>> -
>> /** Service to get order header information as standard results. */
>> public static Map<String, Object> getOrderHeaderInformation(DispatchContext dctx, Map<String, ? extends Object> context) {
>> Delegator delegator = dctx.getDelegator();
>>
>>
>>
>
>
Re: svn commit: r1726542 - in /ofbiz/trunk/applications/order:
script/org/ofbiz/order/communication/ script/org/ofbiz/order/order/
servicedef/ src/org/ofbiz/order/order/
Posted by Jacques Le Roux <ja...@les7arts.com>.
Hi Nicolas, All,
I globally like the changes, I just have a question.
With these changes the OrderSecurityError* properties which were used here are now useless, should we not remove them?
Despite the lost, I think it's reasonable because these errors have very few chances to appear, and if they appear it's not so hard to find your way
through the log.
Jacques
Le 24/01/2016 20:06, nmalin@apache.org a écrit :
> Author: nmalin
> Date: Sun Jan 24 19:06:32 2016
> New Revision: 1726542
>
> URL: http://svn.apache.org/viewvc?rev=1726542&view=rev
> Log:
>
>
> I converted following services from simple to entity-auto :
>
> createOrderNotificationLog
> createOrderItemBilling
> createOrderAdjustment
> updateOrderAdjustment
> createOrderAdjustmentBilling
> createOrderShipment
> updateOrderShipment
> deleteOrderShipment
> createCommunicationEventOrder
> removeCommunicationEventOrder
> createOrderItemShipGroup
> createOrderContactMech
> removeOrderContactMech
> createOrderTerm
> removeOrderTerm
> createOrderRequirementCommitment
>
> And from java to entity-auto for createOrderPaymentPreference
> Related issue OFBIZ-6854.
>
> Removed:
> ofbiz/trunk/applications/order/script/org/ofbiz/order/communication/
> Modified:
> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
> ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
> ofbiz/trunk/applications/order/servicedef/services.xml
> ofbiz/trunk/applications/order/servicedef/services_requirement.xml
> ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
>
> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml (original)
> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderServices.xml Sun Jan 24 19:06:32 2016
> @@ -88,56 +88,8 @@ under the License.
> <field-to-result field="orderInfo.totalOrders" result-name="totalOrders"/>
> </if-not-empty>
> </simple-method>
> - <simple-method method-name="createOrderShipment" short-description="Create OrderShipment">
> - <set value="Create OrderShipment" field="operationName"/>
> - <check-permission permission="ORDERMGR" action="_CREATE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderShipment"/>
> - </check-permission>
> - <check-errors/>
> -
> - <make-value value-field="newEntity" entity-name="OrderShipment"/>
> - <set-pk-fields map="parameters" value-field="newEntity"/>
> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> -
> - <create-value value-field="newEntity"/>
> - </simple-method>
> - <simple-method method-name="updateOrderShipment" short-description="Update OrderShipment">
> - <set value="Update OrderShipment" field="operationName"/>
> - <check-permission permission="ORDERMGR" action="_UPDATE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
> - </check-permission>
> - <check-errors/>
> -
> - <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
> - <store-value value-field="lookedUpValue"/>
> - </simple-method>
> - <simple-method method-name="deleteOrderShipment" short-description="Delete OrderShipment">
> - <set value="Delete OrderShipment" field="operationName"/>
> - <check-permission permission="ORDERMGR" action="_DELETE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderShipment"/>
> - </check-permission>
> - <check-errors/>
> -
> - <entity-one entity-name="OrderShipment" value-field="lookedUpValue"/>
> - <remove-value value-field="lookedUpValue"/>
> - </simple-method>
>
> <!-- order requirement methods -->
> - <simple-method method-name="createOrderRequirementCommitment" short-description="Create OrderRequirementCommitment">
> - <!-- The permission checking is commented out to make this service work also when triggered from ecommerce orders -->
> - <!--
> - <check-permission permission="ORDERMGR" action="_CREATE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCheckCreateOrderRequirementCommitment"/>
> - </check-permission>
> - <check-errors/>
> - -->
> - <make-value value-field="newEntity" entity-name="OrderRequirementCommitment"/>
> - <set-pk-fields map="parameters" value-field="newEntity"/>
> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> - <create-value value-field="newEntity"/>
> - </simple-method>
> -
> <simple-method method-name="createRequirementAndCommitment" short-description="create a requirement and commitment for it">
> <set value="PRODUCT_REQUIREMENT" field="inputMap.requirementTypeId"/>
>
> @@ -759,20 +711,6 @@ under the License.
> </simple-method>
>
> <!-- OrderItemShipGroup -->
> - <simple-method method-name="createOrderItemShipGroup" short-description="Create OrderItemShipGroup">
> - <check-permission permission="ORDERMGR" action="_CREATE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderItemShipGroup"/>
> - </check-permission>
> - <check-errors/>
> - <make-value value-field="newEntity" entity-name="OrderItemShipGroup"/>
> - <set-pk-fields map="parameters" value-field="newEntity"/>
> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> - <if-empty field="newEntity.shipGroupSeqId">
> - <make-next-seq-id value-field="newEntity" seq-field-name="shipGroupSeqId"/> <!-- this finds the next sub-sequence ID -->
> - <field-to-result field="newEntity.shipGroupSeqId" result-name="shipGroupSeqId"/>
> - </if-empty>
> - <create-value value-field="newEntity"/>
> - </simple-method>
> <simple-method method-name="updateOrderItemShipGroup" short-description="Update OrderItemShipGroup">
> <check-permission permission="ORDERMGR" action="_UPDATE">
> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderItemShipGroup"/>
> @@ -832,16 +770,6 @@ under the License.
> <field-to-result field="orderItemShipGroupInvRes.promisedDatetime" result-name="estimatedShipDate"/>
> </simple-method>
>
> - <simple-method method-name="createOrderContactMech" short-description="Create OrderContactMech">
> - <check-permission permission="ORDERMGR" action="_CREATE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderContactMech"/>
> - </check-permission>
> - <check-errors/>
> - <make-value value-field="newEntity" entity-name="OrderContactMech"/>
> - <set-pk-fields map="parameters" value-field="newEntity"/>
> - <create-value value-field="newEntity"/>
> - </simple-method>
> -
> <simple-method method-name="removeOrderContactMech" short-description="Remove OrderContactMech">
> <check-permission permission="ORDERMGR" action="_DELETE">
> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderContactMech"/>
> @@ -861,39 +789,6 @@ under the License.
> <store-value value-field="orderHeaderNote"/>
> </simple-method>
>
> - <simple-method method-name="createOrderTerm" short-description="Create an OrderTerm">
> - <check-permission permission="ORDERMGR" action="_CREATE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunCreateOrderTerm"/>
> - </check-permission>
> - <check-errors/>
> -
> - <make-value value-field="newEntity" entity-name="OrderTerm"/>
> - <set-pk-fields map="parameters" value-field="newEntity"/>
> - <set value="_NA_" set-if-empty="true" field="newEntity.orderItemSeqId"/>
> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> -
> - <create-value value-field="newEntity"/>
> - </simple-method>
> - <simple-method method-name="updateOrderTerm" short-description="Update OrderTerm">
> - <check-permission permission="ORDERMGR" action="_DELETE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderTerm"/>
> - </check-permission>
> - <check-errors/>
> -
> - <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
> - <store-value value-field="lookedUpValue"/>
> - </simple-method>
> - <simple-method method-name="removeOrderTerm" short-description="Remove OrderTerm">
> - <check-permission permission="ORDERMGR" action="_DELETE">
> - <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunRemoveOrderTerm"/>
> - </check-permission>
> - <check-errors/>
> -
> - <entity-one entity-name="OrderTerm" value-field="lookedUpValue"/>
> - <remove-value value-field="lookedUpValue"/>
> - </simple-method>
> -
> <simple-method method-name="addPaymentMethodToOrder" short-description="Create an PaymentMethodToOrder">
> <check-permission permission="ORDERMGR" action="_CREATE">
> <fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunAddPaymentMethodToOrder"/>
> @@ -1373,7 +1268,7 @@ under the License.
> </iterate>
> </iterate>
> </simple-method>
> -
> +
> <simple-method method-name="updateShippingMethodAndCharges" short-description="Updates shipping method and shipping charges from Order View page when Shipment is in picked status and items of Order are packed">
> <!-- splitting shipmentMethodAndAmount request parameter value that contains "*" symbol
> into "shipmentMethod" and "newAmount".
> @@ -1424,6 +1319,7 @@ under the License.
> </else>
> </if-compare-field>
> </simple-method>
> +
> <simple-method method-name="productAvailabalityByFacility" short-description="Calculate ATP and Qoh According For each facility">
> <set field="facilityMap.ownerPartyId" from-field="parameters.ownerPartyId"/>
> <find-by-and map="facilityMap" list="facilityList" entity-name="Facility"/>
> @@ -1442,7 +1338,7 @@ under the License.
> </iterate>
> <field-to-result field="availabalityList"/>
> </simple-method>
> -
> +
> <simple-method method-name="createOrderPaymentApplication" short-description="Create Order Payment Application">
> <entity-one value-field="paymentMap" entity-name="Payment"/>
> <set field="createCtx.amountApplied" from-field="paymentMap.amount" type="BigDecimal"/>
> @@ -1459,7 +1355,7 @@ under the License.
> <call-service service-name="createPaymentApplication" in-map-name="createCtx"/>
> </if-not-empty>
> </simple-method>
> -
> +
> <simple-method method-name="MoveItemBetweenShipGroups" short-description="Move order items between ship groups">
> <entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
> <field-map field-name="orderId" from-field="parameters.orderId"/>
> @@ -1483,7 +1379,7 @@ under the License.
> <set field="map.shipGroupSeqId" from-field="parameters.toGroupIndex"/>
> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity + parameters.quantity}" type="BigDecimal"/>
> <call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
> -
> +
> <entity-one entity-name="OrderItemShipGroupAssoc" value-field="orderItemShipGroupAssoc">
> <field-map field-name="orderId" from-field="parameters.orderId"/>
> <field-map field-name="orderItemSeqId" from-field="parameters.orderItemSeqId"/>
> @@ -1495,7 +1391,7 @@ under the License.
> </add-error>
> </if-empty>
> <check-errors/>
> -
> +
> <clear-field field="map"/>
> <set field="map.orderId" from-field="parameters.orderId"/>
> <set field="map.orderItemSeqId" from-field="parameters.orderItemSeqId"/>
> @@ -1503,5 +1399,5 @@ under the License.
> <set field="map.quantity" value="${orderItemShipGroupAssoc.quantity - parameters.quantity}" type="BigDecimal"/>
> <call-service service-name="updateOrderItemShipGroupAssoc" in-map-name="map"/>
> </simple-method>
> -
> +
> </simple-methods>
>
> Modified: ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml (original)
> +++ ofbiz/trunk/applications/order/script/org/ofbiz/order/order/OrderSimpleMethods.xml Sun Jan 24 19:06:32 2016
> @@ -44,33 +44,12 @@ under the License.
> </else>
> </if-compare>
> </simple-method>
> -
> - <simple-method method-name="createOrderAdjustment" short-description="Create an OrderAdjustment">
> -
> - <make-value entity-name="OrderAdjustment" value-field="newEntity"/>
> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> -
> - <sequenced-id sequence-name="OrderAdjustment" field="newEntity.orderAdjustmentId"/>
> - <field-to-result field="newEntity.orderAdjustmentId" result-name="orderAdjustmentId"/>
> -
> - <now-timestamp field="newEntity.createdDate"/>
> - <set from-field="userLogin.userLoginId" field="newEntity.createdByUserLogin"/>
> -
> - <create-value value-field="newEntity"/>
> - </simple-method>
> - <simple-method method-name="updateOrderAdjustment" short-description="Update an OrderAdjustment">
> - <check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderAdjustement"/></check-permission>
> - <check-errors/>
> - <entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
> - <set-nonpk-fields map="parameters" value-field="lookedUpValue"/>
> - <store-value value-field="lookedUpValue"/>
> - </simple-method>
> <simple-method method-name="deleteOrderAdjustment" short-description="Delete an OrderAdjustment">
> <check-permission permission="ORDERMGR" action="_DELETE"><fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunDeleteOrderAdjustement"/></check-permission>
> <check-errors/>
> <entity-one entity-name="OrderAdjustment" value-field="lookedUpValue"/>
> <remove-value value-field="lookedUpValue"/>
> -
> +
> <if-not-empty field="parameters.productPromoCodeId">
> <entity-one entity-name="OrderProductPromoCode" value-field="lookedUpValue"/>
> <if-not-empty field="lookedUpValue">
> @@ -79,30 +58,6 @@ under the License.
> </if-not-empty>
> </simple-method>
>
> - <simple-method method-name="createOrderAdjustmentBilling" short-description="Create an OrderAdjustmentBilling">
> - <make-value entity-name="OrderAdjustmentBilling" value-field="newEntity"/>
> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> - <set-pk-fields map="parameters" value-field="newEntity"/>
> - <create-value value-field="newEntity"/>
> - </simple-method>
> -
> - <simple-method method-name="createOrderItemBilling" short-description="Create an OrderItemBilling">
> - <make-value entity-name="OrderItemBilling" value-field="newEntity"/>
> - <set-nonpk-fields map="parameters" value-field="newEntity"/>
> - <set-pk-fields map="parameters" value-field="newEntity"/>
> - <create-value value-field="newEntity"/>
> - </simple-method>
> -
> - <simple-method method-name="createNotificationLog" short-description="Log an order notification" login-required="false">
> - <make-value entity-name="OrderNotification" value-field="orderNotification"/>
> - <sequenced-id sequence-name="OrderNotification" field="orderNotification.orderNotificationId"/>
> - <set from-field="parameters.orderId" field="orderNotification.orderId"/>
> - <set from-field="parameters.emailType" field="orderNotification.emailType"/>
> - <set from-field="parameters.comments" field="orderNotification.comments"/>
> - <now-timestamp field="orderNotification.notificationDate"/>
> - <create-value value-field="orderNotification"/>
> - </simple-method>
> -
> <simple-method method-name="updateOrderStatusFromReceipt" short-description="Update Order Status From ShipmentReceipt">
> <check-permission permission="ORDERMGR" action="_UPDATE"><fail-property resource="OrderErrorUiLabels" property="OrderSecurityErrorToRunUpdateOrderStatusFromReceipt"/></check-permission>
> <check-errors/>
>
> Modified: ofbiz/trunk/applications/order/servicedef/services.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/servicedef/services.xml (original)
> +++ ofbiz/trunk/applications/order/servicedef/services.xml Sun Jan 24 19:06:32 2016
> @@ -76,8 +76,7 @@ under the License.
> <attribute name="assignedRoleTypeId" type="String" mode="IN" optional="true"/>
> </service>
>
> - <service name="createOrderNotificationLog" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createNotificationLog">
> + <service name="createOrderNotificationLog" engine="entity-auto" default-entity-name="OrderNotification" invoke="create">
> <description>Logs when a notification was sent</description>
> <attribute name="orderId" type="String" mode="IN" optional="false"/>
> <attribute name="emailType" type="String" mode="IN" optional="false"/>
> @@ -164,17 +163,10 @@ under the License.
> <attribute name="orderId" type="String" mode="OUT" optional="false"/>
> </service>
>
> - <service name="createOrderItemBilling" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderItemBilling">
> + <service name="createOrderItemBilling" default-entity-name="OrderItemBilling" engine="entity-auto" invoke="create">
> <description>Create a new order item billing record</description>
> - <attribute name="orderId" type="String" mode="IN" optional="false"/>
> - <attribute name="orderItemSeqId" type="String" mode="IN" optional="false"/>
> - <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
> - <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
> - <attribute name="quantity" type="BigDecimal" mode="IN" optional="true"/>
> - <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
> - <attribute name="itemIssuanceId" type="String" mode="IN" optional="true"/>
> - <attribute name="shipmentReceiptId" type="String" mode="IN" optional="true"/>
> + <auto-attributes mode="IN" include="pk"/>
> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
> </service>
>
> <service name="orderAdjustmentPermissionCheck" engine="simple"
> @@ -183,8 +175,7 @@ under the License.
> <implements service="permissionInterface"/>
> </service>
>
> - <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustment">
> + <service name="createOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="create">
> <description>Creates a new order adjustment record</description>
> <permission-service service-name="orderAdjustmentPermissionCheck" main-action="CREATE"/>
> <auto-attributes mode="OUT" include="pk" optional="false"/>
> @@ -192,9 +183,9 @@ under the License.
> <override name="orderAdjustmentTypeId" optional="false"/>
> <override name="orderId" optional="false"/>
> </service>
> - <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="updateOrderAdjustment">
> + <service name="updateOrderAdjustment" default-entity-name="OrderAdjustment" engine="entity-auto" invoke="update">
> <description>Update an order adjustment record</description>
> + <permission-service service-name="orderAdjustmentPermissionCheck" main-action="UPDATE"/>
> <auto-attributes mode="IN" include="pk" optional="false"/>
> <auto-attributes mode="IN" include="nonpk" optional="true"/>
> <override name="orderId" optional="false"/>
> @@ -207,13 +198,10 @@ under the License.
> <attribute name="productPromoCodeId" type="String" mode="IN" optional="true"/>
> </service>
>
> - <service name="createOrderAdjustmentBilling" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderSimpleMethods.xml" invoke="createOrderAdjustmentBilling">
> + <service name="createOrderAdjustmentBilling" default-entity-name="OrderAdjustmentBilling" engine="entity-auto" invoke="create">
> <description>Create a new order adjustment billing record</description>
> - <attribute name="orderAdjustmentId" type="String" mode="IN" optional="false"/>
> - <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
> - <attribute name="invoiceItemSeqId" type="String" mode="IN" optional="false"/>
> - <attribute name="amount" type="BigDecimal" mode="IN" optional="true"/>
> + <auto-attributes mode="IN" include="pk"/>
> + <auto-attributes mode="IN" include="nonpk" optional="true"/>
> </service>
>
> <service name="createPaymentFromPreference" engine="java"
> @@ -435,15 +423,10 @@ under the License.
> <attribute name="roleTypeId" type="String" mode="IN"/>
> </service>
>
> - <service name="createOrderPaymentPreference" engine="java"
> - location="org.ofbiz.order.order.OrderServices" invoke="createPaymentPreference">
> + <service name="createOrderPaymentPreference" default-entity-name="OrderPaymentPreference" engine="entity-auto" invoke="create">
> <description>Creates an order payment preference</description>
> - <attribute name="orderId" type="String" mode="IN"/>
> - <attribute name="paymentMethodTypeId" type="String" mode="IN"/>
> - <attribute name="paymentMethodId" type="String" mode="IN" optional="true"/>
> - <attribute name="statusId" type="String" mode="IN" optional="true"/>
> - <attribute name="maxAmount" type="BigDecimal" mode="IN" optional="true"/>
> - <attribute name="orderPaymentPreferenceId" type="String" mode="OUT"/>
> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
> + <auto-attributes include="pk" mode="OUT"/>
> </service>
>
> <service name="createOrderNote" engine="java"
> @@ -568,20 +551,17 @@ under the License.
> </service>
>
> <!-- OrderShipment Services -->
> - <service name="createOrderShipment" default-entity-name="OrderShipment" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderShipment" auth="true">
> + <service name="createOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="create" auth="true">
> <description>Create OrderShipment</description>
> <auto-attributes include="pk" mode="IN" optional="false"/>
> <auto-attributes include="nonpk" mode="IN" optional="true"/>
> </service>
> - <service name="updateOrderShipment" default-entity-name="OrderShipment" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderShipment" auth="true">
> + <service name="updateOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="update" auth="true">
> <description>Update OrderShipment</description>
> <auto-attributes include="pk" mode="IN" optional="false"/>
> <auto-attributes include="nonpk" mode="IN" optional="true"/>
> </service>
> - <service name="deleteOrderShipment" default-entity-name="OrderShipment" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="deleteOrderShipment" auth="true">
> + <service name="deleteOrderShipment" engine="entity-auto" default-entity-name="OrderShipment" invoke="delete" auth="true">
> <description>Delete OrderShipment</description>
> <auto-attributes include="pk" mode="IN" optional="false"/>
> </service>
> @@ -667,21 +647,27 @@ under the License.
> </service>
>
> <!-- CommunicationEvents related to orders -->
> - <service name="createCommunicationEventOrder" engine="simple"
> - location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="createCommunicationEventOrder" auth="true">
> + <service name="createCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="create" auth="true">
> <description>Create a Communication Event Order</description>
> - <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="CREATE"/>
> + </required-permissions>
> + <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
> </service>
> - <service name="removeCommunicationEventOrder" engine="simple"
> - location="component://order/script/org/ofbiz/order/communication/CommunicationEventServices.xml" invoke="removeCommunicationEventOrder" auth="true">
> + <service name="removeCommunicationEventOrder" default-entity-name="CommunicationEventOrder" engine="entity-auto" invoke="delete" auth="true">
> <description>Remove a Communication Event Order</description>
> - <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN" optional="false"/>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="DELETE"/>
> + </required-permissions>
> + <auto-attributes entity-name="CommunicationEventOrder" include="pk" mode="IN"/>
> </service>
>
> <!-- Order Shipping and Contacts -->
> - <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderItemShipGroup" auth="true">
> + <service name="createOrderItemShipGroup" default-entity-name="OrderItemShipGroup" engine="entity-auto" invoke="create" auth="true">
> <description>Creates a new OrderItemShipGroup.</description>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="CREATE"/>
> + </required-permissions>
> <auto-attributes mode="IN" include="nonpk" optional="true"/>
> <attribute name="orderId" type="String" mode="IN" optional="false"/>
> <attribute name="shipGroupSeqId" type="String" mode="INOUT" optional="true"/>
> @@ -696,42 +682,52 @@ under the License.
> <attribute name="contactMechPurposeTypeId" type="String" mode="IN" optional="true"/>
> </service>
>
> - <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderContactMech" auth="true">
> + <service name="createOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="create" auth="true">
> <description>Create Order Contact Mech</description>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="CREATE"/>
> + </required-permissions>
> <auto-attributes include="pk" mode="IN" optional="false"/>
> </service>
>
> - <service name="updateOrderContactMech" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true" default-entity-name="OrderContactMech">
> + <service name="updateOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
> + location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderContactMech" auth="true">
> <description>Update Order Contact Mech</description>
> <auto-attributes include="pk" mode="IN" optional="false"/>
> <attribute name="oldContactMechId" type="String" mode="IN" optional="true"/>
> </service>
>
> - <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderContactMech" auth="true">
> + <service name="removeOrderContactMech" default-entity-name="OrderContactMech" engine="entity-auto" invoke="delete" auth="true">
> <description>Remove Order Contact Mech</description>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="DELETE"/>
> + </required-permissions>
> <auto-attributes include="pk" mode="IN" optional="false"/>
> </service>
>
> <!-- OrderTerms -->
> - <service name="createOrderTerm" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderTerm" auth="true">
> + <service name="createOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="create" auth="true">
> <description>Create an Order Term</description>
> - <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
> - <auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
> - <override name="orderItemSeqId" optional="true"/>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="CREATE"/>
> + </required-permissions>
> + <auto-attributes include="pk" mode="IN" optional="false"/>
> + <auto-attributes include="nonpk" mode="IN" optional="true"/>
> + <override name="orderItemSeqId" optional="true" default-value="_NA_"/>
> </service>
> - <service name="updateOrderTerm" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="updateOrderTerm" auth="true">
> + <service name="updateOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="update" auth="true">
> <description>Update an Order Term</description>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="UPDATE"/>
> + </required-permissions>
> <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
> <auto-attributes entity-name="OrderTerm" include="nonpk" mode="IN" optional="true"/>
> </service>
> - <service name="removeOrderTerm" engine="simple"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="removeOrderTerm" auth="true">
> + <service name="removeOrderTerm" default-entity-name="OrderTerm" engine="entity-auto" invoke="delete" auth="true">
> <description>Remove an Order Term</description>
> + <required-permissions join-type="AND">
> + <check-permission permission="ORDERMGR" action="DELETE"/>
> + </required-permissions>
> <auto-attributes entity-name="OrderTerm" include="pk" mode="IN" optional="false"/>
> </service>
>
>
> Modified: ofbiz/trunk/applications/order/servicedef/services_requirement.xml
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/servicedef/services_requirement.xml?rev=1726542&r1=1726541&r2=1726542&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/servicedef/services_requirement.xml (original)
> +++ ofbiz/trunk/applications/order/servicedef/services_requirement.xml Sun Jan 24 19:06:32 2016
> @@ -105,8 +105,7 @@ under the License.
> <attribute mode="OUT" name="quantityTotal" type="BigDecimal" optional="true"/>
> <attribute mode="OUT" name="amountTotal" type="BigDecimal" optional="true"/>
> </service>
> - <service name="createOrderRequirementCommitment" engine="simple" auth="true"
> - location="component://order/script/org/ofbiz/order/order/OrderServices.xml" invoke="createOrderRequirementCommitment">
> + <service name="createOrderRequirementCommitment" engine="entity-auto" auth="true" default-entity-name="OrderRequirementCommitment" invoke="create">
> <auto-attributes entity-name="OrderRequirementCommitment" include="pk" mode="IN" optional="false"/>
> <auto-attributes entity-name="OrderRequirementCommitment" include="nonpk" mode="IN" optional="true"/>
> </service>
>
> Modified: ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java?rev=1726542&r1=1726541&r2=1726542&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java (original)
> +++ ofbiz/trunk/applications/order/src/org/ofbiz/order/order/OrderServices.java Sun Jan 24 19:06:32 2016
> @@ -2824,52 +2824,6 @@ public class OrderServices {
> return ServiceUtil.returnSuccess();
> }
>
> - /** Service to create an order payment preference */
> - public static Map<String, Object> createPaymentPreference(DispatchContext ctx, Map<String, ? extends Object> context) {
> - Map<String, Object> result = new HashMap<String, Object>();
> - Delegator delegator = ctx.getDelegator();
> - String orderId = (String) context.get("orderId");
> - String statusId = (String) context.get("statusId");
> - String paymentMethodTypeId = (String) context.get("paymentMethodTypeId");
> - String paymentMethodId = (String) context.get("paymentMethodId");
> - BigDecimal maxAmount = (BigDecimal) context.get("maxAmount");
> - GenericValue userLogin = (GenericValue) context.get("userLogin");
> - Locale locale = (Locale) context.get("locale");
> -
> - String prefId = null;
> -
> - try {
> - prefId = delegator.getNextSeqId("OrderPaymentPreference");
> - } catch (IllegalArgumentException e) {
> - return ServiceUtil.returnError(UtilProperties.getMessage(resource_error,
> - "OrderErrorCouldNotCreateOrderPaymentPreferenceIdGenerationFailure", locale));
> - }
> -
> - Map<String, Object> fields = UtilMisc.<String, Object>toMap("orderPaymentPreferenceId", prefId, "orderId", orderId, "paymentMethodTypeId",
> - paymentMethodTypeId, "paymentMethodId", paymentMethodId, "maxAmount", maxAmount);
> -
> - if (statusId != null) {
> - fields.put("statusId", statusId);
> - }
> -
> - try {
> - GenericValue v = delegator.makeValue("OrderPaymentPreference", fields);
> - v.set("createdDate", UtilDateTime.nowTimestamp());
> - if (userLogin != null) {
> - v.set("createdByUserLogin", userLogin.getString("userLoginId"));
> - }
> - delegator.create(v);
> - } catch (GenericEntityException e) {
> - result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_ERROR);
> - result.put(ModelService.ERROR_MESSAGE, UtilProperties.getMessage(resource,
> - "OrderOrderPaymentPreferencesCannotBeCreated", UtilMisc.toMap("errorString", e.getMessage()), locale));
> - return ServiceUtil.returnFailure();
> - }
> - result.put("orderPaymentPreferenceId", prefId);
> - result.put(ModelService.RESPONSE_MESSAGE, ModelService.RESPOND_SUCCESS);
> - return result;
> - }
> -
> /** Service to get order header information as standard results. */
> public static Map<String, Object> getOrderHeaderInformation(DispatchContext dctx, Map<String, ? extends Object> context) {
> Delegator delegator = dctx.getDelegator();
>
>
>