La page profile.php renvoie une page "hybride" entre un nouvel utilisateur et un ancien

[Thierz]

Détails techniques

Version du forum : phpBB2
Poste occupé : Administrateur
Navigateur(s) concerné(s) : Mozilla Firefox, Google Chrome et sûrement d'autres
Capture d'écran du problème :

Voir l'image:

Personnes concernées par le problème : Plusieurs utilisateurs (les admins)
Problème apparu depuis : Le problème est présent quand un administrateur clique sur le lien dans un e-mail indiquant qu'un nouvel utilisateur s'est enregistré
Lien du forum : https://astronautique.actifforum.com/

Description du problème

Bonjour,

Le problème est présent quand un administrateur clique sur le lien dans un e-mail indiquant qu'un nouvel utilisateur s'est enregistré.

L'e-mail contient en effet un lien vers la fiche de l'utilisateur qui vient de se connecter. Pour l'exemple donné dans la capture d'écran, le lien est :

http://www.forum-conquete-spatiale.fr/profile.php?mode=viewprofile&status=unregistered&u=shawright

Quand l'administrateur clique sur le lien, la page pointée (profile.php) renvoie vers une seconde page dont l'URL est, pour notre exemple :

https://www.forum-conquete-spatiale.fr/u4?status=unregistered

Je précise que le Forum de la Conquête Spatiale compte plus de 4000 membres, mais comme vous pouvez le voir, la page profile.php renvoie vers une URL contenant "u4" c'est à dire l'utilisateur numéro 4 (un des premiers membres du forum). A mon avis, le problème se situe donc dans la fonction PHP qui crée l'URL à partir du nom du profil donné en paramètre à profile.php (soit "shawright" pour notre exemple). Le "u4" dans l'URL est manifestement faux.

Voici les conséquences de ce bug : depuis la page https://www.forum-conquete-spatiale.fr/u4?status=unregistered qui s'ouvre, on a un mélange d'informations du nouveau membre et de l'ancien (membre n°4). Depuis cette page, le lien "Administrer l'utilisateur: Profil" qui renvoie vers le PA affiche dans l'onglet "Modifier" les infos du nouvel utilisateur, et dans l'onglet "Permissions" les permissions de l'ancien.

[Luzz]

Bonjour,

Merci pour votre signalement, vos informations ont été transférées à notre équipe technique. Nous allons voir d'où vient ce problème.

Bien cordialement

[Thierz]

Merci. J'en profite pour ajouter que si on suit ce même lien vers profile.php APRÈS que le membre en question a validé son inscription, la page résultante indique que l'utilisateur n'existe pas (alors qu'il a activé son compte avec succès).

[Neptunia]

Bonjour ^^

Merci. J'en profite pour ajouter que si on suit ce même lien vers profile.php APRÈS que le membre en question a validé son inscription, la page résultante indique que l'utilisateur n'existe pas (alors qu'il a activé son compte avec succès).

Ce point est normal. Le lien pointe vers un membre non enregistré comme l'indique clairement le lien. Une fois validé, le membre est enregistré et il n'y a donc plus de profil de membre non enregistré à consulter

[Thierz]

Oui, je me disais qu'un message "cet utilisateur s'est bien enregistré" serait plus juste que "cet utilisateur n'existe pas". En bonus on pourrait même atterrir sur la page profil du nouveau membre.

[Neptunia]

Ce n'est pas faisable parce que le membre non enregistré ayant choisi le pseudo shawright peut disparaitre de plusieurs façons
- Si l'administrateur supprime le compte
- Si le membre choisit de modifier son pseudo
- Si l'administration impose une modification de pseudo (chez moi par exemple je le renommerais Shawright parce que j'ai horreur des pseudos, donc des noms qui ne commencent pas par une majuscule).

Et surtout c'est beaucoup de développement (donc d'argent et de source potentielle de bogue) pour un détail qui n'apporte strictement rien en terme de fonctionnalité.

[Thierz]

Nous ne permettons pas le changement de pseudo ni n'imposons de changement de pseudo, je serais au contraire ravi de tomber sur la page montrant le profil de ce nouveau venu. Et si l'utilisateur a été supprimé par l'administrateur, c'est tout a fait juste d'avoir le message "cet utilisateur n'existe pas".

Je ne nie pas qu'un tel développement prend du temps, et donc de l'argent, mais puisqu'on intervient sur le code source pour corriger un bug, et qu'il faudra de toute façons faire un test du correctif, ceci me paraît utile de profiter de l'occasion pour aller un peu plus loin. Quelqu'un peut-il soumettre l'idée aux équipes techniques SVP ?

En vous remerciant.

[Pinguino]

Bonjour,

Le problème a été corrigé.

Le lien vers le PA sur les pages d'utilisateurs à valider était incomplet, donc il redirigeait vers un mélange entre la page d'édition d'un utilisateur existant et celle d'un nouvel utilisateur.

Nous vous remercions pour le signalement.

[Thierz]

C'est avec plaisir... Merci pour la correction !