Projet

Général

Profil

Anomalie #742

Graphiques

Ajouté par James Swerm il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
Début:
31/10/2013
Echéance:
% réalisé:

100%

Version utilisée:

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

Révisions associées

Révision e359a90f
Ajouté par Johan Cwiklinski il y a presque 4 ans

Force integer values for charts; fixes #742

Historique

#1 Mis à jour par Johan Cwiklinski il y a presque 4 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...

#2 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#3 Mis à jour par Johan Cwiklinski il y a presque 4 ans

Le correctif appliqué dans le commit devrait corriger le problème ; merci de tester et de me confirmer :)

#4 Mis à jour par Roland Telle il y a presque 4 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 !

#5 Mis à jour par Johan Cwiklinski il y a presque 4 ans

  • Statut changé de Résolu à Fermé
  • Version cible mis à 0.7.7

#6 Mis à jour par James Swerm il y a presque 4 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.

Formats disponibles : Atom PDF