Merge pull request #18 from experius/feature/fix_codescan_errors

[REFACTOR][SCOSA-329] Added PHPMD suppress warnings for unused formal…

Author: iriks-it

Plugin/Magento/Store/App/Request/StorePathInfoValidator.php

@@ -77,6 +77,7 @@ public function __construct(
      * @param $request
      * @param string $pathInfo
      * @return string
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
      */
     public function afterGetValidStoreCode(
         \Magento\Store\App\Request\StorePathInfoValidator $subject,

Plugin/Magento/UrlRewrite/Model/StoreSwitcher/RewriteUrl.php

@@ -50,6 +50,7 @@ public function __construct(
      * @param $redirectUrl
      * @return array
      * @throws \Magento\Framework\Exception\LocalizedException
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
      */
     public function beforeSwitch(
         \Magento\UrlRewrite\Model\StoreSwitcher\RewriteUrl $subject,

Plugin/Store/App/Request/PathInfoProcessor.php

@@ -38,9 +38,9 @@ class PathInfoProcessor
      * @var \Experius\MultipleWebsiteStoreCodeUrl\Helper\Settings
      */
     private $settings;
-    
+
      /**
-     * @var Data 
+     * @var Data
      */
     protected $data;
 
@@ -60,8 +60,16 @@ public function __construct(
         $this->data = $data;
     }
 
+    /**
+     * @param \Magento\Store\App\Request\PathInfoProcessor $subject
+     * @param callable $proceed
+     * @param RequestInterface $request
+     * @param $pathInfo
+     * @return mixed
+     * @throws \Magento\Framework\Exception\LocalizedException
+     * @SuppressWarnings(PHPMD.UnusedFormalParameter)
+     */
     public function aroundProcess(
-        /** @noinspection PhpUnusedParameterInspection */
         \Magento\Store\App\Request\PathInfoProcessor $subject,
         callable $proceed,
         RequestInterface $request,