subscription implementetion way changed, schema required fields removed, gender input type changed

Author: Stepa4man

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

@@ -11,6 +11,7 @@
 use Magento\Customer\Api\AccountManagementInterface;
 use Magento\Customer\Api\Data\CustomerInterface;
 use Magento\Customer\Api\Data\CustomerInterfaceFactory;
+use Magento\CustomerGraphQl\Model\Customer\ChangeSubscriptionStatus;
 use Magento\CustomerGraphQl\Model\Customer\CustomerDataProvider;
 use Magento\Framework\Api\DataObjectHelper;
 use Magento\Framework\Exception\LocalizedException;
@@ -22,26 +23,20 @@
 use Magento\Newsletter\Model\SubscriberFactory;
 use Magento\Store\Model\StoreManagerInterface;
 
-/**
- * Create customer data resolver
- */
 class CreateCustomer implements ResolverInterface
 {
     /**
      * @var CustomerDataProvider
      */
     private $customerDataProvider;
-
     /**
      * @var AccountManagementInterface
      */
     private $accountManagement;
-
     /**
      * @var CustomerInterfaceFactory
      */
     private $customerFactory;
-
     /**
      * @var DataObjectHelper
      */
@@ -54,6 +49,10 @@ class CreateCustomer implements ResolverInterface
      * @var SubscriberFactory
      */
     private $subscriberFactory;
+    /**
+     * @var ChangeSubscriptionStatus
+     */
+    private $changeSubscriptionStatus;
 
     /**
      * @param DataObjectHelper $dataObjectHelper
@@ -62,21 +61,24 @@ class CreateCustomer implements ResolverInterface
      * @param StoreManagerInterface $storeManager
      * @param SubscriberFactory $subscriberFactory
      * @param CustomerDataProvider $customerDataProvider
+     * @param ChangeSubscriptionStatus $changeSubscriptionStatus
      */
     public function __construct(
         DataObjectHelper $dataObjectHelper,
         CustomerInterfaceFactory $customerFactory,
         AccountManagementInterface $accountManagement,
         StoreManagerInterface $storeManager,
         SubscriberFactory $subscriberFactory,
-        CustomerDataProvider $customerDataProvider
+        CustomerDataProvider $customerDataProvider,
+        ChangeSubscriptionStatus $changeSubscriptionStatus
     ) {
         $this->customerDataProvider = $customerDataProvider;
         $this->accountManagement = $accountManagement;
         $this->customerFactory = $customerFactory;
         $this->dataObjectHelper = $dataObjectHelper;
         $this->storeManager = $storeManager;
         $this->subscriberFactory = $subscriberFactory;
+        $this->changeSubscriptionStatus = $changeSubscriptionStatus;
     }
 
     /**
@@ -94,13 +96,14 @@ public function resolve(
         }
         try {
             $customer = $this->createUserAccount($args);
+            $customerId = (int)$customer->getId();
             $this->setUpUserContext($context, $customer);
             if (array_key_exists('is_subscribed', $args['input'])) {
                 if ($args['input']['is_subscribed']) {
-                    $this->subscriberFactory->create()->subscribeCustomerById($customer->getId());
+                    $this->changeSubscriptionStatus->execute($customerId, true);
                 }
             }
-            $data = $this->customerDataProvider->getCustomerById((int)$customer->getId());
+            $data = $this->customerDataProvider->getCustomerById($customerId);
         } catch (LocalizedException $e) {
             throw new GraphQlInputException(__($e->getMessage()));
         }

app/code/Magento/CustomerGraphQl/etc/schema.graphqls

@@ -19,15 +19,15 @@ type CustomerToken {
 
 input CustomerInput {
     prefix: String
-    firstname: String!
-    middlename: String!
-    lastname: String!
+    firstname: String
+    middlename: String
+    lastname: String
     suffix: String
     dob: String
     taxvat: String
-    gender: String
-    email: String!
-    password: String!
+    gender: Int
+    email: String
+    password: String
     is_subscribed: Boolean
 }