Fix options checks (#2)

zloyuser · web-flow · commit b60a7d44c765 · 2022-12-13T14:28:33.000+03:00
diff --git a/src/FormatFactory.php b/src/FormatFactory.php
@@ -22,7 +22,7 @@ public static function create(string $class, array $config = []): Format
                     throw new InvalidArgumentException('Materialize AVRO format "connection" config required');
                 }
 
-                if (isset($config['options']) || !is_array($config['options'])) {
+                if (isset($config['options']) && !is_array($config['options'])) {
                     throw new InvalidArgumentException('Materialize AVRO format "options" config must be array');
                 }
 
@@ -49,7 +49,7 @@ public static function create(string $class, array $config = []): Format
                     throw new InvalidArgumentException('Materialize Protobuf format "connection" config required');
                 }
 
-                if (isset($config['options']) || !is_array($config['options'])) {
+                if (isset($config['options']) && !is_array($config['options'])) {
                     throw new InvalidArgumentException('Materialize Protobuf format "options" config must be array');
                 }
 

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ public static function create(string $class, array $config = []): Format`
`22`	`22`	`throw new InvalidArgumentException('Materialize AVRO format "connection" config required');`
`23`	`23`	`}`
`24`	`24`
`25`		`- if (isset($config['options']) \|\| !is_array($config['options'])) {`
	`25`	`+ if (isset($config['options']) && !is_array($config['options'])) {`
`26`	`26`	`throw new InvalidArgumentException('Materialize AVRO format "options" config must be array');`
`27`	`27`	`}`
`28`	`28`
`@@ -49,7 +49,7 @@ public static function create(string $class, array $config = []): Format`
`49`	`49`	`throw new InvalidArgumentException('Materialize Protobuf format "connection" config required');`
`50`	`50`	`}`
`51`	`51`
`52`		`- if (isset($config['options']) \|\| !is_array($config['options'])) {`
	`52`	`+ if (isset($config['options']) && !is_array($config['options'])) {`
`53`	`53`	`throw new InvalidArgumentException('Materialize Protobuf format "options" config must be array');`
`54`	`54`	`}`
`55`	`55`