yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixes how Umbraco Context is injected into DI so that LightInject doesn't try to control it's lifetime.

Browse Source
This commit is contained in:
Shannon
2019-06-20 19:10:32 +10:00
parent 5ea6ebcc66
commit f4b2ff93fc
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/Umbraco.Web/Runtime/WebInitialComposer.cs
View File

@@ -92,9 +92,9 @@ namespace Umbraco.Web.Runtime
// we should stop injecting UmbracoContext and always inject IUmbracoContextAccessor, however at the moment
// there are tons of places (controllers...) which require UmbracoContext in their ctor - so let's register
// a way to inject the UmbracoContext - and register it per-request to be more efficient
// TODO: stop doing this
composition.Register(factory => factory.GetInstance<IUmbracoContextAccessor>().UmbracoContext, Lifetime.Request);
// a way to inject the UmbracoContext - DO NOT register this as Lifetime.Request since LI will dispose the context
// in it's own way but we don't want that to happen, we manage its lifetime ourselves.
composition.Register(factory => factory.GetInstance<IUmbracoContextAccessor>().UmbracoContext);
composition.Register<IPublishedContentQuery>(factory =>
{