Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add RegisterDelegate with the list of dependencies to inject - not to Resolve #147

Closed
dadhi opened this issue Jul 11, 2019 · 1 comment
Closed

Add RegisterDelegate with the list of dependencies to inject - not to Resolve #147

dadhi opened this issue Jul 11, 2019 · 1 comment
Assignees
@dadhi
Labels
enhancement New feature or request
Milestone
v4.1.0

Comments

@dadhi
Copy link
Owner

dadhi commented Jul 11, 2019

The feature is similar to Grace ExportFactory<Dep1, Dep2, R>((d1, d2) => new R(d1, d2)).

So instead of passing the IResolverContext and relying on Service Locator anti-pattern (basically black-box) to Resolve the arbitrary dependency, we explicitly specify the dependencies to Inject.
@dadhi dadhi added the enhancement New feature or request label Jul 11, 2019
@dadhi dadhi self-assigned this Jul 11, 2019
dadhi added a commit that referenced this issue Jul 11, 2019
@dadhi
added: proof for #147
17ff51e
@dadhi dadhi changed the title Added RegisterDelegate with the list of dependencies to inject - not to Resolve Add RegisterDelegate with the list of dependencies to inject - not to Resolve Jul 15, 2019
@dadhi
Copy link
Owner Author

dadhi commented Aug 12, 2019

The problem is that we could not interpret the typed Func<T1, T2 ..., R> without reflection Invoke but we can do that for Func<object, object, ..., R>.

@dadhi dadhi closed this as completed in e258520 Oct 4, 2019
Leszek-Kowalski pushed a commit to Leszek-Kowalski/DryIoc that referenced this issue Oct 11, 2019
@dadhi @Leszek-Kowalski
added: proof for dadhi#147
fa852ad
Leszek-Kowalski pushed a commit to Leszek-Kowalski/DryIoc that referenced this issue Oct 11, 2019
@dadhi @Leszek-Kowalski
fixed: dadhi#147 Add RegisterDelegate with the list of dependencies t…
eadb836 
…o inject - not to Resolve

enhancement
@dadhi dadhi added this to the v4.1.0 milestone Dec 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dadhi dadhi

Labels
enhancement New feature or request
Projects
None yet
Milestone
v4.1.0
Development

No branches or pull requests
1 participant
@dadhi