MAGETWO-34993: Refactor controllers from the list (Part1)

Author: Yurii Torbyk

app/code/Magento/Backend/Controller/Adminhtml/Cache/CleanImages.php

@@ -26,7 +26,7 @@ public function execute()
     }
 
     /**
-     * Redirect user to the previous or main page
+     * {@inheritdoc}
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
      */

app/code/Magento/Backend/Controller/Adminhtml/Cache/CleanMedia.php

@@ -26,7 +26,7 @@ public function execute()
     }
 
     /**
-     * Redirect user to the previous or main page
+     * {@inheritdoc}
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
      */

app/code/Magento/Backend/Controller/Adminhtml/Cache/MassDisable.php

@@ -14,6 +14,7 @@ class MassDisable extends \Magento\Backend\Controller\Adminhtml\Cache
      * Mass action for cache disabling
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
+     * @throws \Magento\Framework\Exception\LocalizedException|\Exception
      */
     public function execute()
     {
@@ -39,7 +40,7 @@ public function execute()
     }
 
     /**
-     * Redirect user to the previous or main page
+     * {@inheritdoc}
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
      */

app/code/Magento/Backend/Controller/Adminhtml/Cache/MassEnable.php

@@ -14,6 +14,7 @@ class MassEnable extends \Magento\Backend\Controller\Adminhtml\Cache
      * Mass action for cache enabling
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
+     * @throws \Magento\Framework\Exception\LocalizedException|\Exception
      */
     public function execute()
     {
@@ -38,7 +39,7 @@ public function execute()
     }
 
     /**
-     * Redirect user to the previous or main page
+     * {@inheritdoc}
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
      */

app/code/Magento/Backend/Controller/Adminhtml/Cache/MassRefresh.php

@@ -14,6 +14,7 @@ class MassRefresh extends \Magento\Backend\Controller\Adminhtml\Cache
      * Mass action for cache refresh
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
+     * @throws \Magento\Framework\Exception\LocalizedException|\Exception
      */
     public function execute()
     {
@@ -36,7 +37,7 @@ public function execute()
     }
 
     /**
-     * Redirect user to the previous or main page
+     * {@inheritdoc}
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
      */

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

@@ -26,6 +26,7 @@ public function __construct(
 
     /**
      * @return \Magento\Backend\Model\View\Result\Redirect
+     * @throws \Magento\Framework\Exception\LocalizedException|\Exception
      */
     public function execute()
     {
@@ -39,7 +40,7 @@ public function execute()
     }
 
     /**
-     * Redirect user to the previous or main page
+     * {@inheritdoc}
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
      */

app/code/Magento/Backend/Controller/Adminhtml/System/Account/Save.php

@@ -68,7 +68,7 @@ public function execute()
     }
 
     /**
-     * Redirect user to the previous or main page
+     * {@inheritdoc}
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
      */

app/code/Magento/Backend/Controller/Adminhtml/System/Store/DeleteGroupPost.php

@@ -10,6 +10,7 @@ class DeleteGroupPost extends \Magento\Backend\Controller\Adminhtml\System\Store
 {
     /**
      * @return \Magento\Backend\Model\View\Result\Redirect
+     * @throws \Magento\Framework\Exception\LocalizedException|\Exception
      */
     public function execute()
     {
@@ -35,18 +36,4 @@ public function execute()
         $this->messageManager->addSuccess(__('The store has been deleted.'));
         return $redirectResult->setPath('adminhtml/*/');
     }
-
-    /**
-     * Redirect user to the previous or main page
-     *
-     * @return \Magento\Backend\Model\View\Result\Redirect
-     */
-    public function getDefaultRedirect()
-    {
-        $resultRedirect = $this->resultRedirectFactory->create();
-        return $resultRedirect->setPath(
-            'adminhtml/*/editGroup',
-            ['group_id' => $this->getRequest()->getParam('item_id')]
-        );
-    }
 }

app/code/Magento/Backend/Controller/Adminhtml/System/Store/DeleteStorePost.php

@@ -12,6 +12,7 @@ class DeleteStorePost extends \Magento\Backend\Controller\Adminhtml\System\Store
      * Delete store view post action
      *
      * @return \Magento\Backend\Model\View\Result\Redirect
+     * @throws \Magento\Framework\Exception\LocalizedException|\Exception
      */
     public function execute()
     {
@@ -39,18 +40,4 @@ public function execute()
         $this->messageManager->addSuccess(__('The store view has been deleted.'));
         return $redirectResult->setPath('adminhtml/*/');
     }
-
-    /**
-     * Redirect user to the previous or main page
-     *
-     * @return \Magento\Backend\Model\View\Result\Redirect
-     */
-    public function getDefaultRedirect()
-    {
-        $resultRedirect = $this->resultRedirectFactory->create();
-        return $resultRedirect->setPath(
-            'adminhtml/*/editStore',
-            ['store_id' => $this->getRequest()->getParam('item_id')]
-        );
-    }
 }

app/code/Magento/Backend/Controller/Adminhtml/System/Store/DeleteWebsitePost.php

@@ -10,6 +10,7 @@ class DeleteWebsitePost extends \Magento\Backend\Controller\Adminhtml\System\Sto
 {
     /**
      * @return \Magento\Backend\Model\View\Result\Redirect
+     * @throws \Magento\Framework\Exception\LocalizedException|\Exception
      */
     public function execute()
     {
@@ -37,18 +38,4 @@ public function execute()
         $this->messageManager->addSuccess(__('The website has been deleted.'));
         return $redirectResult->setPath('adminhtml/*/');
     }
-
-    /**
-     * Redirect user to the previous or main page
-     *
-     * @return \Magento\Backend\Model\View\Result\Redirect
-     */
-    public function getDefaultRedirect()
-    {
-        $resultRedirect = $this->resultRedirectFactory->create();
-        return $resultRedirect->setPath(
-            'adminhtml/*/editWebsite',
-            ['website_id' => $this->getRequest()->getParam('item_id')]
-        );
-    }
 }