[jira] Updated: (ARIES-48) BeanRecipe has two destroy methods, the one that invokes BeanProcessor is unused.

["Jeremy Hughes (JIRA)" <ji...@apache.org>]

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

Jeremy Hughes updated ARIES-48:
-------------------------------

    Fix Version/s: 0.1

> 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
>             Fix For: 0.1
>
>
> 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.