Merge remote-tracking branch 'remotes/mainline/2.4-develop' into MC-40624

Author: paliarush

app/code/Magento/Backend/Block/Dashboard/Bar.php

@@ -5,6 +5,7 @@
  */
 namespace Magento\Backend\Block\Dashboard;
 
+use Magento\Directory\Model\Currency;
 use Magento\Store\Model\Store;
 
 /**
@@ -20,10 +21,15 @@ class Bar extends \Magento\Backend\Block\Dashboard\AbstractDashboard
     protected $_totals = [];
 
     /**
-     * @var \Magento\Directory\Model\Currency|null
+     * @var Currency|null
      */
     protected $_currentCurrencyCode = null;
 
+    /**
+     * @var Currency
+     */
+    private $_currency;
+
     /**
      * Get totals
      *
@@ -67,7 +73,7 @@ public function format($price)
     /**
      * Setting currency model
      *
-     * @param \Magento\Directory\Model\Currency $currency
+     * @param Currency $currency
      * @return void
      */
     public function setCurrency($currency)
@@ -78,7 +84,7 @@ public function setCurrency($currency)
     /**
      * Retrieve currency model if not set then return currency model for current store
      *
-     * @return \Magento\Directory\Model\Currency
+     * @return Currency
      * @SuppressWarnings(PHPMD.RequestAwareBlockMethod)
      */
     public function getCurrency()

app/code/Magento/Backend/Controller/Adminhtml/Dashboard/RefreshStatistics.php

@@ -6,25 +6,33 @@
 
 namespace Magento\Backend\Controller\Adminhtml\Dashboard;
 
+use Magento\Backend\App\Action\Context;
 use Magento\Framework\App\Action\HttpPostActionInterface;
+use Magento\Framework\Stdlib\DateTime\Filter\Date;
 use Magento\Reports\Controller\Adminhtml\Report\Statistics;
