[GROOVY-9240] Better signature for ResourceGroovyMethods.traverse(File, Map<String, Object>, Closure) (and overloadings)

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.5.8
• Fix Version/s: 2.5.9, 3.0.0-rc-1
• Component/s: groovy-jdk
• Assignee: Eric Milles
• Reporter: Mauro Molinari
• Created: 30/Aug/19 15:52
• Updated: 24/Nov/19 02:44
• Resolved: 10/Sep/19 00:06

Description

When you try to perform the following invocation in a statically checked/compile Groovy class you'll get a compilation error:

myFolder.traverse([
			type: FileType.FILES,
			nameFilter: ~/.*\.(?i)pdf/
		]) { println it }

The error is:

Groovy:[Static type checking] - Cannot call java.io.File#traverse(java.util.Map <java.lang.String, java.lang.Object>, groovy.lang.Closure) with arguments [java.util.LinkedHashMap <java.lang.String, java.io.Serializable>, groovy.lang.Closure] 

I think that more flexible and static type checker-friendly signatures for the traverse methods would be:
ResourceGroovyMethods.traverse(File, Map<String, ?>, Closure)
ResourceGroovyMethods.traverse(File, Map<String, ?>)

Issue Links

• links to: GitHub Pull Request #1001