Some phpStan fixes

app/Mage.php

@@ -924,9 +924,11 @@ public static function log($message, $level = null, $file = '', $forceLog = fals
             }
 
             if (is_array($message) || is_object($message)) {
+                // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
                 $message = print_r($message, true);
             }
 
+            // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
             $message = addcslashes($message, '<?');
             $loggers[$file]->log($message, $level);
         } catch (Exception $e) {

app/code/core/Mage/Admin/Model/Block.php

@@ -37,7 +37,7 @@ protected function _construct()
     }
 
     /**
-     * @return array|bool
+     * @return array|true
      * @throws Exception
      * @throws Zend_Validate_Exception
      */

app/code/core/Mage/Adminhtml/Block/Catalog/Product/Edit/Tab/Super/Config.php

@@ -308,6 +308,7 @@ protected function _getRequiredAttributesIds()
      */
     public function escapeJs($string)
     {
+        // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
         return addcslashes($string, "'\r\n\\");
     }
 

app/code/core/Mage/Adminhtml/Block/Customer/Edit/Tab/Newsletter/Grid/Renderer/Action.php

@@ -44,6 +44,7 @@ public function render(Varien_Object $row)
 
     protected function _getEscapedValue($value)
     {
+        // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
         return addcslashes(htmlspecialchars($value), '\\\'');
     }
 

app/code/core/Mage/Adminhtml/Block/Newsletter/Template/Edit.php

@@ -269,6 +269,7 @@ public function getJsTemplateName()
         if ($templateCode === null) {
             return '';
         }
+        // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
         return addcslashes($this->escapeHtml($templateCode), "\"\r\n\\");
     }
 

app/code/core/Mage/Adminhtml/Block/Sales/Reorder/Renderer/Action.php

@@ -44,6 +44,7 @@ public function render(Varien_Object $row)
 
     protected function _getEscapedValue($value)
     {
+        // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
         return addcslashes(htmlspecialchars($value), '\\\'');
     }
 

app/code/core/Mage/Adminhtml/Block/System/Email/Template/Grid/Renderer/Action.php

@@ -38,6 +38,7 @@ public function render(Varien_Object $row)
 
     protected function _getEscapedValue($value)
     {
+        // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
         return addcslashes(htmlspecialchars($value), '\\\'');
     }
 

app/code/core/Mage/Api2/Model/Route/ApiType.php

@@ -38,13 +38,15 @@ class Mage_Api2_Model_Route_ApiType extends Mage_Api2_Model_Route_Abstract imple
      * @param Zend_Translate|null $translator Translator to use for this instance
      * @param mixed $locale
      */
+    // phpcs:ignore Generic.CodeAnalysis.UnusedFunctionParameter.FoundInExtendedClassAfterLastUsed
     public function __construct(
         $route,
         $defaults = [],
         $reqs = [],
         ?Zend_Translate $translator = null,
         $locale = null
     ) {
+        // phpcs:ignore Ecg.Security.ForbiddenFunction.Found
         parent::__construct([Mage_Api2_Model_Route_Abstract::PARAM_ROUTE => str_replace('.php', '', basename(getenv('SCRIPT_FILENAME'))) . '/:api_type']);
     }
 }

app/code/core/Mage/Bundle/Model/Product/Price.php

@@ -79,7 +79,6 @@ public function getTotalBundleItemsPrice($product, $qty = null)
                 $selectionIds = unserialize($customOption->getValue(), ['allowed_classes' => false]);
                 /** @var Mage_Bundle_Model_Product_Type $productType */
                 $productType = $product->getTypeInstance(true);
-                /** @var Mage_Bundle_Model_Resource_Selection_Collection $selections */
                 $selections = $productType->getSelectionsByIds($selectionIds, $product);
                 $selections->addTierPriceData();
                 Mage::dispatchEvent('prepare_catalog_product_collection_prices', [

app/code/core/Mage/Captcha/Helper/Data.php

@@ -62,7 +62,7 @@ public function isEnabled(): bool
      * Get Captcha
      *
      * @param string $formId
-     * @return Mage_Captcha_Model_Interface
+     * @return Mage_Captcha_Model_Zend
      */
     public function getCaptcha($formId)
     {
@@ -76,7 +76,7 @@ public function getCaptcha($formId)
     /**
      * Returns value of the node with respect to current area (frontend or backend)
      *
-     * @param string $id The last part of XML_PATH_$area_CAPTCHA_ constant (case insensitive)
+     * @param string $id The last part of XML_PATH_$area_CAPTCHA_ constant (case-insensitive)
      * @param Mage_Core_Model_Store $store
      * @return Mage_Core_Model_Config_Element
      */

app/code/core/Mage/Catalog/Helper/Data.php

@@ -44,7 +44,7 @@ class Mage_Catalog_Helper_Data extends Mage_Core_Helper_Abstract
     /**
      * Breadcrumb Path cache
      *
-     * @var string
+     * @var string|array<string, array<string, string|null>>
      */
     protected $_categoryPath;
 
@@ -236,7 +236,7 @@ public function isPriceGlobal()
     /**
      * Indicate whether to save URL Rewrite History or not (create redirects to old URLs)
      *
-     * @param int $storeId Store View
+     * @param null|string|bool|int|Mage_Core_Model_Store $storeId Store View
      * @return bool
      */
     public function shouldSaveUrlRewritesHistory($storeId = null)

app/code/core/Mage/Catalog/Model/Api/Resource.php

@@ -75,7 +75,7 @@ protected function _isAllowedAttribute($attribute, $attributes = null)
 
     /**
      * Retrieves store id from store code, if no store id specified,
-     * it use seted session or admin store
+     * it uses set session or admin store
      *
      * @param string|int $store
      * @return int

app/code/core/Mage/Catalog/Model/Category.php

@@ -441,14 +441,14 @@ public function getLayoutUpdateHandle()
     /**
      * Return store id.
      *
-     * If store id is underfined for category return current active store id
+     * If store id is undefined for category return current active store id
      *
      * @return int
      */
     public function getStoreId()
     {
         if ($this->hasData('store_id')) {
-            return $this->_getData('store_id');
+            return (int)$this->_getData('store_id');
         }
         return Mage::app()->getStore()->getId();
     }

app/code/core/Mage/Catalog/Model/Convert/Adapter/Product.php

@@ -368,6 +368,7 @@ public function parse()
         $importIds = $batchImportModel->getIdCollection();
 
         foreach ($importIds as $importId) {
+            // phpcs:ignore Ecg.Performance.Loop.ModelLSD
             $batchImportModel->load($importId);
             $importData = $batchImportModel->getBatchData();
 
@@ -511,6 +512,7 @@ public function save()
                         // if product is new, create default values first
                         if (!$model->getId()) {
                             $new = true;
+                            // phpcs:ignore Ecg.Performance.Loop.ModelLSD
                             $model->save();
 
                             // if new product and then store is not default
@@ -520,6 +522,7 @@ public function save()
                                 $default = Mage::getModel('catalog/product');
                                 $default->setData($data);
                                 $default->setStoreId(0);
+                                // phpcs:ignore Ecg.Performance.Loop.ModelLSD
                                 $default->save();
                                 unset($default);
                             } // end
@@ -533,6 +536,7 @@ public function save()
                                     $storeId
                                 );
                             }
+                            // phpcs:ignore Ecg.Performance.Loop.ModelLSD
                             $model->save();
                         }
 
@@ -564,6 +568,7 @@ public function save()
                                     }
                                 }
                             }
+                            // phpcs:ignore Ecg.Performance.Loop.ModelLSD
                             $stockItem->save();
                             unset($data);
                             unset($stockItem);
