Azine\MailgunWebhooksBundle\Services\AzineMailgunMailerService

Complexity

Total Complexity

10

Size/Duplication

Total Lines	177
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	23	1
B	sendSpamComplaintNotification()	0	49	6
B	sendBlacklistNotification()	0	25	3

<?php


namespace Azine\MailgunWebhooksBundle\Services;

use Azine\MailgunWebhooksBundle\Services\HetrixtoolsService\HetrixtoolsServiceResponse;
use Doctrine\Common\Persistence\ManagerRegistry;
use Symfony\Component\Translation\TranslatorInterface;
use Azine\MailgunWebhooksBundle\Entity\EmailTrafficStatistics;

class AzineMailgunMailerService
{
    /**
     * @var \Swift_Mailer
     */
    private $mailer;

    /**
     * @var \Twig_Environment
     */
    private $twig;

    /**
     * @var TranslatorInterface
     */
    private $translator;

    /**
     * @var string
     */
    private $fromEmail;

    /**
     * @var string
     */
    private $ticketId;

    /**
     * @var string
     */
    private $ticketSubject;

    /**
     * @var string
     */
    private $ticketMessage;

    /**
     * @var string
     */
    private $spamAlertsRecipientEmail;

    /**
     * @var ManagerRegistry
     */
    private $managerRegistry;

    /**
     * @var int
     */
    private $sendNotificationsInterval;

    /**
     * AzineMailgunMailerService constructor.
     *
     * @param \Swift_Mailer $mailer
     * @param \Twig_Environment $twig
     * @param TranslatorInterface $translator
     * @param string $fromEmail
     * @param string $ticketId
     * @param string $ticketSubject
     * @param string $ticketMessage
     * @param string $spamAlertsRecipientEmail
     * @param ManagerRegistry $managerRegistry
     * @param int $sendNotificationsInterval
     */
    public function __construct(
        \Swift_Mailer $mailer,
        \Twig_Environment $twig,
        TranslatorInterface $translator,
        $fromEmail,
        $ticketId,
        $ticketSubject,
        $ticketMessage,
        $spamAlertsRecipientEmail,
        ManagerRegistry $managerRegistry,
        $sendNotificationsInterval
    ) {
        $this->mailer = $mailer;
        $this->twig = $twig;
        $this->translator = $translator;
        $this->fromEmail = $fromEmail;
        $this->ticketId = $ticketId;
        $this->ticketSubject = $ticketSubject;
        $this->ticketMessage = $ticketMessage;
        $this->spamAlertsRecipientEmail = $spamAlertsRecipientEmail;
        $this->managerRegistry = $managerRegistry;
        $this->sendNotificationsInterval = $sendNotificationsInterval;
    }

    /**
     * @param string $eventId
     * @throws \Exception
     * @return int $messagesSent
     */
    public function  sendSpamComplaintNotification($eventId)
    {
        $messagesSent = 0;
        $failedRecipients = [];

        /** @var \Swift_Message $message */
        $message = $this->mailer->createMessage();
        $message->setTo($this->spamAlertsRecipientEmail)
            ->setFrom($this->fromEmail)
            ->setSubject($this->translator->trans('notification.spam_complaint_received'))
            ->setBody(
                $this->twig->render('@AzineMailgunWebhooks/Email/notification.html.twig', array('eventId' => $eventId, 'ticketId' => $this->ticketId)),
                'text/html'
            );

        $lastSpamReport = $this->managerRegistry->getManager()->getRepository(EmailTrafficStatistics::class)
            ->getLastByAction(EmailTrafficStatistics::SPAM_ALERT_SENT);

        if($lastSpamReport instanceof EmailTrafficStatistics) {

            $time = new \DateTime();
            $timeDiff = $time->diff($lastSpamReport->getCreated());

            if($timeDiff->i > $this->sendNotificationsInterval) {

                $messagesSent = $this->mailer->send($message, $failedRecipients);
            }

        }else{

            $messagesSent = $this->mailer->send($message, $failedRecipients);
        }

        if($messagesSent > 0) {

            $spamAlert = new EmailTrafficStatistics();
            $spamAlert->setAction(EmailTrafficStatistics::SPAM_ALERT_SENT);
            $manager = $this->managerRegistry->getManager();
            $manager->persist($spamAlert);
            $manager->flush($spamAlert);
            $manager->clear();
        }

        if($messagesSent == 0 && !empty($failedRecipients)){

            throw new \Exception('Tried to send notification about spam complaint but no messages were sent');
        }
        return $messagesSent;
    }

    /**
     * @param HetrixtoolsServiceResponse $response
     * @param string $ipAddress
     * @return int
     * @throws \Exception
     */
    public function sendBlacklistNotification(HetrixtoolsServiceResponse $response, $ipAddress)
    {
        $messagesSent = 0;

$messagesSent is not used, you could remove the assignment.

        $failedRecipients = [];

        /** @var \Swift_Message $message */
        $message = $this->mailer->createMessage();
        $message->setTo($this->spamAlertsRecipientEmail)
            ->setFrom($this->fromEmail)
            ->setSubject($this->translator->trans('notification.blacklist_received'))
            ->setBody(
                $this->twig->render('@AzineMailgunWebhooks/Email/blacklistNotification.html.twig', array('response' => $response, "ipAddress" => $ipAddress)),
                'text/html'
            )
            ->addPart($this->twig->render('@AzineMailgunWebhooks/Email/blacklistNotification.txt.twig', array('response' => $response, "ipAddress" => $ipAddress),
                'text/plain'));

The call to Twig_Environment::render() has too many arguments starting with 'text/plain'.

        $messagesSent = $this->mailer->send($message, $failedRecipients);

        if($messagesSent == 0 && !empty($failedRecipients)){

            throw new \Exception('Tried to send notification about ip is blacklisted but no messages were sent');
        }
        return $messagesSent;
    }
}