Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 357560

Summary: DataModelFacetCreationWizardPage creates listeners that are not cleaned up in the dispose method
Product: [WebTools] WTP Java EE Tools Reporter: Roberto Sanchez Herrera <shr31223>
Component: wst.webAssignee: Roberto Sanchez Herrera <shr31223>
Status: RESOLVED FIXED QA Contact: Chuck Bridgham <cbridgha>
Severity: normal    
Priority: P3 CC: ccc
Version: 3.2.4Flags: cbridgha: review+
Target Milestone: 3.2.5   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on:    
Bug Blocks: 357845    
Attachments:
Description Flags
Proposed patch none

Description Roberto Sanchez Herrera CLA 2011-09-13 16:48:12 EDT 
The class org.eclipse.wst.web.ui.internal.wizards.DataModelFacetCreationWizardPage creates listeners (IFacetedProjectListener) that are not
cleaned up in the dispose method. This can cause problem in case other classes
are using the data model (containing the faceted project this listeners are listening to) after the page is disposed. In fact, the page
should not act on the model if it has been already disposed.
Comment 1 Roberto Sanchez Herrera CLA 2011-09-13 17:14:17 EDT 
Created attachment 203305 [details]
Proposed patch

Patch to remove the listeners on dispose. The model is set to null as well,
given that this class should not act on the model after being disposed
Comment 2 Chuck Bridgham CLA 2011-09-14 10:38:35 EDT 
approved
Comment 3 Roberto Sanchez Herrera CLA 2011-09-14 12:02:49 EDT 
Code committed to R3_2_maintenance and HEAD for WTP 3.2.5 and 3.4. I'll keep this defect open to commit this code to WTP 3.3.2.
Comment 4 Roberto Sanchez Herrera CLA 2011-09-15 13:31:08 EDT 
Created bug 357845 for WTP 3.3.2. Resolving defect
Comment 5 Roberto Sanchez Herrera CLA 2011-09-15 13:31:15 EDT 
Created bug 357845 for WTP 3.3.2. Resolving defect