ACP2E-294: Category Ids Not Available in scheduled Export Filters

Author: dhorytskyi

app/code/Magento/ImportExport/Model/Export/Entity/AbstractEntity.php

@@ -11,6 +11,7 @@
 /**
  * Export entity abstract model
  *
+ * phpcs:ignore Magento2.Classes.AbstractApi
  * @api
  *
  * @SuppressWarnings(PHPMD.TooManyFields)
@@ -48,8 +49,6 @@ abstract class AbstractEntity
     protected $_disabledAttrs = [];
 
     /**
-     * Entity type id.
-     *
      * @var int
      */
     protected $_entityTypeId;
@@ -97,8 +96,6 @@ abstract class AbstractEntity
     protected $_messageTemplates = [];
 
     /**
-     * Parameters.
-     *
      * @var array
      */
     protected $_parameters = [];
@@ -175,8 +172,10 @@ public function __construct(
     protected function _initStores()
     {
         foreach ($this->_storeManager->getStores(true) as $store) {
+            // phpstan:ignore "Access to an undefined property"
             $this->_storeIdToCode[$store->getId()] = $store->getCode();
         }
+        // phpstan:ignore "Access to an undefined property"
         ksort($this->_storeIdToCode);
         // to ensure that 'admin' store (ID is zero) goes first
 
@@ -251,6 +250,8 @@ protected function _initAttrValues()
      * @param \Magento\Eav\Model\Entity\Collection\AbstractCollection $collection
      * @return \Magento\Eav\Model\Entity\Collection\AbstractCollection
      * @SuppressWarnings(PHPMD.CyclomaticComplexity)
+     * @SuppressWarnings(PHPMD.NPathComplexity)
+     * phpcs:disable Generic.Metrics.NestingLevel
      */
     protected function _prepareEntityCollection(\Magento\Eav\Model\Entity\Collection\AbstractCollection $collection)
     {
@@ -335,6 +336,7 @@ protected function _prepareEntityCollection(\Magento\Eav\Model\Entity\Collection
         }
         return $collection;
     }
+    //phpcs:enable Generic.Metrics.NestingLevel
 
     /**
      * Add error with corresponding current data source row number.
@@ -348,6 +350,7 @@ public function addRowError($errorCode, $errorRowNum)
         $errorCode = (string)$errorCode;
         $this->_errors[$errorCode][] = $errorRowNum + 1;
         // one added for human readability
+        // phpstan:ignore "Access to an undefined property"
         $this->_invalidRows[$errorRowNum] = true;
         $this->_errorsCount++;
 
@@ -440,6 +443,7 @@ public function getAttributeOptions(\Magento\Eav\Model\Entity\Attribute\Abstract
                         }
                     }
                 }
+                // phpcs:disable Magento2.CodeAnalysis.EmptyBlock.DetectedCatch
             } catch (\Exception $e) {
                 // ignore exceptions connected with source models
             }
@@ -504,6 +508,7 @@ public function getErrorsCount()
      */
     public function getInvalidRowsCount()
     {
+        // phpstan:ignore "Access to an undefined property"
         return count($this->_invalidRows);
     }
 
@@ -569,6 +574,7 @@ public function setWriter(AbstractAdapter $writer)
 
     /**
      * Clean cached values
+     *
      * @since 100.1.2
      */
     public function __destruct()

dev/tests/integration/testsuite/Magento/CatalogImportExport/Model/Export/ProductTest.php

@@ -9,7 +9,9 @@
 namespace Magento\CatalogImportExport\Model\Export;
 
 use Magento\Catalog\Api\ProductRepositoryInterface;
+use Magento\Catalog\Model\ResourceModel\Product\Attribute\Collection as ProductAttributeCollection;
 use Magento\Catalog\Observer\SwitchPriceAttributeScopeOnConfigChange;
+use Magento\CatalogImportExport\Model\Export\Product\Type\Simple as SimpleProductType;
 use Magento\Framework\App\Config\ReinitableConfigInterface;
 
 /**
@@ -711,6 +713,24 @@ public function testExportProductWithRestrictedWebsite(): void
         $this->assertStringNotContainsString('"Virtual Product With Custom Options"', $exportData);
     }
 
+    public function testFilterAttributeCollection(): void
+    {
+        $collection = $this->objectManager->create(ProductAttributeCollection::class);
+        $collection = $this->model->filterAttributeCollection($collection);
+        $attributes = [];
+        foreach ($collection->getItems() as $attribute) {
+            $attributes[] = $attribute->getAttributeCode();
+        }
+
+        $simpleProductType = $this->objectManager->create(SimpleProductType::class);
+        $disabledAttributes = $simpleProductType->getDisabledAttrs();
+        $this->assertEmpty(
+            array_intersect($disabledAttributes, $attributes),
+            'Disabled attributes are not filtered.'
+        );
+        $this->assertContains('category_ids', $attributes);
+    }
+
     /**
      * Perform export
      *