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

Bug 364993

Summary: EJB Client project operation doesn't check supported java version when runtime set
Product: [WebTools] WTP EJB Tools Reporter: Roberto Sanchez Herrera <shr31223>
Component: jst.ejbAssignee: Roberto Sanchez Herrera <shr31223>
Status: RESOLVED FIXED QA Contact: Kaloyan Raev <kaloyan>
Severity: normal    
Priority: P3 CC: cbridgha, ccc
Version: 3.3.1Flags: cbridgha: review+
Target Milestone: 3.3.1 P   
Hardware: PC   
OS: Windows 7   
Whiteboard:
Bug Depends on: 363973, 364499    
Bug Blocks:    
Attachments:
Description Flags
Proposed patch
ccc: iplog+
Null check for runtime ccc: iplog+

Description Roberto Sanchez Herrera CLA 2011-11-28 13:14:31 EST 
An adopter is requesting this bug to be fixed in 3.3.1 P

+++ This bug was initially created as a clone of Bug #364499 +++

+++ 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 15:19:27 EST 
Created attachment 207624 [details]
Proposed patch

Patch created using R3_3_1
Comment 2 Chuck Bridgham CLA 2011-11-28 20:20:59 EST 
same fix reviewed earlier...  I approve
Comment 3 Chuck Bridgham CLA 2011-11-28 20:27:12 EST 
Correction - this was an EJB problem.. properly categorizing
Comment 4 Roberto Sanchez Herrera CLA 2011-11-30 16:51:44 EST 
Created attachment 207763 [details]
Null check for runtime

An NPE was detected when running the JUnits. A null check must be done for the cases where the project does not have runtime. 

Note: This patch does not make obsolete the previous patch:  it should be applied on top of it.
Comment 5 Carl Anderson CLA 2011-12-01 15:58:37 EST 
Committed to R3_3_1_patches