Skip to content

Commit e4a1b8a

Browse files
committed
Merging the PR #21480
1 parent 96f6a61 commit e4a1b8a

File tree

1 file changed

+40
-9
lines changed
  • app/code/Magento/Sales/Controller/Guest

1 file changed

+40
-9
lines changed

app/code/Magento/Sales/Controller/Guest/Form.php

Lines changed: 40 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -4,40 +4,71 @@
44
* Copyright © Magento, Inc. All rights reserved.
55
* See COPYING.txt for license details.
66
*/
7+
declare(strict_types=1);
8+
79
namespace Magento\Sales\Controller\Guest;
810

11+
use Magento\Customer\Model\Session as CustomerSession;
12+
use Magento\Framework\App\Action\Context;
13+
use Magento\Framework\App\ObjectManager;
14+
use Magento\Framework\Controller\Result\Redirect;
15+
use Magento\Framework\View\Result\Page;
16+
use Magento\Framework\View\Result\PageFactory;
17+
use Magento\Sales\Helper\Guest as GuestHelper;
18+
19+
/**
20+
* Class Form
21+
*/
922
class Form extends \Magento\Framework\App\Action\Action
1023
{
1124
/**
12-
* @var \Magento\Framework\View\Result\PageFactory
25+
* @var PageFactory
1326
*/
1427
protected $resultPageFactory;
1528

1629
/**
17-
* @param \Magento\Framework\App\Action\Context $context
18-
* @param \Magento\Framework\View\Result\PageFactory $resultPageFactory
30+
* @var CustomerSession|null
31+
*/
32+
private $customerSession;
33+
34+
/**
35+
* @var GuestHelper|null
36+
*/
37+
private $guestHelper;
38+
39+
/**
40+
* @param Context $context
41+
* @param PageFactory $resultPageFactory
42+
* @param CustomerSession|null $customerSession
43+
* @param GuestHelper|null $guestHelper
1944
*/
2045
public function __construct(
21-
\Magento\Framework\App\Action\Context $context,
22-
\Magento\Framework\View\Result\PageFactory $resultPageFactory
46+
Context $context,
47+
PageFactory $resultPageFactory,
48+
CustomerSession $customerSession = null,
49+
GuestHelper $guestHelper = null
2350
) {
2451
parent::__construct($context);
2552
$this->resultPageFactory = $resultPageFactory;
53+
$this->customerSession = $customerSession ?: ObjectManager::getInstance()->get(CustomerSession::class);
54+
$this->guestHelper = $guestHelper ?: ObjectManager::getInstance()->get(GuestHelper::class);
2655
}
2756

2857
/**
2958
* Order view form page
3059
*
31-
* @return \Magento\Framework\Controller\Result\Redirect|\Magento\Framework\View\Result\Page
60+
* @return Redirect|Page
3261
*/
3362
public function execute()
3463
{
35-
if ($this->_objectManager->get(\Magento\Customer\Model\Session::class)->isLoggedIn()) {
64+
if ($this->customerSession->isLoggedIn()) {
3665
return $this->resultRedirectFactory->create()->setPath('customer/account/');
3766
}
67+
3868
$resultPage = $this->resultPageFactory->create();
3969
$resultPage->getConfig()->getTitle()->set(__('Orders and Returns'));
40-
$this->_objectManager->get(\Magento\Sales\Helper\Guest::class)->getBreadcrumbs($resultPage);
70+
$this->guestHelper->getBreadcrumbs($resultPage);
71+
4172
return $resultPage;
4273
}
43-
}
74+
}

0 commit comments

Comments
 (0)