Auth\Listener\SendRegistrationNotifications

Complexity

Total Complexity

4

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	__invoke()	0	25	3

<?php
/**
 * YAWIK
 *
 * @filesource
 * @license MIT
 * @copyright  2013 - 2017 Cross Solution <http://cross-solution.de>
 */
  
/** */
namespace Auth\Listener;

use Auth\Listener\Events\AuthEvent;
use Auth\Options\ModuleOptions;
use Core\Mail\MailService;

/**
 * Listener to send registration notifications.
 * 
 * @author Mathias Gelhausen <gelhausen@cross-solution.de>
 * @todo write test
 * @since 0.30
 */
class SendRegistrationNotifications 
{

    /**
     *
     *
     * @var \Core\Mail\MailService
     */
    private $mailer;

    /**
     *
     *
     * @var \Auth\Options\ModuleOptions
     */
    private $options;

    public function __construct(MailService $mailer, ModuleOptions $options)
    {
        $this->mailer = $mailer;
        $this->options = $options;
    }

    public function __invoke(AuthEvent $e)
    {
        /* @var \Core\Mail\HTMLTemplateMessage $mail */
        $tmpl = sprintf(
            'auth/mail/%s.%s',
            $e->getName() == AuthEvent::EVENT_USER_REGISTERED

The method getName does only exist in Auth\Listener\Events\AuthEvent, but not in Exception.

                ? 'new-registration'
                : 'user-confirmed',
            $this->options->getNotificationLanguage()
        );

        $mail = $this->mailer->get('htmltemplate');
        $mail->setTemplate($tmpl);
        $mail->setVariable('user', $e->getUser());

The method getUser does only exist in Auth\Listener\Events\AuthEvent, but not in Exception.

        $mail->setTo($this->options->getNotificationEmail());
        $mail->renderBodyText(true);

        try {
            $this->mailer->send($mail);

$mail is of type object|array, but the function expects a string|object<Zend\Mail\Message>.

        } catch (\Exception $e) {

Consider adding a comment why this CATCH block is empty.

            /* Silently ignore all exceptions.
             * @todo Logging
             */
        }
    }
}