search

sallesma / IF-BackOffice

Back Office for the Festival's mobile app
1 stars 1 forks source link

erreur info.isDisplayedOnMap #11

Closed sallesma closed 10 years ago

sallesma commented 10 years ago

Lorsqu'on supprime un point, s'il a une info liée, le champ 'isDisplayedOnMap' de l'info est mis à 0.

Cela peut introduire une erreur si DEUX points étaient liés à cet info (il en reste toujours un alors que l'attribut indique 0)

flo-sch commented 10 years ago

Je pense qu'il serait mieux de remplacer ce champ booléen par une relation (many informations to one point), et de voir le nombre lors des requêtes, plutôt que de checker si ce champ est à 0 ou 1.

sallesma commented 10 years ago

A l'origine on l'avait rajouté pour éviter de devoir faire une requête plus compliquée dans l'appli (qui est le vrai produit proposé aux vrais utilisateurs finaux du backoffice). Mais c'est vrai que c'est une source non négligeable de bugs :)

Du coup la relation existe déjà puisque mapitem contient un champ infoId qui le lie à l'info qu'il représente.

Je pense donc supprimer simplement ce champs isDisplayedOnMap et modifier les requêtes pour que l'on puisse quand même afficher pour une info si un point la représente ou non.

Ca demandera peut être de toucher à EntityManager ou à surcharger quelque chose dans InfosManager