Discussion :

[bndd24]

Salut à tous,

Je ne comprends pas pourquoi j'ai cette erreur :

Warning: get_class() expects parameter 1 to be object, int given

alors oui, je comprends l'erreur, il veut un objet et je lui donne un entier ok mais j'ai passé 2 heures ce soir dessus sans succès !
Je veux passer un entier dans la méthode de mon repo et non un objet.

Repo :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    public function unreadCount(int $userId)
    {
        return $this->createQueryBuilder('m')
            ->where('m.recipient', $userId)
            ->groupBy('m.author')
            ->select("m.author, count(m.id) as nb")
            ->andWhere("m.readAt is NULL")
            ->getQuery()->execute(null, Query::HYDRATE_ARRAY);
 
            //->getArrayResult();
            //->getResult();
    }

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
 
    /**
     * @Route("/inbox/{user}", name="messages_show", methods={"GET"})
     */
    public function show(User $user)
    {
        $message = new Message();
 
        $form = $this->createForm(MessageType::class, $message);
 
        return $this->render('message/show.html.twig', [
            'form' => $form->createView(),
            'users' => $this->r->getMessages($this->getUser()->getId()),
            'user' => $user,
            'messages' => $this->r->getMessagesFor($this->getUser()->getId(), $user->getId()),
            'unread' => $this->r->unreadCount($this->getUser()->getId()),
        ]);
    }

mon code me semble correct... quelqu'un aurait-il une idée ?

Je vous remercie.

[Alexandre T]

Oui, tu as bien compris l'erreur.

Option 1 : Tu t'y plies et tu passes l'objet

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public function unreadCount(UserInterface $user)
    {
        return $this->createQueryBuilder('m')
            ->where('m.recipient', $user)
            ->groupBy('m.author')
            ->select("m.author, count(m.id) as nb")
            ->andWhere("m.readAt is NULL")
            ->getQuery()->execute(null, Query::HYDRATE_ARRAY);
 
            //->getArrayResult();
            //->getResult();
    }

Option 2 : Tu passes l'id que seul l'entité user connait (j'ai bien dit entité et pas table, car oui la table message connais l'user_id, mais après l'ORM l'entité message ne connait que l'objet user, pas la propriété id de l'objet user)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public function unreadCount(int $userId)
    {
        return $this->createQueryBuilder('m')
            ->join('m.recipient')            
            ->where('r.id', $userId)
             ->groupBy('m.author')
             ->select("m.author, count(m.id) as nb") 
            ->andWhere("m.readAt is NULL") 
            ->getQuery()->execute(null, Query::HYDRATE_ARRAY);
               //->getArrayResult();             
              //->getResult();     
}

L'option 1 est une meilleure pratique pratique que l'option 2, car

• tu te limites à la couche Objet, alors que dans l'option 2, tu t'encombres de la couche "persistance".
• tu évites une jointure inutile (certes, cette jointure est forcée par l'ORM, le SGBD saurait très bien s'en passer)

le développeur qui code le contrôleur lira une documentation du repository du genre : Tu dois passer l'objet utilisateur. C'est plus compréhensible que : tu dois me passer un entier. D'ici à ce que le développeur te balance, l'entier du message, que ça marche dans ces tests (ou plutôt qu'il ne voit pas le soucis), y'a pas des kilomètres. Je dis pas que c'est parti en production ce genre de bugs... Je le dis pas...

[bndd24]

oui je te rejoins la dessus, la meilleure solution est clairement l'option 1.

j'ai passé l'objet User.

Code du repo :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    public function unreadCount(UserInterface $user)
    {
        return $this->createQueryBuilder('message')
            ->where('message.recipient', $user)
            ->groupBy('message.author')
            ->select("message.author, count(message.id) as nb")
            ->andWhere("message.readAt is NULL")
            ->getQuery()->execute(null, Query::HYDRATE_ARRAY);
 
            //->getArrayResult();
            //->getResult();
    }

Code du controller (on sait jamais) :

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
 
    /**
     * @Route("/inbox/{user}", name="messages_show", methods={"GET"})
     */
    public function show(User $user)
    {
        $message = new Message();
 
        $form = $this->createForm(MessageType::class, $message);
 
        return $this->render('message/show.html.twig', [
            'form' => $form->createView(),
            'users' => $this->r->getMessages($this->getUser()->getId()),
            'user' => $user,
            'messages' => $this->r->getMessagesFor($this->getUser()->getId(), $user->getId()),
            'unread' => $this->r->unreadCount($user),
        ]);
    }

et bien sur, l'erreur affichée explique vraiment bien le problème :

Expression of type 'App\Entity\User' not allowed in this context.

Expression de type «App\Entity\User» non autorisée dans ce contexte.

J'ai une question dans un coin de la tête depuis tout à l'heure.. pourquoi dans le controller je mets "User $user" et dans le repo je mets "UserInterface $user" ? la classe User implémente la UserInterface. Il faut mettre l'un ou l'autre

PS : oui, je ne sais pas pourquoi j'ai loupé ton message hier soir.

[Alexandre T]

J'ai une question dans un coin de la tête depuis tout à l'heure.. pourquoi dans le controller je mets "User $user" et dans le repo je mets "UserInterface $user" ? la classe User implémente la UserInterface. Il faut mettre l'un ou l'autre

Le principe : De préférence on passe toujours l'interface, comme cela si tu crées une autre classe (classe étudiant, ou classe professeur, etc) qui implémente l'interface, tu n'auras pas à changer ton code.

En revanche, dans le cas d'un repository, c'est plus discutable, on peut très bien comprendre que tu passes l'entité dont tu as besoin pour la jointure.

[bndd24]

oui toujours l'interface avant.

en tout cas User ou UserInterface il me sort la même erreur depuis plus d'une semaine

J'ai beau regardé mon code ligne par ligne que je ne trouve pas le problème.

peut-être passer par une requête native ? et non avec le createQueryBuilder.

[Alexandre T]

Je t'ai mal conseillé, j'ai dû répondre un peu vite

->where('r.id', $userId) <== ça ne peut pas marcher, il faut passer un paramètre !

OPTION1

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
    public function unreadCount(UserInterface $user)
    {
        return $this->createQueryBuilder('m')


            ->select("m.author, count(m.id) as nb")
            ->where('m.recipient = :recipient')            
            ->andWhere("m.readAt is NULL") 
            ->groupBy('m.author')
            ->setParameter('recipient', $user)
            ->getQuery()->execute(null, Query::HYDRATE_ARRAY);
              //->getArrayResult();             //->getResult();     }

OPTION2

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public function unreadCount(int $userId)
    {
        return $this->createQueryBuilder('m')
            ->join('m.recipient' ,'r') //Le second parametre c'est le nom de l'alias
            ->where('r.id = :recipientId')
             ->groupBy('m.author')
             ->select("m.author, count(m.id) as nb") 
            ->andWhere("m.readAt is NULL") 
            ->setParameter('recipientId', $userId)
            ->getQuery()->execute(null, Query::HYDRATE_ARRAY);
               //->getArrayResult();             
              //->getResult();     
}