Author: deepak
Date: Tue Nov 15 08:41:09 2016
New Revision: 1769755

URL: http://svn.apache.org/viewvc?rev=1769755&view=rev
Log:
Improved: Converted createInvoiceContactMech service from simple to entity 
auto, and fixed service definition for same, removed unused out parameter 
contactMechId from service definition, I don't understand purpose of this out 
parameter, and this service is used only in 3 places and not SECA rule define 
on it. So I think we can modify service definition.

Modified:
    ofbiz/trunk/applications/accounting/minilang/invoice/InvoiceServices.xml
    ofbiz/trunk/applications/accounting/servicedef/services_invoice.xml

Modified: 
ofbiz/trunk/applications/accounting/minilang/invoice/InvoiceServices.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/minilang/invoice/InvoiceServices.xml?rev=1769755&r1=1769754&r2=1769755&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/minilang/invoice/InvoiceServices.xml 
(original)
+++ ofbiz/trunk/applications/accounting/minilang/invoice/InvoiceServices.xml 
Tue Nov 15 08:41:09 2016
@@ -1013,10 +1013,4 @@ under the License.
         </if-compare-field>
         <field-to-result field="isForeign"/>
     </simple-method>
-    <simple-method method-name="createInvoiceContactMech" 
short-description="Create a ContactMech for an invoice">
-        <make-value value-field="invoiceContactMech" 
entity-name="InvoiceContactMech"/>
-        <set-pk-fields map="parameters" value-field="invoiceContactMech"/>
-        <create-value value-field="invoiceContactMech"/>
-        <field-to-result field="contactMechId" 
result-name="invoiceContactMech"/>
-    </simple-method>
 </simple-methods>

Modified: ofbiz/trunk/applications/accounting/servicedef/services_invoice.xml
URL: 
http://svn.apache.org/viewvc/ofbiz/trunk/applications/accounting/servicedef/services_invoice.xml?rev=1769755&r1=1769754&r2=1769755&view=diff
==============================================================================
--- ofbiz/trunk/applications/accounting/servicedef/services_invoice.xml 
(original)
+++ ofbiz/trunk/applications/accounting/servicedef/services_invoice.xml Tue Nov 
15 08:41:09 2016
@@ -301,12 +301,10 @@ under the License.
         <description>Checks to see if the payments applied to an invoice total 
up to the invoice total; if so sets to PAID</description>
         <attribute name="invoiceId" type="String" mode="IN" optional="false"/>
     </service>
-    <service name="createInvoiceContactMech" engine="simple" 
location="component://accounting/minilang/invoice/InvoiceServices.xml"
-             invoke="createInvoiceContactMech" 
default-entity-name="InvoiceContactMech">
+    <service name="createInvoiceContactMech" engine="entity-auto" 
invoke="create" default-entity-name="InvoiceContactMech">
         <description>Create a ContactMech for an invoice</description>
         <permission-service service-name="acctgInvoicePermissionCheck" 
main-action="CREATE"/>
         <auto-attributes include="pk" mode="IN" optional="false"/>
-        <attribute name="contactMechId" type="String" mode="OUT" 
optional="true"/>
     </service>
     <service name="deleteInvoiceContactMech" engine="entity-auto" 
invoke="delete" default-entity-name="InvoiceContactMech">
         <description>Delete a ContactMech for an invoice</description>


Reply via email to