[RC] My Topics 1.0.9 [compatible 3.2.x]

Pour les extensions destinées à phpBB 3.1.x

Modérateur: Equipe

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

Re: My Topics 1.0.3

Messagepar ABDev » 30 Oct 2015 à 18:49

Bonsoir,
La syntaxe sur cette ligne n'est pas correcte :

Code: Tout sélectionner
$params  "author=" $this->user->data['username'] . "&sf=firstpost&sr=topics"

Il faut utiliser les entités HTML dans les paramètres.
Donc & doit être remplacé par &

Ce qui donnera donc :

Code: Tout sélectionner
$params  "author=" $this->user->data['username'] . "&sf=firstpost&sr=topics"


Par contre, pour des soucis de lecture pour les autres développeurs, ce serait très sympa de respecter la syntaxe du logiciel.
Genre append_sid() au lieu de append_sid () ou encore public function build_url($event) au lieu de public function build_url ($event) pour ne citer que ces quelques exemples ...
Pour se payer le luxe d'être feignant, il faut sans cesse faire travailler ses méninges ! (Korben)

On clique sur le bouton Répondre plutôt que le bouton Citer ...
Un problème avec une extension ? Pas de lien vers celle-ci ? Sujet ignoré !
Avatar de l’utilisateur
ABDev
Programmationateur
Programmationateur
 
Messages: 15689
Enregistré le: 12 Mai 2005 à 22:54

Re: My Topics 1.0.3

Messagepar pierredu » 30 Oct 2015 à 20:50

OK pour les entités HTML. Je vais corriger.
Sinon, chacun ses habitudes.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 946
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: My Topics 1.0.3

Messagepar PeutEtre » 01 Avr 2016 à 21:50

Bonjour pierredu,

Toujours en alpha ou je peux y aller en production ?

Cordialement
PeutEtre
Posteur néophyte
Posteur néophyte
 
Messages: 82
Enregistré le: 20 Fév 2012 à 11:07

Re: My Topics 1.0.3

Messagepar pierredu » 02 Avr 2016 à 07:12

Je n'ai pas eu le temps de passer par la validation, mais ça tourne sur mon forum sans difficultés depuis quelques mois.
Donc je passe ici en RC.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 946
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: My Topics 1.0.3

Messagepar Skouat » 02 Avr 2016 à 09:49

Bonjour,

ABDev a écrit:Par contre, pour des soucis de lecture pour les autres développeurs, ce serait très sympa de respecter la syntaxe du logiciel.
Genre append_sid() au lieu de append_sid () ou encore public function build_url($event) au lieu de public function build_url ($event) pour ne citer que ces quelques exemples ...

pierredu a écrit:Sinon, chacun ses habitudes.


Oui chacun ses habitudes dans la méthode de développement.
Par contre pour le côté "présentation" du code, ceci est défini dans le codes guidelines de phpBB.

Si on va plus loin, l'univers du dev PHP tente à se standardiser au niveau de l'écriture du code. Car comme l'a indiqué ABDev, ça facilite la lecture du code pour les autres développeurs.
Donc pour PHP on s'appuie sur les PSR (PSR-0, PSR-1, PSR-2, etc.)
http://blog.nicolashachet.com/technolog ... tions-psr/

En l'occurence, pour la déclaration des méthodes/fonctions => https://github.com/php-fig/fig-standard ... 43-methods
Method names MUST NOT be declared with a space after the method name.

(pour les non anglophones... qui passeraient par là...)
Les méthodes NE DOIVENT PAS être déclarées avec une espace placée après le nom de la méthode.


A+
Mes MODS

Ultima-World Hébergé par phpBB-Services
Code parrainage : 1241646554
Skouat
Traducteur
Traducteur
 
Messages: 13493
Enregistré le: 02 Avr 2008 à 19:47

Re: My Topics 1.0.3

Messagepar pierredu » 02 Avr 2016 à 10:20

Tu as raison...
Je vais aller faire du ménage.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 946
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: My Topics 1.0.3

Messagepar soda » 02 Avr 2016 à 10:54

Cela étant, phpBB ne respect pas les PSR sur certains points, Donc il ne faut pas les suivre à 100% si l'on souhaite se faire valider son extension
Appelez moi..... Ismaël

phpBB-Services.com: L'hébergement et l'assistance de votre forum phpBB en toute tranquillité.
soda
Posteur expérimenté
Posteur expérimenté
 
Messages: 343
Enregistré le: 25 Déc 2006 à 11:55
Localisation: Sur mon cheval blanc

Re: My Topics 1.0.3

Messagepar Skouat » 02 Avr 2016 à 14:00

@soda:
Tout a fait.
Donc en gros tu suis les PSR, sauf si dans le codes guidelines de phpBB il est dit le contraire.

Exemple : le nommage de methodes/fonctions n'est pas à présenter selon les PSR, mais selon ce que phpBB indique.
Mes MODS

