diff --git a/lib/Models/Newsletter.php b/lib/Models/Newsletter.php index 064b04849d..4190299b0e 100644 --- a/lib/Models/Newsletter.php +++ b/lib/Models/Newsletter.php @@ -589,7 +589,8 @@ class Newsletter extends Model { } // WooCommerce revenues - if ($woocommerce_helper->isWooCommerceActive()) { + $settings = new SettingsController(); + if ($woocommerce_helper->isWooCommerceActive() && $settings->get('display_revenues')) { $currency = $woocommerce_helper->getWoocommerceCurrency(); $row = StatisticsWooCommercePurchases::selectExpr('SUM(order_price_total) AS total') ->where([ diff --git a/tests/DataFactories/Settings.php b/tests/DataFactories/Settings.php index 637ffe8b3a..43d5334616 100644 --- a/tests/DataFactories/Settings.php +++ b/tests/DataFactories/Settings.php @@ -116,4 +116,9 @@ class Settings { $this->settings->set('deactivate_subscriber_after_inactive_days', 180); return $this; } + + function withDisplayRevenuesEnabled() { + $this->settings->set('display_revenues', true); + return $this; + } } diff --git a/tests/acceptance/NewsletterStatisticsCest.php b/tests/acceptance/NewsletterStatisticsCest.php index 23c057e0a4..007b97729a 100644 --- a/tests/acceptance/NewsletterStatisticsCest.php +++ b/tests/acceptance/NewsletterStatisticsCest.php @@ -4,6 +4,7 @@ namespace MailPoet\Test\Acceptance; require_once __DIR__ . '/../DataFactories/Newsletter.php'; require_once __DIR__ . '/../DataFactories/NewsletterLink.php'; +require_once __DIR__ . '/../DataFactories/Settings.php'; require_once __DIR__ . '/../DataFactories/StatisticsClicks.php'; require_once __DIR__ . '/../DataFactories/StatisticsWooCommercePurchases.php'; require_once __DIR__ . '/../DataFactories/Subscriber.php'; @@ -11,6 +12,7 @@ require_once __DIR__ . '/../DataFactories/WooCommerceOrder.php'; use MailPoet\Test\DataFactories\Newsletter; use MailPoet\Test\DataFactories\NewsletterLink; +use MailPoet\Test\DataFactories\Settings; use MailPoet\Test\DataFactories\StatisticsClicks; use MailPoet\Test\DataFactories\StatisticsWooCommercePurchases; use MailPoet\Test\DataFactories\Subscriber; @@ -20,6 +22,7 @@ class NewsletterStatisticsCest { function _before(\AcceptanceTester $I) { $I->activateWooCommerce(); + (new Settings())->withDisplayRevenuesEnabled(); } function showWooCommercePurchaseStatistics(\AcceptanceTester $I) {