[GROOVY-6002] ModuleNode should setScriptBaseClassFromConfig from context or unit

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.0.7, 2.1.1
• Component/s: Compiler
• Assignee: Cédric Champeau
• Reporter: Andrew Eisenberg
• Created: 14/Feb/13 23:18
• Updated: 16/Feb/13 00:47
• Resolved: 15/Feb/13 03:27

Description

Suggestion to replace the body of ModuleNode.setScriptBaseClassFromConfig with this:

        String baseClassName = null;
        if (unit != null) {
            baseClassName = unit.getConfig().getScriptBaseClass();
        } else if (context != null) {
            baseClassName = context.getConfiguration().getScriptBaseClass();
        }
        if(baseClassName != null) {
            if(!cn.getSuperClass().getName().equals(baseClassName)) {
                cn.setSuperClass(ClassHelper.make(baseClassName));
            }
        }

Reason: groovy-eclipse sometimes does not have the unit set when this method gets called, but the context is available.