PISHPS-353: removed error when uninstalling and reinstalling plugin (#…

…842)

src/Migration/Migration1725347559MollieTags.php

@@ -3,6 +3,7 @@
 namespace Kiener\MolliePayments\Migration;
 
 use Doctrine\DBAL\Connection;
+use Doctrine\DBAL\Result;
 use Kiener\MolliePayments\Struct\Tags\AbstractTag;
 use Kiener\MolliePayments\Struct\Tags\SubscriptionTag;
 use Shopware\Core\Framework\Log\Package;
@@ -44,6 +45,13 @@ private function createTag(
         string $id,
         string $name
     ): void {
+        // migration must be able to run multiple times in succession
+        // with every install of the mollie plugin all migrations are run
+        // since tags don't count as plugin data, they're not purged and they must not be created again
+        if ($this->tagExists($connection, $id)) {
+            return;
+        }
+
         $query = <<<SQL
         INSERT INTO tag 
         (id, name, created_at, updated_at) 
@@ -61,4 +69,21 @@ private function createTag(
 
         $stmt->execute($parameters);
     }
+
+    private function tagExists(Connection $connection, string $id): bool
+    {
+        $qb = $connection->createQueryBuilder();
+        $qb->select('id')
+            ->from('tag')
+            ->where('id = :id')
+            ->setParameter('id', Uuid::fromHexToBytes($id));
+
+        $result = $qb->execute();
+
+        if ($result instanceof Result) {
+            return $result->rowCount() > 0;
+        }
+
+        return false;
+    }
 }

src/Service/SettingsService.php

@@ -82,11 +82,13 @@ public function getSettings(?string $salesChannelId = null): MollieSettingStruct
         /** @var array<mixed> $systemConfigData */
         $systemConfigData = $this->systemConfigService->get(self::SYSTEM_CONFIG_DOMAIN, $salesChannelId);
 
-        foreach ($systemConfigData as $key => $value) {
-            if (stripos($key, self::SYSTEM_CONFIG_DOMAIN) !== false) {
-                $structData[substr($key, strlen(self::SYSTEM_CONFIG_DOMAIN))] = $value;
-            } else {
-                $structData[$key] = $value;
+        if (count($systemConfigData) !== 0) {
+            foreach ($systemConfigData as $key => $value) {
+                if (stripos($key, self::SYSTEM_CONFIG_DOMAIN) !== false) {
+                    $structData[substr($key, strlen(self::SYSTEM_CONFIG_DOMAIN))] = $value;
+                } else {
+                    $structData[$key] = $value;
+                }
             }
         }