EmailChangeSubscriber::notifyUser() - Code Metrics - Inspection of "Add event update email / password" - projet-normandie/user-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — develop ( 7ff764...a20136 )

by BENARD

created 2025-05-17 07:42 UTC

EmailChangeSubscriber::notifyUser() A

↳ Parent: EmailChangeSubscriber

Complexity

Conditions	2
Paths	9

Size

Total Lines	64
Code Lines	43

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	43
c	1
b	0
f	0
nc	9
nop	1
dl	0
loc	64
rs	9.232

How to fix Long Method

<?php

namespace ProjetNormandie\UserBundle\EventSubscriber;

use Psr\Log\LoggerInterface;
use ProjetNormandie\UserBundle\Event\EmailChangedEvent;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\Mailer\MailerInterface;
use Symfony\Component\Mime\Email;
use Symfony\Contracts\Translation\TranslatorInterface;

class EmailChangeSubscriber implements EventSubscriberInterface
{
    public function __construct(
        private readonly LoggerInterface $logger,
        private readonly MailerInterface $mailer,
        private readonly TranslatorInterface $translator
    ) {
    }

    /**
     * @return array<string, array<int, array<int, int|string>>>
     */
    public static function getSubscribedEvents(): array
    {
        return [
            EmailChangedEvent::class => [
                ['logEmailChange', 100],
                ['notifyUser', 90],
            ],
        ];
    }

    public function logEmailChange(EmailChangedEvent $event): void
    {
        $user = $event->getUser();
        $oldEmail = $event->getOldEmail();
        $newEmail = $event->getNewEmail();

        $this->logger->info(
            sprintf(
                '##EMAIL_CHANGED##[user=%s/id=%d/old=%s/new=%s]',
                $user->getUsername(),
                $user->getId(),
                $oldEmail,
                $newEmail
            ),
            [
                'user_id' => $user->getId(),
                'username' => $user->getUsername(),
                'old_email' => $oldEmail,
                'new_email' => $newEmail,
                'action' => 'email_change'
            ]
        );
    }

    public function notifyUser(EmailChangedEvent $event): void
    {
        $user = $event->getUser();
        $oldEmail = $event->getOldEmail();
        $newEmail = $event->getNewEmail();

        $this->logger->debug('Sending email change notifications', [
            'user_id' => $user->getId(),
            'username' => $user->getUsername()
        ]);

        try {
            $locale = $user->getLanguage();

            // Email to old email
            $oldEmailBody = $this->translator->trans(
                'email_change.notification_old_email',
                [
                    '%username%' => $user->getUsername(),
                    '%old_email%' => $oldEmail,
                    '%new_email%' => $newEmail
                ],
                'email',
                $locale
            );

            $emailToOld = (new Email())
                ->to($oldEmail)
                ->subject($this->translator->trans('email_change.subject', [], 'email', $locale))
                ->text($oldEmailBody)
                ->html($oldEmailBody);

            $this->mailer->send($emailToOld);

            // Email to new email
            $newEmailBody = $this->translator->trans(
                'email_change.notification_new_email',
                [
                    '%username%' => $user->getUsername(),
                    '%old_email%' => $oldEmail,
                    '%new_email%' => $newEmail
                ],
                'email',
                $locale
            );

            $emailToNew = (new Email())
                ->to($newEmail)
                ->subject($this->translator->trans('email_change.subject', [], 'email', $locale))
                ->text($newEmailBody)
                ->html($newEmailBody);

            $this->mailer->send($emailToNew);

            $this->logger->info('Email change notifications sent successfully', [
                'user_id' => $user->getId(),
                'username' => $user->getUsername()
            ]);
        } catch (\Exception $e) {
            $this->logger->error('Error sending email change notifications', [
                'user_id' => $user->getId(),
                'username' => $user->getUsername(),
                'error' => $e->getMessage(),
                'trace' => $e->getTraceAsString()
            ]);
        }
    }
}


