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

Use SubscribersRepository instead of Subscriber model

Browse Source 
[MAILPOET-2394]
This commit is contained in:
Amine Ben hammou
2019-11-26 21:50:32 +01:00
committed by Jack Kitterhing
parent 924486e914
commit df628ad78d
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
lib/Cron/Workers/StatsNotifications/Worker.php
View File

@ -16,6 +16,7 @@ use MailPoet\Models\ScheduledTask;
use MailPoet\Models\Subscriber; use MailPoet\Models\Subscriber;
use MailPoet\Newsletter\Statistics\NewsletterStatisticsRepository; use MailPoet\Newsletter\Statistics\NewsletterStatisticsRepository;
use MailPoet\Settings\SettingsController; use MailPoet\Settings\SettingsController;
use MailPoet\Subscribers\SubscribersRepository;
use MailPoet\Tasks\Sending; use MailPoet\Tasks\Sending;
use MailPoet\Util\License\Features\Subscribers as SubscribersFeature; use MailPoet\Util\License\Features\Subscribers as SubscribersFeature;
use MailPoet\WP\Functions as WPFunctions; use MailPoet\WP\Functions as WPFunctions;
@ -56,6 +57,9 @@ class Worker {
/** @var SubscribersFeature */ /** @var SubscribersFeature */
private $subscribers_feature; private $subscribers_feature;
/** @var SubscribersRepository */
private $subscribers_repository;
function __construct( function __construct(
Mailer $mailer, Mailer $mailer,
Renderer $renderer, Renderer $renderer,
@ -66,7 +70,8 @@ class Worker {
NewsletterLinkRepository $newsletter_link_repository, NewsletterLinkRepository $newsletter_link_repository,
NewsletterStatisticsRepository $newsletter_statistics_repository, NewsletterStatisticsRepository $newsletter_statistics_repository,
EntityManager $entity_manager, EntityManager $entity_manager,
SubscribersFeature $subscribers_feature SubscribersFeature $subscribers_feature,
SubscribersRepository $subscribers_repository
) { ) {
$this->renderer = $renderer; $this->renderer = $renderer;
$this->mailer = $mailer; $this->mailer = $mailer;
@ -78,6 +83,7 @@ class Worker {
$this->newsletter_link_repository = $newsletter_link_repository; $this->newsletter_link_repository = $newsletter_link_repository;
$this->newsletter_statistics_repository = $newsletter_statistics_repository; $this->newsletter_statistics_repository = $newsletter_statistics_repository;
$this->subscribers_feature = $subscribers_feature; $this->subscribers_feature = $subscribers_feature;
$this->subscribers_repository = $subscribers_repository;
} }
/** @throws \Exception */ /** @throws \Exception */
@ -121,7 +127,7 @@ class Worker {
$opened = ($statistics->getOpenCount() * 100) / $statistics->getTotalSentCount(); $opened = ($statistics->getOpenCount() * 100) / $statistics->getTotalSentCount();
$unsubscribed = ($statistics->getUnsubscribeCount() * 100) / $statistics->getTotalSentCount(); $unsubscribed = ($statistics->getUnsubscribeCount() * 100) / $statistics->getTotalSentCount();
$subject = $newsletter->getLatestQueue()->getNewsletterRenderedSubject(); $subject = $newsletter->getLatestQueue()->getNewsletterRenderedSubject();
$subscribers_count = Subscriber::getTotalSubscribers(); $subscribers_count = $this->subscribers_repository->getTotalSubscribers();
$context = [ $context = [
'subject' => $subject, 'subject' => $subject,
'preheader' => sprintf(_x( 'preheader' => sprintf(_x(