search

YesWiki / yeswiki

YesWiki is a wiki system written in PHP, including extensions making collaboration more simple (databases, maps, easy editing, bootstrap themes,...).
https://yeswiki.net
GNU Affero General Public License v3.0
107 stars 55 forks source link

feat(MigrationService): create events #1156

Closed J9rem closed 5 months ago

J9rem commented 5 months ago

@seballot je te propose cette PR (pas testée de mon côté) pour intégrer deux évènements PHP avant et après migration

mrflos commented 5 months ago

@seballot si c'est ok pour toi, je merge et je teste une dernière fois avant de merger ta PR.
Je trouve utile d'avoir des events, qui a terme pourraient générer des logs et des webhooks, mais par contre, je ne pense pas qu'il faudrait les utiliser dans des actions de postupdate comme l'exemple de recréer les bases bazar du lms. Normalement l'usager n'est pas sensé casser des trucs, et si c'est le cas, vaut mieux rejouer la migration (effacer le triple), que d'alourdir l'upgrade avec des vérifications post install.