[QA] Mise à jour vers Symfony 7.4 #5208
Conversation
emilschn
changed the title
emilschn
changed the title
emilschn
force-pushed
the
feature/5189-symfony-7-4
branch
from
46c45cf to
9282321
Compare
emilschn
changed the title
| $epcis = json_decode($request->get('epcis')); | ||
| $statut = $request->get('statut'); | ||
| $strEtiquettes = json_decode($request->get('etiquettes') ?? '[]'); | ||
| if (Request::METHOD_POST === $request->getMethod()) { |
There was a problem hiding this comment.
J'ai pas compris/trouvé a quel moment on peux etre dans une méthode différente de POST ?
There was a problem hiding this comment.
Dans les tests ; sinon je peux modifier les tests
There was a problem hiding this comment.
Je pense qu'il ne faut pas agir sur le code de prod pour faire fonctionner les tests (surtout que la ca signifie que les tests ne vérifient pas un comportement existant)
There was a problem hiding this comment.
C'est modifié, c'était pas grand chose
| } | ||
| $authorizedTerritories = $user->getPartnersTerritories(); | ||
| $territoryId = $request->get('territoire'); | ||
| if (Request::METHOD_POST === $request->getMethod()) { |
There was a problem hiding this comment.
Idem, je comprend pas quel point d'entrée peux amener au GET ?
There was a problem hiding this comment.
Dans les tests ; sinon je peux modifier les tests
There was a problem hiding this comment.
Je préférerais une modification des tests
There was a problem hiding this comment.
C'est modifié, c'était pas grand chose
| $user, | ||
| $filters, | ||
| (int) $request->get('offset') | ||
| (int) $request->query->get('offset') |
There was a problem hiding this comment.
C'est hors sujet mais je serais curieux de comprendre à quel moment on utilise ca ?
There was a problem hiding this comment.
On à pas de réponse à cette question si je comprends bien. Est-ce qu'on scrée un ticket à part si on veux creuser le sujet et supprimer ?
There was a problem hiding this comment.
J'ai testé dans différents cas, et j'ai supprimé du coup (ainsi que le paramètre de la fonction qui n'a pas d'utilité du coup)
tests/Functional/Controller/Back/SignalementActionControllerTest.php
Outdated
Show resolved
Hide resolved
| array $requestData, | ||
| array $expectedUserData, | ||
| ): void { | ||
| $code = $requestData['code']; |
There was a problem hiding this comment.
j'ai mis ça pour que ce soit envoyé dans l'url comme dans les cas réels ; sinon ça marche en vrai, mais pas dans les tests
There was a problem hiding this comment.
on aurait pu faire un peu plus propre avec cette ligne
$request = new Request([], $requestData, ['code' => $requestData['code']]);
(idem sur le reste des modification de ce fichier, mais c'est pas bloquant)
|
hmeneuvrier
left a comment
hmeneuvrier
left a comment
There was a problem hiding this comment.
Relecture et tests OK
Ticket
#5189
Description
Mise à jour de la version de Symfony vers 7.4
Changements apportés
$request->getpar$request->request->getou$request->query->getselon les casPré-requis
make composerTests