1			<?php
2
3			namespace ProjetNormandie\UserBundle\EventSubscriber;
4
5			use Psr\Log\LoggerInterface;
6			use ProjetNormandie\UserBundle\Event\EmailChangedEvent;
7			use Symfony\Component\EventDispatcher\EventSubscriberInterface;
8			use Symfony\Component\Mailer\MailerInterface;
9			use Symfony\Component\Mime\Email;
10			use Symfony\Contracts\Translation\TranslatorInterface;
11
12			class EmailChangeSubscriber implements EventSubscriberInterface
13			{
14			public function __construct(
15			private readonly LoggerInterface $logger,
16			private readonly MailerInterface $mailer,
17			private readonly TranslatorInterface $translator
18			) {
19			}
20
21			/**
22			* @return array<string, array<int, array<int, int\|string>>>
23			*/
24			public static function getSubscribedEvents(): array
25			{
26			return [
27			EmailChangedEvent::class => [
28			['logEmailChange', 100],
29			['notifyUser', 90],
30			],
31			];
32			}
33
34			public function logEmailChange(EmailChangedEvent $event): void
35			{
36			$user = $event->getUser();
37			$oldEmail = $event->getOldEmail();
38			$newEmail = $event->getNewEmail();
39
40			$this->logger->info(
41			sprintf(
42			'##EMAIL_CHANGED##[user=%s/id=%d/old=%s/new=%s]',
43			$user->getUsername(),
44			$user->getId(),
45			$oldEmail,
46			$newEmail
47			),
48			[
49			'user_id' => $user->getId(),
50			'username' => $user->getUsername(),
51			'old_email' => $oldEmail,
52			'new_email' => $newEmail,
53			'action' => 'email_change'
54			]
55			);
56			}
57
58			public function notifyUser(EmailChangedEvent $event): void
59			{
60			$user = $event->getUser();
61			$oldEmail = $event->getOldEmail();
62			$newEmail = $event->getNewEmail();
63
64			$this->logger->debug('Sending email change notifications', [
65			'user_id' => $user->getId(),
66			'username' => $user->getUsername()
67			]);
68
69			try {
70			$locale = $user->getLanguage();
71
72			// Email to old email
73			$oldEmailBody = $this->translator->trans(
74			'email_change.notification_old_email',
75			[
76			'%username%' => $user->getUsername(),
77			'%old_email%' => $oldEmail,
78			'%new_email%' => $newEmail
79			],
80			'email',
81			$locale
82			);
83
84			$emailToOld = (new Email())
85			->to($oldEmail)
86			->subject($this->translator->trans('email_change.subject', [], 'email', $locale))
87			->text($oldEmailBody)
88			->html($oldEmailBody);
89
90			$this->mailer->send($emailToOld);
91
92			// Email to new email
93			$newEmailBody = $this->translator->trans(
94			'email_change.notification_new_email',
95			[
96			'%username%' => $user->getUsername(),
97			'%old_email%' => $oldEmail,
98			'%new_email%' => $newEmail
99			],
100			'email',
101			$locale
102			);
103
104			$emailToNew = (new Email())
105			->to($newEmail)
106			->subject($this->translator->trans('email_change.subject', [], 'email', $locale))
107			->text($newEmailBody)
108			->html($newEmailBody);
109
110			$this->mailer->send($emailToNew);
111
112			$this->logger->info('Email change notifications sent successfully', [
113			'user_id' => $user->getId(),
114			'username' => $user->getUsername()
115			]);
116			} catch (\Exception $e) {
117			$this->logger->error('Error sending email change notifications', [
118			'user_id' => $user->getId(),
119			'username' => $user->getUsername(),
120			'error' => $e->getMessage(),
121			'trace' => $e->getTraceAsString()
122			]);
123			}
124			}
125			}
126

projet-normandie / user-bundle

Push — develop ( 7ff764...a20136 )

EmailChangeSubscriber::notifyUser() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like