Open blackheaven opened 11 years ago
Est-ce que c'est testé localement ?
Rapidement, malheureusement.
cf le mail de Seb pour les règles de merge, mais l'idée, c'est quand même de tester avant de seulement proposer la Pull Request ;)
C'est moi, ou cette PR non mergée est déjà en production ? Rassurez-moi ...
la production est sur master
Cool, bon réflexe ! Mais le comportement est étrange, l'erreur #24 de master ne semble plus présente ...
C'est pour cette raison qu'on a des branches (profitons du système propre de Git). Pour le #24 : je ne suis pas arrivé à le voir donc je ne peux pas dire grand chose :/
J'ai édité la PR pour donner un bon exemple. Voir le wiki pour les suivantes et le workflow ;)
merci
Testé localement de mon côté, ça marche. Il y a toujours cette histoire d'accents et d'apostrophes évoquée dans #24 . Est-il envisageable de la corriger aussi dans cette PR?
Edit: par ailleurs, je n'ai pas le problème d'accents sur une copie de AEDI/SI fraîche. C'est donc un des commits introduits ici qui rajoute cette erreur. Encodage des chaînes en BDD?
tu as tenté avec une nouvelle entrée ? (je pense que ça vient des données).
@blackheaven, merci de tester localement avant de proposer la PR la prochaine fois C'était une bonne idée d'ajouter des données, vu qu'effectivement les chaînes apparaissent comme étant url encodées désormais...
Nom: Rhéolia Description: Ce sont les inventeurs de l'eau chaude. Ils aiment les accents aigus (comme dans é), les accents graves (comme dans è), les accents circonflexes (comme dans ê), les esperluettes (comme &) et les balises
remplir les champs nécessaires Poste: Rédacteur
je vois ça
Chez moi ça marche
Même en rechargeant la page? Recherche et d& eacute ;veloppement, c'est normal?
Merci de revert le commit SERVER[REMOTE_ADDR], ça a déjà été corrigé par une PR de Seb.
en rechargeant ça plante en effet non ok
Autant pour moi j'avais omis de préciser qu'effectivement j'avais rechargé la page après insertion.
pas de soucis, tant que je peux le reproduire, je peux le corriger
c'est encodeURIComponent qui m'embête, je corrige ça
normalement les accents passent bien (j'ai testé, mais il faut le tester chez vous aussi). j'ai virer mon commit de REMOTE_ADDR.