Portage MOD Introduciator + tutoriel

Aide aux auteurs d'extensions pour phpBB 3.2.x

Modérateur: Equipe

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

Re: Portage MOD Introduciator + tutoriel

Messagepar Informpro » 14 Fév 2019 à 22:51

Si tu veux avancer, je te conseille d'aller sur IRC ou sur discord pour en parler directement avec les intéressés. Ils pourront te guider et t'aiguiller mieux que nous pour que ton patch soit accepté.
Informpro
Apprenti-posteur
Apprenti-posteur
 
Messages: 183
Enregistré le: 06 Aoû 2012 à 20:24

Re: Portage MOD Introduciator + tutoriel

Messagepar galixte » 15 Fév 2019 à 01:17

Communau EzCom
Image
« Traductions d’extensions & styles pour phpBB 3.1.x & 3.2.x ».
ImageListes de toutes les extensions recensées pour phpBB 3.1.x & 3.2.x.
ImageListes de tous les styles recensés pour phpBB 3.1.x & 3.2.x.

Image Tu as un forum et tu veux aussi un site web ? Regarde par ici.
Avatar de l’utilisateur
galixte
Floodeur universel
Floodeur universel
 
Messages: 10342
Enregistré le: 08 Mar 2004 à 01:40
Localisation: Sud de la France.

Re: Portage MOD Introduciator + tutoriel

Messagepar Feneck91 » 15 Fév 2019 à 07:46

Merci Informpro pour les infos (sans les liens) et merci à galixte pour les liens.
Bon, site bloqué par l'entreprise, je verrais ça chez moi.
Feneck91
Maître-posteur
Maître-posteur
 
Messages: 424
Enregistré le: 31 Déc 2007 à 12:02
Localisation: Essonne (91)

Re: Portage MOD Introduciator + tutoriel

Messagepar Feneck91 » 15 Fév 2019 à 21:50

Scritinizer me sont des trucs pas compréhensibles, genre ça :
https://scrutinizer-ci.com/docs/tools/p ... ency-paths

Ou il me demande d'écrire ça :
Code: Tout sélectionner
/** @scrutinizer ignore-call */ get_username_string


Je pense que là je vais stopper les dev et tenter d'améliorer le code...

Déjà je me suis fait défoncer ici : https://www.phpbb.com/community/viewtop ... #p15201031

Je sens que ça va être super chiant comme la dernière fois, ça me fait un peu peur…
Feneck91
Maître-posteur
Maître-posteur
 
Messages: 424
Enregistré le: 31 Déc 2007 à 12:02
Localisation: Essonne (91)

Re: Portage MOD Introduciator + tutoriel

Messagepar Skouat » 16 Fév 2019 à 18:12

Scrutinizer ne vas pas corriger ton code, il met juste en évidence des incohérences.
Dans la cas que tu cites c'est un faux positif. A toi de faire la part des choses tout de même ;)

Concernant le commentaire de 3Di, je le rejoins complètement.
Skouat
Traducteur
Traducteur
 
Messages: 13939
Enregistré le: 02 Avr 2008 à 19:47

Re: Portage MOD Introduciator + tutoriel

Messagepar pierredu » 16 Fév 2019 à 18:42

3Di est un homme posé.
On ne peut pas dire qu'il t'a défoncé.
Juste un peu mouché.

Mets de l'ordre dans ton code et soumets l'extension ici. On lui fera un traitement selon les règles.
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1138
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: Portage MOD Introduciator + tutoriel

Messagepar Feneck91 » 16 Fév 2019 à 20:01

Le pire c'est que je le comprend parfaitement. Je suis en train de pomper le code de Skouat afin de définir un contrôleur par page (en plus simple), mon module est trop long et je le découpe un peu.
Je revois mes commentaires et je tente d'améliorer les choses. Il y a un outils qui permet de faire une passe spécifique pour phpBB comme on avait avant pour les MODS ?
Je relis tout, remet en forme et je reviens vers vous. Le problème que j'ai c'est qu'a part scrutinizer je ne sais pas trop quoi chercher et surtout ce qu'on va me reprocher...
Feneck91
Maître-posteur
Maître-posteur
 
Messages: 424
Enregistré le: 31 Déc 2007 à 12:02
Localisation: Essonne (91)

