Ninject 2.2 multiple bindings

1.1k views Asked by At

I recently updated ASP.NET MVC 3 app to Ninject 2.2.

Previously I had the following interface to implementation binding in my main app:

Bind(typeof(IMyInterface<>)).To(typeof(MyImplementation<>)).InRequestScope();

In addition, I had the following in a different assembly that was being loaded by my main app:

var arg = new ConstructorArgument("info", "something");
Bind<IMyInterface<MyClass>>().To<MyImplementation<BlogComment>>().WithParameter(arg);

This worked fine previously and the more specific implementation (the one with the argument) was being recognized. However, when I upgraded to Ninject 2.2, I received the following error:

Error activating IMyInterface{MyClass}
More than one matching bindings are available.
Activation path:
 2) Injection of dependency IMyInterface{MyClass} into parameter myParam of constructor of type SomeOtherClass
 1) Request for IMyInterface

Suggestions:
 1) Ensure that you have defined a binding for IMyInterface{MyClass} only once.

What change was made from 2.0 to 2.2 that is causing this and is there a work around?

1

There are 1 answers

5
Remo Gloor On BEST ANSWER

Ninject 2.2 ensures that only one matching bindings exists when resolving instances. 2.0 returned an instance of the first matching binding ignoring that there are others. But having multiple bindings if only one is requested reflects a bad configuration and can lead to hard to detect unintended behaviors.

But I see that there should be the possibility to overrule open generic bindings with more specific ones. I'll definitely look into it and it will either be added to a bugfix release or the next major release.