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

Bug 359906

Summary: Folder for deployment descriptor field does not display the correct options when there are two or more mappings containing a DD
Product: [WebTools] WTP Java EE Tools Reporter: Roberto Sanchez Herrera <shr31223>
Component: jst.j2eeAssignee: Roberto Sanchez Herrera <shr31223>
Status: RESOLVED FIXED QA Contact: Chuck Bridgham <cbridgha>
Severity: normal    
Priority: P3 CC: ccc
Version: 3.2.5Flags: cbridgha: review+
Target Milestone: 3.2.5   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on:    
Bug Blocks: 360043, 365138    
Attachments:
Description Flags
Proposed patch none

Description Roberto Sanchez Herrera CLA 2011-10-04 15:52:55 EDT
When I added the new advanced option and the Folder for deployment descriptor field, I incorrectly assumed that only one deployment descriptor file can exist in a project. But there can be cases where a project has two mappings to /, and both point to a folder that contains a DD (e.g. when using m2e-wtp). In this scenario, the Folder for deployment descriptor field only shows the first mapping found in the component file as eligible option.
Comment 1 Roberto Sanchez Herrera CLA 2011-10-04 16:49:23 EDT
Created attachment 204552 [details]
Proposed patch
Comment 2 Chuck Bridgham CLA 2011-10-05 15:26:39 EDT
looks good
Comment 3 Roberto Sanchez Herrera CLA 2011-10-05 16:48:22 EDT
Code committed to R3_2_maintenance for WTP 3.2.5. But I cannot commit to HEAD because this bug depends on bug 357866, I will resolve this bug, and clone it, and mark the clone as depending on bug 357866