Re: Portage MOD Introduciator + tutoriel

Messagepar Skouat » 16 Fév 2019 à 20:34

Feneck91 a écrit:Il y a un outils qui permet de faire une passe spécifique pour phpBB comme on avait avant pour les MODS ?

https://www.phpbb.com/extensions/epv/
Skouat
Traducteur
Traducteur
 
Messages: 13939
Enregistré le: 02 Avr 2008 à 19:47

Re: Portage MOD Introduciator + tutoriel

Messagepar Feneck91 » 17 Fév 2019 à 19:58

Grosse évoution du code, EPV ne me ressort plus d'erreur (un critique mais dû à un commentaire sur le patch de l'event non supporté par phpBB) par contre j'ai plein de warnings du genre :
PHP Warning: preg_match(): Compilation failed: PCRE does not support \L, \l, \N{name}, \U, or \u at offset 67 in D:\Developpement\Web\T
Warning: preg_match(): Compilation failed: PCRE does not support \L, \l, \N{name}, \U, or \u at offset 67 in D:\Developpement\Web\Téléchargements\epv\src\Tests\Tests\epv_test_validate_languages.php on line 61
PHP Warning: preg_match(): Compilation failed: PCRE does not support \L, \l, \N{name}, \U, or \u at offset 67 in D:\Developpement\Web\T
Warning: preg_match(): Compilation failed: PCRE does not support \L, \l, \N{name}, \U, or \u at offset 67 in D:\Developpement\Web\Téléchargements\epv\src\Tests\Tests\epv_test_validate_languages.php on line 61
PHP Warning: preg_match(): Compilation failed: PCRE does not support \L, \l, \N{name}, \U, or \u at offset 67 in D:\Developpement\Web\T


J'ai revu les commentaires, splitter les gros morceaux de code en plusieurs fichiers. J'espère être sur la bonne voie...
Je vous jure je fais des efforts !!!!!!
Feneck91
Maître-posteur
Maître-posteur
 
Messages: 424
Enregistré le: 31 Déc 2007 à 12:02
Localisation: Essonne (91)

Re: Portage MOD Introduciator + tutoriel

Messagepar pierredu » 19 Fév 2019 à 20:38

Et qu'est-ce que tu utilises comme switchs sur ton expression régulière ?
Avatar de l’utilisateur
pierredu
Resp. Extensions
Resp. Extensions
 
Messages: 1138
Enregistré le: 29 Mai 2011 à 05:49
Localisation: Paris

Re: Portage MOD Introduciator + tutoriel

Messagepar Feneck91 » 20 Fév 2019 à 23:30

Les messages sont ceux générés par l'outils https://www.phpbb.com/extensions/epv/

Moi je n'ai pas d'erreur ! ;-)
Feneck91
Maître-posteur
Maître-posteur
 
Messages: 424
Enregistré le: 31 Déc 2007 à 12:02
Localisation: Essonne (91)

Re: Portage MOD Introduciator + tutoriel

Messagepar Feneck91 » 26 Fév 2019 à 04:58

Quelqu'un de plus expérimenté pourrait me faire un rapide check et me donner quelques pistes des trucs qui ne vont pas ?
J'ai corrigé plein de trucs mais je pense que ça ne suffit pas...
Feneck91
Maître-posteur
Maître-posteur
 
Messages: 424
Enregistré le: 31 Déc 2007 à 12:02
Localisation: Essonne (91)

Re: Portage MOD Introduciator + tutoriel

Messagepar Skouat » 26 Fév 2019 à 21:49

Hello,

Dans introduciator_module.php
  • ligne 12 Référence invalide
  • Ligne 68Commentaire incorrect. La référence au controller n'est pas la bonne.
  • Les méthodes switch_mode() et do_action() sont inutiles. Autant remplacer la ligne 82 par
    Code: Tout sélectionner


            
    /** @type \phpbb\request\request $request Request object */
            $request = $phpbb_container->get('request');
            // Requests
            $action = $request->variable('action', '');

            $acp_controller->do_action($mode, $action); 

Dans adm/style
    Il y a un mix de twig/template phpBB. A un moment donné il faut choisir ;)

