Error logs (null object)

eolia

A ajouter.
Je ne sais pas pourquoi chez toi cette variable n'est pas systématiquement remplie.
On peut aussi récupérer l'ID produit comme ça:
$id_product = (int)Tools::getValue('id_product');

Olivier75009

Le problème c'est qu'elle l'est. d((int)$params['product']->id) avant ou après $cache_id me retourne bien l'ID produit.
Vu qu'il ne trouve pas l'index product, ne devrais-je pas faire un array_key_exists ?

Cannot use isset() on the result of an expression (you can use "null !== expression" instead)

eolia

parce que tu fais un d() qui est un die et arrête le script mais s'il est appelé plusieurs fois tu ne vois pas les autres (ou alors faire un p())

Olivier75009

du coup :
if(array_key_exists('product',$params)){
$cache_id = 'crossselling|productfooter|'.(int)$params['product']->id;
}
else{
$cache_id = 'crossselling|productfooter|'.(int)Tools::getValue('id_product');
}

Olivier75009

Bon en fait, si je clique sur un produit du crossselling, même erreur (ligne 256). Donc :

$id_product = (int)Tools::getValue('id_product');
$cache_id = 'crossselling|productfooter|'.$id_product;

    if (!$this->isCached('crossselling.tpl', $this->getCacheId($cache_id))) {
        $final_products_list = $this->getOrderProducts(array($id_product));

Olivier75009

@eolia p(((int)$params['product']->id)) donne bien l'ID puis Notice: Undefined index: product in /var/www/clients/client1/web4/web/modules/crossselling/crossselling.php on line 253

Notice: Trying to get property 'id' of non-object in /var/www/clients/client1/web4/web/modules/crossselling/crossselling.php on line 253
puis 0

eolia

donc tu mets le if(isset(... comme je t'ai dit

Olivier75009

@eolia Cannot use isset() on the result of an expression (you can use "null !== expression" instead)
NB: c'est un peu pénible cette contrainte de temps pour poster une réponse.

if(null==((int)$params['product']->id)){
return;
} ben toujours le problème mais n'affiche plus les produits crossselling

Olivier75009

@Olivier75009 a dit dans Error logs (null object) :

Bon en fait, si je clique sur un produit du crossselling, même erreur (ligne 256). Donc :

$id_product = (int)Tools::getValue('id_product');
$cache_id = 'crossselling|productfooter|'.$id_product;
    if (!$this->isCached('crossselling.tpl', $this->getCacheId($cache_id))) {
        $final_products_list = $this->getOrderProducts(array($id_product));

ça fait le job.

eolia

Tu es à quel endroit là, sur quelle page ?

Olivier75009

Sur la page produit (hookProductFooter de toute façon)

Grrrr:
En tant que nouvel utilisateur, vous ne pouvez poster que toutes les 120 seconde(s) jusqu'à ce que vous obteniez une réputation de 3 - patientez avant de publier de nouveau.

eolia

Ton pb est ailleurs, pas normal que le module soit appelé 2 fois (et à mon avis, c'est un ajax qui l'appelle la seconde fois et forcément il n'y a pas d'id_product).
Mois si je fais un p($params['product']->id); je n'ai qu'une ligne avec le bon id.

Olivier75009

Disons que j'ai une valeur (l'ID) et après la notice un 0.
<xmp style="text-align: left;">5992</xmp>
...
Notice: Undefined index: product in /var/www/clients/client1/web4/web/modules/crossselling/crossselling.php on line 269 Notice: Trying to get property 'id' of non-object in /var/www/clients/client1/web4/web/modules/crossselling/crossselling.php on line 269 
<xmp style="text-align: left;">0</xmp>

Je suppose que le second est $final_products_list = $this->getOrderProducts(array($params['product']->id

eolia

le second te parle de la ligne 269

Olivier75009

la ligne 269 c'est justement : p(((int)$params['product']->id));

eolia

Donc oui, ton hook est appelé 2 fois et ce n'est pas normal !

Olivier75009

Hooks visibles :
displayHeader, displayProductFooter

Hooks invisibles :
actionOrderStatusPostUpdate, displayShoppingCartFooter

!$this->registerHook('productFooter') ||
!$this->registerHook('header') ||
!$this->registerHook('shoppingCart') ||
!$this->registerHook('actionOrderStatusPostUpdate')

eolia

je veux dire que ton module est appelé 2 fois, c'est ça qui n'est pas normal.

Olivier75009

Oui oui, j'avais bien compris.