Move the set of the curr3ent.Value to the ctor from the ExtensionCont…

…ainerRoot

castleproject#563

src/Castle.Windsor.Extensions.DependencyInjection/Scope/ExtensionContainerRootScope.cs

@@ -22,9 +22,7 @@ private ExtensionContainerRootScope() : base(null)
 
 		public static ExtensionContainerRootScope BeginRootScope()
 		{
-			var scope = new ExtensionContainerRootScope();
-			current.Value = scope;
-			return scope;
+			return new ExtensionContainerRootScope();
 		}
 	}
 }

src/Castle.Windsor.Extensions.DependencyInjection/Scope/ExtensionContainerScope.cs

@@ -23,6 +23,9 @@ namespace Castle.Windsor.Extensions.DependencyInjection.Scope
 
 	internal class ExtensionContainerScope : ILifetimeScope, IDisposable
 	{
+
+		/// <summary>Current scope for the thread. Initial scope will be set when calling BeginRootScope from a ExtensionContainerRootScope instance.</summary>
+		/// <exception cref="InvalidOperationException">Thrown when there is no scope available.</exception>
 		internal static ExtensionContainerScope Current
 		{
 			get => current.Value ?? throw new InvalidOperationException("No scope available");
@@ -38,15 +41,14 @@ protected ExtensionContainerScope(ExtensionContainerScope parent)
 			scopeCache = new ScopeCache();
 			RootScope = parent?.RootScope ?? this as ExtensionContainerRootScope;
 			this.parent = parent;
+			current.Value = this;
 		}
 
 		internal ExtensionContainerRootScope RootScope { get; }
 
 		internal static ExtensionContainerScope BeginScope()
 		{
-			var scope = new ExtensionContainerScope(Current);
-			current.Value = scope;
-			return scope;
+			return new ExtensionContainerScope(Current);
 		}
 
 		public void Dispose()