Merge pull request #37 from PrestaShop/dev

Release 2.0.4

Author: Quetzacoalt91

.github/workflows/php.yml

@@ -8,7 +8,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v2.0.0
+        uses: actions/checkout@v5
 
       - name: PHP syntax checker 5.6
         uses: prestashop/github-action-php-lint/5.6@master
@@ -39,13 +39,7 @@ jobs:
           php-version: '7.4'
 
       - name: Checkout
-        uses: actions/checkout@v2.0.0
-
-      - name: Cache dependencies
-        uses: actions/cache@v2
-        with:
-          path: vendor
-          key: php-${{ hashFiles('composer.lock') }}
+        uses: actions/checkout@v5
 
       - name: Install dependencies
         run: composer install
@@ -58,6 +52,7 @@ jobs:
     name: PHPStan
     runs-on: ubuntu-latest
     strategy:
+      fail-fast: false
       matrix:
         presta-versions: ['1.7.1.2', '1.7.2.5', '1.7.3.4', '1.7.4.4', '1.7.5.1', '1.7.6', '1.7.7', '1.7.8', 'latest']
     steps:
@@ -67,21 +62,7 @@ jobs:
           php-version: '7.4'
 
       - name: Checkout
-        uses: actions/checkout@v2.0.0
-
-      # Add vendor folder in cache to make next builds faster
-      - name: Cache vendor folder
-        uses: actions/cache@v1
-        with:
-          path: vendor
-          key: php-${{ hashFiles('composer.lock') }}
-
-      # Add composer local folder in cache to make next builds faster
-      - name: Cache composer folder
-        uses: actions/cache@v1
-        with:
-          path: ~/.composer/cache
-          key: php-composer-cache
+        uses: actions/checkout@v5
 
       - run: composer install
 

config.xml

@@ -2,7 +2,7 @@
 <module>
 	<name>statscheckup</name>
 	<displayName><![CDATA[Catalog evaluation]]></displayName>
-	<version><![CDATA[2.0.3]]></version>
+	<version><![CDATA[2.0.4]]></version>
 	<description><![CDATA[Adds a quick evaluation of your catalog quality to the Stats dashboard.]]></description>
 	<author><![CDATA[PrestaShop]]></author>
 	<tab><![CDATA[analytics_stats]]></tab>

statscheckup.php

@@ -35,7 +35,7 @@ public function __construct()
     {
         $this->name = 'statscheckup';
         $this->tab = 'administration';
-        $this->version = '2.0.3';
+        $this->version = '2.0.4';
         $this->author = 'PrestaShop';
         $this->need_instance = 0;
 
@@ -110,7 +110,6 @@ public function hookDisplayAdminStatsModules()
             1 => '<img src="../modules/' . $this->name . '/img/orange.png" alt="' . $this->trans('Average', [], 'Modules.Statscheckup.Admin') . '" />',
             2 => '<img src="../modules/' . $this->name . '/img/green.png" alt="' . $this->trans('Good', [], 'Modules.Statscheckup.Admin') . '" />',
         ];
-        $token_products = Tools::getAdminToken('AdminProducts' . (int) Tab::getIdFromClassName('AdminProducts') . (int) Context::getContext()->employee->id);
         $divisor = 4;
         $totals = ['products' => 0, 'active' => 0, 'images' => 0, 'sales' => 0, 'stock' => 0];
         foreach ($languages as $language) {
@@ -163,7 +162,7 @@ public function hookDisplayAdminStatsModules()
 		<div class="panel-heading">'
             . $this->displayName . '
 		</div>
-		<form action="' . Tools::safeOutput(AdminController::$currentIndex . '&token=' . Tools::getValue('token') . '&module=' . $this->name) . '" method="post" class="checkup form-horizontal">
+		<form action="' . Tools::safeOutput($this->context->link->getAdminLink('AdminStats', true, [], ['module' => $this->name])) . '" method="post" class="checkup form-horizontal">
 			<table class="table checkup">
 				<thead>
 					<tr>
@@ -205,7 +204,7 @@ public function hookDisplayAdminStatsModules()
 				<i class="icon-save"></i> ' . $this->trans('Save', [], 'Admin.Actions') . '
 			</button>
 		</form>
-		<form action="' . Tools::safeOutput(AdminController::$currentIndex . '&token=' . Tools::getValue('token') . '&module=' . $this->name) . '" method="post" class="form-horizontal alert">
+		<form action="' . Tools::safeOutput($this->context->link->getAdminLink('AdminStats', true, [], ['module' => $this->name])) . '" method="post" class="form-horizontal alert">
 			<div class="row">
 				<div class="col-lg-12">
 					<label class="control-label pull-left">' . $this->trans('Order by', [], 'Modules.Statscheckup.Admin') . '</label>
@@ -270,7 +269,7 @@ public function hookDisplayAdminStatsModules()
             $this->html .= '
 				<tr>
 					<td>' . $row['id_product'] . '</td>
-					<td><a href="' . Tools::safeOutput('index.php?controller=AdminProducts&updateproduct&id_product=' . $row['id_product'] . '&token=' . $token_products) . '">' . Tools::substr($row['name'], 0, 42) . '</a></td>
+					<td><a href="' . Tools::safeOutput($this->context->link->getAdminLink('AdminProducts', true, [], ['updateproduct' => 1, 'id_product' => $row['id_product']])) . '">' . Tools::substr($row['name'], 0, 42) . '</a></td>
 					<td class="center">' . $array_colors[$scores['active']] . '</td>';
             foreach ($languages as $language) {
                 if (isset($row['desclength_' . $language['iso_code']])) {

upgrade/upgrade-2.0.2.php

@@ -1,4 +1,3 @@
-
 <?php
 /**
  * Copyright since 2007 PrestaShop SA and Contributors