This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 522676 - Remove the partial Java EE 8 support from JavaEEProjectUtilities.getLatestVersion()
Summary: Remove the partial Java EE 8 support from JavaEEProjectUtilities.getLatestVer...
Status: RESOLVED FIXED
Alias: None
Product: WTP Java EE Tools
Classification: WebTools
Component: jst.j2ee (show other bugs)
Version: 3.9   Edit
Hardware: PC Mac OS X
: P3 blocker with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: Carl Anderson CLA
QA Contact: Chuck Bridgham CLA
URL:
Whiteboard: JavaEE8 PMC
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-22 17:18 EDT by John Collier CLA
Modified: 2017-11-27 13:56 EST (History)
8 users (show)

See Also:
ccc: pmc_approved? (david_williams)
raghunathan.srinivasan: pmc_approved+
ccc: pmc_approved? (naci.dai)
ccc: pmc_approved? (neil.hauge)
cbridgha: pmc_approved+
ccc: pmc_approved+
vrubezhny: pmc_approved+
eyuen7: pmc_approved+
ccc: review+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description John Collier CLA 2017-09-22 17:18:08 EDT 
We need to remove the partial Java EE 8 support from the JavaEEProjectUtilities.getLatestVersion() function, as this has caused a serious regression in components that access it. This is due to incomplete Java EE 8 support throughout the rest of WTP Java EE Tools, which has not yet been committed (See Bug 518383).
Comment 1 Eclipse Genie CLA 2017-09-22 17:26:35 EDT 
New Gerrit change created: https://git.eclipse.org/r/105645
Comment 2 Carl Anderson CLA 2017-09-25 15:21:14 EDT 
Explain why you believe this is a stop-ship defect. Or, if it is a "hotbug" (requested by an adopter) please document it as such.

Adopters are running into NPEs and other exceptions due to issues exposed by partial Java EE 8 functionality.

Is there a work-around? If so, why do you believe the work-around is insufficient?

There is no work-around.

How has the fix been tested? Is there a test case attached to the bugzilla record? Has a JUnit Test been added?

The fix has been tested in the adopter code and fixes the issue.

Give a brief technical overview. Who has reviewed this fix?

As part of the partial Java EE 8 update, the default Java EE version for undefined instances was raised to Java EE 8.  We are reverting the default Java EE version back to Java EE 7.  Carl Anderson, Chuck Bridgham, and Elson Yuen have reviewed this fix.

What is the risk associated with this fix?

Extremely low risk- we are reverting back to well-tested functionality from Oxygen
Comment 3 Carl Anderson CLA 2017-09-25 15:23:29 EDT 
Note that both WTP PMC and Eclipse PMC approval will be needed to get this change into WTP 3.9.1a
Comment 5 Carl Anderson CLA 2017-09-28 14:41:25 EDT 
Committed to master for WTP 3.9.1a and WTP 3.10.0 M3