Fix Ninject for .NET Standard to not fall over in the constructor strategy when doing default bindings

[einari]

No description provided.

[einari]

Turns out that this is a bug in Ninject - see here

[einari]

Adding a workaround for it by replacing the ConstructorReflectionStrategy through the Components registrations in Ninject by a custom one that leads to a custom ConstructorInjectionDirective that ultimately leads to a custom ParameterTarget which deals with the bug. This way we get to move forward and can rely on the default package out there for version 4 of Ninject.

Considering a pull request back to Ninject.

In Container.cs this will ultimately be:

kernel.Components.Remove<global::Ninject.Planning.Strategies.IPlanningStrategy, global::Ninject.Planning.Strategies.ConstructorReflectionStrategy>();
kernel.Components.Add<global::Ninject.Planning.Strategies.IPlanningStrategy, ConstructorReflectionStrategy>();