This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 223460 - EAR 5 navigator tree does not show bundled libraries properly
Summary: EAR 5 navigator tree does not show bundled libraries properly
Status: CLOSED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.0 M6   Edit
Assignee: Dimitar Giormov CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-03-21 02:39 EDT by Kaloyan Raev CLA
Modified: 2008-05-29 12:56 EDT (History)
1 user (show)

See Also:


Attachments
patch (23.52 KB, patch)
2008-03-24 03:57 EDT, Dimitar Giormov CLA
no flags Details | Diff
improved patch for bundled libs. (26.23 KB, patch)
2008-03-24 08:49 EDT, Dimitar Giormov CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kaloyan Raev CLA 2008-03-21 02:39:49 EDT
Build: I2008032015123u

Steps to reproduce:
1. Create an EAR 5 project. 
2. Call the EJB 3.0 project wizard. 
3. Choose to add the EJB module to the EAR project on the first page. 
4. Leave the "Create an EJB Client..." checkbox as is - checked - on the second page. 
5. Expand the EAR Deployment Descriptor tree in the Project Navigator. 

The EJB project is split in two parts: and EJB module and EJB Client Java project. At the moment both are displayed under "Modules" node, but EJB Client should be displayed under the "Bundled library" node. 

All kind of JARs that are not Java EE modules and are referenced in the J2EE Module Dependencies property page of the EAR should be displayed under the "Bundled library" node. This does not depend on whether the "In Lib Dir" checkbox is checked. However, if the "In Lib Dir" checkbox is checked this somehow should reflect in the DD tree, but this is out of the scope of this bug. 

In addition the label "Bundled library" should be renamed to "Bundled Libraries".
Comment 1 Kaloyan Raev CLA 2008-03-21 02:41:46 EDT
Dimitar, please have a look. 
Comment 2 Dimitar Giormov CLA 2008-03-24 03:57:01 EDT
Created attachment 93254 [details]
patch

fixes the bundled libs node
Added Library Directory as subdirectory of bundled libraries. It represents all libraries that will be packed in default library folder.
Comment 3 Dimitar Giormov CLA 2008-03-24 08:49:50 EDT
Created attachment 93263 [details]
improved patch for bundled libs.

Sorter is added and bug where if in lib dir option is used the tree could not expand.
Comment 4 Kaloyan Raev CLA 2008-03-24 09:38:51 EDT
Thanks for the patches. Dropped and released to HEAD. 
Comment 5 Kaloyan Raev CLA 2008-04-07 07:48:37 EDT
Verified with S-3.0M6-20080404175400
Comment 6 Kaloyan Raev CLA 2008-05-29 12:56:39 EDT
Closing