Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 364499 - EJB Client project operation doesn't check supported java version when runtime set
Summary: EJB Client project operation doesn't check supported java version when runtim...
Status: RESOLVED FIXED
Alias: None
Product: WTP EJB Tools
Classification: WebTools
Component: jst.ejb (show other bugs)
Version: 3.3.1   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 3.3.2   Edit
Assignee: Roberto Sanchez Herrera CLA
QA Contact: Kaloyan Raev CLA
URL:
Whiteboard:
Keywords:
Depends on: 363973
Blocks: 364993
  Show dependency tree
 
Reported: 2011-11-22 12:45 EST by Roberto Sanchez Herrera CLA
Modified: 2011-11-28 20:46 EST (History)
2 users (show)

See Also:
cbridgha: review+


Attachments
Proposed patch (for R3_3_maintenance and HEAD) (3.46 KB, patch)
2011-11-28 13:11 EST, 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-11-22 12:45:02 EST
+++ This bug was initially created as a clone of Bug #363973 +++

Unlike most Faceted project creation operations.  The EJB Client op does not inherit the behavior of synching or reacting to a new runtime set, and checking for the latest java version supported.  This is because it inherits the "default" java version based on the initial fixed facet template.

For instance, if the EJB project version is 3.1, then it defaults to java 6.
Comment 1 Roberto Sanchez Herrera CLA 2011-11-28 13:11:26 EST
Created attachment 207616 [details]
Proposed patch (for R3_3_maintenance and HEAD)

This patch can be applied in both R3_3_maintenance (for 3.3.2) and HEAD (for 3.4)
Comment 2 Chuck Bridgham CLA 2011-11-28 13:58:29 EST
looks good
Comment 3 Chuck Bridgham CLA 2011-11-28 20:25:43 EST
Correction - this was an EJB problem.. properly categorizing
Comment 4 Chuck Bridgham CLA 2011-11-28 20:43:36 EST
committed to 3.3.2 stream
Comment 5 Chuck Bridgham CLA 2011-11-28 20:46:39 EST
and Head...