Skip to content

v2

Maïeul a demandé de fusionner gh-c1f65940/1/unknown/refs/pull/1/head vers master

@marcimat @cerdic besoin de vos avis avant de merger.

La v2 n'apporte fonctionnellement rien, mais elle nettoie un peu la manière dont le code est organisée, pour que cela soit plus conforme aux conventions.

En particulier, je passe des fonctions de pipelines dans le fichier _pipelines.php, et je les nommes homonymes au pipeline. Le but est simplement de pouvoir mieux s'y retrouver dans les futurs devs.

Ce que ca casse :

  • pour des gens qui up leurs plugins sans passer par ?exec=admin_plugins (via git par ex), ca marchera plus sans reinitialisation de tmp/cache_charger_pipelines.php. C'est la principale hésitation de mon côté.
  • pour des gens qui utiliseraient directement la fonction typo_guillemets_remplacements, cela casserait les choses, mais je ne vois pas de cas d'usage, puisque de toute facon la fonction était appellée en post_propre.

Donc : à mon avis faible risque, petit gain de maintenance. Mais je veux pas prendre de risque de me faire engueuler après :)

Rapports de requête de fusion

Chargement en cours