magento/graphql-ce#762: Subscribe for newsletter even when not allowed

lenaorobei · lenaorobei · commit 0768d89e8f2a · 2019-09-09T10:10:30.000-05:00
diff --git a/app/code/Magento/CustomerGraphQl/Model/Resolver/CreateCustomer.php b/app/code/Magento/CustomerGraphQl/Model/Resolver/CreateCustomer.php
@@ -14,6 +14,7 @@
 use Magento\Framework\GraphQl\Query\ResolverInterface;
 use Magento\Framework\GraphQl\Schema\Type\ResolveInfo;
 use Magento\Newsletter\Model\Config;
+use Magento\Store\Model\ScopeInterface;
 
 /**
  * Create customer account resolver
@@ -66,7 +67,7 @@ public function resolve(
             throw new GraphQlInputException(__('"input" value should be specified'));
         }
 
-        if (!$this->newsLetterConfig->isActive()) {
+        if (!$this->newsLetterConfig->isActive(ScopeInterface::SCOPE_STORE)) {
             $args['input']['is_subscribed'] = false;
         }
       

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@`
`14`	`14`	`use Magento\Framework\GraphQl\Query\ResolverInterface;`
`15`	`15`	`use Magento\Framework\GraphQl\Schema\Type\ResolveInfo;`
`16`	`16`	`use Magento\Newsletter\Model\Config;`
	`17`	`+use Magento\Store\Model\ScopeInterface;`
`17`	`18`
`18`	`19`	`/**`
`19`	`20`	`* Create customer account resolver`
`@@ -66,7 +67,7 @@ public function resolve(`
`66`	`67`	`throw new GraphQlInputException(__('"input" value should be specified'));`
`67`	`68`	`}`
`68`	`69`
`69`		`- if (!$this->newsLetterConfig->isActive()) {`
	`70`	`+ if (!$this->newsLetterConfig->isActive(ScopeInterface::SCOPE_STORE)) {`
`70`	`71`	`$args['input']['is_subscribed'] = false;`
`71`	`72`	`}`
`72`	`73`