Skip to content

Commit 909af53

Browse files
author
Stanislav Idolov
authored
ENGCOM-948: Improve Customer GraphQL Model code quality #14158
2 parents a77366b + 11709a7 commit 909af53

File tree

3 files changed

+7
-4
lines changed

3 files changed

+7
-4
lines changed

app/code/Magento/CustomerGraphQl/Model/Resolver/Customer.php

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,12 @@
66

77
namespace Magento\CustomerGraphQl\Model\Resolver;
88

9+
use Magento\Authorization\Model\UserContextInterface;
10+
use Magento\CustomerGraphQl\Model\Resolver\Customer\CustomerDataProvider;
911
use Magento\Framework\Exception\NoSuchEntityException;
1012
use Magento\Framework\GraphQl\Exception\GraphQlAuthorizationException;
1113
use Magento\Framework\GraphQl\Exception\GraphQlNoSuchEntityException;
1214
use Magento\GraphQl\Model\ResolverInterface;
13-
use Magento\Framework\GraphQl\Exception\GraphQlInputException;
1415
use Magento\GraphQl\Model\ResolverContextInterface;
1516

1617
/**
@@ -24,10 +25,10 @@ class Customer implements ResolverInterface
2425
private $customerResolver;
2526

2627
/**
27-
* @param \Magento\CustomerGraphQl\Model\Resolver\Customer\CustomerDataProvider $customerResolver
28+
* @param CustomerDataProvider $customerResolver
2829
*/
2930
public function __construct(
30-
\Magento\CustomerGraphQl\Model\Resolver\Customer\CustomerDataProvider $customerResolver
31+
CustomerDataProvider $customerResolver
3132
) {
3233
$this->customerResolver = $customerResolver;
3334
}
@@ -37,7 +38,7 @@ public function __construct(
3738
*/
3839
public function resolve(array $args, ResolverContextInterface $context)
3940
{
40-
if ((!$context->getUserId()) || $context->getUserType() == 4) {
41+
if ((!$context->getUserId()) || $context->getUserType() == UserContextInterface::USER_TYPE_GUEST) {
4142
throw new GraphQlAuthorizationException(
4243
__(
4344
'Current customer does not have access to the resource "%1"',

app/code/Magento/CustomerGraphQl/composer.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
"require": {
77
"php": "7.0.2|7.0.4|~7.0.6|~7.1.0",
88
"magento/module-customer": "100.2.*",
9+
"magento/module-authorization": "100.3.*",
910
"magento/module-graph-ql": "100.0.*",
1011
"magento/framework": "100.3.*"
1112
},

app/code/Magento/CustomerGraphQl/etc/module.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
<module name="Magento_CustomerGraphQl" >
1010
<sequence>
1111
<module name="Magento_Customer"/>
12+
<module name="Magento_Authorization"/>
1213
<module name="Magento_GraphQl"/>
1314
</sequence>
1415
</module>

0 commit comments

Comments
 (0)