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

Inject services checker

Browse Source 
[MAILPOET-2903]
This commit is contained in:
Jan Jakeš
2020-05-25 13:02:33 +02:00
committed by Veljko V
parent 4279e642b5
commit 47ccd0d3ea
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lib/Analytics/Reporter.php
View File

@@ -21,6 +21,9 @@ class Reporter {
/** @var SegmentsRepository */ /** @var SegmentsRepository */
private $segmentsRepository; private $segmentsRepository;
/** @var ServicesChecker */
private $servicesChecker;
/** @var SettingsController */ /** @var SettingsController */
private $settings; private $settings;
@@ -33,12 +36,14 @@ class Reporter {
public function __construct( public function __construct(
NewslettersRepository $newslettersRepository, NewslettersRepository $newslettersRepository,
SegmentsRepository $segmentsRepository, SegmentsRepository $segmentsRepository,
ServicesChecker $servicesChecker,
SettingsController $settings, SettingsController $settings,
WooCommerceHelper $woocommerceHelper, WooCommerceHelper $woocommerceHelper,
WPFunctions $wp WPFunctions $wp
) { ) {
$this->newslettersRepository = $newslettersRepository; $this->newslettersRepository = $newslettersRepository;
$this->segmentsRepository = $segmentsRepository; $this->segmentsRepository = $segmentsRepository;
$this->servicesChecker = $servicesChecker;
$this->settings = $settings; $this->settings = $settings;
$this->woocommerceHelper = $woocommerceHelper; $this->woocommerceHelper = $woocommerceHelper;
$this->wp = $wp; $this->wp = $wp;
@@ -49,7 +54,6 @@ class Reporter {
$mta = $this->settings->get('mta', []); $mta = $this->settings->get('mta', []);
$newsletters = $this->newslettersRepository->getAnalytics(); $newsletters = $this->newslettersRepository->getAnalytics();
$isCronTriggerMethodWP = $this->settings->get('cron_trigger.method') === CronTrigger::METHOD_WORDPRESS; $isCronTriggerMethodWP = $this->settings->get('cron_trigger.method') === CronTrigger::METHOD_WORDPRESS;
$checker = new ServicesChecker();
$bounceAddress = $this->settings->get('bounce.address'); $bounceAddress = $this->settings->get('bounce.address');
$segments = $this->segmentsRepository->getCountsPerType(); $segments = $this->segmentsRepository->getCountsPerType();
$hasWc = $this->woocommerceHelper->isWooCommerceActive(); $hasWc = $this->woocommerceHelper->isWooCommerceActive();
@@ -81,7 +85,7 @@ class Reporter {
'Bounce email address' => !empty($bounceAddress), 'Bounce email address' => !empty($bounceAddress),
'Newsletter task scheduler (cron)' => $isCronTriggerMethodWP ? 'visitors' : 'script', 'Newsletter task scheduler (cron)' => $isCronTriggerMethodWP ? 'visitors' : 'script',
'Open and click tracking' => (boolean)$this->settings->get('tracking.enabled', false), 'Open and click tracking' => (boolean)$this->settings->get('tracking.enabled', false),
'Premium key valid' => $checker->isPremiumKeyValid(), 'Premium key valid' => $this->servicesChecker->isPremiumKeyValid(),
'New subscriber notifications' => NewSubscriberNotificationMailer::isDisabled($this->settings->get(NewSubscriberNotificationMailer::SETTINGS_KEY)), 'New subscriber notifications' => NewSubscriberNotificationMailer::isDisabled($this->settings->get(NewSubscriberNotificationMailer::SETTINGS_KEY)),
'Number of standard newsletters sent in last 3 months' => $newsletters['sent_newsletters_3_months'], 'Number of standard newsletters sent in last 3 months' => $newsletters['sent_newsletters_3_months'],
'Number of standard newsletters sent in last 30 days' => $newsletters['sent_newsletters_30_days'], 'Number of standard newsletters sent in last 30 days' => $newsletters['sent_newsletters_30_days'],