You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by jl...@apache.org on 2020/03/17 07:48:56 UTC

[ofbiz-plugins] 02/02: Fixed: Convert DimensionServices.xml minilang to groovy

This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/ofbiz-plugins.git

commit 350571deaadef1cecc837ad6688d9cdedd630887
Author: Jacques Le Roux <ja...@les7arts.com>
AuthorDate: Tue Mar 17 08:49:05 2020 +0100

    Fixed: Convert DimensionServices.xml minilang to groovy
    
    (OFBIZ-10948)
    
    There is a bug with loadCurrencyDimension in DimensionServices.groovy
    It shows when running quickInitDataWarehouse.
    
    The problem is this Minilang expression is missing in DimensionServices.groovy:
    <sequenced-id sequence-name="CurrencyDimension" field="currencyDim.dimensionId"/>
    
    Adding
      delegator.setNextSeqId(currencyDim)
    is enough.
---
 bi/groovyScripts/DimensionServices.groovy | 1 +
 1 file changed, 1 insertion(+)

diff --git a/bi/groovyScripts/DimensionServices.groovy b/bi/groovyScripts/DimensionServices.groovy
index f44d0a0..2a1dd01 100644
--- a/bi/groovyScripts/DimensionServices.groovy
+++ b/bi/groovyScripts/DimensionServices.groovy
@@ -88,6 +88,7 @@ def loadCurrencyDimension() {
             }
         } else {
             currencyDim = delegator.makeValue("CurrencyDimension")
+            delegator.setNextSeqId(currencyDim)
             currencyDim.currencyId = currency.uomId
             currencyDim.description = currency.description
             currencyDim.create()