From 95f162592b2bf0d078ebb5caae7438c35b7dd892 Mon Sep 17 00:00:00 2001 From: wxa Date: Mon, 4 Feb 2019 15:49:01 +0300 Subject: [PATCH] Move WooCommerceHelper instantiation to constructor [MAILPOET-1723] --- lib/Analytics/Reporter.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/lib/Analytics/Reporter.php b/lib/Analytics/Reporter.php index da85a2ca3b..a15c650891 100644 --- a/lib/Analytics/Reporter.php +++ b/lib/Analytics/Reporter.php @@ -15,8 +15,12 @@ class Reporter { /** @var SettingsController */ private $settings; + /** @var WooCommerceHelper */ + private $woocommerce_helper; + public function __construct(SettingsController $settings) { $this->settings = $settings; + $this->woocommerce_helper = new WooCommerceHelper; } function getData() { @@ -27,8 +31,7 @@ class Reporter { $checker = new ServicesChecker(); $bounceAddress = $this->settings->get('bounce.address'); $segments = Segment::getAnalytics(); - $woocommerce_helper = new WooCommerceHelper; - $has_wc = $woocommerce_helper->isWooCommerceActive(); + $has_wc = $this->woocommerce_helper->isWooCommerceActive(); $wc_customers_count = 0; if($has_wc) { $wc_customers_count = (int)Newsletter::rawQuery(