From b10f903744cfb912e3d257d854196119e36003cc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Jakes=CC=8C?= <jan@jakes.pro>
Date: Mon, 29 Jun 2020 16:47:34 +0200
Subject: [PATCH] Add fix for false being saved as a string

[MAILPOET-2991]
---
 lib/Config/ServicesChecker.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lib/Config/ServicesChecker.php b/lib/Config/ServicesChecker.php
index 56b3d01627..c19eb992d4 100644
--- a/lib/Config/ServicesChecker.php
+++ b/lib/Config/ServicesChecker.php
@@ -122,7 +122,8 @@ class ServicesChecker {
   public function isMailPoetAPIKeyPendingApproval(): bool {
     $mssActive = Bridge::isMPSendingServiceEnabled();
     $mssKeyValid = $this->isMailPoetAPIKeyValid();
-    $mssKeyPendingApproval = $this->settings->get('mta.mailpoet_api_key_state.data.is_approved') === false;
+    $isApproved = $this->settings->get('mta.mailpoet_api_key_state.data.is_approved');
+    $mssKeyPendingApproval = $isApproved === false || $isApproved === 'false'; // API unfortunately saves this as a string
     return $mssActive && $mssKeyValid && $mssKeyPendingApproval;
   }
 }