#94738 fix errors

SnowdogApps · Jul 8, 2024 · c89c331 · c89c331
1 parent d30d2ac
commit c89c331
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/Model/Menu/Node/Image/File.php b/Model/Menu/Node/Image/File.php
@@ -123,7 +123,11 @@ public function getImageSize(string $file): array
         $mediaDirectory = $this->filesystem->getDirectoryRead(DirectoryList::MEDIA);
         $fileFullPath = $mediaDirectory->getAbsolutePath(self::PATH . $file);
 
-        return getimagesize($fileFullPath);
+        try {
+            return getimagesize($fileFullPath);
+        } catch (Exception $e) {
+            return [];
+        }
     }
 
     private function getAbsolutePath(): string

diff --git a/Model/ResourceModel/Menu/Node.php b/Model/ResourceModel/Menu/Node.php
@@ -23,7 +23,7 @@ protected function _afterSave(\Magento\Framework\Model\AbstractModel $object)
             $nodeCustomerGroups = json_decode($nodeCustomerGroups);
         }
         $insertData = [];
-        foreach ($nodeCustomerGroups as $customerGroup) {
+        foreach ($nodeCustomerGroups ?? [] as $customerGroup) {
             $insertData[] = [
                 'node_id' => $object->getNodeId(),
                 'group_id' => $customerGroup

diff --git a/Model/ResourceModel/Menu/Node/Collection.php b/Model/ResourceModel/Menu/Node/Collection.php
@@ -28,8 +28,8 @@ public function __construct(
         AdapterInterface $connection = null,
         AbstractDb $resource = null
     ) {
-        parent::__construct($entityFactory, $logger, $fetchStrategy, $eventManager, $connection, $resource);
         $this->scopeConfig = $scopeConfig;
+        parent::__construct($entityFactory, $logger, $fetchStrategy, $eventManager, $connection, $resource);
     }
 
     protected function _construct()