Bug 268722 - Fix for bug 250004 causes NoSuchMethod error
Summary: Fix for bug 250004 causes NoSuchMethod error
Status: RESOLVED FIXED
Alias: None
Product: WTP Source Editing
Classification: WebTools
Component: wst.sse (show other bugs)
Version: 3.0.4   Edit
Hardware: PC Windows Vista
: P3 critical (vote)
Target Milestone: 3.0.5   Edit
Assignee: Krzysztof Daniel CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2009-03-16 04:44 EDT by Krzysztof Daniel CLA
Modified: 2009-06-01 11:47 EDT (History)
5 users (show)

See Also:


Attachments
Update of required wst version for 3_0_3 (931 bytes, patch)
2009-03-23 09:12 EDT, Krzysztof Daniel CLA
thatnitind: iplog+
Details | Diff
Update of required wst version for 3_0_maintenance (1.29 KB, patch)
2009-03-23 09:14 EDT, Krzysztof Daniel CLA
thatnitind: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Krzysztof Daniel CLA 2009-03-16 04:44:28 EDT
createComponent(IProject, boolean) was added by the patch in the /org.eclipse.wst.common.modulecore/modulecore-src/org/eclipse/wst/common/componentcore/ComponentCore.java

This method is used by /org.eclipse.jst.jee/earproject/org/eclipse/jst/jee/project/facet/EarFacetInstallDelegate.java.

The patch was released for 3.0.3 stream.

BUT

org.eclipse.jst.jee is a part of org.eclipse.jst.web_core.feature.
org.eclipse.wst.common.modulecore is a part of org.eclipse.wst.web_core.feature.

now, jst feature requires wst feature in version 3.0.0! So it is possible to have jst 3.0.3 and wst in 3.0.0. 

Since the fix for 250004 modifies the API it is necessary to adjust the dependencies.
Comment 1 David Williams CLA 2009-03-19 20:13:47 EDT
This sounds similar to (or maybe the reverse of) a problem Neil saw with Dali 2.1 ... that it would update WST but not JST and thus illustrated some incompatible change between JST and WST. 

Christopher, can you detail how you see this? I'm wondering if it's still a problem with WTP 3.0.4 (And Ganymede SR2) or are you saying that's where you see it? 



Comment 2 Krzysztof Daniel CLA 2009-03-20 04:17:00 EDT
Yes.

The problem occurred when Spring IDE was installed on Eclipse 3.4 based  application with corresponding latest WST & JST.

Spring requested very specific (not the latest) version of WST, so it was installed by P2. Then the environment was launched, newer JST specified it can work with older WST (and it is not true), so you cannot create EAR projects anymore.

I believe that correct version dependency should cause either older JST loaded or some configuration errors at _installation_ time.
Comment 3 Krzysztof Daniel CLA 2009-03-23 09:12:44 EDT
Created attachment 129587 [details]
Update of required wst version for 3_0_3
Comment 4 Krzysztof Daniel CLA 2009-03-23 09:14:00 EDT
Created attachment 129589 [details]
Update of required wst version for 3_0_maintenance
Comment 5 David Williams CLA 2009-03-23 09:42:03 EDT
I thought we got rid of the "<requires>" in feature.xml's ... at least for HEAD, (WTP 3.1 version). 

Perhaps I missed one? 
Comment 6 Krzysztof Daniel CLA 2009-03-23 09:47:01 EDT
Yes, you are right. But that change was done for HEAD and I am not sure how it will affect this issue.

But anyway, maintenance streams should be fixed, because they are not in a perfectly valid state in the HEAD.
Comment 7 Tim deBoer CLA 2009-04-07 13:42:53 EDT
We should fix the feature in this case (since it lists the version), but going forward what really should have happened is that the org.eclipse.jst.jee plugin increased the lower bound of the version dependency against org.eclipse.wst.common.modulecore when EarFacetInstallDelegate started using new API.

This is a very easy thing to miss, especially since the developer may not have even known it was new API when they added the dependency. It won't fix the problem, but at least the user will receive a failure at install time instead of during runtime at some point in the future.
Comment 8 Carl Anderson CLA 2009-04-24 10:55:27 EDT
Nitin,

    The org.eclipse.jst.web_core.feature is in sourceediting.  Can you please apply Christopher's second patch?
Comment 9 Nitin Dahyabhai CLA 2009-04-24 17:30:39 EDT
Yes, and released.
Comment 10 Nitin Dahyabhai CLA 2009-05-05 17:32:26 EDT
Marking as fixed.  Thanks, Krzysztof.