Fixed static tests

sivaschenko · sivaschenko · commit a557683a0c1e · 2018-12-21T15:07:06.000Z
diff --git a/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/Crosssell.php b/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/Crosssell.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\Crosssell as CatalogCrossel;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Crosssell
  *
  * @package Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit
  * @deprecated Not used since cross-sell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/crosssell_product_listing.xml
  */
-class Crosssell extends \Magento\Catalog\Controller\Adminhtml\Product\Crosssell
+class Crosssell extends CatalogCrossel implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/CrosssellGrid.php b/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/CrosssellGrid.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\CrosssellGrid as CatalogCrosssellGrid;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class CrosssellGrid
  *
  * @package Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit
  * @deprecated Not used since cross-sell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/crosssell_product_listing.xml
  */
-class CrosssellGrid extends \Magento\Catalog\Controller\Adminhtml\Product\CrosssellGrid
+class CrosssellGrid extends CatalogCrosssellGrid implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/Related.php b/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/Related.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\Related as CatalogRelated;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Related
  *
  * @package Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit
  * @deprecated Not used since related products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/related_product_listing.xml
  */
-class Related extends \Magento\Catalog\Controller\Adminhtml\Product\Related
+class Related extends CatalogRelated implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/RelatedGrid.php b/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/RelatedGrid.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\RelatedGrid as CatalogRelatedGrid;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class RelatedGrid
  *
  * @package Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit
  * @deprecated Not used since related products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/related_product_listing.xml
  */
-class RelatedGrid extends \Magento\Catalog\Controller\Adminhtml\Product\RelatedGrid
+class RelatedGrid extends CatalogRelatedGrid implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/Upsell.php b/app/code/Magento/Bundle/Controller/Adminhtml/Bundle/Product/Edit/Upsell.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\Upsell as CatalogUpsell;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Upsell
  *
  * @package Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit
  * @deprecated Not used since upsell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/upsell_product_listing.xml
  */
-class Upsell extends \Magento\Catalog\Controller\Adminhtml\Product\Upsell
+class Upsell extends CatalogUpsell implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Catalog/Block/Adminhtml/Product/Edit/Tab/Crosssell.php b/app/code/Magento/Catalog/Block/Adminhtml/Product/Edit/Tab/Crosssell.php
@@ -10,6 +10,8 @@
 use Magento\Catalog\Model\Product;
 
 /**
+ * Crossel product edit tab
+ *
  * @api
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
  * @since 100.0.2
diff --git a/app/code/Magento/Catalog/Block/Adminhtml/Product/Edit/Tab/Related.php b/app/code/Magento/Catalog/Block/Adminhtml/Product/Edit/Tab/Related.php
@@ -9,6 +9,8 @@
 use Magento\Backend\Block\Widget\Grid\Extended;
 
 /**
+ * Related product edit tab
+ *
  * @api
  * @since 100.0.2
  * @deprecated Not used since related products grid moved to UI components.
diff --git a/app/code/Magento/Catalog/Block/Adminhtml/Product/Edit/Tab/Upsell.php b/app/code/Magento/Catalog/Block/Adminhtml/Product/Edit/Tab/Upsell.php
@@ -6,6 +6,8 @@
 namespace Magento\Catalog\Block\Adminhtml\Product\Edit\Tab;
 
 /**
+ * Upsell product edit tab
+ *
  * @api
  * @since 100.0.2
  * @deprecated Not used since upsell products grid moved to UI components.
diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/Crosssell.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/Crosssell.php
@@ -6,14 +6,17 @@
  */
 namespace Magento\Catalog\Controller\Adminhtml\Product;
 
+use Magento\Catalog\Controller\Adminhtml\Product;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Crosssell
  *
  * @package Magento\Catalog\Controller\Adminhtml\Product
  * @deprecated Not used since cross-sell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/crosssell_product_listing.xml
  */
