Merge pull request #2482 from rbayet/feat_2437_allows_0_false_to_be_i…

…ndexed

Feat 2437 allows 0 false to be indexed

src/module-elasticsuite-catalog/Block/Plugin/Adminhtml/Product/Attribute/Edit/Tab/FrontPlugin.php

@@ -20,7 +20,6 @@
 use Magento\Framework\Registry;
 use Smile\ElasticsuiteCatalog\Model\Attribute\Source\FilterBooleanLogic;
 use Smile\ElasticsuiteCatalog\Model\Attribute\Source\FilterSortOrder;
-use Smile\ElasticsuiteCore\Search\Request\BucketInterface;
 use Magento\Framework\Data\Form\Element\Fieldset;
 use Magento\Catalog\Api\Data\EavAttributeInterface;
 
@@ -114,6 +113,16 @@ public function afterSetForm(Front $subject, Front $result, Form $form)
             $form->getElement('is_searchable')->setDisabled(1);
         }
 
+        if (($this->getAttribute()->getSourceModel() == 'Magento\Eav\Model\Entity\Attribute\Source\Boolean')
+            || ($this->getAttribute()->getBackendType() == 'int')
+            || ($this->getAttribute()->getFrontendClass() == 'validate-digits')
+            || ($this->getAttribute()->getBackendType() == 'decimal' || $this->getAttribute()->getFrontendClass() == 'validate-number')
+            || (in_array($this->getAttribute()->getFrontendInput(), ['select', 'multiselect'])
+                || $this->getAttribute()->getSourceModel() != '')
+        ) {
+            $this->addIncludeZeroFalseField($fieldset);
+        }
+
         $this->appendFieldsDependency($subject);
 
         return $result;
@@ -428,6 +437,36 @@ private function appendSliderDisplayRelatedFields($form, $subject)
         return $this;
     }
 
+    /**
+     * Add field allowing to configure if zero/false values should be indexed or ignored.
+     *
+     * @param Fieldset $fieldset Target fieldset
+     *
+     * @return FrontPlugin
+     */
+    private function addIncludeZeroFalseField(Fieldset $fieldset)
+    {
+        $includeZeroFalseNote = __(
+            // phpcs:ignore Generic.Files.LineLength
+            'If set to Yes, zero (integer or numeric attribute) or false (boolean attribute) values will be indexed in the search engine (default is No).'
+            . ' Also applies to source model keys/values of Dropdown/Multiple Select attributes.'
+        );
+        $fieldset->addField(
+            'include_zero_false_values',
+            'select',
+            [
+                'name'   => 'include_zero_false_values',
+                'label'  => __('Include zero or false values'),
+                'values' => $this->booleanSource->toOptionArray(),
+                // phpcs:ignore Generic.Files.LineLength
+                'note'   => $includeZeroFalseNote,
+            ],
+            'used_for_sortby'
+        );
+
+        return $this;
+    }
+
     /**
      * Manage dependency between fields.
      *

src/module-elasticsuite-catalog/Helper/AbstractAttribute.php

@@ -64,6 +64,11 @@ abstract class AbstractAttribute extends Mapping
      */
     private $attributeMappers = [];
 