+use Psr\Log\LoggerInterface;
 
 /**
  * Refresh Dashboard statistics action.
  */
 class RefreshStatistics extends Statistics implements HttpPostActionInterface
 {
     /**
-     * @param \Magento\Backend\App\Action\Context $context
-     * @param \Magento\Framework\Stdlib\DateTime\Filter\Date $dateFilter
+     * @var LoggerInterface
+     */
+    private $logger;
+
+    /**
+     * @param Context $context
+     * @param Date $dateFilter
      * @param array $reportTypes
-     * @param \Psr\Log\LoggerInterface $logger
+     * @param LoggerInterface $logger
      */
     public function __construct(
-        \Magento\Backend\App\Action\Context $context,
-        \Magento\Framework\Stdlib\DateTime\Filter\Date $dateFilter,
+        Context $context,
+        Date $dateFilter,
         array $reportTypes,
-        \Psr\Log\LoggerInterface $logger
+        LoggerInterface $logger
     ) {
         parent::__construct($context, $dateFilter, $reportTypes);
         $this->logger = $logger;

app/code/Magento/Backend/Model/Menu/Config.php

@@ -63,6 +63,11 @@ class Config
      */
     protected $_appState;
 
+    /**
+     * @var Builder
+     */
+    private $_menuBuilder;
+
     /**
      * @param \Magento\Backend\Model\Menu\Builder $menuBuilder
      * @param \Magento\Backend\Model\Menu\AbstractDirector $menuDirector

app/code/Magento/Backup/Model/Fs/Collection.php

@@ -40,6 +40,10 @@ class Collection extends \Magento\Framework\Data\Collection\Filesystem
      */
     protected $_backup = null;
 
+    /**
+     * @var \Magento\Framework\Filesystem
+     */
+    private $_filesystem;
     /**
      * @param \Magento\Framework\Data\Collection\EntityFactory $entityFactory
      * @param \Magento\Backup\Helper\Data $backupData

app/code/Magento/Backup/Test/Mftf/Data/BackupConfigData.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+ /**
+  * Copyright © Magento, Inc. All rights reserved.
+  * See COPYING.txt for license details.
+  */
+-->
+
+<entities xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+          xsi:noNamespaceSchemaLocation="urn:magento:mftf:DataGenerator/etc/dataProfileSchema.xsd">
+    <entity name="EnableBackupFunctionality">
+        <data key="path">system/backup/functionality_enabled</data>
+        <data key="value">1</data>
+    </entity>
+    <entity name="DisableBackupFunctionality">
+        <!-- Magento default value -->
+        <data key="path">system/backup/functionality_enabled</data>
+        <data key="value">0</data>
+    </entity>
+</entities>

app/code/Magento/Backup/Test/Mftf/Test/AdminCreateAndDeleteBackupsTest.xml

@@ -17,10 +17,13 @@
             <severity value="CRITICAL"/>
             <testCaseId value="MAGETWO-94176"/>
             <group value="backup"/>
-            <skip>
-                <issueId value="MC-5807"/>
-            </skip>
         </annotations>
+        <before>
+            <magentoCLI command="config:set {{EnableBackupFunctionality.path}} {{EnableBackupFunctionality.value}}" stepKey="setEnableBackup"/>
+        </before>
+        <after>
+            <magentoCLI command="config:set {{DisableBackupFunctionality.path}} {{DisableBackupFunctionality.value}}" stepKey="setDisableBackup"/>
+        </after>
 
         <!--Login to admin area-->
         <actionGroup ref="AdminLoginActionGroup" stepKey="loginAsAdmin"/>

app/code/Magento/Bundle/Model/ResourceModel/Indexer/BundleOptionStockDataSelectBuilder.php

@@ -17,6 +17,16 @@
  */
 class BundleOptionStockDataSelectBuilder
 {
+    /**
+     * @var \Magento\Framework\App\ResourceConnection
+     */
+    private $resourceConnection;
+
+    /**
+     * @var \Magento\Framework\EntityManager\MetadataPool
+     */
+    private $metadataPool;
+
     /**
      * @param \Magento\Framework\App\ResourceConnection $resourceConnection
      * @param \Magento\Framework\EntityManager\MetadataPool $metadataPool

app/code/Magento/Bundle/Model/ResourceModel/Indexer/StockStatusSelectBuilder.php

@@ -18,6 +18,20 @@
  */
 class StockStatusSelectBuilder
 {
+    /**
+     * @var \Magento\Framework\App\ResourceConnection
+     */
+    private $resourceConnection;
+
+    /**
+     * @var \Magento\Framework\EntityManager\MetadataPool
+     */
+    private $metadataPool;
+
+    /**
+     * @var \Magento\Eav\Model\Config
+     */
+    private $eavConfig;
 
     /**
      * @param \Magento\Framework\App\ResourceConnection $resourceConnection

app/code/Magento/Bundle/Test/Unit/Ui/DataProvider/Product/Form/Modifier/BundlePanelTest.php

@@ -23,6 +23,11 @@
  */
 class BundlePanelTest extends TestCase
 {
+    /**
+     * @var ObjectManager
+     */
+    private $objectManager;
+
     /**
      * @var UrlInterface|MockObject
      */

app/code/Magento/Catalog/Block/Product/View/Options.php

@@ -60,6 +60,11 @@ class Options extends \Magento\Framework\View\Element\Template
      */
     protected $_catalogData;
 
+    /**
+     * @var \Magento\Framework\Stdlib\ArrayUtils
+     */
+    private $arrayUtils;
+
     /**
      * @param \Magento\Framework\View\Element\Template\Context $context
      * @param \Magento\Framework\Pricing\Helper\Data $pricingHelper
@@ -93,7 +98,7 @@ public function __construct(
      * Retrieve product object
      *
      * @return Product
-     * @throws \LogicExceptions
+     * @throws \LogicException
      */
     public function getProduct()
     {

app/code/Magento/Catalog/Model/Attribute/ScopeOverriddenValue.php

@@ -45,6 +45,11 @@ class ScopeOverriddenValue
      */
     private $resourceConnection;
 
+    /**
+     * @var FilterBuilder
+     */
+    private $filterBuilder;
+
     /**
      * ScopeOverriddenValue constructor.
      * @param AttributeRepository $attributeRepository

app/code/Magento/Catalog/Model/Category/AttributeRepository.php

@@ -24,6 +24,11 @@ class AttributeRepository implements CategoryAttributeRepositoryInterface
      */
     protected $eavAttributeRepository;
 
+    /**
+     * @var \Magento\Eav\Model\Config
+     */
+    private $eavConfig;
+
     /**
      * @param \Magento\Framework\Api\SearchCriteriaBuilder $searchCriteriaBuilder
      * @param \Magento\Framework\Api\FilterBuilder $filterBuilder

app/code/Magento/Catalog/Model/Config.php

@@ -44,6 +44,11 @@ class Config extends \Magento\Eav\Model\Config
      */
     protected $_productTypesById;
 
+    /**
+     * @var array
+     */
+    private $_productTypesByName;
+
     /**
      * Array of attributes codes needed for product load
      *
@@ -175,16 +180,6 @@ public function __construct(
         );
     }
 
-    /**
-     * Initialize resource model
-     *
-     * @return void
-     */
-    protected function _construct()
-    {
-        $this->_init(\Magento\Catalog\Model\ResourceModel\Config::class);
-    }
-
     /**
      * Set store id
      *

app/code/Magento/Catalog/Model/CustomOptions/CustomOption.php

@@ -17,6 +17,11 @@
 
 class CustomOption extends AbstractExtensibleModel implements CustomOptionInterface
 {
+    /**
+     * @var FileProcessor
+     */
+    private $fileProcessor;
+
     /**
      * @param Context $context
      * @param Registry $registry

app/code/Magento/Catalog/Model/Layer/Filter/Dynamic/Manual.php

@@ -18,6 +18,41 @@ class Manual implements AlgorithmInterface
 {
     const XML_PATH_RANGE_MAX_INTERVALS = 'catalog/layered_navigation/price_range_max_intervals';
 
+    /**
+     * @var Algorithm
+     */
+    private $algorithm;
+
+    /**
+     * @var \Magento\Catalog\Model\Layer
+     */
+    private $layer;
+
+    /**
+     * @var ScopeConfigInterface
+     */
+    private $scopeConfig;
+
+    /**
+     * @var Render
+     */
+    private $render;
+
+    /**
+     * @var Registry
+     */
+    private $coreRegistry;
+
+    /**
+     * @var Range
+     */
+    private $range;
+
+    /**
+     * @var Price
+     */
+    private $resource;
+
     /**
      * @param Algorithm $algorithm
      * @param Resolver $layerResolver

app/code/Magento/Catalog/Model/Product/Price/TierPriceFactory.php

@@ -56,6 +56,16 @@ class TierPriceFactory
      */
     private $customerGroupsByCode = [];
 
+    /**
+     * @var \Magento\Framework\Api\SearchCriteriaBuilder
+     */
+    private $searchCriteriaBuilder;
+
+    /**
+     * @var \Magento\Framework\Api\FilterBuilder
+     */
+    private $filterBuilder;
+
     /**
      * TierPriceBuilder constructor.
      *

app/code/Magento/Catalog/Model/Product/Price/Validation/InvalidSkuProcessor.php

@@ -11,6 +11,16 @@
  */
 class InvalidSkuProcessor
 {
+    /**
+     * @var \Magento\Catalog\Model\ProductIdLocatorInterface
+     */
+    private $productIdLocator;
+
+    /**
+     * @var \Magento\Catalog\Api\ProductRepositoryInterface
+     */
+    private $productRepository;
+
     /**
      * @param \Magento\Catalog\Model\ProductIdLocatorInterface $productIdLocator
      * @param \Magento\Catalog\Api\ProductRepositoryInterface $productRepository

app/code/Magento/Catalog/Model/Product/Type/Price.php

@@ -298,7 +298,7 @@ public function getTierPrice($qty, $product)
 
         $custGroup = $this->_getCustomerGroupId($product);
         if ($qty) {
-            $prevQty = 1;
+            $prevQty = 0;
             $prevPrice = $product->getPrice();
             $prevGroup = $allGroupsId;
 

app/code/Magento/Catalog/Model/ResourceModel/Product/Indexer/Price/CompositeProductRelationsCalculator.php

@@ -11,6 +11,11 @@
  */
 class CompositeProductRelationsCalculator
 {
+    /**
+     * @var DefaultPrice
+     */
+    private $indexerResource;
+
     /**
      * @param DefaultPrice $indexerResource
      */

app/code/Magento/Catalog/Observer/Compare/BindCustomerLoginObserver.php

@@ -14,6 +14,11 @@
  */
 class BindCustomerLoginObserver implements ObserverInterface
 {
+    /**
+     * @var Item
+     */
+    private $item;
+
     /**
      * @param Item $item
      */