Bug 217709

Summary: Extension Point org.eclipse.wst.jsdt.core.inferrenceSupport does not load its extensions
Product: [WebTools] JSDT Reporter: Matthias Kempka <mkempka>
Component: GeneralAssignee: Phil Berkland <berkland>
Status: RESOLVED FIXED QA Contact: Phil Berkland <berkland>
Severity: normal    
Priority: P3 Keywords: contributed
Version: unspecified   
Target Milestone: 3.0 M6   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Attachments:
Description Flags
Updated extension point schema none

Description Matthias Kempka CLA 2008-02-04 12:34:47 EST
Build ID: CVS Head

Steps To Reproduce:
1. Contribute to org.eclipse.wst.jsdt.core.inferrenceSupport 
2. Notice that it's never loaded


More information:
The InferrenceManager.TAG_INFERENGINE still points to the old element name "inferenceManager", but should be "inferenceProvider"

I'm also attaching an updated extension point schema that tells users more detailled what the extension point expects.
Comment 1 Matthias Kempka CLA 2008-02-04 12:37:22 EST
Created attachment 88798 [details]
Updated extension point schema

this makes the inferenceProvider's class attribute a required element that says that contributions must implement the org.eclipse.wst.jsdt.internal.infer.InferrenceProvider
Comment 2 Phil Berkland CLA 2008-02-06 11:09:19 EST
I have fixed the name, but havent updated the schema yet.
We are still actively working on this extension point, and hope to have it relatively stable in the next week for the M5 milestone.  We will be doing refactoring so that the extension does not use internal classes.
Comment 3 Matthias Kempka CLA 2008-02-08 05:35:11 EST
Not depending on internal classes would be a great thing. Right now I'm suffering from the method InferrenceProvider.applysTo(CompilationUnitDeclaration scriptFile);
which I have to implement as InferrenceProvider, but that forces me to use CompilationUnitDeclaration that is exported to plug-in org.eclipse.wst.jsdt.compiler.tool only.

Comment 4 Phil Berkland CLA 2008-03-26 11:54:08 EDT
Applied patch to schema.