Skouat a écrit : ↑08 juin 2021 à 20:55
Sachant que phpBB permet déjà d'uploader des images sur le forum, donc faire de l'auto-hébergement, c'est quoi le plus de cette extension ?
L'upload par fichier joint ? C'est vrai que pour la gestion des orphelins c'est pas mal mais je ne trouve pas ça très pratique pour l'utilisateur ni très beau avec la bordure autour, pas pratique pour la mise en page. Puis, il n'y a pas l'option pour redimensionner. Enfin, c'est mon avis perso.
Après, peut-être que j'ai loupé quelque chose.
Si mon extension est vraiment inutile, tant pis, je me serai bien amusé ^^
Skouat a écrit : ↑08 juin 2021 à 20:55
Je ne vais pas procéder à une revue de code, mais là rapidement je note que tu n'appliques pas les directives de phpBB en matière de nommage de fichiers, méthodes, classes.
Alors pour les noms des fichiers/classes, en effet, j'ai pas fait gaffe, l'habitude. Par contre, les méthodes, je vois pas bien. Lowercase, explicite, avec des verbes... Il manque le préfixe "phpbb_" ? Je pensais que c'était seulement pour le développement du core pas des extensions.
Je suis sur GitLab donc je n'ai pas les fameuses GitHub Actions. Mais je regarderai quand même voir si je peux adapter à GitLab CI.
De toute façon, à terme, je comptais mettre en place une CI pour le code style et les unit tests (qui sont encore à faire).
J'applique les corrections sur les noms de fichiers/classes dès que possible et je reposte une release.
Merci en tout cas pour votre temps à tous les deux !