Skip to content

Commit 4a30707

Browse files
magento2-login-as-customer/issues/90: Logging update.
1 parent 9cf2a1c commit 4a30707

File tree

2 files changed

+12
-21
lines changed

2 files changed

+12
-21
lines changed

app/code/Magento/LoginAsCustomer/Plugin/AdminAddCommentOnOrderPlacementPlugin.php

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -39,16 +39,18 @@ public function __construct(
3939
public function afterPlace(Order $subject, Order $result): Order
4040
{
4141
$adminUser = $this->userSession->getUser();
42-
$subject->addCommentToStatusHistory(
43-
'Order Placed by Store Administrator',
44-
false,
45-
true
46-
)->setIsCustomerNotified(false);
47-
$subject->addCommentToStatusHistory(
48-
"Order Placed by {$adminUser->getFirstName()} {$adminUser->getLastName()} using Admin Panel",
49-
false,
50-
false
51-
)->setIsCustomerNotified(false);
42+
if ($adminUser) {
43+
$subject->addCommentToStatusHistory(
44+
'Order Placed by Store Administrator',
45+
false,
46+
true
47+
)->setIsCustomerNotified(false);
48+
$subject->addCommentToStatusHistory(
49+
"Order Placed by {$adminUser->getFirstName()} {$adminUser->getLastName()} using Admin Panel",
50+
false,
51+
false
52+
)->setIsCustomerNotified(false);
53+
}
5254

5355
return $result;
5456
}

dev/tests/integration/testsuite/Magento/Sales/Model/AdminOrder/CreateTest.php

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
*/
66
namespace Magento\Sales\Model\AdminOrder;
77

8-
use Magento\Backend\Model\Auth;
98
use Magento\Backend\Model\Session\Quote as SessionQuote;
109
use Magento\Customer\Api\AddressRepositoryInterface;
1110
use Magento\Customer\Api\CustomerRepositoryInterface;
@@ -43,21 +42,11 @@ class CreateTest extends \PHPUnit\Framework\TestCase
4342
*/
4443
private $objectManager;
4544

46-
/**
47-
* @var Auth
48-
*/
49-
private $auth;
50-
5145
protected function setUp()
5246
{
5347
$this->objectManager = Bootstrap::getObjectManager();
5448
$this->messageManager = $this->objectManager->get(ManagerInterface::class);
5549
$this->model =$this->objectManager->create(Create::class, ['messageManager' => $this->messageManager]);
56-
$this->auth = $this->objectManager->create(Auth::class);
57-
$this->auth->login(
58-
\Magento\TestFramework\Bootstrap::ADMIN_NAME,
59-
\Magento\TestFramework\Bootstrap::ADMIN_PASSWORD
60-
);
6150
}
6251

6352
/**

0 commit comments

Comments
 (0)