@@ -734,6 +739,7 @@ public function saveRow(array $importData)
                     if (!in_array($website->getId(), $websiteIds)) {
                         $websiteIds[] = $website->getId();
                     }
+                    // phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
                 } catch (Exception $e) {
                 }
             }

app/code/core/Mage/Catalog/Model/Product.php

@@ -138,7 +138,7 @@
  * @method string getMetaDescription()
  * @method string getMetaKeyword()
  * @method string getMetaTitle()
- * @method $this hasMsrpEnabled(bool $value)
+ * @method $this hasMsrpEnabled()
  * @method bool getMsrpEnabled()
  * @method string getMsrpDisplayActualPriceType()
  *
@@ -217,7 +217,7 @@
  * @method $this setStoreIds(array $storeIds)
  * @method array getSwatchPrices()
  *
- * @method int getTaxClassId()
+ * @method string getTaxClassId()
  * @method string getThumbnail()
  * @method float|null getTaxPercent()
  * @method $this setTaxPercent(float|null $value)
@@ -365,7 +365,7 @@ protected function _initOldFieldsMap()
     public function getStoreId()
     {
         if ($this->hasData('store_id')) {
-            return $this->getData('store_id');
+            return (int)$this->getData('store_id');
         }
         return Mage::app()->getStore()->getId();
     }
