Thanks Jacques, you are fast just completed my testing to commit it. :-) Thanks again!
Rishi Solanki Sr Manager, Enterprise Software Development HotWax Systems Pvt. Ltd. Direct: +91-9893287847 http://www.hotwaxsystems.com www.hotwax.co On Sat, May 12, 2018 at 7:25 PM, Jacques Le Roux < jacques.le.r...@les7arts.com> wrote: > Done at r1831480, Jira updated > > > > Le 12/05/2018 à 15:40, Jacques Le Roux a écrit : > >> Thanks Suraj, Rishi, >> >> Sorry got an issue running tests locally (I had one of the documentation >> PDF file generated open and did not notice the tests failed until I got >> there) >> >> I agree about adding and will do so >> >> Jacques >> >> >> Le 12/05/2018 à 14:41, Rishi Solanki a écrit : >> >>> I would add the *custRequestId* and *custRequestItemSeqId* as IN >>> parameter >>> in createRequirement service. The idea behind the ticket is to not to do >>> functional changes and just to convert the services to entity-auto. >>> >>> Rishi Solanki >>> Sr Manager, Enterprise Software Development >>> HotWax Systems Pvt. Ltd. >>> Direct: +91-9893287847 >>> http://www.hotwaxsystems.com >>> www.hotwax.co >>> >>> On Sat, May 12, 2018 at 5:57 PM, Suraj Khurana < >>> suraj.khur...@hotwaxsystems.com> wrote: >>> >>> Thanks Jacques, >>>> >>>> I was checking *testIntegration* and *createRequirement* service is >>>> failing >>>> due to missing *custRequestId* in it. >>>> *Solution:* Remove this parameter from testCreateRequirement as it is >>>> not >>>> part of Requirement entity. >>>> But, on the same time I found a seca on *createRequirement* calling >>>> *associatedRequirementWithRequestItem* >>>> if *custRequestId* and *custRequestItemSeqId* is not empty. >>>> >>>> Another solution could be add *custRequestId* and >>>> *custRequestItemSeqId* as >>>> IN parameter in createRequirement service attributes. >>>> Please suggest what should be the idle thing to do. >>>> >>>> -- >>>> 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 Sat, May 12, 2018 at 4:53 PM, <jler...@apache.org> wrote: >>>> >>>> Author: jleroux >>>>> Date: Sat May 12 11:23:29 2018 >>>>> New Revision: 1831467 >>>>> >>>>> URL: http://svn.apache.org/viewvc?rev=1831467&view=rev >>>>> Log: >>>>> Improved: Convert Requirement related services from simple to >>>>> entity-auto >>>>> (OFBIZ-10395) >>>>> >>>>> There is nothing extra in these services and they can be made to >>>>> entity-auto. >>>>> >>>>> Thanks: Suraj Khurana >>>>> >>>>> Modified: >>>>> ofbiz/ofbiz-framework/trunk/applications/order/minilang/requirement/ >>>>> RequirementServices.xml >>>>> ofbiz/ofbiz-framework/trunk/applications/order/servicedef/ >>>>> services_requirement.xml >>>>> >>>>> Modified: ofbiz/ofbiz-framework/trunk/applications/order/minilang/ >>>>> requirement/RequirementServices.xml >>>>> URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/ >>>>> applications/order/minilang/requirement/RequirementServices.xml?rev= >>>>> 1831467&r1=1831466&r2=1831467&view=diff >>>>> ============================================================ >>>>> ================== >>>>> --- ofbiz/ofbiz-framework/trunk/applications/order/minilang/requ >>>>> irement/ >>>>> >>>> RequirementServices.xml >>>> >>>>> (original) >>>>> +++ ofbiz/ofbiz-framework/trunk/applications/order/minilang/requ >>>>> irement/ >>>>> >>>> RequirementServices.xml >>>> >>>>> Sat May 12 11:23:29 2018 >>>>> @@ -20,35 +20,6 @@ under the License. >>>>> >>>>> <simple-methods xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance >>>>> " >>>>> xmlns="http://ofbiz.apache.org/Simple-Method" >>>>> xsi:schemaLocation="http://ofbiz.apache.org/Simple-Method >>>>> http://ofbiz.apache.org/dtds/simple-methods.xsd"> >>>>> - <simple-method method-name="createRequirement" >>>>> short-description="Create a new requirement"> >>>>> - <make-value entity-name="Requirement" >>>>> value-field="newEntity"/> >>>>> - <sequenced-id sequence-name="Requirement" >>>>> >>>> field="requirementId"/> >>>> >>>>> - <set field="newEntity.requirementId" >>>>> >>>> from-field="requirementId"/> >>>> >>>>> - <set-nonpk-fields map="parameters" value-field="newEntity"/> >>>>> - <if-empty field="parameters.statusId"> >>>>> - <set value="REQ_CREATED" field="newEntity.statusId"/> >>>>> - </if-empty> >>>>> - <now-timestamp field="nowTimestamp"/> >>>>> - <set field="newEntity.createdDate" from-field="nowTimestamp"/> >>>>> - <set field="newEntity.lastModifiedDate" >>>>> from-field="nowTimestamp"/> >>>>> - <set field="newEntity.createdByUserLogin" >>>>> >>>> from-field="userLogin. >>>> >>>>> userLoginId"/> >>>>> - <set field="newEntity.lastModifiedByUserLogin" >>>>> from-field="userLogin.userLoginId"/> >>>>> - <create-value value-field="newEntity"/> >>>>> - <field-to-result field="requirementId" >>>>> result-name="requirementId"/> >>>>> - </simple-method> >>>>> - >>>>> - <simple-method method-name="updateRequirement" >>>>> short-description="Update a requirement"> >>>>> - <make-value entity-name="Requirement" >>>>> >>>> value-field="lookupPKMap"/> >>>> >>>>> - <set-pk-fields map="parameters" value-field="lookupPKMap"/> >>>>> - <find-by-primary-key entity-name="Requirement" >>>>> map="lookupPKMap" >>>>> value-field="lookedUpValue"/> >>>>> - <field-to-result field="lookedUpValue.statusId" >>>>> result-name="oldStatusId"/> >>>>> - <set-nonpk-fields map="parameters" >>>>> value-field="lookedUpValue"/> >>>>> - <now-timestamp field="nowTimestamp"/> >>>>> - <set field="newEntity.lastModifiedDate" >>>>> from-field="nowTimestamp"/> >>>>> - <set field="newEntity.lastModifiedByUserLogin" >>>>> from-field="userLogin.userLoginId"/> >>>>> - <store-value value-field="lookedUpValue"/> >>>>> - </simple-method> >>>>> - >>>>> <simple-method method-name="deleteRequirement" >>>>> short-description="Delete a requirement"> >>>>> <entity-one entity-name="Requirement" >>>>> value-field="requirement" >>>>> auto-field-map="true"/> >>>>> <check-errors/> >>>>> >>>>> Modified: ofbiz/ofbiz-framework/trunk/applications/order/servicedef/ >>>>> services_requirement.xml >>>>> URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/ >>>>> applications/order/servicedef/services_requirement.xml?rev= >>>>> 1831467&r1=1831466&r2=1831467&view=diff >>>>> ============================================================ >>>>> ================== >>>>> --- ofbiz/ofbiz-framework/trunk/applications/order/servicedef/ >>>>> >>>> services_requirement.xml >>>> >>>>> (original) >>>>> +++ ofbiz/ofbiz-framework/trunk/applications/order/servicedef/ >>>>> >>>> services_requirement.xml >>>> >>>>> Sat May 12 11:23:29 2018 >>>>> @@ -24,33 +24,18 @@ under the License. >>>>> <vendor>OFBiz</vendor> >>>>> <version>1.0</version> >>>>> >>>>> - <!-- Requirement Services --> >>>>> - <service name="requirementInterface" engine="interface" >>>>> location="" >>>>> invoke=""> >>>>> - <description>Requirement Interface</description> >>>>> - <auto-attributes mode="IN" entity-name="Requirement" >>>>> include="nonpk" optional="true"> >>>>> - <exclude field-name="createdDate"/> >>>>> - <exclude field-name="createdByUserLogin"/> >>>>> - <exclude field-name="lastModifiedDate"/> >>>>> - <exclude field-name="lastModifiedByUserLogin"/> >>>>> - </auto-attributes> >>>>> - </service> >>>>> - <service name="createRequirement" engine="simple" >>>>> - location="component://order/minilang/requirement/ >>>>> >>>> RequirementServices.xml" >>>> >>>>> invoke="createRequirement"> >>>>> + <service name="createRequirement" default-entity-name="Requireme >>>>> nt" >>>>> engine="entity-auto" invoke="create" auth="true"> >>>>> <description>Create a new requirement</description> >>>>> - <implements service="requirementInterface"/> >>>>> - <attribute name="requirementTypeId" type="String" mode="IN" >>>>> optional="false"/> >>>>> - <attribute name="custRequestId" type="String" mode="IN" >>>>> optional="true"/> >>>>> - <attribute name="custRequestItemSeqId" type="String" mode="IN" >>>>> optional="true"/> >>>>> - <attribute name="requirementId" type="String" mode="INOUT" >>>>> optional="true"/> >>>>> + <auto-attributes include="pk" mode="INOUT" optional="true"/> >>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/> >>>>> + <override name="statusId" default-value="REQ_CREATED"/> >>>>> </service> >>>>> - <service name="updateRequirement" engine="simple" >>>>> - location="component://order/minilang/requirement/ >>>>> >>>> RequirementServices.xml" >>>> >>>>> invoke="updateRequirement"> >>>>> + <service name="updateRequirement" default-entity-name="Requireme >>>>> nt" >>>>> engine="entity-auto" invoke="update" auth="true"> >>>>> <description>Update an existing requirement</description> >>>>> - <implements service="requirementInterface"/> >>>>> - <attribute name="requirementId" type="String" mode="IN" >>>>> optional="false"/> >>>>> - <attribute name="oldStatusId" type="String" mode="OUT" >>>>> optional="false"/> >>>>> + <auto-attributes include="pk" mode="IN" optional="false"/> >>>>> + <auto-attributes include="nonpk" mode="IN" optional="true"/> >>>>> + <attribute name="oldStatusId" type="String" mode="OUT" >>>>> optional="true"/> >>>>> </service> >>>>> - >>>>> <service name="deleteRequirement" engine="simple" >>>>> location="component://order/minilang/requirement/ >>>>> >>>> RequirementServices.xml" >>>> >>>>> invoke="deleteRequirement"> >>>>> <description>Delete a requirement</description> >>>>> >>>>> >>>>> >>>>> >> >> >