IFAEDI / S.I

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

Annuaire - URL décodage #10

Open benjaminplanche opened 11 years ago

benjaminplanche commented 11 years ago

Mauvais URL encodage/décodage de certains champs (ou tous ?) de l'entreprise et des contacts.

Exemple : voir ici http://ifaedi.asso.insa-lyon.fr/index.php?page=Administration_Annuaire#ADENEO

PandiPanda69 commented 11 years ago

J'avais remarqué ça. Je penche pour un problème niveau PHP, je regarde ça asap.

benjaminplanche commented 11 years ago

Problème toujours présent (repéré sur l'édition et visualisation des données Contact).

blackheaven commented 11 years ago

Je vois pas le soucis, il est à quel niveau ?

blackheaven commented 11 years ago

désolé

benjaminplanche commented 11 years ago

Cool ! ... Mais c'était quoi le problème ? Pas de modifs des sources nécessaires ?

benjaminplanche commented 11 years ago

Oups, pas vu la réouverture entre temps, mon précédent message est caduc. Désolé.

blackheaven commented 11 years ago

désolé (le double tab c'est le mal) c'est à quel niveau que vous voyez l'erreur ?

benjaminplanche commented 11 years ago

J'ai voulu checker le fonctionnement en prod' pour donner un exemple à @PandiPanda69 . Il s'avère que des modifications fantômes (je vois pas d'autres mots pour justifier) ont dû être faites sur le serveur, puisque l'Annuaire est complètement foireux (impossible d'ajouter / éditer des contacts, mauvaise récupération des ID des objets, ...) et inutilisable, même pour checker les bugs relevés précédemment.

Ayant tester à diverses reprises ce module en prod' dans la semaine, j'ai ainsi pu attester de multiples changements dans le comportement de celui-ci, alors que je le connais plutôt bien, relevant de nouveaux bugs (#10 et #24, et maintenant le dysfonctionnement actuel). Bugs qui a priori viennent du serveur. Bugs qui ne s'expliquent a priori pas puisque qu'aucune modification touchant au PHP n'est passé par le repo, toute modification devant passer par là. Il est compréhensible que le serveur puisse nécessiter des maj, voire même que des modifications directes au contenu du site puisse être mises en prod' avant même d'être mergées sur le repo si le besoin se fait urgent, mais si aucune trace n'est là pour étayer les faits, alors que ceux-ci ont des conséquences potentielles importantes sur d'autres chantiers, cela ne peut que nuire au travail collectif et à la maintenabilité (voir dossiers pour l'importance de ce facteur dans ce projet).

@PandiPanda69 ne connais pas la raison non plus, et n'a encore rien touché côté prod'. @blackheaven, qu'en est-il de ton côté ?

blackheaven commented 11 years ago

Tout est dans la #19.

benjaminplanche commented 11 years ago

Je peux comprendre que le bug #24 relevé ce matin soit dû à celui décrit en #19. Makes sense.

Mais.

Mais ce matin, l'Annuaire fonctionnait. Comme décrit en #24, j'ai pu effectué divers tests (ajouts / suppressions / modifications d'entreprises / contacts) pour étudier le comportement du bug. Le comportement à l'heure actuelle est encore différent.

Que le serveur évolue, j'approuve. Que des modifications soient faites direct en prod' sans filet de sauvatage, ca me crispe, mais j'ai aucune légitimité pour critiquer, et je ne le ferais pas. Pas mes oignons. Que ces modifications affectent des pans entiers du projet sans apparaître sur le repo, ruinent la production, bloquent les travaux, corrompent les données utilisateurs, et s'accumulent ... Serait-il possible, s'il te plaît, d'avoir des explications précises de ce qu'il se passe ?

Bonne nuit.

blackheaven commented 11 years ago

pour le coup je n'ai pas d'explications, je n'ai rien touché depuis hier matin/midi.

blackheaven commented 11 years ago

est-ce quelqu'un pourrait fournir un screen du problème ?

benjaminplanche commented 11 years ago

Tant que #19 n'est pas résolue, non. Impossible de reproduire le bug car impossible de faire marcher quoi que ce soit touchant à la BDD.

19 reste la priorité absolue (@blackheaven, vous aviez pas parlé de downgrader le serveur en attendant de fixer d'ailleurs ?).

blackheaven commented 11 years ago

À tester