#28569: Missing store codes in relation to a group and website

- Refactored code to use getList

Author: gallyamov

app/code/Magento/Store/Model/Service/StoreConfigManager.php

@@ -3,24 +3,33 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+
 namespace Magento\Store\Model\Service;
 
+use Magento\Framework\App\Config\ScopeConfigInterface;
+use Magento\Store\Api\Data\StoreConfigInterface;
+use Magento\Store\Api\Data\StoreInterface;
+use Magento\Store\Model\Data\StoreConfig;
+use Magento\Store\Model\Data\StoreConfigFactory;
+use Magento\Store\Model\ResourceModel\Store\CollectionFactory;
+use Magento\Store\Model\Store;
+
 class StoreConfigManager implements \Magento\Store\Api\StoreConfigManagerInterface
 {
     /**
-     * @var \Magento\Store\Model\ResourceModel\Store\CollectionFactory
+     * @var CollectionFactory
      */
     protected $storeCollectionFactory;
 
     /**
-     * @var \Magento\Store\Model\Data\StoreConfigFactory
+     * @var StoreConfigFactory
      */
     protected $storeConfigFactory;
 
     /**
      * Core store config
      *
-     * @var \Magento\Framework\App\Config\ScopeConfigInterface
+     * @var ScopeConfigInterface
      */
     protected $scopeConfig;
 
@@ -38,23 +47,25 @@ class StoreConfigManager implements \Magento\Store\Api\StoreConfigManagerInterfa
     ];
 
     /**
-     * @param \Magento\Store\Model\ResourceModel\Store\CollectionFactory $storeCollectionFactory
-     * @param \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig
-     * @param \Magento\Store\Model\Data\StoreConfigFactory $storeConfigFactory
+     * @param CollectionFactory $storeCollectionFactory
+     * @param ScopeConfigInterface $scopeConfig
+     * @param StoreConfigFactory $storeConfigFactory
      */
     public function __construct(
-        \Magento\Store\Model\ResourceModel\Store\CollectionFactory $storeCollectionFactory,
-        \Magento\Framework\App\Config\ScopeConfigInterface $scopeConfig,
-        \Magento\Store\Model\Data\StoreConfigFactory $storeConfigFactory
+        CollectionFactory $storeCollectionFactory,
+        ScopeConfigInterface $scopeConfig,
+        StoreConfigFactory $storeConfigFactory
     ) {
         $this->storeCollectionFactory = $storeCollectionFactory;
         $this->scopeConfig = $scopeConfig;
         $this->storeConfigFactory = $storeConfigFactory;
     }
 
     /**
+     * Get store configurations
+     *
      * @param string[] $storeCodes list of stores by store codes, will return all if storeCodes is not set
-     * @return \Magento\Store\Api\Data\StoreConfigInterface[]
+     * @return StoreConfigInterface[]
      */
     public function getStoreConfigs(array $storeCodes = null)
     {
@@ -71,12 +82,14 @@ public function getStoreConfigs(array $storeCodes = null)
     }
 
     /**
-     * @param \Magento\Store\Model\Store $store
-     * @return \Magento\Store\Api\Data\StoreConfigInterface
+     * Get store specific configs
+     *
+     * @param Store|StoreInterface $store
+     * @return StoreConfigInterface
      */
-    protected function getStoreConfig($store)
+    public function getStoreConfig($store)
     {
-        /** @var \Magento\Store\Model\Data\StoreConfig $storeConfig */
+        /** @var StoreConfig $storeConfig */
         $storeConfig = $this->storeConfigFactory->create();
 
         $storeConfig->setId($store->getId())

app/code/Magento/StoreGraphQl/Model/Resolver/Store/StoreConfigDataProvider.php

@@ -10,6 +10,7 @@
 use Magento\Framework\App\Config\ScopeConfigInterface;
 use Magento\Store\Api\Data\StoreConfigInterface;
 use Magento\Store\Api\StoreConfigManagerInterface;
+use Magento\Store\Api\StoreRepositoryInterface;
 use Magento\Store\Model\ScopeInterface;
 use Magento\Store\Api\Data\StoreInterface;
 
@@ -33,19 +34,27 @@ class StoreConfigDataProvider
      */
     private $extendedConfigData;
 
+    /**
+     * @var StoreRepositoryInterface
+     */
+    private $storeRepository;
+
     /**
      * @param StoreConfigManagerInterface $storeConfigManager
      * @param ScopeConfigInterface $scopeConfig
+     * @param StoreRepositoryInterface $storeRepository
      * @param array $extendedConfigData
      */
     public function __construct(
         StoreConfigManagerInterface $storeConfigManager,
         ScopeConfigInterface $scopeConfig,
+        StoreRepositoryInterface $storeRepository,
         array $extendedConfigData = []
     ) {
         $this->storeConfigManager = $storeConfigManager;
         $this->scopeConfig = $scopeConfig;
         $this->extendedConfigData = $extendedConfigData;
+        $this->storeRepository = $storeRepository;
     }
 
     /**
@@ -56,11 +65,24 @@ public function __construct(
      */
     public function getStoreConfigData(StoreInterface $store): array
     {
-        $stores = $this->storeConfigManager->getStoreConfigs();
-        $defaultStoreConfig = current($this->storeConfigManager->getStoreConfigs([$store->getCode()]));
+        $defaultStoreConfig = $this->storeConfigManager->getStoreConfig($store);
         $defaultStoreConfigData = $this->prepareStoreConfigData($defaultStoreConfig);
 
-        foreach ($stores as $storeConfig) {
+        return $this->addStores($defaultStoreConfigData);
+    }
+
+    /**
+     * Add all stores
+     *
+     * @param array $defaultStoreConfigData
+     * @return mixed
+     */
+    private function addStores($defaultStoreConfigData)
+    {
+        $stores = $this->storeRepository->getList();
+
+        foreach ($stores as $store) {
+            $storeConfig = $this->storeConfigManager->getStoreConfig($store);
             $defaultStoreConfigData['stores'][] = $this->prepareStoreConfigData($storeConfig);
         }
 
@@ -101,7 +123,7 @@ private function prepareStoreConfigData($storeConfig): array
      * @param int $storeId
      * @return array
      */
-    private function getExtendedConfigData(int $storeId)
+    private function getExtendedConfigData(int $storeId): array
     {
         $extendedConfigData = [];
         foreach ($this->extendedConfigData as $key => $path) {