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 2017/07/22 09:46:47 UTC

svn commit: r1802666 - in /ofbiz/ofbiz-framework/trunk/applications/accounting: groovyScripts/rate/RateServices.groovy servicedef/services_rate.xml

Author: nmalin
Date: Sat Jul 22 09:46:47 2017
New Revision: 1802666

URL: http://svn.apache.org/viewvc?rev=1802666&view=rev
Log:
Fixed: For service getRateAmount move the periodTypeId default value from groovy to service definition,
Thans Scott for this remark

Modified:
    ofbiz/ofbiz-framework/trunk/applications/accounting/groovyScripts/rate/RateServices.groovy
    ofbiz/ofbiz-framework/trunk/applications/accounting/servicedef/services_rate.xml

Modified: ofbiz/ofbiz-framework/trunk/applications/accounting/groovyScripts/rate/RateServices.groovy
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/accounting/groovyScripts/rate/RateServices.groovy?rev=1802666&r1=1802665&r2=1802666&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/applications/accounting/groovyScripts/rate/RateServices.groovy (original)
+++ ofbiz/ofbiz-framework/trunk/applications/accounting/groovyScripts/rate/RateServices.groovy Sat Jul 22 09:46:47 2017
@@ -150,9 +150,6 @@ def getRateAmount() {
     if (!parameters.rateCurrencyUomId) {
         parameters.rateCurrencyUomId = UtilProperties.getPropertyValue('general.properties', 'currency.uom.id.default', 'USD')
     }
-    if (!parameters.periodTypeId) {
-        parameters.periodTypeId = 'RATE_HOUR'
-    }
     String serviceName = null;
     if (parameters.workEffortId && parameters.workEffortId != '_NA_') {
         // workeffort level

Modified: ofbiz/ofbiz-framework/trunk/applications/accounting/servicedef/services_rate.xml
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/applications/accounting/servicedef/services_rate.xml?rev=1802666&r1=1802665&r2=1802666&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/applications/accounting/servicedef/services_rate.xml (original)
+++ ofbiz/ofbiz-framework/trunk/applications/accounting/servicedef/services_rate.xml Sat Jul 22 09:46:47 2017
@@ -63,7 +63,7 @@ under the License.
         <auto-attributes include="pk" mode="IN" optional="true"/>
         <attribute name="level" type="String" mode="OUT" optional="true"/><!-- the level the amount was retrieved from(specific -> general): workEffort, party, emplPositionType, rateType or 'null' if not found-->
         <attribute name="rateAmount" type="BigDecimal" mode="OUT" optional="true"/>
-        <attribute name="periodTypeId" type="String" mode="INOUT" optional="true"/>
+        <attribute name="periodTypeId" type="String" mode="INOUT" optional="true" default-value="RATE_HOUR"/>
         <attribute name="rateCurrencyUomId" type="String" mode="INOUT" optional="true"/>
         <attribute name="fromDate" type="Timestamp" mode="OUT" optional="true"/>
         <override name="rateTypeId" optional="false"/>