Author: nmalin Date: Sat Jul 22 22:42:56 2017 New Revision: 1802712 URL: http://svn.apache.org/viewvc?rev=1802712&view=rev Log: Improved: With new groovy DSL for query one, simplify groovy rate amount services
Modified: ofbiz/ofbiz-framework/trunk/applications/accounting/groovyScripts/rate/RateServices.groovy 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=1802712&r1=1802711&r2=1802712&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 22:42:56 2017 @@ -68,7 +68,7 @@ def expireRateAmount() { if (!lookedUpValue.rateCurrencyUomId) { lookedUpValue.rateCurrencyUomId = UtilProperties.getPropertyValue('general.properties', 'currency.uom.id.default') } - lookedUpValue = from('RateAmount').where(lookedUpValue.getFields(lookedUpValue.getModelEntity().getPkFieldNames())).queryOne() + lookedUpValue = from('RateAmount').where(lookedUpValue).queryOne() if (lookedUpValue) { Timestamp previousDay = UtilDateTime.adjustTimestamp(UtilDateTime.nowTimestamp(), 5, -1) lookedUpValue.thruDate = UtilDateTime.getDayEnd(previousDay) @@ -115,7 +115,7 @@ def deletePartyRate() { return error('delete party rate isn\'t possible, please update your code with service name "expirePartyRate" instead "deletePartyRate"') } def expirePartyRate() { - GenericValue lookedUpValue = from('PartyRate').where([partyId: partyId, rateTypeId: rateTypeId, fromDate: fromDate]).queryOne() + GenericValue lookedUpValue = from('PartyRate').where(parameters).queryOne() if (lookedUpValue) { lookedUpValue.thruDate = UtilDateTime.nowTimestamp() lookedUpValue.store() @@ -213,9 +213,9 @@ def getRatesAmountsFrom(String field) { condition.put(field, parameters.get(field)) List ratesList = from('RateAmount').where(condition).filterByDate().queryList() if (!ratesList) { - GenericValue periodType = from('PeriodType').where([periodTypeId: parameters.periodTypeId]).queryOne() - GenericValue rateType = from('RateType').where([rateTypeId: parameters.rateTypeId]).queryOne() - GenericValue partyNameView = from('PartyNameView').where([partyId: parameters.partyId]).queryOne() + GenericValue periodType = from('PeriodType').where(parameters).queryOne() + GenericValue rateType = from('RateType').where(parameters).queryOne() + GenericValue partyNameView = from('PartyNameView').where(parameters).queryOne() logError('A valid rate entry could be found for rateType:' + rateType.description + ', ' + entityName + ':' + parameters.get(field) + ', party: ' + partyNameView.lastName + partyNameView.middleName + partyNameView.firstName + partyNameView.groupName + ' However.....not for the period:' + periodType.description + ' and currency:' + parameters.rateCurrencyUomId)