[RC] [1.1.0] 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 à 23: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.
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Re: [2.0.1] View Friends

Messagepar Ady » 11 Jan 2020 à 19: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
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Re: [1.0.2] View Friends

Messagepar Sylveretour » 11 Jan 2020 à 20: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
Floodeur mature
Floodeur mature
 
Messages: 1514
Enregistré le: 21 Nov 2017 à 19:24
Localisation: West Coast

Re: [1.0.2] View Friends

Messagepar pierredu » 11 Jan 2020 à 21:56

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

Re: [1.0.2] View Friends

Messagepar Ady » 12 Jan 2020 à 01: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
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Re: [1.0.2] View Friends

Messagepar Skouat » 12 Jan 2020 à 15: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
Resp. Traducteurs
Resp. Traducteurs
 
Messages: 14630
Enregistré le: 02 Avr 2008 à 20:47

Re: [1.0.2] View Friends

Messagepar Ady » 21 Juil 2020 à 12:53

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')),
Modifié en dernier par Steph le 21 Juil 2020 à 14:06, modifié 1 fois.
Raison: Suppression de la citation inutile de message en entier et auquel tu réponds directement
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Re: [1.0.2] View Friends

Messagepar Skouat » 21 Juil 2020 à 20:31

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
Skouat
Resp. Traducteurs
Resp. Traducteurs
 
Messages: 14630
Enregistré le: 02 Avr 2008 à 20:47

Re: [1.0.2] View Friends

Messagepar Ady » 21 Juil 2020 à 21:41

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.
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Re: [1.1.0] View Friends

Messagepar Ady » 27 Juil 2020 à 21:22

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.
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Re: [1.1.0] View Friends

Messagepar Skouat » 28 Juil 2020 à 12:09

Hello,

Donc la prochaine étape... soumettre ton extension dans notre BDD et/ou celle de phpbb.com ?
Skouat
Resp. Traducteurs
Resp. Traducteurs
 
Messages: 14630
Enregistré le: 02 Avr 2008 à 20:47

Re: [1.1.0] View Friends

Messagepar Ady » 28 Juil 2020 à 12:34

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 ^^
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Re: [1.1.0] View Friends

Messagepar Skouat » 28 Juil 2020 à 15:08

Ady a écrit: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.
Skouat
Resp. Traducteurs
Resp. Traducteurs
 
Messages: 14630
Enregistré le: 02 Avr 2008 à 20:47

Re: [1.1.0] View Friends

Messagepar Ady » 28 Juil 2020 à 19:24

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

Merci pour ton aide Skouat :)
Avatar de l’utilisateur
Ady
Posteur néophyte
Posteur néophyte
 
Messages: 72
Enregistré le: 27 Aoû 2018 à 13:00

Précédente

Retourner vers Extensions en développement

Qui est en ligne

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