You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@felix.apache.org by "Bram de Kruijff (JIRA)" <ji...@apache.org> on 2006/12/06 11:25:21 UTC

[jira] Commented: (FELIX-137) ServiceDependency fails to invoke callback methods when implemented in superclass (patch included)

    [ http://issues.apache.org/jira/browse/FELIX-137?page=comments#action_12455957 ] 
            
Bram de Kruijff commented on FELIX-137:
---------------------------------------

It seems now it is all little too enthusiastic :P In my case I now get multiple callbacks on from servicedependencies in the same scenario. The problem is in the recursive invokeMethod that takes a depth first approach and doesn't stop when it's done. I'll attach a new patch to this problem. It's non-recursive and breadth first.. I like it :)

> ServiceDependency fails to invoke callback methods when implemented in superclass (patch included)
> --------------------------------------------------------------------------------------------------
>
>                 Key: FELIX-137
>                 URL: http://issues.apache.org/jira/browse/FELIX-137
>             Project: Felix
>          Issue Type: Bug
>          Components: Dependency Manager
>    Affects Versions: 0.8.0
>            Reporter: Bram de Kruijff
>         Assigned To: Marcel Offermans
>             Fix For: 0.8.0
>
>         Attachments: ServiceDependency_patch1.txt, ServiceDependency_patch2.txt
>
>
> ServiceDependency fails to invoke callback methods when implemented in superclass. The problem is in the invokeCallbackMethod method cause it will never loop. I include 2 patches. The first just fixes the loop, but I was wondering why getDeclaredMethod is used. Using getMethod instead seems to work just fine.
> grz,
> Bram

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira