Author: deepak
Date: Tue Nov 15 08:41:09 2016
New Revision: 1769755
URL:
http://svn.apache.org/viewvc?rev=1769755&view=revLog:
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>