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

Bug 116469

Summary: [UI] Java facet needs install page for source and output dir config
Product: [WebTools] WTP Common Tools Reporter: Konstantin Komissarchik <konstantin>
Component: wst.commonAssignee: Konstantin Komissarchik <konstantin>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: cbridgha, jsholl, kaloyan
Version: 1.0Keywords: plan
Target Milestone: 3.1 M1   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Patch for EJB tooling none

Description Konstantin Komissarchik CLA 2005-11-15 13:07:05 EST
Haven't looked to see how it's hooked up, but the configuring the java source 
directory should be left to the jst.java facet. We should create a jst.java 
facet install page.
Comment 1 Konstantin Komissarchik CLA 2006-05-12 13:47:58 EDT
Neeraj, if you want you can assign this to me and I can take care of creating an install page for the java facet...
Comment 2 Konstantin Komissarchik CLA 2008-07-24 16:16:21 EDT
Ok. This change is ready. I committed and released the changes in the common component (this includes the java facet itself). I committed changes to the j2ee component, but didn't release because there are a tone of unreleased changes in these plugins in HEAD. Will attach a patch for the ejb component. The ejb component patch requires that the j2ee component releases first.

Summary of Changes:

1. Added a wizard page to go with the java facet install action. The page allows the user to configure multiple source directories and the output folder.
2. Verified that back-and-forth synchronization between java facet and module facets worked. Had to add some extra code since existing code only handled one direction.
3. Removed the java source folder field from install pages of the web, connector, appclient and ejb facets.
Comment 3 Konstantin Komissarchik CLA 2008-07-24 16:18:53 EDT
Created attachment 108394 [details]
Patch for EJB tooling

Kaloyan,

Could you release this patch when Chuck releases changes to Java EE plugins in HEAD? I will leave this bug open until all changes are in.
Comment 4 Konstantin Komissarchik CLA 2008-08-04 12:03:16 EDT
Kaloyan,

The changes to the Java EE plugins have all been released last week. Could you review and release the EJB tools patch at this point? That's the last item left before this can be closed as completed.
Comment 5 Kaloyan Raev CLA 2008-08-04 13:10:42 EDT
Patch is committed to HEAD and released to build. 
Comment 6 Konstantin Komissarchik CLA 2008-08-04 13:13:51 EDT
Thanks, Kaloyan!
Comment 7 Konstantin Komissarchik CLA 2008-10-07 20:36:28 EDT
Closing.