[Springframework-rcp-dev] [Spring-JIRA] Commented: (RCP-483) missing to remove view from ApplicationEventMulticaster when view is closed

[Peter De B. (JIRA) <no...@at...>]

    [ http://opensource.atlassian.com/projects/spring/browse/RCP-483?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_26518 ] 

Peter De Bruycker commented on RCP-483:
---------------------------------------

if we add code to remove the view from the ApplicationEventMulticaster, the best place would be AbstractApplicationPage.

The code to add the view as an application listener is in DefaultViewDescriptor.createView method.

Wouldn't it be better if this code is also moved to AbstractApplicationPage (i.e. the createPageComponent method)?

> missing to remove view from ApplicationEventMulticaster when view is closed
> ---------------------------------------------------------------------------
>
>                 Key: RCP-483
>                 URL: http://opensource.atlassian.com/projects/spring/browse/RCP-483
>             Project: Spring Framework Rich Client Project
>          Issue Type: Bug
>          Components: Application Framework
>            Reporter: Christian Heilmann
>            Priority: Critical
>
> In DefaultViewDescriptor.createView() the view is added to the ApplicationEventMulticaster (ApplicationEventMulticaster.addApplicationListener(view)) as Listener.
> But when closing the view, it is missing to remove the view from the ApplicationEventMulticaster (ApplicationEventMulticaster.removeApplicationListener(view)).
> Please remove the view from the ApplicationEventMulticaster when it is closed.

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