Merge pull request #113 from blmage/master

Switch from setup scripts to declarative schema
shoppingflux · Apr 27, 2022 · 7ecdee7 · 7ecdee7
2 parents 0904464 + 423052e
commit 7ecdee7
Show file tree

Hide file tree

Showing 11 changed files with 851 additions and 2,517 deletions.
diff --git a/Model/Account/Store/ConfigManager.php b/Model/Account/Store/ConfigManager.php
@@ -147,7 +147,15 @@ public function importStoreData(StoreInterface $store, array $data)
      */
     public function upgradeStoreData(StoreInterface $store, $moduleVersion)
     {
-        $originalConfigData = $store->getConfiguration()->getData();
+        $oldVersion = $store->getConfiguration()
+            ->getDataByKey('version');
+
+        if (
+            !empty($oldVersion)
+            && (version_compare($oldVersion, $moduleVersion) >= 0)
+        ) {
+            return false;
+        }
 
         $this->feedGeneralConfig->upgradeStoreData($store, $this, $moduleVersion);
         $this->feedExportStateConfig->upgradeStoreData($store, $this, $moduleVersion);
@@ -158,6 +166,9 @@ public function upgradeStoreData(StoreInterface $store, $moduleVersion)
 
         $this->orderGeneralConfig->upgradeStoreData($store, $this, $moduleVersion);
 
-        return $store->getConfiguration()->getData() !== $originalConfigData;
+        $store->getConfiguration()
+            ->setData('version', $moduleVersion);
+
+        return true;
     }
 }