+    /**
+     * @var array
+     */
+    private $attributeCleaners = [];
+
     /**
      * @param Context          $context           Helper context.
      * @param AttributeFactory $attributeFactory  Factory used to create attributes.
@@ -163,6 +168,8 @@ public function getFieldType(AttributeInterface $attribute)
      * multivalued attributes merging on composite products).
      * ES doesn't care of having array of int when it an int is required.
      *
+     * @SuppressWarnings(PHPMD.ElseExpression)
+     *
      * @param AttributeInterface $attribute Product attribute.
      * @param integer            $storeId   Store id.
      * @param mixed              $value     Raw value to be parsed.
@@ -175,13 +182,26 @@ public function prepareIndexValue(AttributeInterface $attribute, $storeId, $valu
         $values = [];
 
         $mapperKey = 'simple_' . $attribute->getId();
-
         if (!isset($this->attributeMappers[$mapperKey])) {
             $this->attributeMappers[$mapperKey] = function ($value) use ($attribute) {
                 return $this->prepareSimpleIndexAttributeValue($attribute, $value);
             };
         }
 
+        if (!isset($this->attributeCleaners[$mapperKey])) {
+            if ($attribute->getIncludeZeroFalseValues()) {
+                // Filter empty values while keeping "0" (int or float) and "false" value.
+                $this->attributeCleaners[$mapperKey] = function ($value) {
+                    return (($value === false) || strlen($value));
+                };
+            } else {
+                // Filter out empty values. Also removes "0" and "false" values.
+                $this->attributeCleaners[$mapperKey] = function ($value) {
+                    return !empty($value);
+                };
+            }
+        }
+
         if ($this->usesSource($attribute) && !is_array($value)) {
             $value = explode(',', $value);
         }
@@ -191,7 +211,7 @@ public function prepareIndexValue(AttributeInterface $attribute, $storeId, $valu
         }
 
         $value = array_map($this->attributeMappers[$mapperKey], $value);
-        $value = array_filter($value);
+        $value = array_filter($value, $this->attributeCleaners[$mapperKey]);
         $value = array_values($value);
         $values[$attributeCode] = $value;
 
@@ -254,7 +274,10 @@ public function getIndexOptionText(AttributeInterface $attribute, $storeId, $opt
         if (!isset($this->attributeOptionTextCache[$storeId][$attributeId][$optionId])) {
             $optionValue = $attribute->getSource()->getIndexOptionText($optionId);
             if ($this->getFieldType($attribute) == FieldInterface::FIELD_TYPE_BOOLEAN) {
-                $optionValue = $attribute->getStoreLabel($storeId);
+                $optionValue = null;
+                if ($optionId == \Magento\Eav\Model\Entity\Attribute\Source\Boolean::VALUE_YES) {
+                    $optionValue = $attribute->getStoreLabel($storeId);
+                }
             }
             $this->attributeOptionTextCache[$storeId][$attributeId][$optionId] = $optionValue;
         }

src/module-elasticsuite-catalog/Model/Layer/Filter/Boolean.php

@@ -22,6 +22,7 @@
 use Smile\ElasticsuiteCatalog\Api\LayeredNavAttributeInterface;
 use Smile\ElasticsuiteCatalog\Helper\ProductAttribute;
 use Smile\ElasticsuiteCatalog\Model\Attribute\LayeredNavAttributesProvider;
+use Smile\ElasticsuiteCore\Search\Request\BucketInterface;
 
 /**
  * Product boolean filter implementation.
@@ -147,6 +148,7 @@ protected function getFilterField()
     /**
      * @SuppressWarnings(PHPMD.CamelCaseMethodName)
      * @SuppressWarnings(PHPMD.ElseExpression)
+     * @SuppressWarnings(PHPMD.CyclomaticComplexity)
      *
      * {@inheritDoc}
      */
@@ -184,6 +186,12 @@ protected function _initItems()
             $item->setApplyFilterValue(array_values($applyValue));
         }
 
+        if (($this->getAttributeModel()->getFacetSortOrder() == BucketInterface::SORT_ORDER_MANUAL)
+            && (count($this->_items) > 1)
+        ) {
+            krsort($this->_items, SORT_NUMERIC);
+        }
+
         return $this;
     }
 }

src/module-elasticsuite-catalog/Plugin/Catalog/Eav/AttributePlugin.php

@@ -44,6 +44,11 @@ class AttributePlugin
      */
     private $indicesConfig;
 
