Eclipse Community Forums: TMF (Xtext) » IProposalConflictHelper not respecting Module binding

Home » Modeling » TMF (Xtext) » IProposalConflictHelper not respecting Module binding(IProposalConflictHelper not being bound to AntlrProposalConflictHelper)

IProposalConflictHelper not respecting Module binding [message #1858610]

Tue, 11 April 2023 16:38

Eclipse User

Hello,

I am creating an IdeContentProposalProvider for the ide project of my Xtext language. I have been debugging the provider to understand how it all works together. For some reason, the IProposalConflictHelper is being bound to the Null implementation (via its @ImplementedBy(IProposalConflictHelper.NullHelper.class) annotation) and not to the AntlrProposalConflictHelper as specified in the AbstractLanguageIdeModule.

I have debugged and it seems that the MethodBasedModule class correctly applies the module binding . Howver, in my IdeContentProposalCreator, the

@Inject
private IProposalConflictHelper conflictHelper;

is injected with the NullHelper. Any ideas? Do we need the @ImplementedBy annotation if the generated modules make the binding anyway?

Re: IProposalConflictHelper not respecting Module binding [message #1858613 is a reply to message #1858610]

Wed, 12 April 2023 00:14

Eclipse User

hi, please note: there are two IProposalConflictHelper in two different packages
can you provide a small reproducing unit test / java main
in domain model example i cannot reproduce.

import org.eclipse.xtext.ide.editor.contentassist.IProposalConflictHelper;

import com.google.inject.Inject;

public class Thing {

@Inject
IProposalConflictHelper h;

public static void main(String[] args) {
Thing t = new DomainmodelIdeSetup()
.createInjectorAndDoEMFRegistration().getInstance(Thing.class);
System.out.println(t.h.getClass());
}

}

Re: IProposalConflictHelper not respecting Module binding [message #1858629 is a reply to message #1858613]

Wed, 12 April 2023 11:02

Eclipse User

Hi ,
I ran the Java main you provided and got:

class org.eclipse.xtext.ide.editor.contentassist.antlr.AntlrProposalConflictHelper

so I guess the binding does work in this minimal example. However, I am still getting the Null binding when running a nested Eclipse. I enabled TRACE logging and found only one binding, the UI:

TRACE BindModule - Adding binding from interface org.eclipse.xtext.ui.editor.contentassist.IProposalConflictHelper to org.eclipse.xtext.ui.editor.contentassist.antlr.AntlrProposalConflictHelper. Declaring Method was 'public java.lang.Class<? extends org.eclipse.xtext.ui.editor.contentassist.IProposalConflictHelper> org.eclipse.emf.emfatic.xtext.ui.AbstractEmfaticUiModule.bindIProposalConflictHelper()' in Module org.eclipse.xtext.service.BindModule

seems the binding is not happening for the IDE plugin.

I am reusing the IDE implementations in the UiModule:

... 
public Class<? extends IContentProposalProvider> bindIContentProposalProvider() {
		return UiToIdeContentProposalProvider.class;
	}
	
	@SingletonBinding(eager = true)
	public Class<? extends IdeContentProposalProvider> bindIdeContentProposalProvider() {
		return EmfaticIdeCPP.class;
	}

So it seems that in the binding process, the IDE IProposalConflictHelper is not being bound correctly.

Perhaps my UiModule also needs to bind the org.eclipse.xtext.ide.editor.contentassist.IProposalConflictHelper, additionally to the org.eclipse.xtext.ui.editor.contentassist.IProposalConflictHelper?

Re: IProposalConflictHelper not respecting Module binding [message #1858632 is a reply to message #1858629]

Wed, 12 April 2023 11:39

Eclipse User

if you use eclipse ui and UiToIdeContentProposalProvider then you needs to to do the ide binding to the ui module.
the ide module is not used in ui

Re: IProposalConflictHelper not respecting Module binding [message #1858650 is a reply to message #1858632]

Thu, 13 April 2023 10:02

Eclipse User

that seems to do the trick. So my UiModule ended up having:

public Class<? extends IProposalConflictHelper> bindIdeIProposalConflictHelper() {
		return AntlrProposalConflictHelper.class;
	}
	
	public Class<? extends IPrefixMatcher> bindIdeIPrefixMatcher() {
		return FQNPrefixMatcher.class;
	}

I added the prefix matcher becuase it also uses the ImplementedBy annotation and we want to make sure it uses the FQN matcher.

Thanks for your help!

Previous Topic:	Xtext 2.31.0.M0 is out
Next Topic:	Auto format Xtext textual model

Goto Forum:

-=] Back to Top [=-

Current Time: Mon Jun 23 03:32:32 EDT 2025

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter