AbstractFunction::setParams() extracts allowed keys from the given array if they are set instead of enforcing them #25

Author: gregor-j

src/AbstractFunction.php

@@ -267,14 +267,9 @@ public function setParam(string $key, float|int|bool|array|string $value): Abstr
     public function setParams(array $params): IFunction
     {
         foreach ($this->getAllowedKeys() as $key) {
-            if (!array_key_exists($key, $params)) {
-                throw new InvalidArgumentException(sprintf(
-                    '%s is missing parameter key %s!',
-                    static::class,
-                    $key
-                ));
+            if (array_key_exists($key, $params)) {
+                $this->set($key, $params[$key]);
             }
-            $this->set($key, $params[$key]);
         }
         return $this;
     }

tests/AbstractFunctionTest.php

@@ -535,12 +535,7 @@ public static function provideInvalidJson(): array
                 . '"params":{"DbF6y6oE":"CLsVlAje"}}'
             ],
             ['7072.8'],
-            ['true'],
-            [
-                '{"name":"gYcOqTNx",'
-                . '"api":[{"type":"string","name":"gixpbEZj","direction":"input","optional":false}],'
-                . '"params":{"8rszwVLE":"dUK6wkp0"}}'
-            ]
+            ['true']
         ];
     }