-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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.
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
Change IoC #25
Comments
Branch for Autofac here: https://github.com/OrchardCMS/Brochard/tree/Autofac |
Moving list to Trello |
Can we reopen this? I need the ability to inject a |
@sfmskywalker you want to change DI? |
Maybe we can add extension methods similar to this issue aspnet/DependencyInjection#98 |
@Jetski5822 Well, I primarily want support for Lazy and Func so yeah, the extension method solution would work too. The only other benefit of using another DI provider I can think of right now is that it might have better support for child containers. |
The current IoC container we use is limiting in it's features, i.e. it doesnt support Lazy<> and wont.
So, look to implement Autofac or DryIoC
The text was updated successfully, but these errors were encountered: