CatalogImportExport categoryProcessor is using default store id values

[piotrekkaminski]

From @koenner01 on October 13, 2016 13:5

When importing product data through the catalogImportExport module, new categories will be created even though categories with the correct path already exist.

Preconditions

1. Magento 2.1
2. PHP7.0

Steps to reproduce

1. Create a category 'Test'
2. Set the category name to something completely different on default storeview
3. Create a csv with catalog product data
4. Set 'Default Category/Test' as the 'categories' value
5. Import

Expected result

1. The product is added to the existing category 'Test' (which we just created)

Actual result

1. The product will not be added to the existing category
2. A new category with the exact same path is created and the product is put in the new category
   OR
3. An error is thrown with message "1. Category "Default Category/Test" has not been created. URL key for specified store already exists. in row(s): 1"

I traced this issue back to \Magento\CatalogImportExport\Model\Import\Product\CategoryProcessor.
In the initCategories function a category collection is loaded with store id equal to the default store id.

protected function initCategories()
{
    if (empty($this->categories)) {
        $collection = $this->categoryColFactory->create();
        $collection->addAttributeToSelect('name')
            ->addAttributeToSelect('url_key')
            ->addAttributeToSelect('url_path');
        /* @var $collection \Magento\Catalog\Model\ResourceModel\Category\Collection */
        foreach ($collection as $category) {
            $structure = explode(self::DELIMITER_CATEGORY, $category->getPath());
            $pathSize = count($structure);

            $this->categoriesCache[$category->getId()] = $category;
            if ($pathSize > 1) {
                $path = [];
                for ($i = 1; $i < $pathSize; $i++) {
                    $path[] = $collection->getItemById((int)$structure[$i])->getName();
                }
                $index = implode(self::DELIMITER_CATEGORY, $path);
                $this->categories[$index] = $category->getId();
            }
        }
    }
    return $this;
}

In my opinion the collection should have a setStoreId(0) because the values for categories in the import csv should be the admin values.

$collection->setStoreId(0)
    ->addAttributeToSelect('name')
    ->addAttributeToSelect('url_key')
    ->addAttributeToSelect('url_path');

Copied from original issue: magento/magento2#6992

[piotrekkaminski]

From @sevos1984 on October 18, 2016 15:4

Internal ticket created MAGETWO-59827, thanks for posting

[piotrekkaminski]

From @MattDelac on December 18, 2016 20:12

Hello everyone,

As @koenner01, I have the same issue (Magento 2.1.2) and I don't know why categories do not work as attributes.
Indeed, it's very important to synchronize by 'Admin value' and to easily change the name in each store view.

Can someone tell me if this issue is resolved with the new update (Magento 2.1.3) ? I did not see in the changelog.
If not, when should it be resolved (approximately) ?

Thank you

[piotrekkaminski]

From @Tchinkatchuk on December 19, 2016 8:30

hello,

I dot agree with @MattDelac & @koenner01.
We should be able to do this. This is minimum valuable for export/import.

Thanks.

[piotrekkaminski]

From @MattDelac on December 19, 2016 8:44

@koenner01

Did you find a hack to do it in another way ?

Thank you