You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@groovy.apache.org by "John Mercier (JIRA)" <ji...@apache.org> on 2017/09/28 16:58:00 UTC

[jira] [Commented] (GROOVY-4610) GroovyInterceptable (AOP) not working with closures

    [ https://issues.apache.org/jira/browse/GROOVY-4610?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16184454#comment-16184454 ] 

John Mercier commented on GROOVY-4610:
--------------------------------------

Is there a workaround for this issue?

> GroovyInterceptable (AOP) not working with closures
> ---------------------------------------------------
>
>                 Key: GROOVY-4610
>                 URL: https://issues.apache.org/jira/browse/GROOVY-4610
>             Project: Groovy
>          Issue Type: Bug
>          Components: groovy-runtime
>         Environment: JDK 1.6.x, Grails 1.2.1
>            Reporter: Sunny Thandassery
>
> I've got a grails app with Service classes that inherit from Groovy's GroovyInterceptable: 
> [code] 
>     class customerSerrvice implements GroovyInterceptable { 
>     private List<Customer> customers 
>     def invokeMethod(String name, args) { 
>         log.debug "=======>INVOKING method [$name] with args:$args" 
>     } 
>     void foo() { 
>         customers.each { doSomething(it) } 
>     } 
>     void doSomething(Customer cust) { log.debug "doSomething invoked with $cust" } 
>     } 
> [/code] 
> If I call foo() or doSomething() **directly** from another class, the invokeMethod gets called like it is supposed to. However, when foo() calls doSomething(), that call is not intercepted in invokeMethod. 
> If I change from   
> `customers.each { doSomething(it) }`   
> to   
> `for(Customer cust: customers) { doSomething(cust) }`   
> then the invokeMethod gets called just fine. 
> There appears to be an ExpandoMetaClass bug that prevents closures and GroovyInterceptable from working together.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)