API for processable installers & minor refactor changes

Author: arimger

Assets/Package/Plugins/Zenject/Runtime/Signals/Zenject-Signals.csproj.meta

@@ -0,0 +1,38 @@
+using System.Collections.Generic;
+using UnityEngine;
+using Zenject;
+
+namespace BroWar.Injection
+{
+    using BroWar.Common;
+
+    [AddComponentMenu("BroWar/Injection/Context Installer")]
+    public class ContextInstaller : MonoInstaller<ContextInstaller>
+    {
+        [SerializeReference, ReferencePicker(TypeGrouping = TypeGrouping.ByFlatName), ReorderableList(elementLabel: "Installer")]
+        private ISubInstaller[] installers;
+
+        protected virtual void HandleInstallers()
+        {
+            var count = installers?.Length ?? 0;
+            for (var i = 0; i < count; i++)
+            {
+                var installer = installers[i];
+                if (installer == null)
+                {
+                    LogHandler.Log($"[Injection][{name}] Installer at '{i}' is invalid.", LogType.Warning);
+                    continue;
+                }
+
+                installer.Install(Container);
+            }
+        }
+
+        public override void InstallBindings()
+        {
+            HandleInstallers();
+        }
+
+        public IReadOnlyList<ISubInstaller> Installers => installers;
+    }
+}

Assets/Package/Runtime/ContextInstaller.cs

@@ -0,0 +1,72 @@
+using UnityEngine;
+using Zenject;
+
+namespace BroWar.Injection
+{
+    using BroWar.Common;
+
+    /// <summary>
+    /// Installer that can be exposed to a parent context, e.g. from <see cref="GameObjectContext"/> to <see cref="SceneContext"/>.
+    /// </summary>
+    public abstract class ExposableInstaller : ISubInstaller
+    {
+        [SerializeField, Tooltip("If true parent container will be used for binding." +
+            " This is useful when we want to expose references outside our context.")]
+        private bool bindToParent;
+
+        private DiContainer GetParentContainer(DiContainer target)
+        {
+            var parents = target.ParentContainers;
+            if (parents == null || parents.Length == 0)
+            {
+                return null;
+            }
+
+            return parents[0];
+        }
+
+        protected abstract void OnInstall(DiContainer container);
+
+        public void Install(DiContainer container)
+        {
+            TargetContainer = container;
+            ParentContainer = GetParentContainer(container);
+            if (BindToParent)
+            {
+                if (ParentContainer != null)
+                {
+                    container = ParentContainer;
+                }
+                else
+                {
+                    LogHandler.Log("[Game] Cannot find parent container.", LogType.Warning);
+                }
+            }
+
+            OnInstall(container);
+        }
+
+        protected bool BindToParent
+        {
+            get => bindToParent;
+            set => bindToParent = value;
+        }
+
+        /// <summary>
+        /// Default <see cref="DiContainer"/> associated to "our" context.
+        /// </summary>
+        protected DiContainer TargetContainer
+        {
+            get; private set;
+        }
+
+        /// <summary>
+        /// Parent <see cref="DiContainer"/> associated with higher context 
+        /// (e.g. <see cref="SceneContext"/> if installer is placed within <see cref="GameObjectContext"/>).
+        /// </summary>
+        protected DiContainer ParentContainer
+        {
+            get; private set;
+        }
+    }
+}

…Package/Runtime/GeneralInstaller.cs.meta …Package/Runtime/ContextInstaller.cs.metaAssets/Package/Runtime/GeneralInstaller.cs.meta renamed to Assets/Package/Runtime/ContextInstaller.cs.meta Assets/Package/Runtime/GeneralInstaller.cs.meta renamed to Assets/Package/Runtime/ContextInstaller.cs.meta

@@ -2,7 +2,7 @@
 
 namespace BroWar.Injection
 {
-    public interface IMinorInstaller
+    public interface ISubInstaller
     {
         void Install(DiContainer container);
     }

Assets/Package/Runtime/ExposableInstaller.cs

@@ -0,0 +1,16 @@
+namespace BroWar.Injection
+{
+    public interface ISubProcessor<T>
+    {
+        /// <summary>
+        /// Called right before the installation process.
+        /// </summary>
+        /// <param name="data">Data used to initialize the context installer.</param>
+        void OnBeginInstallation(T data);
+        /// <summary>
+        /// Called right after the installation process.
+        /// </summary>
+        /// <param name="data">Data used to initialize the context installer.</param>
+        void OnCloseInstallation(T data);
+    }
+}