Discussion :

[Yvesremort]

Bonjour à tous,

je débute sur Symonfy 4, j'essaye de faire mettre en place un envoi automatique de mail lors de la diffusion d'un article sur un blog.

Je gère l'envoi des mails avec Swift Mailer. Je souhaiterai récupérer les adresses mails des utilisateurs en base, et les passer à "setTo" mais j'ai une erreur. Je comprend celle-ci cependant je ne trouve pas de solution pour la résoudre, pouvez-vous m'aider ?

Merci d'avance pour votre temps et votre aide.



ensuite mon controller :

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
20
21
22
23
24
25
26
27
28
29
30
31
32
<?php
 
namespace App\Controller;
 
 
use App\Repository\UserRepository;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\Routing\Annotation\Route;
 
class AdminMailController extends AbstractController
{
    /**
     * @Route("/admin/mail", name="admin_mail")
     */
 
    public function index ( \Swift_Mailer $mailer,UserRepository $userRepo){
 
        $mailUser= $userRepo->findMailUsers();
        dump($mailUser);
 
        $message=(new \Swift_Message('Message de test '))
            ->setFrom('xxx@gmail.com')
            ->setTo($mailUser)
            ->setBody('Un nouvel article  est paru, il n\'attend que votre lecture !' );
        $mailer -> send($message);
        dump($message);
 
        return $this ->render(null);
    }
 
 
}

Le UserRepository :

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
<?php
 
namespace App\Repository;
 
use App\Entity\User;
use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
use Symfony\Bridge\Doctrine\RegistryInterface;
 
/**
 * @method User|null find($id, $lockMode = null, $lockVersion = null)
 * @method User|null findOneBy(array $criteria, array $orderBy = null)
 * @method User[]    findAll()
 * @method User[]    findBy(array $criteria, array $orderBy = null, $limit = null, $offset = null)
 */
class UserRepository extends ServiceEntityRepository
{
    public function __construct(RegistryInterface $registry)
    {
        parent::__construct($registry, User::class);
    }
 
    public function findMailUsers(){
        return $this ->createQueryBuilder('u')
            ->select('u.email')
            ->getQuery()
            ->getResult();
    }
 
    // /**
    //  * @return User[] Returns an array of User objects
    //  */
    /*
    public function findByExampleField($value)
    {
        return $this->createQueryBuilder('u')
            ->andWhere('u.exampleField = :val')
            ->setParameter('val', $value)
            ->orderBy('u.id', 'ASC')
            ->setMaxResults(10)
            ->getQuery()
            ->getResult()
        ;
    }
    */
 
    /*
    public function findOneBySomeField($value): ?User
    {
        return $this->createQueryBuilder('u')
            ->andWhere('u.exampleField = :val')
            ->setParameter('val', $value)
            ->getQuery()
            ->getOneOrNullResult()
        ;
    }
    */
}

Mon entité :

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
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
<?php
 
namespace App\Entity;
 
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
use Doctrine\ORM\Mapping as ORM;
use Symfony\Bridge\Doctrine\Validator\Constraints\UniqueEntity;
use Symfony\Component\Security\Core\User\UserInterface;
use Symfony\Component\Validator\Constraints as Assert;
 
/**
 * @ORM\Entity(repositoryClass="App\Repository\UserRepository")
 * @UniqueEntity(
 *     fields={"email"},
 *     message="L'email indiqué est déjà utilisé"
 * )
 */
class User implements UserInterface
{
    /**
     * @ORM\Id()
     * @ORM\GeneratedValue()
     * @ORM\Column(type="integer")
     */
    private $id;
 
    /**
     * @ORM\Column(type="string", length=255)
     * @Assert\NotBlank(message="Merci de préciser un email")
     * @Assert\Email(message="L'email n'est pas au bon format",
     *               checkMX=true)
     *
     */
    private $email;
 
    /**
     * @ORM\Column(type="string", length=255)
     * @Assert\NotBlank(message="Merci de préciser un nom d'utilisateur")
     * @Assert\Length(min="5",
     *     minMessage="Nom d'utilisateur trop court")
     *
     */
    private $username;
 
    /**
     * @ORM\Column(type="string", length=255)
     * @Assert\Length(min="8",
     *     minMessage="Le mot de passe doit faire minimum 8 caractères")
     * @Assert\NotBlank(message="Merci de préciser un mot de passe")
     * @Assert\EqualTo(propertyPath="confirm_password",
     * message="Les mots de passes doivent être identiques ")
     */
    private $password;
 
    /**
     * @Assert\NotBlank(message="Merci de confirmer le mot de passe")
     * @Assert\EqualTo(propertyPath="password",
     *     message="Les mots de passes doivent être identiques")
     */
    public $confirm_password;
 
