Bug #1661

Affichage des compteurs

Added by Dominique FERRAND over 9 years ago. Updated over 9 years ago.

Status:ClosedStart date:04/26/2012
Priority:NormalDue date:
Assignee:Jérôme TRUFFOT% Done:

100%

Category:Saisie des services
Target version:0.2

Description

Bonjour,
J'ai activé l'affichage des compteurs, le pavé apparait bien au bas de la page mais vide.
Il y a bien un service saisi pour l'enseignant concerné et je ne vois pas d'erreur dans les logs.
J'ai tracé la session et exécuté les requêtes que j'ai trouvé dans la trace avec succès.
merci de votre aide.

compt-globaux.png - compteur globaux : valeur NaN (28.4 KB) Harold Moyse, 05/22/2012 05:32 PM

History

#1 Updated by Jérôme TRUFFOT over 9 years ago

  • Category set to Saisie des services
  • Assignee set to Jérôme TRUFFOT
  • Target version set to 0.2

Bonjour,

Je vois 2 possibilités :

  • le problème est côté client. Y a-t-il une erreur dans la console javascript de votre navigateur ?
  • le problème est côté serveur. Les compteurs globaux sont passés en même temps que la liste des services annuels dans le contrôleur org.esupportail.geisha.web.rest.san.SansResource, ligne 89 :
compteurs = getDomainService().getCompteurs(individu, annee);

Si les compteurs sont corrects à cet endroit, le problème est nécessairement côté client.

#2 Updated by Dominique FERRAND over 9 years ago

Bonjour,
J'ai bien coché l'affichage de la console Java mais elle ne s'affiche pas avec cette application (Elle s'affiche avec Harpege par exemple).
Je suis sous IE 8.0(j'étais avec Mozilla auparavant), java 1.6.0_18 et je vois une erreur sur la page :
Détails de l’erreur de la page Web

Agent utilisateur : Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0; BTRS124153; InfoPath.1; .NET CLR 1.1.4322; .NET CLR 2.0.50727; .NET CLR 3.0.4506.2152; .NET CLR 3.5.30729)
Horodateur : Thu, 3 May 2012 14:00:03 UTC

Message : 'null' a la valeur Null ou n'est pas un objet.
Ligne : 6
Caractère : 53919
Code : 0
URI : http://crinum.univ-st-etienne.fr:8090/media/esup-geisha/geisha.js?dt=1334849407

Je ne sais donc pas si c'est une erreur client ou serveur.

Dominique.

#3 Updated by Jérôme TRUFFOT over 9 years ago

De mon coté, j'ai essayé avec IE7, IE8 et IE9 : aucun problème avec les compteurs.

Je n'ai malheureusement pas d'idée comme ça pouvant expliquer ce phénomène. Ce que je peux vous proposez, c'est d'essayer d'en savoir un peu plus :
- en remplaçant geisha.js par geisha-debug.js dans le fichier properties/tags/tags.xml. Ca permettra d'avoir une indication plus précise de l'endroit où à lieu l'erreur javascript.
- en loguant les compteurs envoyés par le contrôleur org.esupportail.geisha.web.rest.san.SansResource (ligne 89). Ca permettra de s'assurer que les compteurs sont corrects, même si dans le cas contraire, côté client, des tests sont prévus pour éviter les erreurs...

#4 Updated by Dominique FERRAND over 9 years ago

Merci, je vais essayer de faire ces tests cette semaine, je n'ai pas tous les outils pour travailler facilement de chez moi.

#5 Updated by Harold Moyse over 9 years ago

Hello,

Nous avons nous aussi la même erreur sur l'affichage des compteurs.
Nous avons activé geisha-debug.js et l'erreur se situe à la ligne 3335 ( au niveau du return qui est null semble-t-il) :

tpl = new Ext.XTemplate(
'<div class="geisha-compteurs">',
compteur('serviceAssure'),
compteur('serviceCompta'),
compteur('partTPenTD'),
compteur('serviceStatut'),
compteur('serviceDu'),
compteur({
name: 'hcNonPayables',
value: '{[this.toFixed(values.serviceAssure - values.serviceCompta)]} h'
}),
compteur({
name: 'hcPayables',
cls: '{[values.limite !== undefined && values.hcPayables > values.limite ? "geisha-hc-limite" : "geisha-hc"]}'
}),
compteur('limite'),
'<tpl if="values.limite !== undefined && values.hcPayables !== undefined && hcPayables > limite">',
'<div class="geisha-hc-limite">' + me.texts.depassement + '</div>',
'</tpl>',
'</div>', {
toFixed: function (value) {
return value.toFixed(2);
}
}
);

Harold

#6 Updated by Jérôme TRUFFOT over 9 years ago

Bonjour,

C'est étrange que je n'ai pas rencontré ce problème. Un ou plusieurs compteurs doivent être dans un état bizarre.

Pourriez-vous essayer de remplacer la ligne

return value.toFixed(2);

par

return Ext.isNumber(value) ? value.toFixed(2) : 'NaN';

Si ça fonctionne, un ou plusieurs compteurs devraient afficher 'NaN' ("Not a Number"). Ca devrait me donner une piste pour comprendre ce qu'il se passe.

Et en passant, je vais sûrement intégrer ce test dans les futures versions. Au pire, il ne serait qu'inutile...

#7 Updated by Harold Moyse over 9 years ago

Effectivement, ça fonctionne mieux.
Voir le screenshot ci-joint pour le compteur NaN.
Merci.

#8 Updated by Jérôme TRUFFOT over 9 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Merci pour le test et le screenshot. Ca m'a permis de comprendre le problème. En fait, je le rencontre également pour l'une de nos universités si j'affiche les compteurs (actuellement, il m'a été demandé de les cacher).

En fait, c'est le compteur "dont TP = TD" qui ne devrait pas s'afficher et mes tests qui sont mauvais.

J'ai corrigé ces tests et ça semble fonctionner correctement maintenant. Si vous voulez tester, vous pouvez regarder les différences de la révision r371 et les appliquer au fichier geisha-debug.js.

Encore merci pour votre aide.

#9 Updated by Harold Moyse over 9 years ago

Excellent! Je confirme que ce fix résout le bug.

#10 Updated by Jérôme TRUFFOT over 9 years ago

  • Status changed from Resolved to Closed

Fixé dans la version 0.2.7.

Also available in: Atom PDF