fix - loadModule reinitializes modules if the module has already been…

src/ModuleManager.php

@@ -6,8 +6,10 @@
 
 use Laminas\EventManager\EventManager;
 use Laminas\EventManager\EventManagerInterface;
+use ReflectionClass;
 use Traversable;
 
+use function class_exists;
 use function current;
 use function get_class;
 use function is_array;
@@ -128,10 +130,26 @@ public function loadModule($module)
             $module     = current($module);
         }
 
+        // search for a specific name
         if (isset($this->loadedModules[$moduleName])) {
             return $this->loadedModules[$moduleName];
         }
 
+        // when no specific name is found, try the complete class-string
+        $verifiedModulName = $this->getVerifiedModuleName($moduleName);
+        if (isset($this->loadedModules[$verifiedModulName])) {
+            return $this->loadedModules[$verifiedModulName];
+        }
+
+        // when no class-string is found, try search for a namespace
+        if (class_exists($verifiedModulName)) {
+            $moduleReflection = new ReflectionClass($verifiedModulName);
+
+            if (isset($this->loadedModules[$moduleReflection->getNamespaceName()])) {
+                return $this->loadedModules[$moduleReflection->getNamespaceName()];
+            }
+        }
+
         /*
          * Keep track of nested module loading using the $loadFinished
          * property.
@@ -317,4 +335,21 @@ protected function attachDefaultListeners($events)
     {
         $events->attach(ModuleEvent::EVENT_LOAD_MODULES, [$this, 'onLoadModules']);
     }
+
+    /**
+     * determines the class string of the module
+     *
+     * @param string $moduleName
+     * @return string
+     */
+    private function getVerifiedModuleName($moduleName)
+    {
+        $verifiedModulName = $moduleName;
+
+        if (! class_exists($moduleName) && class_exists($moduleName . '\Module')) {
+            $verifiedModulName = $moduleName . '\Module';
+        }
+
+        return $verifiedModulName;
+    }
 }

test/ModuleManagerTest.php

@@ -100,6 +100,34 @@ public function testModuleLoadingBehavior()
         self::assertSame(1, count($modules));
     }
 
+    public function testModuleLoadingBehaviorWithModuleClassStrings()
+    {
+        $moduleManager = new ModuleManager(['SomeModule'], $this->events);
+        $this->defaultListeners->attach($this->events);
+        $modules = $moduleManager->getLoadedModules();
+        self::assertSame(0, count($modules));
+        $modules = $moduleManager->getLoadedModules(true);
+        self::assertSame(1, count($modules));
+        $moduleManager->loadModules(); // should not cause any problems
+        $moduleManager->loadModule(Module::class); // should not cause any problems
+        $modules = $moduleManager->getLoadedModules(true); // BarModule already loaded so nothing happens
+        self::assertSame(1, count($modules));
+    }
+
+    public function testModuleLoadingBehaviorWithModuleClassStringsVersion2()
+    {
+        $moduleManager = new ModuleManager([Module::class], $this->events);
+        $this->defaultListeners->attach($this->events);
+        $modules = $moduleManager->getLoadedModules();
+        self::assertSame(0, count($modules));
+        $modules = $moduleManager->getLoadedModules(true);
+        self::assertSame(1, count($modules));
+        $moduleManager->loadModules(); // should not cause any problems
+        $moduleManager->loadModule('SomeModule'); // should not cause any problems
+        $modules = $moduleManager->getLoadedModules(true); // BarModule already loaded so nothing happens
+        self::assertSame(1, count($modules));
+    }
+
     public function testConstructorThrowsInvalidArgumentException()
     {
         $this->expectException(InvalidArgumentException::class);