Fixed wrong use of array_key_exists with ArrayObject

Author: thomasvargiu

phpstan.neon.dist

@@ -15,6 +15,5 @@ parameters:
         - "#Casting to [a-z]+ something that's already [a-z]+#"
         - '#PHPDoc tag @throws with type Interop\\Container\\Exception\\ContainerException is not subtype of Throwable#'
         - '#PHPDoc tag @throws with type ([\w\\\|]+\|)?Psr\\Container\\ContainerExceptionInterface(\|[\w\\\|]+)? is not subtype of Throwable#'
-        - '#Parameter \#2 \$search of function array_key_exists expects array, (array\|)?ArrayObject given#'
 # AbstractPluginManager::__construct() accepts more types:
         - '#Result of && is always false#'

src/AbstractFactory/ConfigAbstractFactory.php

@@ -26,10 +26,16 @@ final class ConfigAbstractFactory implements AbstractFactoryInterface
      */
     public function canCreate(\Psr\Container\ContainerInterface $container, $requestedName)
     {
-        if (! $container->has('config') || ! array_key_exists(self::class, $container->get('config'))) {
+        if (! $container->has('config')) {
             return false;
         }
+
         $config = $container->get('config');
+
+        if (! isset($config[self::class])) {
+            return false;
+        }
+
         $dependencies = $config[self::class];
 
         return is_array($dependencies) && array_key_exists($requestedName, $dependencies);
@@ -50,7 +56,7 @@ public function __invoke(\Psr\Container\ContainerInterface $container, $requeste
             throw new ServiceNotCreatedException('Config must be an array or an instance of ArrayObject');
         }
 
-        if (! array_key_exists(self::class, $config)) {
+        if (! isset($config[self::class])) {
             throw new ServiceNotCreatedException('Cannot find a `' . self::class . '` key in the config array');
         }