Re: [Galette-devel] Les plugins peuvent-ils étendre le comportement de G

Bonjour Johan,

Merci pour ton retour.

Pour la migration vers Slim4, je ne pense pas que ce soit un gros sujet, car il semble que seul l'appel au container sera à revoir (l'event dispatcher est utilisé en injection de dépendance)..
Pour la compatibilité PHP7.4, je vais vérifier ce soir. Pour le moment je n'ai testé qu'en 7.2.
Pour le déclenchement, au départ je souhaitais les déclencher depuis la méthode store de l'entité Adhérent (je suis d'accord avec toi, ce serait plus logique), mais sauf erreur de ma part, je n'ai plus accès à l'application Slim à ce moment-là, c'est pour ça que j'ai pris le raccourci en l'ajoutant à la route. Je pense rester à cet endroit sauf si tu m'indiques comment l'intégrer aux Entity.
Je ne suis pas très à l'aise avec les tests, d'autant plus qu'ils devraient toucher les plugins et des événements asynchrones... Je regarderai si j'arrive à en ajouter, mais je ne promets rien ;-)

Je devrai pouvoir te faire un Pull-Request d'ici la fin de semaine.

Bonne semaine,

Nicolas.

Le dim. 16 févr. 2020 à 20:23, Johan Cwiklinski <johan@xxxxxxxx> a écrit :

Hello,

Le 16/02/2020 à 18:02, Nicolas Béhier-Dévigne a écrit :
> Bonjour Johan,
> Je crois avoir réussi le test. J'ai proposé une solution :
> https://github.com/nbehier/galette/commit/473b19cb04fa87a920d8235b3c2ff6d8158733ff
> J'ai ajouté une méthode loadEventProviders au chargement des plugins,
> qui charge tous les écouteurs d'événements des plugins s'ils possèdent
> la classe PluginEventProvider.
> J'ai ajouté deux événements member.add et member.update pour les tests.
> J'ai créé un plugin avec le fichier lib/PluginEventProvider.php sur la
> base de https://event.thephpleague.com/2..0/listeners/providers/ et je
> récupère bien l'objet member et je peux en faire ce que je veux (sur la
> capture ci-jointe, tu peux voir que j'ai ajouté un log avec le fullname
> de l'adhérent).
> Est-ce que cela te paraît aller dans le bon sens ?

Ouais, tout à fait ; ça me semble nickel :)

Je n'ai pas eu le loisir de regarder de très près, mais l'approche me
semble tout à faire correcte, et s'intègre sans trop de difficultés (le
diff reste très light, c'est plutôt bon !).

Juste une remarque au sujet de la lib, qui ne semble plus maintenue
(rien depuis 2 ans, alors que slmi4 existe depuis un moment maintenant).
Bon, dans l'absolu, ça ne fait qu'un problème de plus pour la migration
à Slim4, ce n'est as si grave.

En revanche, l'auteur indique clairement que sa lib n'est pas trop
prévue pour fonctionner en PHP 7... Je ne sais pas trop pourquoi, mais
il faut clairement que les libs supportent les mêmes versions de PHP que
Galette.

Ceci étant dit, je n'ai pas de solution alternative à proposer, si ça
fait le boulot, y compris en PHP 7.4, ce sera bon.

Enfin, juste une note technique ; je ne suis pas certain que les
évènements devraient être déclenchés depuis les routes. L'ajout d'un
adhérent pourrait venir d'un autre endroit (même si ce n'est pas le cas
actuellement), et dans ce cas les évènements ne seraient pas lancés.
Par ailleurs, quelques tests sur cette partie pourraient être
intéressants ; et ne sont pas possibles sur les routes :/

Merci en tous cas ! :)

++
--
Johan

--
Galette development - http://galette.eu/

List documentation: https://listengine.tuxfamily.org/lists.galette.eu/devel/