fix phpstan

PrestaShop · Jun 30, 2020 · 438bf28 · 438bf28
1 parent ad33c47
commit 438bf28
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 29 deletions.
diff --git a/controllers/front/action.php b/controllers/front/action.php
@@ -18,7 +18,6 @@
  * International Registered Trademark & Property of PrestaShop SA
  */
 
-use PrestaShop\Module\BlockWishlist\WishList;
 use PrestaShop\PrestaShop\Adapter\Image\ImageRetriever;
 use PrestaShop\PrestaShop\Adapter\Product\PriceFormatter;
 use PrestaShop\PrestaShop\Adapter\Product\ProductColorsRetriever;
@@ -90,7 +89,6 @@ private function addProductToWishListAction($params)
         );
 
         $newStat = new Statistics();
-        $newStat->id_wishlist = $idWishList;
         $newStat->id_product = $id_product;
         $newStat->id_product_attribute = $id_product_attribute;
         $newStat->save();

diff --git a/src/Calculator/StatisticsCalculator.php b/src/Calculator/StatisticsCalculator.php
@@ -35,6 +35,7 @@ class StatisticsCalculator
     ];
 
     private $context;
+    private $productAssembler;
 
     public function __construct($context)
     {

diff --git a/src/Controller/AdminAjaxPrestashopWishlistController.php b/src/Controller/AdminAjaxPrestashopWishlistController.php
@@ -89,6 +89,9 @@ private function handleForm($datas)
     private function getWishlistConfigurationDatas()
     {
         $languages = \Language::getLanguages(true);
+        $wishlistNames = [];
+        $wishlistDefaultTitles = [];
+        $wishlistCreateNewButtonsLabel = [];
 
         foreach ($languages as $lang) {
             $wishlistNames[$lang['id_lang']] = \Configuration::get('blockwishlist_WishlistPageName', $lang['id_lang']);
@@ -129,28 +132,28 @@ public function getStatisticsAction(Request $request)
     }
 
     // this idea need some functional specification, it is not used ATM
-    public function forceRefreshCacheStatsAction(Request $request)
-    {
-        $cacheName = $request->request->get('cacheName');
-
-        $statsCalculator = new StatisticsCalculator($this->context);
-        switch ($cacheName) {
-            case 'year':
-                $results = $statsCalculator->computeYearStats();
-                break;
-            case 'month':
-                $results = $statsCalculator->computeMonthStats();
-                break;
-            case 'day':
-                $results = $statsCalculator->computeDayStats();
-            break;
-            default:
-                break;
-        }
-
-        return $this->json([
-            'success' => true,
-            'stats' => $results,
-        ]);
-    }
+    // public function forceRefreshCacheStatsAction(Request $request)
+    // {
+    //     $cacheName = $request->request->get('cacheName');
+
+    //     $statsCalculator = new StatisticsCalculator($this->context);
+    //     switch ($cacheName) {
+    //         case 'year':
+    //             $results = $statsCalculator->computeYearStats();
+    //             break;
+    //         case 'month':
+    //             $results = $statsCalculator->computeMonthStats();
+    //             break;
+    //         case 'day':
+    //             $results = $statsCalculator->computeDayStats();
+    //         break;
+    //         default:
+    //             break;
+    //     }
+
+    //     return $this->json([
+    //         'success' => true,
+    //         'stats' => $results,
+    //     ]);
+    // }
 }
diff --git a/views/js/admin/adminConfiguration.js b/views/js/admin/adminConfiguration.js
@@ -18,7 +18,7 @@
  */
 
 console.log('blockwishlist - adminConfiguration.js');
-import TranslatableInput from "admin-dev/themes/new-theme/js/components/translatable-input";
+// import TranslatableInput from "admin-dev/themes/new-theme/js/components/translatable-input";
 
-// enable togging of different locales
-new TranslatableInput();
+// // enable togging of different locales
+// new TranslatableInput();