Updates existing migration to address read-only scenario

barrelstrength · May 24, 2024 · 3d82820 · 3d82820
1 parent d8ea0ad
commit 3d82820
Show file tree

Hide file tree

Showing 11 changed files with 36 additions and 6 deletions.
diff --git a/CHANGELOG/CHANGELOG-META.md b/CHANGELOG/CHANGELOG-META.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.2.5 - 2024-05-24
+
+### Fixed
+
+- Fixed migration issue when read-only mode enabled
+
 ## 4.2.4 - 2024-05-18
 
 ### Added

diff --git a/CHANGELOG/CHANGELOG-REDIRECTS.md b/CHANGELOG/CHANGELOG-REDIRECTS.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.2.5 - 2024-05-24
+
+### Fixed
+
+- Fixed migration issue when read-only mode enabled
+
 ## 4.2.4 - 2024-05-18
 
 ### Added

diff --git a/CHANGELOG/CHANGELOG-SENT-EMAIL.md b/CHANGELOG/CHANGELOG-SENT-EMAIL.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.2.5 - 2024-05-24
+
+### Fixed
+
+- Fixed migration issue when read-only mode enabled
+
 ## 4.2.4 - 2024-05-18
 
 ### Fixed

diff --git a/CHANGELOG/CHANGELOG-SITEMAPS.md b/CHANGELOG/CHANGELOG-SITEMAPS.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.2.5 - 2024-05-24
+
+### Fixed
+
+- Fixed migration issue when read-only mode enabled
+
 ## 4.2.5 - 2024-05-18
 
 ### Fixed

diff --git a/CHANGELOG/CHANGELOG-TRANSACTIONAL.md b/CHANGELOG/CHANGELOG-TRANSACTIONAL.md
@@ -1,5 +1,11 @@
 # Changelog
 
+## 4.2.5 - 2024-05-24
+
+### Fixed
+
+- Fixed migration issue when read-only mode enabled
+
 ## 4.2.4 - 2024-05-18
 
 ### Fixed

diff --git a/src/forms/migrations/m211101_000000_run_install_migration.php b/src/forms/migrations/m211101_000000_run_install_migration.php
@@ -57,7 +57,7 @@ public function safeUp(): void
 
         $keyExists = Craft::$app->getProjectConfig()->get($moduleSettingsKey);
 
-        if ($keyExists) {
+        if (!$keyExists) {
             // @todo - fix default settings to import
             Craft::$app->getProjectConfig()->set($moduleSettingsKey, [
                 'defaultSidebarTab' => 'submissions',

diff --git a/src/meta/migrations/m211101_000000_run_install_migration.php b/src/meta/migrations/m211101_000000_run_install_migration.php
@@ -33,7 +33,7 @@ public function safeUp(): void
 
         $keyExists = Craft::$app->getProjectConfig()->get($moduleSettingsKey);
 
-        if ($keyExists) {
+        if (!$keyExists) {
             Craft::$app->getProjectConfig()->set($moduleSettingsKey, [
                 'renderMetadata' => true,
                 'maxMetaDescriptionLength' => 160,

diff --git a/src/redirects/migrations/m211101_000000_run_install_migration.php b/src/redirects/migrations/m211101_000000_run_install_migration.php
@@ -35,7 +35,7 @@ public function safeUp(): void
 
         $keyExists = Craft::$app->getProjectConfig()->get($moduleSettingsKey);
 
-        if ($keyExists) {
+        if (!$keyExists) {
             Craft::$app->getProjectConfig()->set($moduleSettingsKey, [
                 'matchDefinition' => self::URL_WITHOUT_QUERY_STRINGS,
                 'queryStringStrategy' => self::REMOVE_QUERY_STRINGS,

diff --git a/src/sentemail/migrations/m211101_000000_run_install_migration.php b/src/sentemail/migrations/m211101_000000_run_install_migration.php
@@ -23,7 +23,7 @@ public function safeUp(): void
 
         $keyExists = Craft::$app->getProjectConfig()->get($moduleSettingsKey);
 
-        if ($keyExists) {
+        if (!$keyExists) {
             Craft::$app->getProjectConfig()->set($moduleSettingsKey, [
                 'sentEmailsLimit' => 2500,
                 'cleanupProbability' => 1000,

diff --git a/src/sitemaps/migrations/m211101_000000_run_install_migration.php b/src/sitemaps/migrations/m211101_000000_run_install_migration.php
@@ -26,7 +26,7 @@ public function safeUp(): void
 
         $keyExists = Craft::$app->getProjectConfig()->get($moduleSettingsKey);
 
-        if ($keyExists) {
+        if (!$keyExists) {
             Craft::$app->getProjectConfig()->set($moduleSettingsKey, [
                 'enableContentQuerySitemaps' => false,
                 'enableCustomPagesSitemap' => false,

diff --git a/src/transactional/migrations/m211101_000000_run_install_migration.php b/src/transactional/migrations/m211101_000000_run_install_migration.php
@@ -45,7 +45,7 @@ public function safeUp(): void
 
         $keyExists = Craft::$app->getProjectConfig()->get($coreModuleSettingsKey);
 
-        if ($keyExists) {
+        if (!$keyExists) {
             Craft::$app->getProjectConfig()->set($coreModuleSettingsKey, [
                 'enabled' => true,
             ]);