Soucis de liaison reclamations <-> produits sur une commande

[Shimrra]

Salut,

 

Lorsque l'on fait une réclamation sur une commande passée, on peut choisir un produit sur lequel porte la réclamation. Soucis ? La liaison ne se fait pas !

 

Pourquoi ?

=> /controllers/front/OrderDetailController.php

l.76 dans la fonction postProcess

if ($id_product = (int)Tools::getValue('id_product') && $order->orderContainProduct((int)$id_product))

 

Ma correction :

if (($id_product = (int)Tools::getValue('id_product')) && $order->orderContainProduct((int)$id_product))

 

Explication : le test && se fait avant d'attribuer la valeur de $id_product. Du coup on a une notice (si activé) et la fonction orderContainProduct() reçoit une variable non définie et retourne donc false.

 

n'ayant pas réussi à faire la remontée sur github, si vous pouviez remonter ça au dev. merci.

 

Edit : on m'a expliqué comment faire sur Github, j'ai donc proposé ma correction si jamais personne ne transmets d'ici

Edited June 27, 2013 by Shimrra (see edit history)