Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 365138 - Folder for deployment descriptor field does not display the correct options when there are two or more mappings containing a DD
Summary: Folder for deployment descriptor field does not display the correct options w...
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.2.5   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.3.1 P   Edit
Assignee: Roberto Sanchez Herrera CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on: 359906
Blocks: 360043
  Show dependency tree
 
Reported: 2011-11-29 16:21 EST by Roberto Sanchez Herrera CLA
Modified: 2011-12-01 16:01 EST (History)
1 user (show)

See Also:
cbridgha: review+

Attachments
Proposed patch for 3.3.1P (1.42 KB, patch)
2011-11-29 16:25 EST, Roberto Sanchez Herrera CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Roberto Sanchez Herrera CLA 2011-11-29 16:21:03 EST 
+++ This bug was initially created as a clone of Bug #359906 +++

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-11-29 16:21:52 EST 
An adopter requests to port this fix to WTP 3.3.1P
Comment 2 Roberto Sanchez Herrera CLA 2011-11-29 16:25:28 EST 
Created attachment 207692 [details]
Proposed patch for 3.3.1P
Comment 3 Chuck Bridgham CLA 2011-11-30 15:23:30 EST 
approved
Comment 4 Carl Anderson CLA 2011-12-01 16:01:01 EST 
Committed to R3_3_1_patches