Petites erreurs de programmation ? [Réglé]
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
J'ai remarqué ceci sur mon forum : [img]http://imagebin.org/91615[/img]
J'ai voulu corriger moi-même cette faute d'orthographe, mais je me suis aperçu qu'il y avait peut-être erreur ou incohérence dans la prog.
Dans le template forum_bottom.tpl, concernant les stats sur les utilisateurs en ligne, nous avons :
Code TPL :
Or, la variable L_USER est défini dans forum.php :
Code PHP :
//Listes les utilisateurs en lignes. list($users_list, $total_admin, $total_modo, $total_member, $total_visit, $total_online) = forum_list_user_online("AND s.session_script = '/forum/forum.php' AND s.session_script_get LIKE '%id=" . $id_get . "%'"); $Template->assign_vars(array( 'TOTAL_ONLINE' => $total_online, 'USERS_ONLINE' => (($total_online - $total_visit) == 0) ? '<em>' . $LANG['no_member_online'] . '</em>' : $users_list, 'ADMIN' => $total_admin, 'MODO' => $total_modo, 'USER' => $total_member, 'GUEST' => $total_visit, 'SELECT_CAT' => forum_list_cat($id_get, $CAT_FORUM[$id_get]['level']), 'L_USER' => ($total_online > 1) ? $LANG['user_s'] : $LANG['user'], 'L_ADMIN' => ($total_admin > 1) ? $LANG['admin_s'] : $LANG['admin'], 'L_MODO' => ($total_modo > 1) ? $LANG['modo_s'] : $LANG['modo'], 'L_USER' => ($total_member > 1) ? $LANG['member_s'] : $LANG['member'], 'L_GUEST' => ($total_visit > 1) ? $LANG['guest_s'] : $LANG['guest'], 'L_AND' => $LANG['and'], 'L_ONLINE' => strtolower($LANG['online']) ));
Si vous remarquez, on attribue deux valeurs différentes à la même variable. Donc, la première attribution n'apparaîtra jamais. De plus, il semble que la condition ne fonctionne pas et je n'ai pas testé à savoir si cela est causé par cette double attribution de valeur. En effet, si le nombre de membres > 1, la valeur de L_USER devrait être membres au pluriel, soit la variable member_s. Or, elle ne prend pas cette valeur comme vous le constatez sur l'image fournie.
Quelqu'un a une idée ?
Édité par DaaX Le 07/04/2010 à 18h52
alain91 Membre non connecté
Booster Missile
-
Booster Missile
- Voir le profil du membre alain91
- Inscrit le : 26/09/2010
- Groupes :
Ton analyse est pertinente et le code montre effectivement un problème.
Dans l'attente d'une intégration dans une prochaine version de PHPBOOST, je pense pouvoir proposer un patch sous la forme d'un zip avec les fichiers modifiés.
Le pessimisme est d'humeur. L'optimisme est de volonté (Philosophe Alain).
ben.popeye Membre non connecté
-
Modérateur
- Voir le profil du membre ben.popeye
- Inscrit le : 04/08/2005
- Site internet
- Groupes :
-
Equipe Historique
Un problème, une question ? Cherchez dans la FAQ ou la documentation. Si vous ne trouvez pas la réponse, demandez du support sur le forum.
Bjarne Stroustrup, inventeur du C++ :"There are two ways to write error-free programs; only the third works."
alain91 Membre non connecté
Booster Missile
-
Booster Missile
- Voir le profil du membre alain91
- Inscrit le : 26/09/2010
- Groupes :
Ce qui conduit à ne prendre en compte que la dernière assignation (membre) et donc l'affichage est perturbé par rapport à ce qu'il devrait être.
ci-joints deux fichiers patch (un pour les php et un pour le template). J'ai commité dans le svn ces modifications.
forum.zip
templates-forum_bottom.zip
Édité par alain91 Le 07/04/2010 à 20h12
Le pessimisme est d'humeur. L'optimisme est de volonté (Philosophe Alain).
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
Eh bien merci pour les patchs.
Quelle rapidité !
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
Il y avait donc d'autres incohérences. Je vois que ton patch inclus le fichier forum.php et bien plus...
En effet, je n'avais pas vu que la variables est un peu partout hein !
Devrait-on l'annoncer à tous ?
Édité par DaaX Le 08/04/2010 à 00h27
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
Bon, il y a un autre souci.... À moins qu'il y ait un truc que je n'aie pas vu que tu as changé dans les fichiers php, mais maintenant, la correction est ok en partie. C'est bien utilisateurs au pluriel qui est indiqué au lieu de membre.
Cependant, dans la première ligne où l'on a :
2 Utilisateurs en ligne :: 1 Administrateur, 0 Modérateur, Membre et 0 Visiteur
Je n'ai plus de valeur pour membre. Pourtant, la variable est bien là dans le template....
On dirait qu'il ne calcule pas le nombre de membres.
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
Tu as une idée pourquoi le nombre de membre ne s'affiche pas ?
alain91 Membre non connecté
Booster Missile
-
Booster Missile
- Voir le profil du membre alain91
- Inscrit le : 26/09/2010
- Groupes :
Il faut chercher les causes ce qui prend un peu de temps.
Ne soyez pas impatient.
Le pessimisme est d'humeur. L'optimisme est de volonté (Philosophe Alain).
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
alain91 Membre non connecté
Booster Missile
-
Booster Missile
- Voir le profil du membre alain91
- Inscrit le : 26/09/2010
- Groupes :
Est-ce systématique ou seulement sur certaines pages ?
Si c'est dans certaines pages peux-tu donner l'url pour que je retrouve le ficher concerné ?
Le pessimisme est d'humeur. L'optimisme est de volonté (Philosophe Alain).
alain91 Membre non connecté
Booster Missile
-
Booster Missile
- Voir le profil du membre alain91
- Inscrit le : 26/09/2010
- Groupes :
Si c'est dans certaines pages peux-tu donner l'url pour que je retrouve le ficher concerné ?
Après vérification il y a aussi le fichier move.php à modifier :
move.zip
Édité par alain91 Le 10/04/2010 à 22h41
Le pessimisme est d'humeur. L'optimisme est de volonté (Philosophe Alain).
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
Dans toutes les pages.
Regardes ici : http://www.mandrivalinux-online.org/forum/index.php
Merci pour le fichier move, je vais faire le changement.
alain91 Membre non connecté
Booster Missile
-
Booster Missile
- Voir le profil du membre alain91
- Inscrit le : 26/09/2010
- Groupes :
Peux tu confirmer que tu utilises bien le nouveau fichier template forum_bottom.tpl que j'ai joins avec les fichiers php ?
Le pessimisme est d'humeur. L'optimisme est de volonté (Philosophe Alain).
DaaX Membre non connecté
Booster Mortier
-
Booster Mortier
- Voir le profil du membre DaaX
- Inscrit le : 08/09/2009
- Site internet
Je n'avais pas vu ta réponse. Je te confirme ça ce soir sans faute. Mais je ne crois pas m'être trompé.
Répondre
Vous n'êtes pas autorisé à écrire dans cette catégorie