Merge pull request #696 from akeneo/release/104.3.8

Release/104.3.8

Author: magentix

CHANGELOG.md

@@ -487,3 +487,6 @@
 
 ### Version 104.3.7 :
 * PGTO-430: Allow to assign an image to the parent or child only
+
+### Version 104.3.8 :
+* PGTO-433: Product default name logic

Job/Product.php

@@ -314,12 +314,6 @@ class Product extends JobImport
      * @var CollectionFactory $categoryCollectionFactory
      */
     protected $categoryCollectionFactory;
-    /**
-     * The default name value when empty
-     *
-     * @var ?array $defaultNameValue
-     */
-    protected $defaultNameValue = null;
 
     /**
      * Product constructor.
@@ -784,8 +778,6 @@ public function insertData()
                     }
                 }
 
-                $product = $this->handleNoName($product);
-
                 /** @var bool $result */
                 $result = $this->entitiesHelper->insertDataFromApi(
                     $product,
@@ -2457,6 +2449,8 @@ public function setValues()
             );
         }
 
+        $this->handleNoName();
+
         if ($this->configHelper->isAdvancedLogActivated()) {
             $this->logImportedEntities($this->logger, true, 'identifier');
         }
@@ -5132,29 +5126,28 @@ private function updateProductVisibility(string $tmpTable, array $mappings): arr
     }
 
     /**
-     * If product has no name in Akeneo, give it an empty string name
+     * Set default product name for default if empty
      */
-    protected function handleNoName(array $product): array
+    protected function handleNoName(): void
     {
-        if (array_key_exists(ProductInterface::NAME, $product['values'])) {
-            return $product;
-        }
+        $connection = $this->entitiesHelper->getConnection();
 
-        if ($this->defaultNameValue === null) {
-            try {
-                $attribute = $this->akeneoClient->getAttributeApi()->get(ProductInterface::NAME);
-                $this->defaultNameValue = [
-                    'locale' => ($attribute['localizable'] ?? false) ? $this->storeHelper->getAdminLang() : null,
-                    'scope' => ($attribute['scopable'] ?? false) ? $this->configHelper->getAdminDefaultChannel() : null,
-                    'data' => '',
-                ];
-            } catch (Exception) {
-                $this->defaultNameValue = ['locale' => null, 'scope' => null, 'data' => ''];
-            }
+        $columnIdentifier = $this->entitiesHelper->getColumnIdentifier(
+            $this->entitiesHelper->getTable('catalog_product_entity')
+        );
+
+        $attribute = $this->eavConfig->getAttribute('catalog_product', ProductInterface::NAME);
+        if (!$attribute) {
+            return;
         }
 
-        $product['values'][ProductInterface::NAME][0] = $this->defaultNameValue;
+        $tmpTable = $this->entitiesHelper->getTableName($this->jobExecutor->getCurrentJob()->getCode());
+        $entityTable = $this->entitiesHelper->getTable('catalog_product_entity');
+        $entityVarcharTable = $this->entitiesHelper->getTable('catalog_product_entity_varchar');
 
-        return $product;
+        $connection->query('INSERT IGNORE INTO `' . $entityVarcharTable . '`
+            (`attribute_id`, `store_id`, `value`, `' . $columnIdentifier . '`)
+            SELECT ' . $attribute->getId() . ', 0, NULL, `' . $columnIdentifier . '` FROM `' . $entityTable . '` e
+            INNER JOIN `' . $tmpTable . '` t ON e.`entity_id` = t.`_entity_id`');
     }
 }

composer.json

@@ -14,7 +14,7 @@
     "nyholm/psr7": "^1.5"
   },
   "type": "magento2-module",
-  "version": "104.3.7",
+  "version": "104.3.8",
   "license": [
     "OSL-3.0",
     "AFL-3.0"