    /**
     * @ORM\ManyToMany(targetEntity="App\Entity\Role", mappedBy="users")
     */
    private $userRoles;
 
    public function __construct()
    {
        $this->userRoles = new ArrayCollection();
    }
 
    public function getId(): ?int
    {
        return $this->id;
    }
 
    public function getEmail(): ?string
    {
        return $this->email;
    }
 
    public function setEmail(string $email): self
    {
        $this->email = $email;
 
        return $this;
    }
 
    public function getUsername(): ?string
    {
        return $this->username;
    }
 
    public function setUsername(string $username): self
    {
        $this->username = $username;
 
        return $this;
    }
 
    public function getPassword(): ?string
    {
        return $this->password;
    }
 
    public function setPassword(string $password): self
    {
        $this->password = $password;
 
        return $this;
    }
 
 
    public function getRoles()
    {
        $roles = $this ->userRoles->map(function ($role){
            return $role->getTitle();
        })->toArray();
 
        $roles[] ='ROLE_USER';
       return $roles;
    }
 
 
    public function getSalt()
    {
 
    }
 
 
    public function eraseCredentials()
    {
 
    }
 
    /**
     * @return Collection|Role[]
     */
    public function getUserRoles(): Collection
    {
        return $this->userRoles;
    }
 
    public function addUserRole(Role $userRole): self
    {
        if (!$this->userRoles->contains($userRole)) {
            $this->userRoles[] = $userRole;
            $userRole->addUser($this);
        }
 
        return $this;
    }
 
    public function removeUserRole(Role $userRole): self
    {
        if ($this->userRoles->contains($userRole)) {
            $this->userRoles->removeElement($userRole);
            $userRole->removeUser($this);
        }
 
        return $this;
    }
}

[_Thomas]

Salut,

D'après le message d'erreur, tu passes un tableau de valeur sous la forme suivante :

Code php :

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

['email' => 'test@test.com']

Or d'après la doc (voir ici), "setTo" attend un tableau associatif dont chaque clé est une adresse mail et la valeur un alias "Prénom Nom" (facultatif), donc que le tableau passé ressemble à quelque chose comme ceci :

Code php :

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

['test@test.com', 'test2@test2.com' => 'test2@test2.com', 'test3@test3.com' => 'Prénom Nom',]

(les trois exemples devraient être valides, le 1 et le 2 sont équivalents ; à toi de voir ce que tu préfères)

Tu peux donc boucler sur les valeurs de ton tableau actuel $mailUser= $userRepo->findMailUsers(); pour créer un nouveau tableau correspond à la structure ci-dessus attendue par "setTo".

[Yvesremort]

Merci Thomas pour les conseils et encore une fois pour ton temps.

J'ai mis en place un double foreach () mais je n'obtiens qu'un seul retour au lieu de la liste de mail, je pense pourtant ne pas me tromper.

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
 public function index ( \Swift_Mailer $mailer, UserRepository $userRepo)
    {
        $listMail = $userRepo->findMailUsers();
        //dump($listmail);
        foreach ($listMail as $email) {
            //  dump($mail);
            foreach ($email as $mailingList){
                dump($mailingList);
            }
        }
                $message = (new \Swift_Message('Message de test en provenance d\'test'))
                    ->setFrom('test.com')
                    ->setTo('toto@toto.com')
                    // ->setTo()
                    ->setBody('Un nouvel article test est paru, il n\'attend que votre lecture ! ');
                $mailer->send($message);
                dump($message);
                return $this->render('admin/mail/mail.html.twig');
    }

[_Thomas]

D'après le code du UserRepository, findMailUsers retourne des objets de type User sur lesquels il faut ensuite boucler pour extraire les adresses mails (comme ceci par exemple) :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
public function index ( \Swift_Mailer $mailer, UserRepository $userRepo)
{
    $users = $userRepo->findMailUsers();
    $destinataires = [];
    foreach ($users as $user) {
        $destinataires[] = $user['email'];
    }
 
    $message = (new \Swift_Message('Message de test en provenance d\'test'))
        ->setFrom('test.com')
        ->setTo($destinataires)
        ->setBody('Un nouvel article test est paru, il n\'attend que votre lecture ! ');
    $mailer->send($message);
    return $this->render('admin/mail/mail.html.twig');
}

Si tu n'as qu'un résultat au lieu d'une liste, c'est soit que tu n'as qu'un utilisateur dans la base, soit que tu écrases la liste des adresses mail à chaque tour de boucle.

[Yvesremort]

ah oui j'ai grandement compliqué les choses... les joies du lundi !

Je vois ou j'ai fait l'erreur, c'est plus clair à présent.

Merci Thomas pour cet échange et pour ta patience.