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

Bug 254992

Summary: EARVirtualComponent is storing for archive root always "/"
Product: [WebTools] WTP Java EE Tools Reporter: Dimitar Giormov <dimitar.giormov>
Component: jst.j2eeAssignee: Dimitar Giormov <dimitar.giormov>
Status: CLOSED INVALID QA Contact: Chuck Bridgham <cbridgha>
Severity: major    
Priority: P3 CC: ccc, jsholl, kaloyan, kariem, manderse, stryker
Version: 3.0.3   
Target Milestone: 3.1.2   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
initial patch none

Description Dimitar Giormov CLA 2008-11-12 02:52:45 EST
EARVirtualComponent is storing for archive root always "/" and if the archive is in a directory the name of the library is returned: "/[folder]/[archiveName]"
Comment 1 Dimitar Giormov CLA 2008-11-12 02:53:41 EST
Bug is initially found in:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=254638
Comment 2 Dimitar Giormov CLA 2008-11-12 03:06:16 EST
Created attachment 117635 [details]
initial patch

As Jason mentioned in the original bug, it is not sure if someone already relies on the logic in the current implementation. I will try to investigate that.
Comment 3 Carl Anderson CLA 2008-11-18 12:51:03 EST
Assigning this to Dimitar while he investigates it.
Comment 4 Max Rydahl Andersen CLA 2009-04-08 07:20:00 EDT
Wouldn't it be best to just fix this and see what breaks ?

Comment 5 Rob Stryker CLA 2009-04-08 22:31:53 EDT
I would imagine the unit tests would catch any regression ;) amirite? 
Comment 6 Rob Stryker CLA 2009-04-17 04:40:03 EDT
kinda funny when I think about it...  getName() is returning a URI, and getModuleURI() is returning the name. 

WEIRD. 
Comment 7 Rob Stryker CLA 2009-05-22 01:05:17 EDT
This is related to the following:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=276825
Comment 8 Dimitar Giormov CLA 2009-09-04 07:25:17 EDT
I could not reproduce the issue anymore, neither in head or 3_1_maintenance
Closing the issue.