fix(profile): champ pseudo_ingame en lecture seule (#501)#529
Merged
darknoon29 merged 8 commits intodevelopfrom Apr 6, 2026
Merged
fix(profile): champ pseudo_ingame en lecture seule (#501)#529darknoon29 merged 8 commits intodevelopfrom
darknoon29 merged 8 commits intodevelopfrom
Conversation
Le pseudo in-game est désormais exclusivement géré par l'extension xtense. Le champ du profil utilisateur était affiché comme éditable mais ne sauvegardait jamais rien (code mort). - views/profile.php: attribut readonly sur l'input pseudo_ingame, texte explicatif via PROFILE_PLAYERNAME_XTENSE_INFO, suppression de l'appel help() dont la clé lang était absente (affichait 'Aide Introuvable') - includes/user.php: suppression de la variable globale pub_pseudo_ingame et de sa validation dans member_user_set() - lang/*/lang_profile.php: ajout de la clé PROFILE_PLAYERNAME_XTENSE_INFO dans les 10 langues Closes #501
Contributor
There was a problem hiding this comment.
Pull request overview
Cette PR corrige un comportement trompeur dans le profil utilisateur : le champ “Pseudo dans le Jeu” semblait éditable mais n’était jamais persisté (car géré par xtense), et l’aide associée pointait vers une clé inexistante.
Changes:
- Rend
pseudo_ingameen lecture seule dans la vue profil et ajoute un texte explicatif i18n. - Supprime la validation/prise en compte serveur de
pseudo_ingamelors de la sauvegarde du profil. - Ajoute la clé de traduction
PROFILE_PLAYERNAME_XTENSE_INFOdans les fichiers de langue concernés.
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| views/profile.php | Rend le champ pseudo in-game non éditable et affiche une info “géré par xtense”. |
| includes/user.php | Retire pseudo_ingame des globals/validations côté sauvegarde profil. |
| lang/fr/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/en/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/en_US/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/es/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/it/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/pt_BR/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/bs/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/bs_BA/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/hr/lang_profile.php | Ajoute la traduction de l’info xtense. |
| lang/hr_HR/lang_profile.php | Ajoute la traduction de l’info xtense. |
- Reorganisation des sections : galaxy/systeme et ip_check montent dans la section 'Informations OGSpy' (donnees editables par l'user) - Section 'Donnees OGame' (ex 'Informations du Jeu') ne contient plus que les donnees gerees par xtense : pseudo, classe, officiers - Note explicative xtense en debut de section (colspan=2) a la place du <small> inline apres le champ pseudo_ingame - Classe (select) et officiers (checkboxes) passes en disabled - Section 'Divers' supprimee (ip_check deplace, bouton save conserve) - PROFILE_GAME renomme 'Donnees OGame' dans les 10 langues
…Game - Position de la planete principale deplacee dans 'Donnees du Jeu' (avant la note xtense, reste editable) - PROFILE_GAME renomme sans reference a OGame (copyright) : 'Donnees du Jeu' / 'Game Data' / 'Datos del Juego' etc. dans les 10 fichiers de langue
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…ce OGame dans les 10 langues
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problème
Closes #501
Le champ "Pseudo dans le Jeu" du profil utilisateur était affiché comme éditable, mais ne sauvegardait jamais les modifications (code mort). De plus, la clé d'aide
help_profile_pseudo_ingameétait absente de tous les fichiers de langue, ce qui affichait "Aide Introuvable" au clic.Le pseudo in-game est exclusivement géré par l'extension xtense via
INSERT ... ON DUPLICATE KEY UPDATEsur la tableogspy_game_player. Tous les utilisateurs utilisent xtense — aucun besoin d'édition manuelle.Solution
Rendre le champ en lecture seule (
readonly) et afficher une phrase explicative indiquant que la valeur est gérée par xtense.Changements
views/profile.phpreadonlyajouté sur l'inputpseudo_ingame<small>ajouté avec la cléPROFILE_PLAYERNAME_XTENSE_INFOhelp("profile_pseudo_ingame")dont la clé était absenteincludes/user.php$pub_pseudo_ingamede la déclarationglobaldansmember_user_set()!check_var($pub_pseudo_ingame, "Pseudo_ingame")du bloc de validationlang/*/lang_profile.php(10 fichiers)PROFILE_PLAYERNAME_XTENSE_INFOdans toutes les langues (fr, en, en_US, es, it, pt_BR, bs, bs_BA, hr, hr_HR)Vérification
pseudo_ingame