IFAEDI / S.I

Mise en place d'un S.I avancé pour l'AEDI
3 stars 2 forks source link

Annuaire - PHP - Mauvaise stringification #24

Open benjaminplanche opened 11 years ago

benjaminplanche commented 11 years ago

[Merci d'essayer de résoudre ce probleme majeur urgemment, avec, si possible, #10 par la meme occasion.]

Contexte

L'Annuaire repose sur de nombreuses requetes Ajax pour fonctionner : récupération des données ou insertion/modifications de celles-ci. Des controleurs PHP (dossier annuaire/ajax) assurent le travail coté serveur.

Comportement

Un changement récent dans le PHP semble avoir altéré le comportement des controleurs. Les données textuelles ( string ) renvoyées dans les json sont encadrées par des apostrophes en trop ( ' ) . Ce probleme a 2 conséquences majeures :

Ce comportement est récent car :

De cette 1ere observation, on peut a priori en conclure que l'ajout des apostrophes fautives ne se fait pas lors de la récupération des données en BDD, mais lors de leur ajout/modification. En effet, si ce n'était que lors de la récupération, les données plus anciennes devraient aussi etre affectées. Je n'ai pas acces a la BDD pour vérifier la théorie cependant. Si elle est avérée, le probleme est urgent, car toute donnée ajoutée/modifiée entre temps sera corrompue et devra etre nettoyée apres résolution.

J'ai vérifié les requetes POST émises par le JS pour voir si l'erreur venait du coté client, mais je n'ai a priori pas observé d'erreurs (pas d'apostrophes il semblerait). A vérifier néanmoins si le PHP s'avere non-fautif.

Chose étrange, les controleurs PHP n'ont pas été modifié, sur le repo, depuis début Octobre, et il me semble donc que cette derniere version était déja en ligne. Soit je me trompe (le PHP n'est pas fautif ou le code n'a été mis en prod que récemment), soit 2 possibilités, selon mes yeux novices :

Désolé, flemme d'ajouter certains accents chers a notre langue, clavier qwerty oblige.

benjaminplanche commented 11 years ago

Précisions : J'ai testé les différentes actions et champs possibles. Etrangement, seuls les numéros de téléphone des contacts, leurs adresses emails, et les catégories affectées aux deux, sont concernées.

benjaminplanche commented 11 years ago

Donc comme référencé dans l'issue #13, ce bug d'apostrophes touche aussi la page d'administration des utilisateurs : http://ifaedi.asso.insa-lyon.fr/index.php?page=Administration_Utilisateurs. Vous pourrez ainsi constaté que mon compte en a fait les frais (j'ai voulu testé l'édition pour checker le bug #13).

J'ai perso l'impression que ca confirme la theorie que l'erreur vient du serveur, et affecte donc potentiellement tous les formulaires du site (ca va etre sympa a tout vérifier).

blackheaven commented 11 years ago

ça vient d'une mise à jour de PHP, ça se résoud avec le #19

blackheaven commented 11 years ago

À tester

bnjbvr commented 11 years ago

Toujours présente: dans l'administration de l'annuaire, voir l'entrée qui commence par un O. Problèmes d'accents aussi, semblerait-il.

blackheaven commented 11 years ago

je le vois !