Dans acp_configuration_controller
Dans acp_explanation_controller
  • ligne 192 ces noms de domaines sont suceptibles d'exister. Ce n'est pas la cas aujourd'hui,
    mais rien ne dit que demain quelqu'un ne vas pas profiter de ton extension pour exploiter ces URL.
    donc à remplacer par http://domain.tld car le domain TLD .tld ne sera jamais mis en exploitation

Dans acp_general_controller

Dans introductiator_explain

Dans helper/introduciator_helper.php
    Ligne 247/248 Le typage de la variable n'est pas indiqué, donc tous les appels à cette méthode remonte un problème de typage.
Dans language/xx/info_acp_introduciator
  • Ce fichier étant chargé de manière globale, il ne doit contenir que les informations nécessaire au chargement des fichiers de langues de l'extension. Toutes les clés de langues dédiées à un module sont à placer dans un fichier dédié à se dernier et à charger à la demande.

Dans migrations/introductiator_migration_2_1_0
    Ligne 82 pas de type de déclaré. Voir ligne 65

Dans composer.json
  • l'indentation ne doit pas être faites avec des tabulations.

Attention, je n'ai fait qu'une relecture du code. Aucun tests n'a été effectué.

A+
Skouat
Traducteur
Traducteur
 
Messages: 13939
Enregistré le: 02 Avr 2008 à 19:47

Re: Portage MOD Introduciator + tutoriel

Messagepar Feneck91 » 27 Fév 2019 à 05:33

Merci BEAUCOUP !
Pas mal de petites merdes et des trucs qui n'existe plus .... bizarre bizarre... ça aurait du planter... comme le "return_display()", je me demande si ce n'est pas un reliquat de phpBB 3.2.x...
C'est tout ce qu'il me faut pour le moment, pour les tests j'ai beaucoup testé en tests unitaires (à la main), il faut que je voie pour faire des tests plus automatisés : beaucoup d'options et de cas différents.

Skouat a écrit:ligne 192 ces noms de domaines sont suceptibles d'exister. Ce n'est pas la cas aujourd'hui,
mais rien ne dit que demain quelqu'un ne vas pas profiter de ton extension pour exploiter ces URL.
donc à remplacer par http://domain.tld car le domain TLD .tld ne sera jamais mis en exploitation[/list]

Typiquement ce n'est pas une faille de sécurité, c'est juste un 'tag' pour moi qui est remplacé par la "bonne" URL quand je récupère les paramètres, donc on ne peut pas en faire grand chose. Pour autant, si tld ne pourra jamais être utilisé autant que je le change. C'est quoi tld et pourquoi celui-là ne pourra jamais être utilisé ? (question culture personnelle)

Il faut aussi que je m'occupe de ça car ça n'a pas l'air de bouger... Tu as déjà proposé des patchs ?
T'as vu j'ai "honteusement" pompé une partie de ton code pour splitter mes pages ACP en petits fichiers plus petits.
Feneck91
Maître-posteur
Maître-posteur
 
Messages: 424
Enregistré le: 31 Déc 2007 à 12:02
Localisation: Essonne (91)

Re: Portage MOD Introduciator + tutoriel

Messagepar galixte » 27 Fév 2019 à 16:02

TLD pour https://en.wikipedia.org/wiki/Top-level_domain; Pour l’instant il ne fait pas partie de la liste de ceux existants : http://stats.research.icann.org/dns/tld_report/. Ce sigle ne parle que à peu de personnes et est utilisé pour illustrer tout domaine de premier niveau dans les docs difficile d’imaginer qu’il puisse être libéré à la vente mais sait-on jamais tout est possible avec l’ICANN pour faire du beurre. ;)
Communau EzCom
Image
« Traductions d’extensions & styles pour phpBB 3.1.x & 3.2.x ».
ImageListes de toutes les extensions recensées pour phpBB 3.1.x & 3.2.x.
ImageListes de tous les styles recensés pour phpBB 3.1.x & 3.2.x.

Image Tu as un forum et tu veux aussi un site web ? Regarde par ici.
Avatar de l’utilisateur
galixte
Floodeur universel
Floodeur universel
 
Messages: 10342
Enregistré le: 08 Mar 2004 à 01:40
Localisation: Sud de la France.

PrécédenteSuivante

Retourner vers Aide au développement

 


  • Articles en relation
    Réponses
    Vues
    Dernier message

Qui est en ligne

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

cron