phpstan fixes

Author: thomasvargiu

src/AbstractFactory/ReflectionBasedAbstractFactory.php

@@ -114,7 +114,9 @@ public function __invoke(ContainerInterface $container, $requestedName, array $o
     {
         $reflectionClass = new ReflectionClass($requestedName);
 
-        if (null === ($constructor = $reflectionClass->getConstructor())) {
+        $constructor = $reflectionClass->getConstructor();
+
+        if (null === $constructor) {
             return new $requestedName();
         }
 
@@ -213,7 +215,9 @@ private function resolveParameter(ReflectionParameter $parameter, ContainerInter
             return [];
         }
 
-        if (! $parameter->getClass()) {
+        $parameterClass = $parameter->getClass();
+
+        if (! $parameterClass) {
             if (! $parameter->isDefaultValueAvailable()) {
                 throw new ServiceNotFoundException(sprintf(
                     'Unable to create service "%s"; unable to resolve parameter "%s" '
@@ -226,7 +230,7 @@ private function resolveParameter(ReflectionParameter $parameter, ContainerInter
             return $parameter->getDefaultValue();
         }
 
-        $type = $parameter->getClass()->getName();
+        $type = $parameterClass->getName();
         $type = $this->aliases[$type] ?? $type;
 
         if ($container->has($type)) {

src/AbstractFactoryInterface.php

@@ -40,8 +40,8 @@ interface AbstractFactoryInterface extends Factory\AbstractFactoryInterface
      * Determine if we can create a service with name
      *
      * @param ServiceLocatorInterface $serviceLocator
-     * @param $name
-     * @param $requestedName
+     * @param string $name
+     * @param string $requestedName
      * @return bool
      */
     public function canCreateServiceWithName(ServiceLocatorInterface $serviceLocator, $name, $requestedName);
@@ -50,8 +50,8 @@ public function canCreateServiceWithName(ServiceLocatorInterface $serviceLocator
      * Create service with name
      *
      * @param ServiceLocatorInterface $serviceLocator
-     * @param $name
-     * @param $requestedName
+     * @param string $name
+     * @param string $requestedName
      * @return mixed
      */
     public function createServiceWithName(ServiceLocatorInterface $serviceLocator, $name, $requestedName);

src/InitializerInterface.php

@@ -32,7 +32,7 @@ interface InitializerInterface extends Initializer\InitializerInterface
     /**
      * Initialize
      *
-     * @param $instance
+     * @param mixed $instance
      * @param ServiceLocatorInterface $serviceLocator
      * @return mixed
      */

src/ServiceManager.php

@@ -69,7 +69,7 @@ class ServiceManager implements ServiceLocatorInterface
     /**
      * Whether or not changes may be made to this instance.
      *
-     * @param bool
+     * @var bool
      */
     protected $allowOverride = false;
 
@@ -645,7 +645,7 @@ private function createDelegatorFromName($name, array $options = null)
             };
         }
 
-        return $creationCallback($this->creationContext, $name, $creationCallback, $options);
+        return $creationCallback();
     }
 
     /**

src/Tool/ConfigDumper.php

@@ -43,7 +43,7 @@ class ConfigDumper
 EOC;
 
     /**
-     * @var ContainerInterface
+     * @var ContainerInterface|null
      */
     private $container;
 
@@ -74,11 +74,12 @@ public function createDependencyConfig(array $config, $className, $ignoreUnresol
         }
 
         // class has no constructor, treat it as an invokable
-        if (! $reflectionClass->getConstructor()) {
+        $constructor = $reflectionClass->getConstructor();
+        if (! $constructor) {
             return $this->createInvokable($config, $className);
         }
 
-        $constructorArguments = $reflectionClass->getConstructor()->getParameters();
+        $constructorArguments = $constructor->getParameters();
         $constructorArguments = array_filter(
             $constructorArguments,
             function (ReflectionParameter $argument) {
@@ -121,7 +122,7 @@ function (ReflectionParameter $argument) {
     }
 
     /**
-     * @param $className
+     * @param string $className
      * @throws InvalidArgumentException if class name is not a string or does
      *     not exist.
      */

src/Tool/FactoryCreator.php

@@ -69,7 +69,7 @@ public function createFactory($className)
     }
 
     /**
-     * @param $className
+     * @param string $className
      * @return string
      */
     private function getClassName($className)
@@ -86,11 +86,12 @@ private function getConstructorParameters($className)
     {
         $reflectionClass = new ReflectionClass($className);
 
-        if (! $reflectionClass || ! $reflectionClass->getConstructor()) {
+        $reflectionConstructor = $reflectionClass->getConstructor();
+        if (! $reflectionConstructor) {
             return [];
         }
 
-        $constructorParameters = $reflectionClass->getConstructor()->getParameters();
+        $constructorParameters = $reflectionConstructor->getParameters();
 
         if (empty($constructorParameters)) {
             return [];
@@ -120,7 +121,9 @@ function (ReflectionParameter $argument) {
         }
 
         return array_map(function (ReflectionParameter $parameter) {
-            return $parameter->getClass()->getName();
+            /** @var ReflectionClass $class */
+            $class = $parameter->getClass();
+            return $class->getName();
         }, $constructorParameters);
     }