Dans un fichier de langue, tu ne doit pas reprendre une clef de langue du core de phpbb car elle serait écrasée.
Ne pas mettre
L_ devant, cela est pour l'affichage de la clef de langue dans un fichier html
- Code: Tout sélectionner
'L_USERNAME' => 'Les amis de %s',
doit être plutôt :
- Code: Tout sélectionner
'VIEWFRIENDS_USERNAME' => 'Les amis de %s',
Ainsi la clef sera unique et correspondra à ton extension
composer.json :
- Code: Tout sélectionner
"license": "GPL-2.0",
Plus accepté depuis longtemps déjà, il doit être :
- Code: Tout sélectionner
"license": "GPL-2.0-only",
- Code: Tout sélectionner
"phpbb/phpbb": "3.2.0"
Il faut mettre supérieur ou égal à :
- Code: Tout sélectionner
"phpbb/phpbb": ">=3.2.0"
et devrait se trouver aussi dans la partie "require": car la version minimum de phpbb doit être indiquée pour éviter par exemple une installation dans phpbb 3.1
migrations/viewfriends_1_0_0.php
Alors que tu indique une version de phpbb 3.2, tu fait référence à la version 3.1.0
- Code: Tout sélectionner
return array('\phpbb\db\migration\data\v310\dev');
Je te conseille d'utiliser la version 3.2.8 ou 3.2.9
- Code: Tout sélectionner
return array('\phpbb\db\migration\data\v32x\v328');
puis, tu indique une version 1.0.0 alors que dans le composer, tu indique une version 1.0.2
event/listener.php
Tu fait charger ton fichier de langue pour la totalité du forum alors que tu n'en a besoin que dans une seule page avec la fonction load_language_on_setup
Il est préférable de le charger dans ta fonction render_memberlist_view_profile :
- Code: Tout sélectionner
$this->lang->add_lang('common', 'ady/viewfriends');
tu doit utiliser la fonction lang comme elle est prévue, pas de sprintf :
- Code: Tout sélectionner
'VIEWFRIENDS_USERNAME' => $this->lang->lang('VIEWFRIENDS_USERNAME', $member['username']),
Ne pas employer
$this->table_prefix . 'users mais la globale
USERS_TABLEPlutôt que faire :
- Code: Tout sélectionner
usort($friends, 'self::sortFriendsByUsername');
tu devrait utiliser
ORDER_BY u.username ASC dans ta requête.
- Code: Tout sélectionner
if ($row = $this->db->sql_fetchrow($result)) {
Ne jamais mettre
{ dans la même ligne mais dans une nouvelle ligne
A suivre...