diff --git a/mailpoet/lib/Tasks/Sending.php b/mailpoet/lib/Tasks/Sending.php index f584d7f836..67bcdd8ad1 100644 --- a/mailpoet/lib/Tasks/Sending.php +++ b/mailpoet/lib/Tasks/Sending.php @@ -39,9 +39,6 @@ class Sending { /** @var SendingQueue */ private $queue; - /** @var Subscribers */ - private $taskSubscribers; - private $queueFields = [ 'id', 'task_id', @@ -94,7 +91,6 @@ class Sending { $this->task = $task; $this->queue = $queue; - $this->taskSubscribers = new Subscribers($task); $this->scheduledTaskSubscribersRepository = ContainerWrapper::getInstance()->get(ScheduledTaskSubscribersRepository::class); $this->scheduledTasksRepository = ContainerWrapper::getInstance()->get(ScheduledTasksRepository::class); $this->sendingQueuesRepository = ContainerWrapper::getInstance()->get(SendingQueuesRepository::class); @@ -245,10 +241,6 @@ class Sending { return $this->task; } - public function taskSubscribers() { - return $this->taskSubscribers; - } - public function getSubscribers($processed = null) { if (is_null($processed)) { $subscribers = $this->scheduledTaskSubscribersRepository->findBy(['task' => $this->task->id]); diff --git a/mailpoet/lib/Tasks/Subscribers.php b/mailpoet/lib/Tasks/Subscribers.php deleted file mode 100644 index fc0e61bef7..0000000000 --- a/mailpoet/lib/Tasks/Subscribers.php +++ /dev/null @@ -1,20 +0,0 @@ -task = $task; - } - - public function getSubscribers() { - return ScheduledTaskSubscriber::where('task_id', $this->task->id); - } -} diff --git a/mailpoet/tests/integration/Tasks/SendingTest.php b/mailpoet/tests/integration/Tasks/SendingTest.php index 5cc464d0b4..97ea484445 100644 --- a/mailpoet/tests/integration/Tasks/SendingTest.php +++ b/mailpoet/tests/integration/Tasks/SendingTest.php @@ -10,7 +10,6 @@ use MailPoet\Models\SendingQueue; use MailPoet\Newsletter\Sending\ScheduledTasksRepository; use MailPoet\Newsletter\Sending\ScheduledTaskSubscribersRepository; use MailPoet\Tasks\Sending as SendingTask; -use MailPoet\Tasks\Subscribers; use MailPoet\Test\DataFactories\Newsletter as NewsletterFactory; use MailPoet\Test\DataFactories\Subscriber as SubscriberFactory; use MailPoet\WP\Functions as WPFunctions; @@ -66,7 +65,6 @@ class SendingTest extends \MailPoetTest { verify($sending)->instanceOf(SendingTask::class); verify($sending->queue())->instanceOf(SendingQueue::class); verify($sending->task())->instanceOf(ScheduledTask::class); - verify($sending->taskSubscribers())->instanceOf(Subscribers::class); } public function testItCanBeConstructedOnlyWithAProperTaskType() {