Anomalie #742
fermé
Graphiques
100%
Description
Bonjour,
Lorsque j'ai testé la partie graphiques sur galette, les aperçus n'étaient pas justes.
En creusant un peu je me suis aperçu que les valeurs envoyées sur le jquery qui s'occupe des graphiques étaient mauvaises. De type [["membres actifs","96"]... au lieu de ["membres actifs",96]... (sans guillemets sur la valeur).
Je me suis fait une petite rustine via un foreach sur le tableau chart couplé avec un intval dans la partie /lib/galette/IO/Charts.php pour résoudre le problème, mais il y a peut être un moyen de faire un code un peu plus propre. Je ne comprends pas pourquoi le compte renvoyé par la base de données est affiché comme caractère est pas entier.
Voilà peut être que le bug était déjà signalé ou corrigé, désolé je n'ai pas trouvé l'information.
Cordialement
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Assigné à mis à Johan Cwiklinski
James Swerm a écrit :
Je ne comprends pas pourquoi le compte renvoyé par la base de données est affiché comme caractère est pas entier.
Visiblement, MySQL/Zend_Db considèrent que le résultat d'un count(mon_champ) est une chaîne de caractère...
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit e359a90f47212b01e7ff4f792eb9fec4745f2685.
Mis à jour par Johan Cwiklinski il y a environ 11 ans
Le correctif appliqué dans le commit devrait corriger le problème ; merci de tester et de me confirmer :)
Mis à jour par Roland Telle il y a environ 11 ans
Johan Cwiklinski a écrit :
Le correctif appliqué dans le commit devrait corriger le problème ; merci de tester et de me confirmer :)
Test fait sur une version de développement, les libellés sont corrects et les calculs très vraisemblables ; j'émets cette petite restriction dans la mesure où - par exemple - dans le camembert "Adhérent par statut d'adhésion" Exempt de cotisation et A jour sont distincts alors que dans la liste des adhérents de Galette, la sélection Adhérents à jour contient les exempts de cotisation. Tout reste cohérent dans les deux langues. Selon moi, c'est ok. Bravo !
Mis à jour par Johan Cwiklinski il y a environ 11 ans
- Statut changé de Résolu à Fermé
- Version cible mis à 0.7.7
Mis à jour par James Swerm il y a environ 11 ans
Le test est ok pour moi également.
Merci pour la rapidité, plutôt étonnant ce count de mysql/zend DB...
J'en profite pour glisser une suggestion : ne prendre dans ces graphiques que les adhérents actuellement en activité.
Bonne journée et bonne continuation.