phpstan: Sitemap, Newsletter, ...

app/Mage.php

@@ -828,7 +828,7 @@ public static function isInstalled($options = [])
      *
      * @param array|object|string $message
      * @param int $level
-     * @param string $file
+     * @param string|null $file
      * @param bool $forceLog
      */
     public static function log($message, $level = null, $file = '', $forceLog = false)

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

@@ -33,14 +33,14 @@ class Mage_AdminNotification_Helper_Data extends Mage_Core_Helper_Abstract
     /**
      * Last Notice object
      *
-     * @var Mage_AdminNotification_Model_Inbox
+     * @var Mage_AdminNotification_Model_Inbox|null
      */
     protected $_latestNotice;
 
     /**
      * count of unread notes by type
      *
-     * @var array
+     * @var array|null
      */
     protected $_unreadNoticeCounts;
 

app/code/core/Mage/AdminNotification/Model/Feed.php

@@ -36,7 +36,7 @@ class Mage_AdminNotification_Model_Feed extends Mage_Core_Model_Abstract
     /**
      * Feed url
      *
-     * @var string
+     * @var string|null
      */
     protected $_feedUrl;
 

app/code/core/Mage/Contacts/controllers/IndexController.php

@@ -34,7 +34,7 @@ class Mage_Contacts_IndexController extends Mage_Core_Controller_Front_Action
     public const XML_PATH_ENABLED          = 'contacts/contacts/enabled';
 
     /**
-     * @return void
+     * @return $this
      */
     public function preDispatch()
     {
@@ -43,6 +43,7 @@ public function preDispatch()
         if (!Mage::getStoreConfigFlag(self::XML_PATH_ENABLED)) {
             $this->norouteAction();
         }
+        return $this;
     }
 
     public function indexAction()

app/code/core/Mage/Core/Controller/Varien/Action.php

@@ -196,7 +196,7 @@ public function getFlag($action, $flag = '')
      *
      * @param   string $action
      * @param   string $flag
-     * @param   string|int $value
+     * @param   string|int|bool $value
      * @return  $this
      */
     public function setFlag($action, $flag, $value)

app/code/core/Mage/Core/Model/Email/Template/Filter.php

@@ -131,7 +131,7 @@ public function getPlainTemplateMode()
     /**
      * Setter
      *
-     * @param int $storeId
+     * @param Mage_Core_Model_Store|int $storeId
      * @return $this
      */
     public function setStoreId($storeId)

app/code/core/Mage/Customer/Model/Customer.php

@@ -463,7 +463,7 @@ public function getAttributes()
      * Get customer attribute model object
      *
      * @param string $attributeCode
-     * @return Mage_Customer_Model_Entity_Attribute|null
+     * @return Mage_Customer_Model_Customer|null
      * @throws Mage_Core_Exception
      */
     public function getAttribute($attributeCode)

app/code/core/Mage/Newsletter/Model/Problem.php

@@ -46,7 +46,7 @@ class Mage_Newsletter_Model_Problem extends Mage_Core_Model_Abstract
     /**
      * Current Subscriber
      *
-     * @var Mage_Newsletter_Model_Subscriber
+     * @var Mage_Newsletter_Model_Subscriber|null
      */
     protected $_subscriber = null;
 
