Magento 2.2.5 changes in indexing `catalog_category_product_index`

[jasperzeinstra]

Tweakwise export isn't compatibale with Magento 2.2.5. They've changed the way the category products are indexed. More information can be found here: https://devdocs.magento.com/guides/v2.2/config-guide/cli/config-cli-subcommands-index.html#config-cli-subcommands-index-reindex-parallel

In the database the table catalog_category_product_index isn't filled any more, instead they've created a table for each store (catalog_category_product_index_store1, catalog_category_product_index_store2, …). Tweakwise export isn't expecting this and isn't finding any category product relation anymore. Resulting in empty categories.

Environment

• [X] PHP Version: (7.0)
• [X] Magento Version: 2.2.5
• [X] Magento Enterprise: (yes)
• [X] Tweakwise Export Version: 1.1.3
• [X] Magento Deploy Mode: (development)
• [X] Third party modules: -

Steps to reproduce

1. Install Magento from master branch.
2. Run indexer
3. Export to tweakwise

Actual result

1. Notice that product in XML export file doesn't contain any category ids.

Expected result

• [ ] Linked categories of product should be in the export XML file.

[martijnrmediact]

@Fgruntjes could you look into this? We have multiple clients with the same issue.

[hostep]

Confirmed, we also see this problem after upgrading to Magento 2.2.5 (from 2.2.2).

Dirty and ugly fix (not very well tested):

--- a/src/Model/Write/Products/CollectionDecorator/CategoryReference.php
+++ b/src/Model/Write/Products/CollectionDecorator/CategoryReference.php
@@ -18,7 +18,7 @@ class CategoryReference extends AbstractDecorator
     {
         $query = $this->getConnection()
             ->select()
-            ->from($this->getTableName(AbstractAction::MAIN_INDEX_TABLE), ['category_id', 'product_id'])
+            ->from($this->getTableName(AbstractAction::MAIN_INDEX_TABLE . '_store' . $collection->getStoreId()), ['category_id', 'product_id'])
             ->where('store_id = ?', $collection->getStoreId())
             ->where('product_id IN(' . implode(',', $collection->getIds()) . ')')
             ->query();

You can't add this like this to the module, since this module needs to support Magento < 2.2.5 & Magento >= 2.2.5, so a proper solution is probably to rewrite this select and use some kind of built-in function from Magento to get this data (if that exists).

[jasperzeinstra]

@hostep we've already created a nice fix in the way Magento indented to get the table name based on a store, see: https://github.com/EmicoEcommerce/Magento2TweakwiseExport/pull/54

[hostep]

@jasperzeinstra: ah, I failed to look into the open PR's 😄 Looks much better then my hack.

Thanks for the heads up! 🙂

Update: left a comment behind in the PR, since it's not the correct solution if we want this module to be compatible with Magento 2.2.4 and lower.

[edwinljacobs]

This fix (a variant thereof) will soon be released, update will follow.

[edwinljacobs]

Fix for this issue has been released in 1.2.0