Discussion :

[Bruno13]

Bonjour à tous,

Mon but est d'envoyer un mail à chaque succès de connexion, je me suis dit rien de plus simple
J'ai donc fait:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

php bin/console make:subscriber

Je choisis le onLoginSuccess event.

Jusqu'ici tout se passe bien.

Un dd($event); s'affiche bien uniquement quand je m'identifie avec succès.

Dans mon fichier LoginSubscriber.php, j'ajoute dans la fonction ci-dessous MailerInterface $mailer

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
    public function onLoginSuccessEvent(
        LoginSuccessEvent $event,
        MailerInterface $mailer,
 
    ): void
    {
 
        dd($mailer);
 
    }

et là hélas j'obtiens un message d'erreur qui est le suivant:

App\EventSubscriber\LoginSubscriber::onLoginSuccessEvent(): Argument #2 ($mailer) must be of type Symfony\Component\Mailer\MailerInterface, string given, called in /home/bruno/workspace/pimotion/BO2/bo/vendor/symfony/event-dispatcher/EventDispatcher.php on line 270

J'ai bien dans les use:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

use Symfony\Component\Mailer\MailerInterface;

Et $mailer avec phpstorm est bien reconnu comme MailerInterface

Ai-je loupé quelque chose ? A-t-on le droit d'ajouter des paramètres dans la fonction onLoginSuccessEvent ?

D'avance merci pour vos conseils !

Bruno

[Bruno13]

ok j'ai trouvé !

Il ne faut pas passer de paramètre supplémentaire à la fonction onLoginSuccessEvent, mais utiliser la fonction magique __construct comme ci-dessous !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
class LoginSubscriber implements EventSubscriberInterface
{
 
    private MailerInterface $mailer;
 
    public function __construct(MailerInterface $mailer)
    {
        $this->mailer = $mailer;
    }
 
    public function onLoginSuccessEvent(
        LoginSuccessEvent $event
    ): void
    {
 
        dd($this->mailer);
 
    }
}