@@ -98,7 +98,7 @@ public function addErrorData(Exception $e)
     /**
      * Retrieve Subscriber
      *
-     * @return Mage_Newsletter_Model_Subscriber
+     * @return Mage_Newsletter_Model_Subscriber|null
      */
     public function getSubscriber()
     {

app/code/core/Mage/Newsletter/Model/Queue.php

@@ -56,13 +56,13 @@ class Mage_Newsletter_Model_Queue extends Mage_Core_Model_Template
     /**
      * Newsletter Template object
      *
-     * @var Mage_Newsletter_Model_Template
+     * @var Mage_Newsletter_Model_Template|null
      */
     protected $_template;
 
     /**
      * Subscribers collection
-     * @var Mage_Newsletter_Model_Resource_Subscriber_Collection
+     * @var Mage_Newsletter_Model_Resource_Subscriber_Collection|null
      */
     protected $_subscribersCollection = null;
 

app/code/core/Mage/Newsletter/Model/Resource/Queue.php

@@ -169,7 +169,7 @@ public function getStores(Mage_Newsletter_Model_Queue $queue)
     /**
      * Saving template after saving queue action
      *
-     * @param Mage_Core_Model_Abstract|Mage_Newsletter_Model_Queue $queue
+     * @param Mage_Newsletter_Model_Queue $queue
      * @return $this
      */
     protected function _afterSave(Mage_Core_Model_Abstract $queue)

app/code/core/Mage/Newsletter/Model/Resource/Subscriber/Collection.php

@@ -75,7 +75,7 @@ protected function _construct()
         $this->_map['fields']['customer_middlename'] = 'customer_middlename_table.value';
         $this->_map['fields']['customer_firstname']  = 'customer_firstname_table.value';
         $this->_map['fields']['type']                = $this->getResource()->getReadConnection()
-            ->getCheckSql('main_table.customer_id = 0', 1, 2);
+            ->getCheckSql('main_table.customer_id = 0', '1', '2');
         $this->_map['fields']['website_id']          = 'store.website_id';
         $this->_map['fields']['group_id']            = 'store.group_id';
         $this->_map['fields']['store_id']            = 'main_table.store_id';
@@ -189,7 +189,7 @@ public function _getFieldTableAlias($field)
         }
 
         if ($field == 'type') {
-            return $this->getConnection()->getCheckSql('main_table.customer_id = 0', 1, 2);
+            return $this->getConnection()->getCheckSql('main_table.customer_id = 0', '1', '2');
         }
 
         if (in_array($field, ['website_id', 'group_id'])) {

app/code/core/Mage/Newsletter/Model/Subscriber.php

@@ -186,7 +186,7 @@ public function setStatus($value)
     /**
      * Set the error messages scope for subscription
      *
-     * @param bool $scope
+     * @param string $scope
      * @return $this
      */
 

app/code/core/Mage/Newsletter/Model/Template.php

@@ -65,7 +65,7 @@ class Mage_Newsletter_Model_Template extends Mage_Core_Model_Email_Template_Abst
     /**
      * Mail object
      *
-     * @var Zend_Mail
+     * @var Zend_Mail|null
      */
     protected $_mail;
 
@@ -173,7 +173,7 @@ public function isPreprocessed()
     /**
      * Check Template Text Preprocessed
      *
-     * @return bool
+     * @return string
      */
     public function getTemplateTextPreprocessed()
     {

app/code/core/Mage/Sitemap/Model/Sitemap.php

@@ -47,7 +47,7 @@ class Mage_Sitemap_Model_Sitemap extends Mage_Core_Model_Abstract
     /**
      * Real file path
      *
-     * @var string
+     * @var string|null
      */
     protected $_filePath;
 

app/code/core/Zend/Db/Select.php

@@ -758,7 +758,7 @@ public function reset($part = null)
      * Gets the Zend_Db_Adapter_Abstract for this
      * particular Zend_Db_Select object.
      *
-     * @return Zend_Db_Adapter_Abstract
+     * @return Varien_Db_Adapter_Interface|Zend_Db_Adapter_Abstract
      */
     public function getAdapter()
     {

lib/Varien/Data/Collection.php

@@ -440,7 +440,7 @@ public function clear()
      *
      * Returns array with results of callback for each item
      *
-     * @param callable $callback
+     * @param string|callable $callback
      * @param array $args
      * @return array
      */

lib/Varien/Http/Adapter/Curl.php

@@ -196,7 +196,7 @@ public function write($method, $url, $http_ver = '1.1', $headers = [], $body = '
     /**
      * Read response from server
      *
-     * @return string
+     * @return string|bool
      */
     public function read()
     {