Issue 36196: Customer attribute and customer address attribute alway require when select 'Use Default' in config

Author: rogerdz

app/code/Magento/Customer/Model/Config/Backend/Show/Customer.php

@@ -3,15 +3,20 @@
  * Copyright © Magento, Inc. All rights reserved.
  * See COPYING.txt for license details.
  */
+declare(strict_types=1);
+
 namespace Magento\Customer\Model\Config\Backend\Show;
 
+use Magento\Config\App\Config\Source\ModularConfigSource;
 use Magento\Eav\Model\Entity\Attribute\AbstractAttribute;
 use Magento\Framework\App\Config\ScopeConfigInterface;
+use Magento\Framework\App\ObjectManager;
 
 /**
  * Customer Show Customer Model
  *
  * @author     Magento Core Team <[email protected]>
+ * @SuppressWarnings(PHPMD.UnusedPrivateField)
  */
 class Customer extends \Magento\Framework\App\Config\Value
 {
@@ -32,6 +37,11 @@ class Customer extends \Magento\Framework\App\Config\Value
      */
     private $telephoneShowDefaultValue = 'req';
 
+    /**
+     * @var ModularConfigSource
+     */
+    private $configSource;
+
     /**
      * @var array
      */
@@ -52,6 +62,8 @@ class Customer extends \Magento\Framework\App\Config\Value
      * @param \Magento\Framework\Model\ResourceModel\AbstractResource $resource
      * @param \Magento\Framework\Data\Collection\AbstractDb $resourceCollection
      * @param array $data
+     * @param ModularConfigSource|null $configSource
+     * @SuppressWarnings(PHPMD.ExcessiveParameterList)
      */
     public function __construct(
         \Magento\Framework\Model\Context $context,
@@ -62,11 +74,13 @@ public function __construct(
         \Magento\Eav\Model\Config $eavConfig,
         \Magento\Framework\Model\ResourceModel\AbstractResource $resource = null,
         \Magento\Framework\Data\Collection\AbstractDb $resourceCollection = null,
-        array $data = []
+        array $data = [],
+        ModularConfigSource $configSource = null
     ) {
         $this->_eavConfig = $eavConfig;
         parent::__construct($context, $registry, $config, $cacheTypeList, $resource, $resourceCollection, $data);
         $this->storeManager = $storeManager;
+        $this->configSource = $configSource ?: ObjectManager::getInstance()->get(ModularConfigSource::class);
     }
 
     /**
@@ -140,7 +154,8 @@ public function afterDelete()
                 $attributeObject->save();
             }
         } elseif ($this->getScope() == ScopeConfigInterface::SCOPE_TYPE_DEFAULT) {
-            $valueConfig = $this->getValueConfig($this->telephoneShowDefaultValue);
+            $defaulValue = $this->configSource->get(ScopeConfigInterface::SCOPE_TYPE_DEFAULT . '/' . $this->getPath());
+            $valueConfig = $this->getValueConfig($defaulValue === [] ? '' : $defaulValue);
             foreach ($this->_getAttributeObjects() as $attributeObject) {
                 $attributeObject->setData('is_required', $valueConfig['is_required']);
                 $attributeObject->setData('is_visible', $valueConfig['is_visible']);