From 0931d0b770f340acf816b95f0a56b01a1779c402 Mon Sep 17 00:00:00 2001
From: wxa <wxa@users.noreply.github.com>
Date: Mon, 28 Dec 2020 20:23:02 +0300
Subject: [PATCH] Keep custom unsubscribe confirmation page setting
 [MAILPOET-3242]

---
 lib/Config/Populator.php | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/lib/Config/Populator.php b/lib/Config/Populator.php
index 1f52a77806..04062be623 100644
--- a/lib/Config/Populator.php
+++ b/lib/Config/Populator.php
@@ -194,14 +194,16 @@ class Populator {
         'captcha' => $mailpoetPageId,
         'confirm_unsubscribe' => $mailpoetPageId,
       ]);
-    } elseif (
-      (empty($subscription['captcha']) || $subscription['captcha'] !== $mailpoetPageId)
-      || (empty($subscription['confirm_unsubscribe']) || $subscription['confirm_unsubscribe'] !== $mailpoetPageId)
-    ) {
+    } else {
       // For existing installations
+      $captchaPageSetting = (empty($subscription['captcha']) || $subscription['captcha'] !== $mailpoetPageId)
+        ? $mailpoetPageId : $subscription['captcha'];
+      $confirmUnsubPageSetting = empty($subscription['confirm_unsubscribe'])
+        ? $mailpoetPageId : $subscription['confirm_unsubscribe'];
+
       $this->settings->set('subscription.pages', array_merge($subscription, [
-        'captcha' => $mailpoetPageId,
-        'confirm_unsubscribe' => $mailpoetPageId,
+        'captcha' => $captchaPageSetting,
+        'confirm_unsubscribe' => $confirmUnsubPageSetting,
       ]));
     }
   }