[Dev] [1.0.2] View Friends

Pour les extensions destinées à phpBB 3.2.x

Modérateur: Equipe

Règles du forum
A lire impérativement : Règlement de phpBB-fr.com

Re: [2.0.1] View Friends

Messagepar Ady » 26 Sep 2019 à 22:35

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.
Actuellement très occupé, de retour fin octobre.
Toujours disponible pour répondre si besoin cependant.
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 64
Enregistré le: 27 Aoû 2018 à 12:00

Re: [2.0.1] View Friends

Messagepar Ady » 11 Jan 2020 à 18:24

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
Actuellement très occupé, de retour fin octobre.
Toujours disponible pour répondre si besoin cependant.
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 64
Enregistré le: 27 Aoû 2018 à 12:00

Re: [1.0.2] View Friends

Messagepar Sylveretour » 11 Jan 2020 à 19:38

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...
Avatar de l’utilisateur
Sylveretour
Apprenti-floodeur
Apprenti-floodeur
 
Messages: 1185
Enregistré le: 21 Nov 2017 à 18:24
Localisation: West Coast

Re: [1.0.2] View Friends

Messagepar pierredu » 11 Jan 2020 à 20:56

Pour l'exploitation du résultat, j'aime mieux un while (row =...
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1264
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: [1.0.2] View Friends

Messagepar Ady » 12 Jan 2020 à 00:28

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
Actuellement très occupé, de retour fin octobre.
Toujours disponible pour répondre si besoin cependant.
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 64
Enregistré le: 27 Aoû 2018 à 12:00

Re: [1.0.2] View Friends

Messagepar Skouat » 12 Jan 2020 à 14:55

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 >.
Skouat
Traducteur
Traducteur
 
Messages: 14113
Enregistré le: 02 Avr 2008 à 19:47

Précédente

Retourner vers Extensions en développement

Qui est en ligne

Utilisateurs parcourant ce forum : Aucun utilisateur enregistré et 1 invité