Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 348951 - Uncheck and check of the Java Facet in the FacetConfigurationDialog causes the Java output folder to change in the Dynamic Web Project wizard.
Summary: Uncheck and check of the Java Facet in the FacetConfigurationDialog causes th...
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.2.4   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2.5   Edit
Assignee: Rosendo Martinez CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 358652
  Show dependency tree
 
Reported: 2011-06-09 16:05 EDT by Musa CLA
Modified: 2011-09-22 15:31 EDT (History)
4 users (show)

See Also:
cbridgha: review+

Attachments
This patch contains a fix for this defect (1.95 KB, application/octet-stream)
2011-07-25 15:39 EDT, Rosendo Martinez CLA 		no flags Details
Fix for this defect (1.95 KB, patch)
2011-07-26 11:24 EDT, Rosendo Martinez CLA 		no flags Details | Diff
New patch with project creation filter (2.58 KB, patch)
2011-08-19 13:20 EDT, Rosendo Martinez CLA 		no flags Details | Diff
This patch handles all possible project cases (Java output folder content) (3.38 KB, patch)
2011-08-22 13:52 EDT, Rosendo Martinez CLA 		shr31223: iplog+
Details | Diff
Show Obsolete (3) 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 Musa CLA 2011-06-09 16:05:30 EDT 
Build Identifier: Build id: 20110324215508

The default output folder in the Dynamic Web Project changes to "build\classes" when the Java Facet is unchecked and checked again. The default output folder should remain as "WebContent\WEB-INF\classes".

Reproducible: Always

Steps to Reproduce:
1. Open the New Dynamic Web Project wizard
2. Click Modify button to open the FacetsConfigurationDialog
3. Unlock the Java Facet. uncheck and check it again.
4. Click OK on the FacetsConfigurationDialog.
5. Click next on the wizard.
6. The default output folder in the Java Config page will appear as build\classes
Comment 1 Rosendo Martinez CLA 2011-07-25 15:39:07 EDT 
Created attachment 200309 [details]
This patch contains a fix for this defect
Comment 2 Rosendo Martinez CLA 2011-07-26 11:24:57 EDT 
Created attachment 200370 [details]
Fix for this defect
Comment 3 Chuck Bridgham CLA 2011-08-17 15:43:56 EDT 
looks good
Comment 4 Rosendo Martinez CLA 2011-08-19 13:20:52 EDT 
Created attachment 201819 [details]
New patch with project creation filter
Comment 5 Rosendo Martinez CLA 2011-08-19 13:24:38 EDT 
Hi Chuck,

Thanks for review the patch. But I realized that EJB projects would be affected with this fix. So I created a new patch and I included a filer to distinguish between EJB Project creation and Dynamic Web project creation.

Could you take and look at it?
Comment 6 Rosendo Martinez CLA 2011-08-22 13:52:12 EDT 
Created attachment 201943 [details]
This patch handles all possible project cases (Java output folder content)
Comment 7 Chuck Bridgham CLA 2011-08-24 22:14:49 EDT 
thanks - approved
Comment 8 Roberto Sanchez Herrera CLA 2011-09-05 11:40:43 EDT 
Code committed to R3_2_maintenance for WTP 3.2.5
Comment 9 Roberto Sanchez Herrera CLA 2011-09-05 16:07:21 EDT 
Code committed to HEAD for WTP 3.2.4. Given that 3.3.1 is in PMC +2 at this moment, we should wait until 3.3.2 to commit to R3_3_maintenance. Therefore, I'm keeping this defect open.
Comment 10 Roberto Sanchez Herrera CLA 2011-09-22 15:31:12 EDT 
Created bug 358652 to commit the code to WTP 3.3.2. I'm resolving this defect.