From eb0f996372ff86e2ea9cd87ad560897c77afbe80 Mon Sep 17 00:00:00 2001 From: Mikhail <99481254+DCFApixels@users.noreply.github.com> Date: Sat, 7 Sep 2024 22:17:08 +0800 Subject: [PATCH] fix inject module bug --- src/EcsPipeline.Builder.cs | 27 +++++++++++++-------------- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/src/EcsPipeline.Builder.cs b/src/EcsPipeline.Builder.cs index 268fda2..0b59a08 100644 --- a/src/EcsPipeline.Builder.cs +++ b/src/EcsPipeline.Builder.cs @@ -161,25 +161,24 @@ namespace DCFApixels.DragonECS } private Builder AddModule_Internal(IEcsModule module, AddParams settedAddParams) { - if (settedAddParams.flags.IsNoImport()) + if (settedAddParams.flags.IsNoImport() == false) { - return this; - } - AddParams prms = _defaultAddParams; - if (module is IEcsDefaultAddParams overrideInterface) - { - prms = prms.Overwrite(overrideInterface.AddParams); - } - var oldDefaultAddParams = _defaultAddParams; - _defaultAddParams = prms.Overwrite(settedAddParams); + AddParams prms = _defaultAddParams; + if (module is IEcsDefaultAddParams overrideInterface) + { + prms = prms.Overwrite(overrideInterface.AddParams); + } + var oldDefaultAddParams = _defaultAddParams; + _defaultAddParams = prms.Overwrite(settedAddParams); - module.Import(this); - if(module is IInjectionUnit injectionUnit) + module.Import(this); + _defaultAddParams = oldDefaultAddParams; + } + + if (module is IInjectionUnit injectionUnit) { Injector.Inject(injectionUnit); } - - _defaultAddParams = oldDefaultAddParams; return this; } #endregion