[GROOVY-3015] Seems like ClosureMetaClass#invokeMethod does not respect the interceptable nature of the owner/delegate

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.5.6
• Fix Version/s: 4.0.0-beta-1, 3.0.9
• Component/s: groovy-runtime
• Environment: Windows XP
• Assignee: Eric Milles
• Reporter: Erick Erickson
• Created: 21/Aug/08 10:32
• Updated: 03/Feb/22 22:34
• Resolved: 18/May/21 13:41

Description

Sample program and output. The closure and the method both call 'outer'
which calls 'inner', but the closure does NOT trace the call to 'outer'

See comments, including Jochen "blackdrag" Theodorou's at
http://www.nabble.com/implementing-GroovyInterceptable-behaves-differently-in-closures-and-methods-td19072057.html

************program output

Entering shouldTraceOuterAndInnerClosure
Entering inner
Leaving inner
Leaving shouldTraceOuterAndInnerClosure

Entering shouldTraceOuterAndInnerMethod
Entering outer
Entering inner
Leaving inner
Leaving outer
Leaving shouldTraceOuterAndInnerMethod

*************end program output

****************code starts
import org.codehaus.groovy.runtime.StringBufferWriter
import org.codehaus.groovy.runtime.InvokerHelper

class Traceable implements GroovyInterceptable {
private static int indent = 1
Writer writer = new PrintWriter(System.out)
Object invokeMethod(String name, Object args)

{ def result def metaClass = InvokerHelper.getMetaClass(this) writer.write("\n" + ' ' * indent + "Entering $name") ++indent result = metaClass.invokeMethod(this, name, args) --indent writer.write("\n" + ' ' * indent + "Leaving $name") return result }

}

class Whatever extends Traceable {
int outer()

{ return inner() }

int inner()

{ return 1 }

def shouldTraceOuterAndInnerClosure =

{ return outer() }

int shouldTraceOuterAndInnerMethod() { return outer() }

}

def log = new StringBuffer()
def traceMe = new Whatever(writer : new StringBufferWriter(log))

traceMe.shouldTraceOuterAndInnerClosure()

traceMe.shouldTraceOuterAndInnerMethod()

println log.toString()

****************program ends*************

Issue Links

• depends upon: GROOVY-2503 MOP 2.0 design inflluencing issues