Cavemanon/piratepoet
8
Fork 0
Code Pull Requests Actions Releases 16 Activity

Fix PHP fatal error caused by using mailpoet_settings DB table before it's available

Browse Source 
MAILPOET-6382
This commit is contained in:
Oluwaseun Olorunsola
2024-12-12 18:39:28 +01:00
committed by Ján Mikláš
parent 1ca3044639
commit 8a4ea2d899
1 changed files with 12 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
mailpoet/lib/Config/Hooks.php
View File

@@ -159,7 +159,12 @@ class Hooks {
} }
public function setupSubscriptionEvents() { public function setupSubscriptionEvents() {
$subscribe = $this->settings->get('subscribe', []); // In some cases on multisite instance, this code may run before DB migrator and settings table is not ready at that time
try {
$subscribe = $this->settings->get('subscribe', []);
} catch (\Exception $e) {
$subscribe = [];
}
// Subscribe in comments // Subscribe in comments
if ( if (
isset($subscribe['on_comment']['enabled']) isset($subscribe['on_comment']['enabled'])
@@ -325,7 +330,12 @@ class Hooks {
} }
public function setupWooCommerceSubscriptionEvents() { public function setupWooCommerceSubscriptionEvents() {
$optInEnabled = (bool)$this->settings->get(Subscription::OPTIN_ENABLED_SETTING_NAME, false); // In some cases on multisite instance, this code may run before DB migrator and settings table is not ready at that time
try {
$optInEnabled = (bool)$this->settings->get(Subscription::OPTIN_ENABLED_SETTING_NAME, false);
} catch (\Exception $e) {
$optInEnabled = false;
}
// WooCommerce: subscribe on checkout // WooCommerce: subscribe on checkout
if ($optInEnabled) { if ($optInEnabled) {
$optInPosition = $this->settings->get(Subscription::OPTIN_POSITION_SETTING_NAME, self::DEFAULT_OPTIN_POSITION); $optInPosition = $this->settings->get(Subscription::OPTIN_POSITION_SETTING_NAME, self::DEFAULT_OPTIN_POSITION);