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.

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

Closed
Jetski5822 opened this issue Jun 30, 2015 · 6 comments
Closed

Change IoC #25

Jetski5822 opened this issue Jun 30, 2015 · 6 comments
Assignees

Comments

@Jetski5822
Copy link
Member

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

@Jetski5822
Copy link
Member Author

Branch for Autofac here: https://github.com/OrchardCMS/Brochard/tree/Autofac

@Jetski5822 Jetski5822 self-assigned this Jul 14, 2015
@Jetski5822 Jetski5822 added this to the DI milestone Jul 14, 2015
@Jetski5822
Copy link
Member Author

Moving list to Trello

@sfmskywalker
Copy link
Member

Can we reopen this? I need the ability to inject a Func<IEnumerable<TSomething>>. I'll submit a PR.

@Jetski5822
Copy link
Member Author

@sfmskywalker you want to change DI?

@Jetski5822
Copy link
Member Author

Maybe we can add extension methods similar to this issue aspnet/DependencyInjection#98

@sfmskywalker
Copy link
Member

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants