Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 318515 - Lib directory not being used as Default deploy path for utility projects
Summary: Lib directory not being used as Default deploy path for utility projects
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.2   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2.1   Edit
Assignee: Scott Huff CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-30 15:01 EDT by Scott Huff CLA
Modified: 2010-07-01 15:10 EDT (History)
1 user (show)

See Also:
cbridgha: review+
jsholl: review+

Attachments
Patch file alters AddComponentToEnterpriseApplicationDataModelProvider.java (2.26 KB, patch)
2010-06-30 15:01 EDT, Scott Huff CLA 		no flags Details | Diff
patch (2.36 KB, patch)
2010-06-30 23:02 EDT, Chuck Bridgham CLA 		no flags Details | Diff
Show Obsolete (1) 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 Scott Huff CLA 2010-06-30 15:01:23 EDT 
Created attachment 173146 [details]
Patch file alters AddComponentToEnterpriseApplicationDataModelProvider.java

RESOLVED by:  returning a map of utility project components to the lib directory when the default component deployment path map is generated. previously a null map was returned, resulting in "/" for all component types being added to the ear. 

Bug occurrence:
when creating a new Utility project from the new utility project wizard with "add to ear" selected, OR when adding an existing utility project to an Ear during Ear creation, the Utility project should be deployed to the lib directory by default
Comment 1 Chuck Bridgham CLA 2010-06-30 23:02:58 EDT 
Created attachment 173161 [details]
patch

Tweaked previous patch...  checking for null lib dir
Comment 2 Chuck Bridgham CLA 2010-06-30 23:03:16 EDT 
approved
Comment 3 Chuck Bridgham CLA 2010-07-01 09:54:51 EDT 
This was dropped to HEAD