Repository navigation
Dependency properties don't work with keys #119
Description
Dependency properties don't seem to work with keys.
public class Test : ITest
{
[Import(Key = "abc")]
public IDep MyDep1 { protected get; set; }
[Import(Key = "def")]
public IDep MyDep2 { protected get; set; }
public void DoAThing()
{
Console.WriteLine("Did a thing");
}
}
...
var container = new DependencyInjectionContainer();
container.Configure(c => c.ImportMember<object>(MembersThat.HaveAttribute<ImportAttribute>()));
container.Configure(c => c.Export<Dep1>().AsKeyed<IDep>("abc"));
container.Configure(c => c.Export<Dep2>().AsKeyed<IDep>("def"));
container.Configure(c => c.Export<Test>().As<ITest>());
IDep myDep = container.Locate<IDep>(withKey: "abc"); // Works
ITest test = container.Locate<ITest>(); // Fails, can't import the dependency property
Exception is:
Grace.DependencyInjection.Exceptions.LocateException: 'Could not locate Type TestConsoleApp.IDep
Activity
ipjohnson commentedon Sep 9, 2017
I see why that didn't work. The ImportMembers isn't taking into account the ImportAttribute it's only being used for filtering purposes (i.e the ImportAttribute isn't being inspected).
changes for ImportMembers for issue #119
RandomEngy commentedon Sep 11, 2017
This code is still failing for me in 6.3.0.
ipjohnson commentedon Sep 11, 2017
Hmm now I'm a little flummexed because I added the following unit test for this use case.
Maybe it's a matter of
ImportMembers()vsImportMembers<T>()Just to check the ImportAttribute we are talking about is the one found in Grace not the ImportAttribute from MEF?
RandomEngy commentedon Sep 11, 2017
Oh my bad, I was doing
ImportMember<T>instead ofImportMembers<T>. It's working now.ipjohnson commentedon Sep 12, 2017
Just to come full circle on why
ImportMember<T>didn't work, I marked the old method as Obsolete because the naming convention is different than the one inExport<T>and left the old functionality the exact same (not processing attributes as I didn't want to introduce new behavior and break older code).