sameday-courier / prestashop-plugin

Prestashop (1.6 and 1.7 versions) plugin for Sameday Courier shipping
Other
4 stars 7 forks source link

Folosire operator === #111

Open FreezeS opened 1 year ago

FreezeS commented 1 year ago

Salutare,

Am instalat versiunea 1.5.8 pe PS 8.1.1 cu PHP 8.1.21. Dupa ce am setat si judetul pe Romania, curierii tot nu erau vizibili pentru livrarea comenzii.

Problema principala era de la folosirea operatorului "===" in functia "carrierDeliveryAvailable": private function carrierDeliveryAvailable($service): bool { return $service && $service['live_mode'] === Configuration::get('SAMEDAY_LIVE_MODE', 0); }

$service['live_mode'] este de tip int iar Configuration returneaza string. Operatorul === este strict, daca tipurile difera returneaza false.

Aici sunt mai multe solutii, de aia nu am dat pull request. Se poate folosi (int)Configuration::get('SAMEDAY_LIVE_MODE', 0) sau $service['live_mode'] == Configuration::get('SAMEDAY_LIVE_MODE', 0).

Mai este si aici similar: if (Configuration::get('SAMEDAY_LIVE_MODE', 0) === 0) {

As verifica toate locurile unde se foloseste Configuration::get si convertit corespunzator rezultatul.