Ultima-World Hébergé par phpBB-Services
Code parrainage : 1241646554
Skouat
Traducteur
Traducteur
 
Messages: 13493
Enregistré le: 02 Avr 2008 à 19:47

Re: My Topics 1.0.3

Messagepar PeutEtre » 02 Avr 2016 à 14:49

pierredu a écrit:Je n'ai pas eu le temps de passer par la validation, mais ça tourne sur mon forum sans difficultés depuis quelques mois. Donc je passe ici en RC.

Re,

Tentative d'installation depuis le download de GitHub

/ext/ développeur / nom de l'extension

Qu'est-ce que je mets en nom de développeur
Qu'est-ce que je mets en nom de l'extension

Pour l'instant elle n'est pas reconnue.

Cordialement
PeutEtre
Posteur néophyte
Posteur néophyte
 
Messages: 82
Enregistré le: 20 Fév 2012 à 11:07

Re: My Topics 1.0.3

Messagepar pierredu » 02 Avr 2016 à 15:03

C'est indiqué dans le fichier readme.md : lmdi et mytop.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 946
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: My Topics 1.0.3

Messagepar Skouat » 02 Avr 2016 à 15:48

Plop !

PB de nommage de clé de langue ?
https://github.com/pierrdu/lmdi_mytopic ... op.php#L21
MY_TOPICS => your topics
Ce n'est pas très logique :roll:

https://github.com/pierrdu/lmdi_mytopic ... hp#L53-L54
la déclaration de $params n'a pas de raison d'être.
Hormis le côté esthétique, là tu déclare une variable (donc consomation de mémoire) alors qu'elle ne sert qu'à la ligne suivante.
Mes MODS

Ultima-World Hébergé par phpBB-Services
Code parrainage : 1241646554
Skouat
Traducteur
Traducteur
 
Messages: 13493
Enregistré le: 02 Avr 2008 à 19:47

Re: My Topics 1.0.4

Messagepar pierredu » 02 Avr 2016 à 16:10

Je ne doutais pas que quelqu'un soulèverait la question.
Dans le menu d'accès rapide, la chaîne française est 'Mes messages'. Du coup, j'ai mis 'Mes sujets'.

Dans le sujet ouvert sur phpbb.com, quelqu'un m'a fait remarquer que la formulation anglaise est 'Your posts' et que le pendant logique devrait donc être 'Your topics'.

Pour la coupure en trois lignes, c'est vrai que l'on pourrait tout mettre en une seule ligne. Mais la lisibilité y gagne.
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 946
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: My Topics 1.0.4

Messagepar Skouat » 02 Avr 2016 à 17:06

pierredu a écrit:Dans le sujet ouvert sur phpbb.com, quelqu'un m'a fait remarquer que la formulation anglaise est 'Your posts' et que le pendant logique devrait donc être 'Your topics'.

Effectivement c'est "Your topics"
Dans ce cas contourne la "problèmatique"...
Dans le pack de langue "Your posts" est assigné à la clé de langue SEARCH_SELF
nomme ta clé de langue POST_SELF :wink:
pierredu a écrit:Pour la coupure en trois lignes, c'est vrai que l'on pourrait tout mettre en une seule ligne. Mais la lisibilité y gagne.

Pas forcément sur 1 seule ligne, mais sur 2.
La 1ère ligne ne fait que concaténer une chaine de caractère.
La 2ème et 3ème font appel à des fonctions/méthodes, donc il est "logique" de les laisser séparées.
Mes MODS

Ultima-World Hébergé par phpBB-Services
Code parrainage : 1241646554
Skouat
Traducteur
Traducteur
 
Messages: 13493
Enregistré le: 02 Avr 2008 à 19:47

Re: My Topics 1.0.4

Messagepar PeutEtre » 03 Avr 2016 à 09:24

Bonjour,
Installation sans problème.
Reconnaissance de langue sans problème.
Par contre, sous prosilver, extensions-developpement-en-cours-phpbb31/sujet206168.html serait réellement utile et logique.
Est-ce que c'est jouable côté extension ou est-ce que l'utilisateur doit faire un hack (ce qui serait contraire à l'esprit des extensions).

Est-ce qu'il est possible d'avoir 3 liens d'accès rapides :
Mes topics
Les topics où j'ai contribué
Mes messages

Merci
PeutEtre
Posteur néophyte
Posteur néophyte
 
Messages: 82
Enregistré le: 20 Fév 2012 à 11:07

Re: My Topics 1.0.4

Messagepar pierredu » 03 Avr 2016 à 10:32

Ben ça ferait un peu double emploi.
Mes sujets est nouveau.
Mes messages est déjà là.
Les sujets auxquels j'ai contribué, c'est la même chose que mes messages, non ?
Avatar de l’utilisateur
pierredu
Extensions
Extensions
 
Messages: 946
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

PrécédenteSuivante

Retourner vers Extensions en développement

 


  • Articles en relation
    Réponses
    Vues
    Dernier message

Qui est en ligne

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