Hi Suraj,

You've missed to add default-entity-name for the entity-auto engine for the
above services.

Could you please handle?
-- 
Thanks & Regards
Pawan Verma
Technical Consultant
*HotWax Systems*
*Enterprise open source experts*
http://www.hotwaxsystems.com


On Thu, Apr 30, 2020 at 5:05 PM <sur...@apache.org> wrote:

> This is an automated email from the ASF dual-hosted git repository.
>
> surajk pushed a commit to branch trunk
> in repository https://gitbox.apache.org/repos/asf/ofbiz-framework.git
>
>
> The following commit(s) were added to refs/heads/trunk by this push:
>      new e1a16aa  Improved: Converted all ShipmentReceiptRole service from
> crud to entity-auto (#85)
> e1a16aa is described below
>
> commit e1a16aa9fe4bacded92a0c4c34a0af7c02d0a325
> Author: Suraj Khurana <64310724+surajkhur...@users.noreply.github.com>
> AuthorDate: Thu Apr 30 17:05:39 2020 +0530
>
>     Improved: Converted all ShipmentReceiptRole service from crud to
> entity-auto (#85)
>
>     * Improved: Converted all ShipmentReceiptRole service from crud to
> entity-auto
>     (OFBIZ-11614)
> ---
>  .../shipment/receipt/ShipmentReceiptServices.xml       | 15
> ---------------
>  applications/product/servicedef/services.xml           |  2 +-
>  applications/product/servicedef/services_shipment.xml  | 18
> ++++++------------
>  3 files changed, 7 insertions(+), 28 deletions(-)
>
> diff --git
> a/applications/product/minilang/shipment/receipt/ShipmentReceiptServices.xml
> b/applications/product/minilang/shipment/receipt/ShipmentReceiptServices.xml
> index 2dade39..4334dc5 100644
> ---
> a/applications/product/minilang/shipment/receipt/ShipmentReceiptServices.xml
> +++
> b/applications/product/minilang/shipment/receipt/ShipmentReceiptServices.xml
> @@ -63,21 +63,6 @@ under the License.
>          <field-to-result field="affectAccounting"
> result-name="affectAccounting"/>
>      </simple-method>
>
> -    <simple-method method-name="createShipmentReceiptRole"
> short-description="Create a ShipmentReceipt Role">
> -        <make-value entity-name="ShipmentReceiptRole"
> value-field="newEntity"/>
> -        <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="removeShipmentReceiptRole"
> short-description="Remove a ShipmentReceipt Role">
> -        <make-value entity-name="ShipmentReceiptRole"
> value-field="lookupPKMap"/>
> -        <set-pk-fields map="parameters" value-field="lookupPKMap"/>
> -        <find-by-primary-key entity-name="ShipmentReceiptRole"
> map="lookupPKMap" value-field="lookedUpValue"/>
> -        <remove-value value-field="lookedUpValue"/>
> -    </simple-method>
> -
>      <simple-method method-name="receiveInventoryProduct"
> short-description="Receive Inventory in new Inventory Item(s)">
>          <!-- NOTES
>              - for serialized items with a serial number passed in: the
> quantityAccepted _should_ always be 1
> diff --git a/applications/product/servicedef/services.xml
> b/applications/product/servicedef/services.xml
> index 6015638..46a8f7c 100644
> --- a/applications/product/servicedef/services.xml
> +++ b/applications/product/servicedef/services.xml
> @@ -1003,7 +1003,7 @@ under the License.
>          <description>Create ProductFeature-DataResource</description>
>          <auto-attributes include="pk" mode="IN" optional="false"/>
>      </service>
> -    <service name="removeProductFeatureDataResource"
> default-entity-name="ProductFeatureDataResource" engine="entity-auto"
> invoke="remove" auth="true">
> +    <service name="removeProductFeatureDataResource"
> default-entity-name="ProductFeatureDataResource" engine="entity-auto"
> invoke="delete" auth="true">
>          <description>Remove ProductFeature-DataResource</description>
>          <auto-attributes include="pk" mode="IN" optional="false"/>
>      </service>
> diff --git a/applications/product/servicedef/services_shipment.xml
> b/applications/product/servicedef/services_shipment.xml
> index 1b33dc6..ff49a88 100644
> --- a/applications/product/servicedef/services_shipment.xml
> +++ b/applications/product/servicedef/services_shipment.xml
> @@ -915,23 +915,17 @@ under the License.
>          <attribute name="statusId" type="String" mode="IN"
> optional="true"/>
>      </service>
>
> -    <service name="interfaceShipmentReceiptRole" engine="interface"
> location="" invoke="">
> -        <description>Interface for ShipmentReceiptRole</description>
> -        <attribute name="receiptId" type="String" mode="IN"
> optional="false"/>
> -        <attribute name="partyId" type="String" mode="IN"
> optional="false"/>
> -        <attribute name="roleTypeId" type="String" mode="IN"
> optional="false"/>
> -    </service>
> -    <service name="createShipmentReceiptRole" engine="simple"
> -
> location="component://product/minilang/shipment/receipt/ShipmentReceiptServices.xml"
> invoke="createShipmentReceiptRole" auth="true">
> +    <service name="createShipmentReceiptRole" engine="entity-auto"
> invoke="create" auth="true">
>          <description>Create a ShipmentReceipt Role entry</description>
>          <permission-service service-name="facilityGenericPermission"
> main-action="CREATE"/>
> -        <implements service="interfaceShipmentReceiptRole"/>
> +        <auto-attributes include="pk" mode="INOUT" optional="false"/>
> +        <auto-attributes include="nonpk" mode="IN" optional="true"/>
>      </service>
> -    <service name="removeShipmentReceiptRole" engine="simple"
> -
> location="component://product/minilang/shipment/receipt/ShipmentReceiptServices.xml"
> invoke="removeShipmentReceiptRole" auth="true">
> +    <service name="removeShipmentReceiptRole" engine="entity-auto"
> invoke="delete" auth="true">
>          <description>Remove a ShipmentReceipt Role entry</description>
>          <permission-service service-name="facilityGenericPermission"
> main-action="DELETE"/>
> -        <implements service="interfaceShipmentReceiptRole"/>
> +        <auto-attributes include="pk" mode="IN" optional="false"/>
> +        <auto-attributes include="nonpk" mode="IN" optional="true"/>
>      </service>
>
>      <!-- Shipment Estimate (Rate) Services -->
>
>

Reply via email to