Project

General

Profile

Anomalie #742

Graphiques

Added by James Swerm about 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
Start date:
10/31/2013
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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

Associated revisions

Revision e359a90f (diff)
Added by Johan Cwiklinski about 6 years ago

Force integer values for charts; fixes #742

History

#1

Updated by Johan Cwiklinski about 6 years ago

  • Assignee set to 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

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100
#3

Updated by Johan Cwiklinski about 6 years ago

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

#4

Updated by Roland Telle about 6 years ago

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

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Résolu to Fermé
  • Target version set to 0.7.7
#6

Updated by James Swerm about 6 years ago

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.

Also available in: Atom PDF