Skip to content

Centralize QCD page builder hook registration#979

Merged
CySSoO merged 1 commit intomasterfrom
cyssoo/complete-installhooks-for-qcd-hooks
Feb 26, 2026
Merged

Centralize QCD page builder hook registration#979
CySSoO merged 1 commit intomasterfrom
cyssoo/complete-installhooks-for-qcd-hooks

Conversation

@CySSoO
Copy link
Contributor

@CySSoO CySSoO commented Feb 26, 2026

Motivation

  • Assurer l'enregistrement explicite et cohérent des hooks QCD utilisés par le module pour éviter des divergences entre install, upgrade et le flux d'auto-réparation.
  • Éviter les enregistrements redondants disséminés dans checkHooks() et fournir une méthode idempotente partagée pour faciliter la maintenance.

Description

  • installHooks() a été modifiée pour déléguer l'enregistrement des hooks QCD à une nouvelle méthode dédiée et retirer l'inscription redondante de filterQcdPageBuilderDeclarativeBlocks de la liste locale.
  • Ajout de la méthode privée registerQcdBuilderHooks(): bool qui centralise l'enregistrement idempotent des hooks filterQcdPageBuilderBackOfficeTargets, filterQcdPageBuilderDeclarativeBlocks et filterQcdPageBuilderThirdPartyBlockFrontRender, et ajoute conditionnellement filterQcdPageBuilderThirdPartyBlockFrontAssets si le hook correspondant est implémenté.
  • checkHooks() conserve son rôle pour la rétrocompatibilité mais délègue désormais l'inscription des hooks QCD à registerQcdBuilderHooks() afin d'éviter la duplication et les divergences entre les flux d'installation/upgrade/auto-réparation.

Testing

  • Exécution de la vérification de syntaxe PHP avec php -l everblock.php qui a renvoyé "No syntax errors detected".
  • Le fichier everblock.php a été modifié et validé localement sans erreurs de syntaxe.

Codex Task

@CySSoO CySSoO merged commit ad58369 into master Feb 26, 2026
2 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant