Problème d'override

[trofex]

Bonjour à tous!! :-)

 

Alors j'aurais une petite question, si il y aurait une âme charitable, ça serait grandement apprécié!

 

Je ne peux pas dire depuis quand le message à commencer à apparaître mais sur la page order quand j'ajoute un produit au panier,  j'ai ce message:

Warning: array_merge(): Argument #1 is not an array in /home/xxxxxx/public_html/override/classes/Carrier.php on line 84

Par contre c'est seulement quand je ne suis pas connecté en tant que client, lorsque je me connecte le message disparaît.

 

 

 

Donc quelqu'un pourrait m'éclairer sur la façon dont je pourrait corriger la situation ??

 

Merci d'avance!!!

 

 

[trofex]

<?php

 

class Carrier extends CarrierCore

{

public static function getAvailableCarrierList(Product $product, $id_warehouse, $id_address_delivery = null, $id_shop = null, $cart = null, &$error = array())

{

static $ps_country_default = null;

 

if ($ps_country_default === null)

$ps_country_default = Configuration::get('PS_COUNTRY_DEFAULT');

 

if (is_null($id_shop))

$id_shop = Context::getContext()->shop->id;

if (is_null($cart))

$cart = Context::getContext()->cart;

 

$id_address = (int)((!is_null($id_address_delivery) && $id_address_delivery != 0) ? $id_address_delivery : $cart->id_address_delivery);

if ($id_address)

{

$id_zone = Address::getZoneById($id_address);

 

// Check the country of the address is activated

if (!Address::isCountryActiveById($id_address))

return array();

}

else

{

// changed for Presto-Changeo carrier modules --->

$cookie = Context::getContext()->cookie;

$cookie_country = $cookie->id_country ? $cookie->id_country : $cookie->pc_dest_country;

$country = new Country((isset($cookie_country) && strlen($cookie_country) ? $cookie_country : $ps_country_default));

// <--- changed for Presto-Changeo carrier modules

$id_zone = $country->id_zone;

}

 

// Does the product is linked with carriers?

$cache_id = 'Carrier::getAvailableCarrierList_'.(int)$product->id.'-'.(int)$id_shop;

if (!Cache::isStored($cache_id))

{

$query = new DbQuery();

$query->select('id_carrier');

$query->from('product_carrier', 'pc');

$query->innerJoin('carrier', 'c',

'c.id_reference = pc.id_carrier_reference AND c.deleted = 0 AND c.active = 1');

$query->where('pc.id_product = '.(int)$product->id);

$query->where('pc.id_shop = '.(int)$id_shop);

 

$carriers_for_product = Db::getInstance(_PS_USE_SQL_SLAVE_)->executeS($query);

Cache::store($cache_id, $carriers_for_product);

}

else

$carriers_for_product = Cache::retrieve($cache_id);

 

$carrier_list = array();

if (!empty($carriers_for_product))

{

//the product is linked with carriers

foreach ($carriers_for_product as $carrier) //check if the linked carriers are available in current zone

if (Carrier::checkCarrierZone($carrier['id_carrier'], $id_zone))

$carrier_list[$carrier['id_carrier']] = $carrier['id_carrier'];

if (empty($carrier_list))

return array();//no linked carrier are available for this zone

}

 

// The product is not dirrectly linked with a carrier

// Get all the carriers linked to a warehouse

if ($id_warehouse)

{

$warehouse = new Warehouse($id_warehouse);

$warehouse_carrier_list = $warehouse->getCarriers();

}

 

$available_carrier_list = array();

$cache_id = 'Carrier::getAvailableCarrierList_getCarriersForOrder_'.(int)$id_zone.'-'.(int)$cart->id;

if (!Cache::isStored($cache_id))

{

$customer = new Customer($cart->id_customer);

$carrier_error = array();

$carriers = Carrier::getCarriersForOrder($id_zone, $customer->getGroups(), $cart, $carrier_error);

Cache::store($cache_id, array($carriers, $carrier_error));

} else

list($carriers, $carrier_error) = Cache::retrieve($cache_id);

 

$error = array_merge($error, $carrier_error); <-----------------------------Ligne 84-----------------

 

foreach ($carriers as $carrier)

$available_carrier_list[$carrier['id_carrier']] = $carrier['id_carrier'];

 

if ($carrier_list)

$carrier_list = array_intersect($available_carrier_list, $carrier_list);

else

$carrier_list = $available_carrier_list;

 

if (isset($warehouse_carrier_list))

$carrier_list = array_intersect($carrier_list, $warehouse_carrier_list);

 

$cart_quantity = 0;

 

foreach ($cart->getProducts(false, $product->id) as $cart_product)

if ($cart_product['id_product'] == $product->id)

$cart_quantity += $cart_product['cart_quantity'];

 

if ($product->width > 0 || $product->height > 0 || $product->depth > 0 || $product->weight > 0)

{

foreach ($carrier_list as $key => $id_carrier)

{

$carrier = new Carrier($id_carrier);

 

// Get the sizes of the carrier and the product and sort them to check if the carrier can take the product.

$carrier_sizes = array((int)$carrier->max_width, (int)$carrier->max_height, (int)$carrier->max_depth);

$product_sizes = array((int)$product->width, (int)$product->height, (int)$product->depth);

rsort($carrier_sizes, SORT_NUMERIC);

rsort($product_sizes, SORT_NUMERIC);

 

if (($carrier_sizes[0] > 0 && $carrier_sizes[0] < $product_sizes[0])

|| ($carrier_sizes[1] > 0 && $carrier_sizes[1] < $product_sizes[1])

|| ($carrier_sizes[2] > 0 && $carrier_sizes[2] < $product_sizes[2]))

{

$error[$carrier->id] = Carrier::SHIPPING_SIZE_EXCEPTION;

unset($carrier_list[$key]);

}

 

if ($carrier->max_weight > 0 && $carrier->max_weight < $product->weight * $cart_quantity)

{

$error[$carrier->id] = Carrier::SHIPPING_WEIGHT_EXCEPTION;

unset($carrier_list[$key]);

}

}

}

return $carrier_list;

}

}

 

Edited August 25, 2017 by trofex (see edit history)

[trofex]

Oui Désolé alors voilà !!

 

J'ai installé un module d'optimisation de htaccess et database ... mais je ne suis pas sûr que ça provient de là ... On dirait qu'il cherche la table mais quand je suis connecté alors le problème disparaît ...

[trofex]

J'avais mis par inadvertance à ON : ini_set('display_errors', 'off');

 

Donc l'erreur ne devait être trop critique car maintenant tout fonctionne bien ...

 

Alors résolut !!

[doekia]

par inadvertance évite l'EPO

[trofex]

Que voulez vous dire ?? par inadvertance évite l'EPO

 

* EPO peut désigner : Érythropoïétine : hormone pouvant être utilisée comme agent dopant.    

[doekia]

regarde plutôt la définition de "par inadvertance"

 

quand au erreur que l'on cache sous le tapis, c'est un peu la marque de fabrique de Prestashop, mais cela ne veux pas dire que ce n'est pas sérieux.

 

ici la fonction est appelée avec un paramètre $error (par référence), qui bien que devant être un tableau, n'en est pas un et explose donc en ligne 84

[trofex]

ok super !!   

 

Voilà la réponse que je cherchais !!

 

Je vais bientôt passer un 1.7 donc j'imagine que le tableau en question apparaîtra comme par magie !!   

 

Sinon j'investiguerais plus à la source !!

 

Et que mon n'inadvertance me rende plus grand et plus fort !!   

 

Merci !!