-class Crosssell extends \Magento\Catalog\Controller\Adminhtml\Product
+class Crosssell extends Product implements HttpPostActionInterface
 {
     /**
      * @var \Magento\Framework\View\Result\LayoutFactory
diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/CrosssellGrid.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/CrosssellGrid.php
@@ -6,14 +6,17 @@
  */
 namespace Magento\Catalog\Controller\Adminhtml\Product;
 
+use Magento\Catalog\Controller\Adminhtml\Product;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class CrosssellGrid
  *
  * @package Magento\Catalog\Controller\Adminhtml\Product
  * @deprecated Not used since cross-sell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/crosssell_product_listing.xml
  */
-class CrosssellGrid extends \Magento\Catalog\Controller\Adminhtml\Product
+class CrosssellGrid extends Product implements HttpPostActionInterface
 {
     /**
      * @var \Magento\Framework\View\Result\LayoutFactory
diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/Related.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/Related.php
@@ -7,14 +7,17 @@
  */
 namespace Magento\Catalog\Controller\Adminhtml\Product;
 
+use Magento\Catalog\Controller\Adminhtml\Product;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Related
  *
  * @package Magento\Catalog\Controller\Adminhtml\Product
  * @deprecated Not used since related products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/related_product_listing.xml
  */
-class Related extends \Magento\Catalog\Controller\Adminhtml\Product
+class Related extends Product implements HttpPostActionInterface
 {
     /**
      * @var \Magento\Framework\View\Result\LayoutFactory
@@ -36,6 +39,8 @@ public function __construct(
     }
 
     /**
+     * Execute
+     *
      * @return \Magento\Framework\View\Result\Layout
      */
     public function execute()
diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/RelatedGrid.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/RelatedGrid.php
@@ -7,13 +7,15 @@
  */
 namespace Magento\Catalog\Controller\Adminhtml\Product;
 
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class RelatedGrid
  *
  * @package Magento\Catalog\Controller\Adminhtml\Product
  * @deprecated Not used since related products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/related_product_listing.xml
  */
-class RelatedGrid extends Related
+class RelatedGrid extends Related implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/Upsell.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/Upsell.php
@@ -6,14 +6,17 @@
  */
 namespace Magento\Catalog\Controller\Adminhtml\Product;
 
+use Magento\Catalog\Controller\Adminhtml\Product;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Upsell
  *
  * @package Magento\Catalog\Controller\Adminhtml\Product
  * @deprecated Not used since upsell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/upsell_product_listing.xml
  */
-class Upsell extends \Magento\Catalog\Controller\Adminhtml\Product
+class Upsell extends Product implements HttpPostActionInterface
 {
     /**
      * @var \Magento\Framework\View\Result\LayoutFactory
diff --git a/app/code/Magento/Catalog/Controller/Adminhtml/Product/UpsellGrid.php b/app/code/Magento/Catalog/Controller/Adminhtml/Product/UpsellGrid.php
@@ -6,14 +6,17 @@
  */
 namespace Magento\Catalog\Controller\Adminhtml\Product;
 
+use Magento\Catalog\Controller\Adminhtml\Product;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class UpsellGrid
  *
  * @package Magento\Catalog\Controller\Adminhtml\Product
  * @deprecated Not used since upsell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/upsell_product_listing.xml
  */
-class UpsellGrid extends \Magento\Catalog\Controller\Adminhtml\Product
+class UpsellGrid extends Product implements HttpPostActionInterface
 {
     /**
      * @var \Magento\Framework\View\Result\LayoutFactory
diff --git a/app/code/Magento/Deploy/Collector/Collector.php b/app/code/Magento/Deploy/Collector/Collector.php
@@ -8,6 +8,7 @@
 use Magento\Deploy\Source\SourcePool;
 use Magento\Deploy\Package\Package;
 use Magento\Deploy\Package\PackageFactory;
+use Magento\Deploy\Package\PackageFile;
 use Magento\Framework\Module\Manager;
 use Magento\Framework\View\Asset\PreProcessor\FileNameResolver;
 
@@ -65,18 +66,19 @@ class Collector implements CollectorInterface
      * @param SourcePool $sourcePool
      * @param FileNameResolver $fileNameResolver
      * @param PackageFactory $packageFactory
+     * @param Manager|null $moduleManager
      */
     public function __construct(
         SourcePool $sourcePool,
         FileNameResolver $fileNameResolver,
         PackageFactory $packageFactory,
-        Manager $moduleManager  = null
+        Manager $moduleManager = null
     ) {
         $this->sourcePool = $sourcePool;
         $this->fileNameResolver = $fileNameResolver;
         $this->packageFactory = $packageFactory;
         $this->moduleManager = $moduleManager ?: \Magento\Framework\App\ObjectManager::getInstance()
-                              ->get(\Magento\Framework\Module\Manager::class);
+            ->get(\Magento\Framework\Module\Manager::class);
     }
 
     /**
@@ -92,18 +94,7 @@ public function collect()
                     continue;
                 }
                 $file->setDeployedFileName($this->fileNameResolver->resolve($file->getFileName()));
-                $params = [
-                    'area' => $file->getArea(),
-                    'theme' => $file->getTheme(),
-                    'locale' => $file->getLocale(),
-                    'module' => $file->getModule(),
-                    'isVirtual' => (!$file->getLocale() || !$file->getTheme() || !$file->getArea())
-                ];
-                foreach ($this->packageDefaultValues as $name => $value) {
-                    if (!isset($params[$name])) {
-                        $params[$name] = $value;
-                    }
-                }
+                $params = $this->getParams($file);
                 $packagePath = "{$params['area']}/{$params['theme']}/{$params['locale']}";
                 if (!isset($packages[$packagePath])) {
                     $packages[$packagePath] = $this->packageFactory->create($params);
@@ -115,4 +106,27 @@ public function collect()
         }
         return $packages;
     }
+
+    /**
+     * Retrieve package params
+     *
+     * @param PackageFile $file
+     * @return array
+     */
+    private function getParams(PackageFile $file)
+    {
+        $params = [
+            'area' => $file->getArea(),
+            'theme' => $file->getTheme(),
+            'locale' => $file->getLocale(),
+            'module' => $file->getModule(),
+            'isVirtual' => (!$file->getLocale() || !$file->getTheme() || !$file->getArea())
+        ];
+        foreach ($this->packageDefaultValues as $name => $value) {
+            if (!isset($params[$name])) {
+                $params[$name] = $value;
+            }
+        }
+        return $params;
+    }
 }
diff --git a/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/Crosssell.php b/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/Crosssell.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\Crosssell as CatalogCrosssell;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Crosssell
  *
  * @package Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit
  * @deprecated Not used since cross-sell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/crosssell_product_listing.xml
  */
-class Crosssell extends \Magento\Catalog\Controller\Adminhtml\Product\Crosssell
+class Crosssell extends CatalogCrosssell implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/CrosssellGrid.php b/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/CrosssellGrid.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\CrosssellGrid as CatalogCrosssellGrid;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class CrosssellGrid
  *
  * @package Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit
  * @deprecated Not used since cross-sell products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/crosssell_product_listing.xml
  */
-class CrosssellGrid extends \Magento\Catalog\Controller\Adminhtml\Product\CrosssellGrid
+class CrosssellGrid extends CatalogCrosssellGrid implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/Related.php b/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/Related.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\Related as CatalogRelated;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class Related
  *
  * @package Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit
  * @deprecated Not used since related products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/related_product_listing.xml
  */
-class Related extends \Magento\Catalog\Controller\Adminhtml\Product\Related
+class Related extends CatalogRelated implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/RelatedGrid.php b/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/RelatedGrid.php
@@ -6,13 +6,16 @@
  */
 namespace Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit;
 
+use Magento\Catalog\Controller\Adminhtml\Product\RelatedGrid as CatalogRelatedGrid;
+use Magento\Framework\App\Action\HttpPostActionInterface;
+
 /**
  * Class RelatedGrid
  *
  * @package Magento\Downloadable\Controller\Adminhtml\Downloadable\Product\Edit
  * @deprecated Not used since related products grid moved to UI components.
  * @see Magento_Catalog::view/adminhtml/ui_component/related_product_listing.xml
  */
-class RelatedGrid extends \Magento\Catalog\Controller\Adminhtml\Product\RelatedGrid
+class RelatedGrid extends CatalogRelatedGrid implements HttpPostActionInterface
 {
 }
diff --git a/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/Upsell.php b/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/Upsell.php
diff --git a/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/UpsellGrid.php b/app/code/Magento/Downloadable/Controller/Adminhtml/Downloadable/Product/Edit/UpsellGrid.php
diff --git a/app/code/Magento/Vault/Model/PaymentTokenRepository.php b/app/code/Magento/Vault/Model/PaymentTokenRepository.php

Original file line number	Diff line number	Diff line change
`@@ -6,13 +6,16 @@`
`6`	`6`	`*/`
`7`	`7`	`namespace Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit;`
`8`	`8`
	`9`	`+use Magento\Catalog\Controller\Adminhtml\Product\Crosssell as CatalogCrossel;`
	`10`	`+use Magento\Framework\App\Action\HttpPostActionInterface;`
	`11`	`+`
`9`	`12`	`/**`
`10`	`13`	`* Class Crosssell`
`11`	`14`	`*`
`12`	`15`	`* @package Magento\Bundle\Controller\Adminhtml\Bundle\Product\Edit`
`13`	`16`	`* @deprecated Not used since cross-sell products grid moved to UI components.`
`14`	`17`	`* @see Magento_Catalog::view/adminhtml/ui_component/crosssell_product_listing.xml`
`15`	`18`	`*/`
`16`		`-class Crosssell extends \Magento\Catalog\Controller\Adminhtml\Product\Crosssell`
	`19`	`+class Crosssell extends CatalogCrossel implements HttpPostActionInterface`
`17`	`20`	`{`
`18`	`21`	`}`