Skip to content

Harmonise le choix de template des blocs pages et factorise le markup#985

Merged
CySSoO merged 1 commit intomasterfrom
cyssoo/refactor-page-templates-and-logic
Feb 26, 2026
Merged

Harmonise le choix de template des blocs pages et factorise le markup#985
CySSoO merged 1 commit intomasterfrom
cyssoo/refactor-page-templates-and-logic

Conversation

@CySSoO
Copy link
Contributor

@CySSoO CySSoO commented Feb 26, 2026

Motivation

  • Eviter d’écraser implicitement context['template'] pour les blocs « latest pages » et garder pages.tpl comme source par défaut sauf règle métier explicite.
  • Permettre d’appliquer le template alternatif pages-alt.tpl uniquement quand une condition claire est fournie (par ex. template_variant=alt).
  • Réduire la duplication entre pages.tpl et pages-alt.tpl en factorisant le rendu des cards de pages.

Description

  • Ajout de la méthode shouldUseAlternatePagesTemplate() dans src/Service/QcdThirdPartyBlockRenderer.php qui vérifie template_variant dans le contexte.
  • renderLatestPagesContext() ne force plus context['template'] systématiquement et ne définit pages-alt.tpl que si shouldUseAlternatePagesTemplate() renvoie true.
  • Factorisation du markup commun dans un partial Smarty views/templates/front/_partials/pages-list-items.tpl et inclusion de ce partial depuis views/templates/front/pages.tpl et views/templates/front/pages-alt.tpl.
  • Ajout d’un fichier de sécurité views/templates/front/_partials/index.php et mise à jour de la whitelist config/allowed_files.php pour inclure les nouveaux fichiers templates/partials.

Testing

  • php -l src/Service/QcdThirdPartyBlockRenderer.php a été exécuté et a réussi sans erreurs de syntaxe.
  • php -l config/allowed_files.php a été exécuté et a réussi sans erreurs de syntaxe.
  • Tentative d’un screenshot via Playwright contre http://localhost a échoué (échec attendu dans cet environnement car le service local n’est pas accessible), donc aucun rendu visuel n’a été validé ici.

Codex Task

@CySSoO CySSoO merged commit ccc8d11 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