[ 
https://issues.apache.org/jira/browse/ARIES-48?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrew Osborne reassigned ARIES-48:
-----------------------------------

    Assignee: Andrew Osborne

> BeanRecipe has two destroy methods, the one that invokes BeanProcessor is 
> unused.
> ---------------------------------------------------------------------------------
>
>                 Key: ARIES-48
>                 URL: https://issues.apache.org/jira/browse/ARIES-48
>             Project: Aries
>          Issue Type: Bug
>          Components: Blueprint
>            Reporter: Andrew Osborne
>            Assignee: Andrew Osborne
>
> BeanRecipe declares 
>   destroy(Object instance) which invokes the destroy method if any.. 
> and
>   destroyInstance(Object obj) which does the same, but also invokes the 
> BeanProcessor before/after destroy calls. 
> destroyInstance has no callers within the codebase, without which the 
> BeanProcessor will never be invoked for before/after destroy.
> SVN history has the destroyInstance method present as far back as I can check 
> for, with the seperate destroy method being brought in as part of the Recipe 
> refactoring that removed DestroyCallback, although DestroyCallback did not 
> invoke destroyInstance either.
> Suspect the fix to this is to remove destroy(Object instance) and rename 
> destroyInstance(Object obj) to destroy(Object obj)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to