Eolia sur PhenixTweet

2 mois

Alors j'ai un casse tête. je dois passer à coté de quelque chose.
j'essaie d'ajouter des infos dans un produit

$this->registerHook('displayProducFooter')

public function hookDisplayProductFooter($params)
{
return 'Dans la place';
}
rien ne s'affiche
pourtant ok dans "positions"
et ok dans product.tpl

des idées ?

exemple tiré du bouquin de serny

Eolia @eolia

2 mois

a répondu au Message de herve_02 il y a 2 mois

Disons que le code exact est displayFooterProduct et non pas displayProductFooter.

Ca me surprend que Fabien ait fait l'erreur.

herve_02 @herve_02

2 mois

a répondu au Message de Eolia - 2 mois

en fait le livre parle de
displayProductTabContent
et HookDisplayProductTabContent
mais cela ne marchait pas et comme il semble y avoir un mic mac avec displayProductTab et Tabcontent, je me suis rabattu sur le footer..
que javais trouvé dans le fichier product.tpl du theme par defaut (je suis parti de lui)

{if isset($HOOK_PRODUCT_FOOTER) && $HOOK_PRODUCT_FOOTER}{$HOOK_PRODUCT_FOOTER}{/if}

Eolia @eolia

2 mois

a répondu au Message de herve_02 il y a 2 mois

Cette variable Smarty est définie dans ProductController.php

CODE
$this->context->smarty->assign(array(
            'HOOK_PRODUCT_FOOTER' => Hook::exec('displayFooterProduct', array(
                'product' => $this->product, 
                'category' => $this->category
            ))
        ));

herve_02 @herve_02

2 mois

a répondu au Message de Eolia - 2 mois

Je ne suis pas un dieu du nomage, mais appeler un truc HOOK_PRODUCT_FOOTER qui en fait appelle displayFooterProduct faut le vouloir

Eolia @eolia

2 mois

a répondu au Message de herve_02 il y a 2 mois

Il faut connaitre l'historique.
Les premiers hooks étaient tous écrits de cette façon dans les tpl des 1ères 1.4 puis tout le système a été repensé en différenciant les hooks de type display et ceux de type action. Sauf que vu le turn-over qu'il y a eu chez Presta c'était rarement les mêmes qui finissaient les correctifs que ceux qui les avaient commencés...
Il n'y a jamais eu de roadmap chez Presta et le temps que les stagiaires connaissent le code maison, leur stage touchait à leur fin.

Dernier exemple (et oui, j'essaye toujours de les empêcher de faire des bêtises^^)
https://github.com/PrestaS...
Oh, il y a une étoile rouge dans le tpl qui signifie "requis" alors on force ces 2 champs à être obligatoires !

Sauf que...

BO Countries : Set two fields `call_prefix` & `zip_code_format` as non-required by Progi1984 · Pull Request #38846 · PrestaShop/PrestaShop · GitHub

Questions Answers Branch? 9.0.x Description? BO Countries : Set two fields call_prefix & zip_code_format as non-requiredAs explained by @Eolia, Country can have no zip code format, and the ca...

https://github.com/PrestaShop/PrestaShop/pull/38846

Eolia @eolia

a répondu au Message de Eolia - 2 mois

Je pourrais écrire un bouquin sur ce type d'anecdotes, comme les 2 colonnes de ps_order_detail qui s'appellent product_id et product_attribute_id alors que DANS TOUTES LES AUTRES TABLES elles s'appellent toujours id_product et id_product_attribute parce que le stagiaire de l'époque ne savait pas faire une jointure.

Bon, quand on s'est fait avoir 3 ou 4 fois dans nos requêtes SQL à cause de cette table on finit par s'en souvenir...

S'identifier

BO Countries : Set two fields `call_prefix` & `zip_code_format` as non-required by Progi1984 · Pull Request #38846 · PrestaShop/PrestaShop · GitHub