Merge pull request #19 from experius/410-response

[BUGFIX] Forward to 410 instate of redirecting to the controller to p…
experius · Jan 11, 2018 · d5292bb · d5292bb
2 parents 93346ed + 7d230cb
commit d5292bb
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/Observer/Controller/ActionPredispatch.php b/Observer/Controller/ActionPredispatch.php
@@ -22,20 +22,24 @@ class ActionPredispatch implements \Magento\Framework\Event\ObserverInterface
 
     protected $urlParts = [];
 
+    private $resultFactory;
+
     public function __construct(
         \Magento\Framework\UrlInterface $url,
         \Experius\PageNotFound\Model\PageNotFoundFactory $pageNotFoundFactory,
         \Magento\Framework\App\ResponseInterface $response,
         \Magento\Framework\App\ActionFactory $actionFactory,
         \Magento\Framework\App\Cache\State $cacheState,
-        \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig
+        \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig,
+        \Magento\Framework\Controller\ResultFactory $resultFactory
     ) {
         $this->url = $url;
         $this->pageNotFoundFactory = $pageNotFoundFactory;
         $this->response = $response;
         $this->actionFactory = $actionFactory;
         $this->cacheState = $cacheState;
         $this->scopeConfig = $scopeConfig;
+        $this->resultFactory = $resultFactory;
     }
 
     private function isEnabled(){
@@ -151,7 +155,8 @@ protected function redirect($url)
     {
 
         if($url=='410'){
-            $url = $this->url->getUrl('experius_pagenotfound/response/gone', ['_secure' => true]);
+            $result = $this->resultFactory->create(\Magento\Framework\Controller\ResultFactory::TYPE_FORWARD);
+            return $result->setModule('experius_pagenotfound')->setController('response')->forward('gone');
         } else {
             // add all configured params to redirect url. <included_params_redirect>
             $queryStart = ($this->urlHasParams($url)) ? '&' : '?';