Fix psalm (#108)

webgriffe · Jan 24, 2022 · 4e66991 · 4e66991
1 parent f4b64af
commit 4e66991
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/src/ValueHandler/ChannelPricingValueHandler.php b/src/ValueHandler/ChannelPricingValueHandler.php
@@ -13,6 +13,7 @@
 use Sylius\Component\Resource\Repository\RepositoryInterface;
 use Symfony\Component\PropertyAccess\PropertyAccessorInterface;
 use Webgriffe\SyliusAkeneoPlugin\ValueHandlerInterface;
+use Webmozart\Assert\Assert;
 
 final class ChannelPricingValueHandler implements ValueHandlerInterface
 {

diff --git a/src/ValueHandler/MetricPropertyValueHandler.php b/src/ValueHandler/MetricPropertyValueHandler.php
@@ -48,6 +48,7 @@ public function __construct(
     public function supports($subject, string $attribute, array $value): bool
     {
         if (!array_key_exists(0, $value) ||
+            !is_array($value[0]) ||
             !array_key_exists('data', $value[0])
         ) {
             return false;
@@ -56,6 +57,7 @@ public function supports($subject, string $attribute, array $value): bool
         $metricValueData = $value[0]['data'];
         if (
             $metricValueData !== null && (
+                !is_array($value[0]['data']) ||
                 !array_key_exists('amount', $value[0]['data']) ||
                 !array_key_exists('unit', $value[0]['data']) ||
                 !is_string($value[0]['data']['amount']) ||
@@ -87,6 +89,7 @@ public function handle($subject, string $attribute, array $value): void
         $hasBeenSet = false;
 
         if (!array_key_exists(0, $value) ||
+            !is_array($value[0]) ||
             !array_key_exists('data', $value[0])
         ) {
             throw new \InvalidArgumentException('Invalid data argument');
@@ -95,6 +98,7 @@ public function handle($subject, string $attribute, array $value): void
         $metricValueData = $value[0]['data'];
         if (
             $metricValueData !== null && (
+                !is_array($value[0]['data']) ||
                 !array_key_exists('amount', $value[0]['data']) ||
                 !array_key_exists('unit', $value[0]['data']) ||
                 !is_string($value[0]['data']['amount']) ||