[GROOVY-7697] GroovyScriptEngine.loadScriptByName doesn't support environment variable "groovy.ast"

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.4.8
• Assignee: John Wagenleitner
• Reporter: Marc Ewert
• Created: 01/Dec/15 14:35
• Updated: 01/Feb/17 23:18
• Resolved: 07/Jul/16 20:34

Description

When setting the environment variable "groovy.ast" to "xml" and calling GroovyScriptEngine.loadScriptByName a FileNotFoundException is thrown (which is caught internally), because a FileWriter is created from a String representation of an URI.

In class SourceUnit method saveAsXML there should be some code added like:

FileWriter writer;
if (name.startsWith("file:")) {
  writer = new FileWriter(new File(new URI(name + ".xml")));
} else {
  writer = new FileWriter(name + ".xml")
}
...

Or however URIs are handled in Groovy. Exception handling still has to be added.

Issue Links

• links to: GitHub Pull Request #330