search

2p2r / velobs_web

Application web permettant aux cyclistes de signaler les problèmes sur les aménagements cyclables.
GNU General Public License v3.0
22 stars 14 forks source link

mise à jour impossible de la priorité #6

Closed bikepunk closed 7 years ago

bikepunk commented 7 years ago

Issue by gerald2545 Friday Sep 23, 2016 at 05:06 GMT Originally opened as https://github.com/2p2r/velobs/issues/6

Je voulais passer des fiches velobs à DONE, mais les modifs n'ont pas été prises en compte Quand je choisis DONE dans la liste déroulante, le champs passe à la la valeur 6, mais si je navigue ensuite entre les pages du tableau et que je reviens sur ma fiche, celle-ci est à la même priorité qu'auparavant

bikepunk commented 7 years ago

Comment by bikepunk Wednesday Oct 26, 2016 at 13:34 GMT

j'ai le même problème aujourd'hui sur l'observation 2574, j'essaye de comprendre

bikepunk commented 7 years ago

Comment by bikepunk Wednesday Oct 26, 2016 at 13:49 GMT

le changement de priorité ne fonctionne pas dans le tableau, mais dans l'interface popup ( http://velobs.2p2r.org/admin.php?id=2574 ), je peux effectivement changer la priorité !

à voir comment fixer ça

gerald2545 commented 7 years ago

Dans lib/php/admin/adminfunction.php, fonction updatePoi, le sql pour mettre à jour le POI n'est jamais exécuté, contrairement aux fonctions updatePoiComcomCarto, updatePoiPoleTechCarto....

gerald2545 commented 7 years ago

oups, il est bien appelé juste avant la fin de la fonction, mais la variable de mise à jour du POI ($sql) est écrasée entre temps ... revoir ces fonctions de mise à jour de POI pour mutualiser le code et le clarifier avec des commentaires

gerald2545 commented 7 years ago

Corrigé et passé en prod