[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 >.

[Steph]

Désolé, je me permets de citer parce que j'ai eu une longue absence.

J'ai pris en compte tes changements.
Juste, une précision. Si je comprends bien, ça veut dire qu'on a plus besoin non plus de préciser la version de l'extension dans la DB ?

Code: Tout sélectionner

array('config.add', array('viewfriends_version', '1.0.2')),

[Skouat]

Tout à fait. Il n'est plus nécessaire de renseigner le numéro de version dans la BDD étant donné que la version de référence est celle indiquée dans le fichier composer.json

[Ady]

OK très bien. Ça simplifie la chose en effet. Je fais cette dernière modif et je préviens quand c'est ok si quelqu'un veut rejeter un coup d'œil.

[Ady]

Hello !

Voilà tout est réglé au niveau des précédents retours (je n'ai rien oublié normalement).
J'en ai profité pour faire une des idées notées au premier message de ce sujet. À savoir, pouvoir supprimer un ami directement via cette liste, sur sa propre page de profile.

J'ai passé l'extension en RC, à vous !

PS : Je mettrai à jour les captures d'écrans dès que possible.

[Skouat]

Hello,

Donc la prochaine étape... soumettre ton extension dans notre BDD et/ou celle de phpbb.com ?

[Ady]

Yep c'est ça. Sauf si vous avez des propositions ? La seule autre idée que j'avais, le tooltip, je suis pas certain de l'utilité. Je laisse ça là au cas où mais je pense pas le faire pour l'instant.
Bref du coup oui prochaine étape soumette déjà à vous puis pourquoi pas phpbb.com. Mais c'est là où concrètement j'ai pas bien compris ce que je dois faire pour soumettre ^^

[Skouat]

j'ai pas bien compris ce que je dois faire pour soumettre ^^

C'est super compliqué...
Tu vas là : http://www.phpbb-fr.com/customise/db/
Et tu crées une nouvelle contribution.

[Ady]

Voilà c'est fait !
http://www.phpbb-fr.com/customise/db/ex ... w_friends/

Merci pour ton aide Skouat