[Ady]

Hello,
Je passe juste pour prévenir que j'ai pas lâchement abandonné. Je suis juste pas mal occupé en ce moment mais au maximum dans un mois j'aurai plus de temps.
Je dis pas que mes dev sont indispensables mais bon toujours sympa de prévenir ^^
Désolé pour le temps.

[Ady]

Hello !

J'ai repris du service ^^

Le lien du repo a été mis à jour au premier message de ce sujet. J'ai migré tous mes projets de GitHub vers GitLab.
J'ai appliqué toutes les recommandations et autres de ABDev. Merci à lui pour son aide.

Encore une fois, n'hésitez pas à me remonter quoi que ce soit, y compris vos potentielles idées d'amélioration

https://gitlab.com/adynemo/viewfriends/-/tags/v1.0.2-rc

[Sylveretour]

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_TABLE
Plutô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...

[pierredu]

Pour l'exploitation du résultat, j'aime mieux un while (row =...

[Ady]

Voilà, j'ai pris en compte tous vos retours. Le master est à jour.
J'ai pas bien compris le système des variables de langue avec le L_, j'ai regardé la doc aussi. J'ai vu sur d'autres extensions qu'ils utilisaient {{ lang(VARIABLE) }} alors je suis parti là-dessus

[Skouat]

Hello,

En plus des prérequis indiqués dans le fichier composer.json, il est recommandé de contrôler ces prérequis au niveau de phpBB via la création d'un fichier ext.php à la racine du répertoire de l'extension.
ex : https://github.com/Skouat/ext_paypal_do ... hp#L25-L40

La création d'une entrée dans la BDD pour indiquer la version de l'extension ne sert plus à rien.
Cette information est récupérée depuis le fichier composer.json.
Donc le fichier de migration peut être supprimé.

concernant le fichier composer.json et dans l'optique où le fichier de migration serait conservé, alors il y a une incohérence de version entre la dépendance indiquée dans le fichier de migration (3.2.8 ) et les versions de phpBB (3.2.0)/PHP (5.3.3) indiquées dans composer.json.

Compat HTML5 : dans listener.php la balise img ne doit pas être fermée avec /> mais avec >.