@@ -417,7 +417,7 @@ public function validate()
     /**
      * Get product name
      *
-     * @return string
+     * @return string|null
      */
     public function getName()
     {
@@ -442,16 +442,18 @@ public function getPrice()
      * Set Price calculation flag
      *
      * @param bool $calculate
+     * @return $this
      */
     public function setPriceCalculation($calculate = true)
     {
         $this->_calculatePrice = $calculate;
+        return $this;
     }
 
     /**
      * Get product type identifier
      *
-     * @return string
+     * @return string|null
      */
     public function getTypeId()
     {
@@ -876,7 +878,7 @@ public function cleanCache()
     /**
      * Get product price model
      *
-     * @return Mage_Catalog_Model_Product_Type_Price|Mage_Bundle_Model_Product_Price
+     * @return Mage_Bundle_Model_Product_Price
      */
     public function getPriceModel()
     {

app/code/core/Mage/Catalog/Model/Resource/Category/Flat/Collection.php

@@ -237,16 +237,6 @@ public function addAttributeToSelect($attribute = '*')
         return $this;
     }
 
-    /**
-     * Retrieve resource instance
-     *
-     * @inheritDoc
-     */
-    public function getResource()
-    {
-        return parent::getResource();
-    }
-
     /**
      * Add attribute to sort order
      *
@@ -325,6 +315,7 @@ public function addPathsFilter($paths)
         $select = $this->getSelect();
         $cond   = [];
         foreach ($paths as $path) {
+            // phpcs:ignore Ecg.Sql.SlowQuery.SlowRawSql
             $cond[] = $this->getResource()->getReadConnection()->quoteInto('main_table.path LIKE ?', "$path%");
         }
         if ($cond) {

app/code/core/Mage/Catalog/Model/Resource/Category/Tree.php

@@ -93,7 +93,7 @@ public function __construct()
      */
     public function setStoreId($storeId)
     {
-        $this->_storeId = (int) $storeId;
+        $this->_storeId = $storeId;
         return $this;
     }
 
@@ -467,13 +467,15 @@ public function loadByIds($ids, $addCollectionData = true, $updateAnchorProductC
             ->where('entity_id IN (?)', $ids);
         $where = [$levelField . '=0' => true];
 
+        // phpcs:ignore Ecg.Performance.FetchAll.Found
         foreach ($this->_conn->fetchAll($select) as $item) {
             if (!preg_match("#^[0-9\/]+$#", $item['path'])) {
                 $item['path'] = '';
             }
             $pathIds  = explode('/', $item['path']);
             $level = (int)$item['level'];
             while ($level > 0) {
+                // phpcs:ignore Ecg.Performance.Loop.ArraySize
                 $pathIds[count($pathIds) - 1] = '%';
                 $path = implode('/', $pathIds);
                 $where["$levelField=$level AND $pathField LIKE '$path'"] = true;
@@ -493,6 +495,7 @@ public function loadByIds($ids, $addCollectionData = true, $updateAnchorProductC
         $select->where(implode(' OR ', $where));
 
         // get array of records and add them as nodes to the tree
+        // phpcs:ignore Ecg.Performance.FetchAll.Found
         $arrNodes = $this->_conn->fetchAll($select);
         if (!$arrNodes) {
             return false;
@@ -535,6 +538,7 @@ public function loadBreadcrumbsArray($path, $addCollectionData = true, $withRoot
             $select
                 ->where('e.entity_id IN(?)', $pathIds)
                 ->order($this->_conn->getLengthSql('e.path') . ' ' . Varien_Db_Select::SQL_ASC);
+            // phpcs:ignore Ecg.Performance.FetchAll.Found
             $result = $this->_conn->fetchAll($select);
             $this->_updateAnchorProductCount($result);
         }
@@ -557,7 +561,7 @@ protected function _updateAnchorProductCount(&$data)
 
     /**
      * Obtain select for categories with attributes.
-     * By default everything from entity table is selected
+     * By default, everything from entity table is selected
      * + name, is_active and is_anchor
      * Also the correct product_count is selected, depending on is the category anchor or not.
      *
@@ -623,6 +627,7 @@ protected function _createCollectionDataSelect($sorted = true, $optionalAttribut
                 ['COUNT(DISTINCT scp.product_id)']
             )
             ->where('see.entity_id = e.entity_id')
+            // phpcs:ignore Ecg.Sql.SlowQuery.SlowRawSql
             ->orWhere('see.path LIKE ?', $subConcat);
         $select->columns(['product_count' => $subSelect]);
 

app/code/core/Mage/Catalog/Model/Resource/Eav/Attribute.php

@@ -222,7 +222,7 @@ public function getStoreId()
         if ($dataObject) {
             return $dataObject->getStoreId();
         }
-        return $this->getData('store_id');
+        return (int)$this->getData('store_id');
     }
 
     /**

app/code/core/Mage/Catalog/Model/Url.php

@@ -211,7 +211,7 @@ public function setShouldSaveRewritesHistory($flag)
     /**
      * Indicate whether to save URL Rewrite History or not (create redirects to old URLs)
      *
-     * @param int $storeId Store View
+     * @param null|string|bool|int|Mage_Core_Model_Store $storeId Store View
      * @return bool
      */
     public function getShouldSaveRewritesHistory($storeId = null)

app/code/core/Mage/Checkout/Model/Api/Resource.php

@@ -72,7 +72,7 @@ protected function _isQuoteExist($quoteId)
 
     /**
      * Retrieves store id from store code, if no store id specified,
-     * it use set session or admin store
+     * it uses set session or admin store
      *
      * @param string|int $store
      * @return int

app/code/core/Mage/Checkout/Model/Cart/Product/Api/V2.php

@@ -24,7 +24,7 @@ class Mage_Checkout_Model_Cart_Product_Api_V2 extends Mage_Checkout_Model_Cart_P
     /**
      * Return an Array of Object attributes.
      *
-     * @param mixed $data
+     * @param object|array $data
      * @return array
      */
     protected function _prepareProductsData($data)

app/code/core/Mage/Core/Helper/Data.php

@@ -331,7 +331,7 @@ public function getVersionHash(Mage_Core_Model_Encryption $encryptionModel)
     /**
      * Retrieve store identifier
      *
-     * @param   mixed $store
+     * @param   bool|int|Mage_Core_Model_Store|null|string $store
      * @return  int
      */
     public function getStoreId($store = null)