Merge pull request #135 from Hlavtox/fix-returns

Fix automated tests

Author: Hlavtox

classes/Hook/HookActionProductCancel.php

@@ -24,6 +24,7 @@
 use Context;
 use OrderDetail;
 use Ps_Googleanalytics;
+use Validate;
 
 class HookActionProductCancel implements HookInterface
 {
@@ -57,6 +58,12 @@ public function run()
         // Display GA refund product
         $orderDetail = new OrderDetail($this->params['id_order_detail']);
 
+        // Check if the hook provided us with a valid existing ID of order detail.
+        // An example are automatic tests, which do not provide it unfortunately.
+        if (!Validate::isLoadedObject($orderDetail)) {
+            return;
+        }
+
         $idProduct = empty($orderDetail->product_attribute_id) ? $orderDetail->product_id : $orderDetail->product_id . '-' . $orderDetail->product_attribute_id;
         if ((bool) Configuration::get('GA_V4_ENABLED')) {
             $js = $this->getGoogleAnalytics4(