Refonte classe Afup\Site\Forum\AppelConferencier#2231
Conversation
| string $order = 's.date_soumission', | ||
| array $onlyIds = [], | ||
| ): array { | ||
| $sql = 'SELECT COUNT(co.id) as commentaires_nombre, ' . $fields . ' |
There was a problem hiding this comment.
question : c'est possible de faire autrement pour éviter l'injection SQL ?
Ou alors c'est pour le chantier migration vers doctrine ?
There was a problem hiding this comment.
Oulah je l'avais pas vu celle là ! Je vais reprendre la requête proprement avec du binding de paramètre Ting.
| public function getSpeakersBySession(int $sessionId): CollectionInterface | ||
| { | ||
| $sql = ' SELECT '; | ||
| $sql .= ' LOWER(CONCAT(SUBSTRING(c.prenom, 1, 1), c.nom)) as code, '; |
There was a problem hiding this comment.
question : est-ce que code est mappé dans l'entité Speaker ?
There was a problem hiding this comment.
Non en l'état il ne l'est pas, je vais regarder ça.
There was a problem hiding this comment.
J'ai la possibilité de mapper le champ code sur l'entité, en revanche ça va impliquer de revoir toutes les actions et templates qui utilisent les nouvelles méthodes.
Est-ce que je le fais dans cette MR ou lors de la migration Doctrine ?
There was a problem hiding this comment.
Est-ce que ce code est utilisé en l'état ?
Avant c'était un array, donc on pouvait avoir l'info.
Le plus simple n'est pas de retourner un tableau ?
There was a problem hiding this comment.
Ouais, je vais faire un array classique, ce sera plus simple à relire.
vgreb
force-pushed
the
refacto/class-appelconferencier
branch
2 times, most recently
from
44ac74d to
dd1afd0
Compare
vgreb
force-pushed
the
refacto/class-appelconferencier
branch
from
dd1afd0 to
e00e47a
Compare
2 participants
resolves #2230