Merge pull request #7303 from magento-trigger/ph-delivery

[Platform Health] Updates for PHP8.1 and JS libs

Author: xmav

app/code/Magento/CatalogImportExport/Model/Import/Product/Validator.php

@@ -205,14 +205,15 @@ public function isAttributeValid($attrCode, array $attrParams, array $rowData)
             return $valid;
         }
 
-        if (!strlen(trim($rowData[$attrCode]))) {
+        if ($rowData[$attrCode] === null || trim($rowData[$attrCode]) === '') {
             return true;
         }
 
         if ($rowData[$attrCode] === $this->context->getEmptyAttributeValueConstant() && !$attrParams['is_required']) {
             return true;
         }
 
+        $valid = false;
         switch ($attrParams['type']) {
             case 'varchar':
             case 'text':

app/code/Magento/Customer/Block/Adminhtml/Edit/Tab/View/PersonalInfo.php

@@ -23,39 +23,29 @@ class PersonalInfo extends \Magento\Backend\Block\Template
      * since his last activity. Used only if it's impossible to get such setting
      * from configuration.
      */
-    const DEFAULT_ONLINE_MINUTES_INTERVAL = 15;
+    public const DEFAULT_ONLINE_MINUTES_INTERVAL = 15;
 
     /**
-     * Customer
-     *
      * @var \Magento\Customer\Api\Data\CustomerInterface
      */
     protected $customer;
 
     /**
-     * Customer log
-     *
      * @var \Magento\Customer\Model\Log
      */
     protected $customerLog;
 
     /**
-     * Customer logger
-     *
      * @var \Magento\Customer\Model\Logger
      */
     protected $customerLogger;
 
     /**
-     * Customer registry
-     *
      * @var \Magento\Customer\Model\CustomerRegistry
      */
     protected $customerRegistry;
 
     /**
-     * Account management
-     *
      * @var AccountManagementInterface
      */
     protected $accountManagement;
@@ -68,43 +58,31 @@ class PersonalInfo extends \Magento\Backend\Block\Template
     protected $groupRepository;
 
     /**
-     * Customer data factory
-     *
      * @var \Magento\Customer\Api\Data\CustomerInterfaceFactory
      */
     protected $customerDataFactory;
 
     /**
-     * Address helper
-     *
      * @var \Magento\Customer\Helper\Address
      */
     protected $addressHelper;
 
     /**
-     * Date time
-     *
      * @var \Magento\Framework\Stdlib\DateTime
      */
     protected $dateTime;
 
     /**
-     * Core registry
-     *
      * @var \Magento\Framework\Registry
      */
     protected $coreRegistry;
 
     /**
-     * Address mapper
-     *
      * @var Mapper
      */
     protected $addressMapper;
 
     /**
-     * Data object helper
-     *
      * @var \Magento\Framework\Api\DataObjectHelper
      */
     protected $dataObjectHelper;
@@ -439,7 +417,8 @@ public function getLastLoginDate()
      */
     public function getStoreLastLoginDate()
     {
-        $date = strtotime($this->getCustomerLog()->getLastLoginAt());
+        $lastLogin = $this->getCustomerLog()->getLastLoginAt();
+        $date = $lastLogin !== null ? strtotime($lastLogin) : false;
 
         if ($date) {
             $date = $this->_localeDate->scopeDate($this->getCustomer()->getStoreId(), $date, true);

app/code/Magento/Customer/Model/Options.php

@@ -49,7 +49,7 @@ public function getNamePrefixOptions($store = null)
     {
         return $this->prepareNamePrefixSuffixOptions(
             $this->addressHelper->getConfig('prefix_options', $store),
-            $this->addressHelper->getConfig('prefix_show', $store) == NooptreqSource::VALUE_OPTIONAL
+            $this->addressHelper->getConfig('prefix_show', $store) === NooptreqSource::VALUE_OPTIONAL
         );
     }
 
@@ -63,7 +63,7 @@ public function getNameSuffixOptions($store = null)
     {
         return $this->prepareNamePrefixSuffixOptions(
             $this->addressHelper->getConfig('suffix_options', $store),
-            $this->addressHelper->getConfig('suffix_show', $store) == NooptreqSource::VALUE_OPTIONAL
+            $this->addressHelper->getConfig('suffix_show', $store) === NooptreqSource::VALUE_OPTIONAL
         );
     }
 
@@ -93,13 +93,12 @@ protected function _prepareNamePrefixSuffixOptions($options, $isOptional = false
      */
     private function prepareNamePrefixSuffixOptions($options, $isOptional = false)
     {
-        $options = trim($options);
-        if (empty($options)) {
+        if ($options === null || empty(trim($options))) {
             return false;
         }
-
         $result = [];
-        $options = explode(';', $options);
+        $options = explode(';', trim($options));
+
         foreach ($options as $value) {
             $result[] = $this->escaper->escapeHtml(trim($value)) ?: ' ';
         }

app/code/Magento/Dhl/Model/Carrier.php

@@ -482,11 +482,12 @@ public function setRequest(\Magento\Framework\DataObject $request)
         );
 
         $shippingWeight = $request->getPackageWeight();
+        $destStreet = $request->getDestStreet() !== null ? str_replace("\n", '', $request->getDestStreet()) : '';
 
         $requestObject->setValue(sprintf('%.2f', $request->getPackageValue()))
             ->setValueWithDiscount($request->getPackageValueWithDiscount())
             ->setCustomsValue($request->getPackageCustomsValue())
-            ->setDestStreet($this->string->substr(str_replace("\n", '', $request->getDestStreet()), 0, 35))
+            ->setDestStreet($this->string->substr($destStreet, 0, 35))
             ->setDestStreetLine2($request->getDestStreetLine2())
             ->setDestCity($request->getDestCity())
             ->setOrigCompanyName($request->getOrigCompanyName())
@@ -939,7 +940,7 @@ protected function _getDimension($dimension, $configWeightUnit = false)
             );
         }
 
-        return round($dimension, 3);
+        return round((float) $dimension, 3);
     }
 
     /**

app/code/Magento/Elasticsearch/composer.json

@@ -12,7 +12,7 @@
         "magento/module-store": "*",
         "magento/module-catalog-inventory": "*",
         "magento/framework": "*",
-        "elasticsearch/elasticsearch": "~7.15.0"
+        "elasticsearch/elasticsearch": "~7.16.0"
     },
     "suggest": {
         "magento/module-config": "*"

app/code/Magento/Elasticsearch6/composer.json

@@ -8,7 +8,7 @@
         "magento/module-catalog-search": "*",
         "magento/module-search": "*",
         "magento/module-elasticsearch": "*",
-        "elasticsearch/elasticsearch": "~7.15.0"
+        "elasticsearch/elasticsearch": "~7.16.0"
     },
     "suggest": {
         "magento/module-config": "*"

app/code/Magento/Elasticsearch7/composer.json

@@ -5,7 +5,7 @@
         "php": "~7.4.0||~8.0.0",
         "magento/framework": "*",
         "magento/module-elasticsearch": "*",
-        "elasticsearch/elasticsearch": "~7.15.0",
+        "elasticsearch/elasticsearch": "~7.16.0",
         "magento/module-advanced-search": "*",
         "magento/module-catalog-search": "*"
     },

app/code/Magento/GoogleAnalytics/Block/Ga.php

@@ -17,8 +17,6 @@
 class Ga extends \Magento\Framework\View\Element\Template
 {
     /**
-     * Google analytics data
-     *
      * @var \Magento\GoogleAnalytics\Helper\Data
      */
     protected $_googleAnalyticsData = null;
@@ -261,8 +259,8 @@ public function getOrdersTrackingData()
     private function getOptPageUrl()
     {
         $optPageURL = '';
-        $pageName = trim($this->getPageName());
-        if ($pageName && substr($pageName, 0, 1) == '/' && strlen($pageName) > 1) {
+        $pageName = $this->getPageName() !== null ? trim($this->getPageName()) : '';
+        if ($pageName && substr($pageName, 0, 1) === '/' && strlen($pageName) > 1) {
             $optPageURL = ", '" . $this->escapeHtmlAttr($pageName, false) . "'";
         }
         return $optPageURL;

app/code/Magento/Payment/Model/Method/Cc.php

@@ -286,7 +286,8 @@ public function assignData(\Magento\Framework\DataObject $data)
             [
                 'cc_type' => $additionalData->getCcType(),
                 'cc_owner' => $additionalData->getCcOwner(),
-                'cc_last_4' => substr($additionalData->getCcNumber(), -4),
+                'cc_last_4' => $additionalData->getCcNumber() !== null ?
+                    substr($additionalData->getCcNumber(), -4) : '',
                 'cc_number' => $additionalData->getCcNumber(),
                 'cc_cid' => $additionalData->getCcCid(),
                 'cc_exp_month' => $additionalData->getCcExpMonth(),
@@ -323,8 +324,9 @@ public function validateCcNum($ccNumber)
     {
         $cardNumber = strrev($ccNumber);
         $numSum = 0;
+        $length = strlen($cardNumber);
 
-        for ($i = 0; $i < strlen($cardNumber); $i++) {
+        for ($i = 0; $i < $length; $i++) {
             $currentNum = substr($cardNumber, $i, 1);
 
             /**

app/code/Magento/Reports/Block/Adminhtml/Config/Form/Field/YtdStart.php

@@ -24,7 +24,7 @@ protected function _getElementHtml(AbstractElement $element)
     {
         $_months = [];
         for ($i = 1; $i <= 12; $i++) {
-            $month = $this->_localeDate->date(mktime(null, null, null, $i, 1))
+            $month = $this->_localeDate->date(mktime(0, 0, 0, $i, 1))
                 ->format('m');
             $_months[$month] = $month;
         }