Merge pull request #185 from Markus-Gurkcity/gdprajax

remove isset condition because variables always set
PrestaShop · Feb 22, 2023 · f263e2a · f263e2a
2 parents d49fa6a + bca23f8
commit f263e2a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/controllers/front/FrontAjaxGdpr.php b/controllers/front/FrontAjaxGdpr.php
@@ -43,12 +43,12 @@ public function display()
 
         if ($customer->isLogged() === true) {
             $token = sha1($customer->secure_key);
-            if (!isset($customer_token) || $customer_token == $token) {
+            if ($customer_token === $token) {
                 GDPRLog::addLog($id_customer, 'consent', $id_module);
             }
         } else {
             $token = sha1('psgdpr' . Context::getContext()->cart->id_guest . $_SERVER['REMOTE_ADDR'] . date('Y-m-d'));
-            if (!isset($guest_token) || $guest_token == $token) {
+            if ($guest_token === $token) {
                 GDPRLog::addLog($id_customer, 'consent', $id_module, $id_guest);
             }
         }