Sign upLogin

Aerendir/aws-ses-monitor-bundle

View on GitHub
Star
src/Processor/NotificationProcessor.php

Summary

Maintainability
A
2 hrs
Test Coverage

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.
Open

        BounceNotificationHandler $bounceNotificationHandler,
        ComplaintNotificationHandler $complaintNotificationHandler,
        DeliveryNotificationHandler $deliveryNotificationHandler,
        EntityManagerInterface $entityManager,
        MessageHelper $messageHelper
Severity: Minor
Found in src/Processor/NotificationProcessor.php - About 35 mins to fix

    Avoid too many return statements within this method.
    Open

                    return $this->deliveryNotificationHandler->processNotification($notificationData, $mailMessage);
    Severity: Major
    Found in src/Processor/NotificationProcessor.php - About 30 mins to fix

      Avoid too many return statements within this method.
      Open

                      return $this->complaintNotificationHandler->processNotification($notificationData, $mailMessage);
      Severity: Major
      Found in src/Processor/NotificationProcessor.php - About 30 mins to fix

        Avoid too many return statements within this method.
        Open

                        return new Response('Notification type not understood', 403);
        Severity: Major
        Found in src/Processor/NotificationProcessor.php - About 30 mins to fix

          There are no issues that match your filters.

          Category
          Status