Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 341303 - Add limited support for Project classpath entries in classpath containers
Summary: Add limited support for Project classpath entries in classpath containers
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.2.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.2.4   Edit
Assignee: Roberto Sanchez Herrera CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-29 23:33 EDT by Roberto Sanchez Herrera CLA
Modified: 2011-04-07 00:09 EDT (History)
1 user (show)

See Also:
cbridgha: review+
ccc: review+


Attachments
Proposed patch (21.00 KB, patch)
2011-03-29 23:43 EDT, Roberto Sanchez Herrera CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Roberto Sanchez Herrera CLA 2011-03-29 23:33:49 EDT
Build Identifier: WTP 3.2.3

This bug is to add limited support for project classpath entries included in classpath containers.

This support would allow classpath containers tagged as classpath dependencies  to include project classpath entries (only faceted projects)


Reproducible: Always
Comment 1 Roberto Sanchez Herrera CLA 2011-03-29 23:43:20 EDT
Created attachment 192155 [details]
Proposed patch

Summary of the changes:

- AddClasspathLibReferencesParticipant.java: Add logic to gather project classpath entries provided by child module to parent.
- J2EEModuleVirtualComponent.java: Create virtual references to project classpath entries
- ClasspathDependencyValidator.java: Do not signal an error when adding a project classpath entry that refers to a faceted project
- ClasspathDependencyUtil.java: Include project classpath entries
- IFlattenParticipant.java: Add method to get child modules that participant thinks should be added to virtual component
- AbstractFlattenParticipant.java: Default implementation for new method
- FlatVirtualComponent.java: Include child modules from participants
Comment 2 Carl Anderson CLA 2011-03-30 18:33:57 EDT
I ran the entire Java EE JUnit bucket against this patch.

I believe that this adds in minimal, required support for this scenario- something that many adopters and users are asking for.

I do have concerns about this behavioral change in a maintenance stream, but weighing that concern against the desires of the community and the minimal nature of this change, I agree to its inclusion.
Comment 3 Chuck Bridgham CLA 2011-03-30 21:02:16 EDT
I also have investigated these changes, and have run tests.  The behavior change is simply "including" referenced projects from within a mapped classpath container, where they were ignored before.  This is an important incremental step in this maintenance release, and will revisit for inclusion in later releases.
Comment 4 Carl Anderson CLA 2011-04-07 00:09:38 EDT
Committed to R3_2_maintenance and HEAD for WTP 3.2.4 and WTP 3.3 M7