+    /**
+     * @var \Magento\Framework\Indexer\IndexerRegistry
+     */
+    private $indexerRegistry;
+
     /**
      * AttributePlugin constructor.
      *
@@ -84,6 +89,10 @@ public function afterSave(
             $invalidateIndex = true;
         }
 
+        if ($subject->dataHasChangedFor('include_zero_false_values')) {
+            $invalidateIndex = true;
+        }
+
         if ($cleanCache) {
             $this->indicesConfig->reset();
         }

src/module-elasticsuite-catalog/Setup/CatalogSetup.php

@@ -583,6 +583,32 @@ public function addFilterBooleanLogicField(SchemaSetupInterface $setup)
         );
     }
 
+    /**
+     * Add "include_zero_false_values" field to catalog_eav_attribute table.
+     *
+     * @param \Magento\Framework\Setup\SchemaSetupInterface $setup Schema Setup
+     *
+     * @return void
+     */
+    public function addIncludeZeroFalseValues(SchemaSetupInterface $setup)
+    {
+        $connection = $setup->getConnection();
+        $table      = $setup->getTable('catalog_eav_attribute');
+
+        // Append a column 'include_zero_false_values' into the db.
+        $connection->addColumn(
+            $table,
+            'include_zero_false_values',
+            [
+                'type'     => \Magento\Framework\DB\Ddl\Table::TYPE_BOOLEAN,
+                'nullable' => false,
+                'default'  => 0,
+                'size'     => 1,
+                'comment'  => 'Should the search engine index zero (integer or decimal attribute) or false (boolean attribute) values',
+            ]
+        );
+    }
+
     /**
      * Update attribute value for an entity with a default value.
      * All existing values are erased by the new value.

src/module-elasticsuite-catalog/Setup/InstallSchema.php

@@ -75,6 +75,9 @@ public function install(SchemaSetupInterface $setup, ModuleContextInterface $con
         // Introduced in version 1.6.0.
         $this->catalogSetup->addFilterBooleanLogicField($setup);
 
+        // Introduced in version 1.6.1.
+        $this->catalogSetup->addIncludeZeroFalseValues($setup);
+
         $setup->endSetup();
     }
 }

src/module-elasticsuite-catalog/Setup/UpgradeSchema.php

@@ -48,6 +48,7 @@ public function __construct(CatalogSetupFactory $catalogSetupFactory)
      * @param ModuleContextInterface $context Context
      *
      * @return void
+     * @SuppressWarnings(PHPMD.NPathComplexity)
      */
     public function upgrade(
         SchemaSetupInterface $setup,
@@ -84,6 +85,10 @@ public function upgrade(
             $this->catalogSetup->addFilterBooleanLogicField($setup);
         }
 
+        if (version_compare($context->getVersion(), '1.6.1', '<')) {
+            $this->catalogSetup->addIncludeZeroFalseValues($setup);
+        }
+
         $setup->endSetup();
     }
 }

src/module-elasticsuite-catalog/etc/module.xml

@@ -16,7 +16,7 @@
  */
  -->
 <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd">
-    <module name="Smile_ElasticsuiteCatalog" setup_version="1.6.0">
+    <module name="Smile_ElasticsuiteCatalog" setup_version="1.6.1">
         <sequence>
             <module name="Smile_ElasticsuiteCore" />
             <module name="Magento_Search" />

src/module-elasticsuite-catalog/i18n/en_US.csv

@@ -106,3 +106,5 @@ Attributes,Attributes
 "Facet internal logic","Facet internal logic"
 "Enable adaptive slider","Enable adaptive slider"
 "If enabled, when necessary to support the presence of outlier values in the navigation context (for instance, a very high price amidst a majority of low prices), the price slider behavior changes so that the middle of the slider range corresponds to the median price instead of the price at the middle of the range.","If enabled, when necessary to support the presence of outlier values in the navigation context (for instance, a very high price amidst a majority of low prices), the price slider behavior changes so that the middle of the slider range corresponds to the median price instead of the price at the middle of the range."
+"Include zero or false values","Include zero or false values"
+"If set to Yes, zero (integer or numeric attribute) or false (boolean attribute) values will be indexed in the search engine (default is No). Also applies to source model keys/values of Dropdown/Multiple Select attributes.","If set to Yes, zero (integer or numeric attribute) or false (boolean attribute) values will be indexed in the search engine (default is No). Also applies to source model keys/values of Dropdown/Multiple Select attributes."

src/module-elasticsuite-catalog/i18n/fr_FR.csv

@@ -106,3 +106,5 @@ Attributes,Attributs
 "Facet internal logic","Logique interne de la facette"
 "Enable adaptive slider","Activer le slider adaptatif"
 "If enabled, when necessary to support the presence of outlier values in the navigation context (for instance, a very high price amidst a majority of low prices), the price slider behavior changes so that the middle of the slider range corresponds to the median price instead of the price at the middle of the range.","Si activé, lorsque c'est nécessaire pour supporter la présence de valeurs extrêmes dans le contexte de navigation (par exemple, un prix très élevé au milieu d'une majorité de prix bas), le comportement du slider de prix change de façon à ce que le milieu de la plage du slider corresponde au prix médian au lieu du prix situé au milieu de la plage de prix."
+"Include zero or false values","Inclure les valeurs à zéro ou 'false'"
+"If set to Yes, zero (integer or numeric attribute) or false (boolean attribute) values will be indexed in the search engine (default is No). Also applies to source model keys/values of Dropdown/Multiple Select attributes.","Si réglé à Oui, les valeurs à zéro (attribute entier ou numérique) ou 'false/Non' (attribut booléen) seront indexées dans le moteur de recherche (ce n'est pas le cas par défaut). S'applique également aux clefs/valeurs des modèles sources des attributs Dropdown/Multiple Select."