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

Bug 369708

Summary: ModifyModulesComposite cannot block removal of modules
Product: [WebTools] WTP ServerTools Reporter: Elson Yuen <eyuen7>
Component: wst.serverAssignee: Elson Yuen <eyuen7>
Status: RESOLVED FIXED QA Contact: Elson Yuen <eyuen7>
Severity: normal    
Priority: P3 CC: ccc, stryker
Version: 3.2   
Target Milestone: 3.2.5 P   
Hardware: PC   
OS: Linux   
Whiteboard:
Bug Depends on: 368503    
Bug Blocks:    
Attachments:
Description Flags
v1.0 none

Description Elson Yuen CLA 2012-01-25 12:06:33 EST 
+++ This bug was initially created as a clone of Bug #368503 +++

The ModifyModulesComposite class automatically checks if addition of a module to a server can proceed. If it cannot (if canModifyModules returns false) it puts up an error preventing the addition of the module to the right side of the wizard. 

It does not perform this check on removal of modules. If canModifyModules(null, selectedRemoval) would return an error status, the composite never checks it and so does not block the removal. 

The result is that a user thinks they can remove the module. Upon pressing 'finish', the removal cannot proceed, and an error message comes up. 

It would be much better if the composite checks this itself, as it does for additions.
Comment 1 Elson Yuen CLA 2012-01-25 12:16:06 EST 
Created attachment 210069 [details]
v1.0

Porting changes from bug 368503 to the 3.2.5 Patch stream.
Comment 2 Carl Anderson CLA 2012-01-26 17:27:33 EST 
Committed to R3_2_5_patches