/[Apache-SVN]
ViewVC logotype

Revision 1854652


Jump to revision: Previous Next
Author: mthl
Date: Sat Mar 2 17:02:32 2019 UTC (5 years, 1 month ago)
Changed paths: 2
Log Message:
Improved: Remove Groovy Base script redundancy (OFBIZ-10806)

According to Groovy documentation [1] the following construction:

    CompilerConfiguration cfg = new CompilerConfiguration();
    cfg.setScriptBaseClass(scriptBaseClass);
    groovyClassLoader =
        new GroovyClassLoader(GroovyUtil.class.getClassLoader(), cfg);

is equivalent to running the a script containing the ‘@BaseScript’
annotation.  Previously in ‘org.apache.ofbiz.base.util.GroovyUtil’
instead of choosing one solution, we were doing both.  Now we only
rely on the construction described above.

[1] http://docs.groovy-lang.org/latest/html/documentation/#_script_base_classes


Changed paths

Path Details
Directoryofbiz/ofbiz-framework/trunk/framework/base/groovyScripts/GroovyInit.groovy deleted
Directoryofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/util/GroovyUtil.java modified , text changed

infrastructure at apache.org
ViewVC Help
Powered by ViewVC 1.1.26