Merge pull request #1643 from algolia/release/3.14.3

3.14.3 to main

Author: damcou

Block/Algolia.php

@@ -10,6 +10,7 @@
 use Algolia\AlgoliaSearch\Helper\Entity\ProductHelper;
 use Algolia\AlgoliaSearch\Helper\Entity\SuggestionHelper;
 use Algolia\AlgoliaSearch\Helper\LandingPageHelper;
+use Algolia\AlgoliaSearch\Model\LandingPage as LandingPageModel;
 use Algolia\AlgoliaSearch\Registry\CurrentCategory;
 use Algolia\AlgoliaSearch\Registry\CurrentProduct;
 use Algolia\AlgoliaSearch\Service\Product\SortingTransformer;
@@ -215,7 +216,7 @@ protected function getAddToCartUrl($additional = []): string
         return $this->_urlBuilder->getUrl('checkout/cart/add', $routeParams);
     }
 
-    protected function getCurrentLandingPage(): LandingPage|null|false
+    protected function getCurrentLandingPage(): LandingPageModel|null|false
     {
         $landingPageId = $this->getRequest()->getParam('landing_page_id');
         if (!$landingPageId) {

CHANGELOG.md

@@ -1,5 +1,19 @@
 # CHANGE LOG
 
+## 3.14.3
+
+### Updates
+- Updated PHP client version in the composer file
+- Tests: added new integration scenarios including multi-stores ones.
+
+### Bug Fixes
+- Fixed landing page typing error
+- Improved query method for alternate root categories - Thank you @igorfigueiredogen
+- Fixed an error where a missing prefix can throw errors with strong types added to ConfigHelper
+- Fixed an error where stale cache data was preventing ranking applied on replica
+- Fixed reindexing issue with NeuralSearch
+- Tests : Fixed current integration tests
+
 ## 3.14.2
 
 ### Updates

Console/Command/ReplicaSyncCommand.php

@@ -11,7 +11,7 @@
 use Algolia\AlgoliaSearch\Exceptions\ExceededRetriesException;
 use Algolia\AlgoliaSearch\Helper\Entity\ProductHelper;
 use Algolia\AlgoliaSearch\Service\StoreNameFetcher;
-use Magento\Framework\App\State;
+use Magento\Framework\App\State as AppState;
 use Magento\Framework\Console\Cli;
 use Magento\Framework\Exception\LocalizedException;
 use Magento\Framework\Exception\NoSuchEntityException;
@@ -27,12 +27,12 @@ public function __construct(
         protected ReplicaManagerInterface $replicaManager,
         protected ProductHelper           $productHelper,
         protected StoreManagerInterface   $storeManager,
-        State                             $state,
+        AppState                          $appState,
         StoreNameFetcher                  $storeNameFetcher,
         ?string                           $name = null
     )
     {
-        parent::__construct($state, $storeNameFetcher, $name);
+        parent::__construct($appState, $storeNameFetcher, $name);
     }
 
     protected function getReplicaCommandName(): string

Helper/AlgoliaHelper.php

@@ -6,6 +6,7 @@
 use Algolia\AlgoliaSearch\Configuration\SearchConfig;
 use Algolia\AlgoliaSearch\Exceptions\AlgoliaException;
 use Algolia\AlgoliaSearch\Exceptions\ExceededRetriesException;
+use Algolia\AlgoliaSearch\Model\Search\SearchRulesResponse;
 use Algolia\AlgoliaSearch\Response\AbstractResponse;
 use Algolia\AlgoliaSearch\Response\BatchIndexingResponse;
 use Algolia\AlgoliaSearch\Response\MultiResponse;
@@ -299,7 +300,7 @@ public function moveIndex(string $fromIndexName, string $toIndexName): void
                 'destination' => $toIndexName
             ]
         );
-        self::setLastOperationInfo($fromIndexName, $response);
+        self::setLastOperationInfo($toIndexName, $response);
     }
 
     /**
@@ -360,6 +361,10 @@ public function mergeSettings($indexName, $settings, $mergeSettingsFrom = '')
 
         $removes = ['slaves', 'replicas', 'decompoundedAttributes'];
 
+        if (isset($onlineSettings['mode']) && $onlineSettings['mode'] == 'neuralSearch') {
+            $removes[] = 'mode';
+        }
+
         if (isset($settings['attributesToIndex'])) {
             $settings['searchableAttributes'] = $settings['attributesToIndex'];
             unset($settings['attributesToIndex']);
@@ -453,6 +458,19 @@ public function saveRule(array $rule, string $indexName, bool $forwardToReplicas
         self::setLastOperationInfo($indexName, $res);
     }
 
+    /**
+     * @param string $indexName
+     * @param array $rules
+     * @param bool $forwardToReplicas
+     * @return void
+     */
+    public function saveRules(string $indexName, array $rules, bool $forwardToReplicas = false): void
+    {
+        $res = $this->client->saveRules($indexName, $rules, $forwardToReplicas);
+
+        self::setLastOperationInfo($indexName, $res);
+    }
+
 
     /**
      * @param string $indexName
@@ -521,6 +539,21 @@ public function copyQueryRules(string $fromIndexName, string $toIndexName): void
         self::setLastOperationInfo($fromIndexName, $response);
     }
 
+    /**
+     * @param string $indexName
+     * @param array|null $searchRulesParams
+     *
+     * @return SearchRulesResponse|mixed[]
+     *
+     * @throws AlgoliaException
+     */
+    public function searchRules(string $indexName, array$searchRulesParams = null)
+    {
+        $this->checkClient(__FUNCTION__);
+
+        return $this->client->searchRules($indexName, $searchRulesParams);
+    }
+
     /**
      * @param $methodName
      * @return void

Helper/ConfigHelper.php

@@ -446,7 +446,7 @@ public function getAutocompleteSections($storeId = null)
     }
 
     protected function serialize(array $value): string {
-        return $this->serializer->serialize($value);
+        return $this->serializer->serialize($value) ?: '';
     }
 
     /**
@@ -1157,7 +1157,7 @@ public function getRawSortingValue(?int $storeId = null): string
      * @param int|null $scopeId
      * @return void
      */
-    public function setSorting(array $sorting, ?string $scope = null, ?int $scopeId = null): void
+    public function setSorting(array $sorting, string $scope = Magento\Framework\App\Config\ScopeConfigInterface::SCOPE_TYPE_DEFAULT, ?int $scopeId = null): void
     {
         $this->configWriter->save(
             self::SORTING_INDICES,
@@ -1242,7 +1242,7 @@ public function getSearchOnlyAPIKey($storeId = null)
      */
     public function getIndexPrefix(int $storeId = null): string
     {
-        return $this->configInterface->getValue(self::INDEX_PREFIX, ScopeInterface::SCOPE_STORE, $storeId);
+        return (string) $this->configInterface->getValue(self::INDEX_PREFIX, ScopeInterface::SCOPE_STORE, $storeId);
     }
 
     /**

Helper/Entity/CategoryHelper.php

@@ -125,7 +125,7 @@ public function getCategoryCollectionQuery($storeId, $categoryIds = null)
     {
         /** @var \Magento\Store\Model\Store $store */
         $store = $this->storeManager->getStore($storeId);
-        $storeRootCategoryPath = sprintf('%d/%d', $this->getRootCategoryId(), $store->getRootCategoryId());
+        $storeRootCategoryPath = sprintf('%d/%d/', $this->getRootCategoryId(), $store->getRootCategoryId());
 
         $unserializedCategorysAttrs = $this->getAdditionalAttributes($storeId);
         $additionalAttr = array_column($unserializedCategorysAttrs, 'attribute');

Helper/Entity/PageHelper.php

@@ -59,7 +59,7 @@ public function getPages($storeId, array $pageIds = null)
             $magentoPages->addFieldToFilter('page_id', ['in' => $pageIds]);
         }
 
-        $excludedPages = $this->getExcludedPageIds();
+        $excludedPages = $this->getExcludedPageIds($storeId);
         if (count($excludedPages)) {
             $magentoPages->addFieldToFilter('identifier', ['nin' => $excludedPages]);
         }
@@ -116,9 +116,9 @@ public function getPages($storeId, array $pageIds = null)
         return $pages;
     }
 
-    public function getExcludedPageIds()
+    public function getExcludedPageIds($storeId = null)
     {
-        $excludedPages = array_values($this->configHelper->getExcludedPages());
+        $excludedPages = array_values($this->configHelper->getExcludedPages($storeId));
         foreach ($excludedPages as &$excludedPage) {
             $excludedPage = $excludedPage['attribute'];
         }

Helper/Entity/ProductHelper.php

@@ -359,9 +359,12 @@ public function setSettings(string $indexName, string $indexNameTmp, int $storeI
             $this->logger->log('Pushing the same settings to TMP index as well');
         }
 
-        $this->setFacetsQueryRules($indexName);
+        $this->setFacetsQueryRules($indexName, $storeId);
+        $this->algoliaHelper->waitLastTask();
+
         if ($saveToTmpIndicesToo) {
-            $this->setFacetsQueryRules($indexNameTmp);
+            $this->setFacetsQueryRules($indexNameTmp, $storeId);
+            $this->algoliaHelper->waitLastTask();
         }
 
         $this->replicaManager->syncReplicasToAlgolia($storeId, $indexSettings);
@@ -1204,17 +1207,16 @@ protected function getAttributesForFaceting($storeId)
 
     /**
      * @param $indexName
+     * @param $storeId
      * @return void
      * @throws AlgoliaException
      */
-    protected function setFacetsQueryRules($indexName)
+    protected function setFacetsQueryRules($indexName, $storeId = null)
     {
-        $client = $this->algoliaHelper->getClient();
-
         $this->clearFacetsQueryRules($indexName);
 
         $rules = [];
-        $facets = $this->configHelper->getFacets();
+        $facets = $this->configHelper->getFacets($storeId);
         foreach ($facets as $facet) {
             if (!array_key_exists('create_rule', $facet) || $facet['create_rule'] !== '1') {
                 continue;
@@ -1245,7 +1247,7 @@ protected function setFacetsQueryRules($indexName)
 
         if ($rules) {
             $this->logger->log('Setting facets query rules to "' . $indexName . '" index: ' . json_encode($rules));
-            $client->saveRules($indexName, $rules, true);
+            $this->algoliaHelper->saveRules($indexName, $rules, true);
         }
     }
 
@@ -1260,8 +1262,7 @@ protected function clearFacetsQueryRules($indexName): void
             $hitsPerPage = 100;
             $page = 0;
             do {
-                $client = $this->algoliaHelper->getClient();
-                $fetchedQueryRules = $client->searchRules($indexName, [
+                $fetchedQueryRules = $this->algoliaHelper->searchRules($indexName, [
                     'context' => 'magento_filters',
                     'page' => $page,
                     'hitsPerPage' => $hitsPerPage,
@@ -1273,7 +1274,7 @@ protected function clearFacetsQueryRules($indexName): void
                 }
 
                 foreach ($fetchedQueryRules['hits'] as $hit) {
-                    $client->deleteRule($indexName, $hit[AlgoliaHelper::ALGOLIA_API_OBJECT_ID], true);
+                    $this->algoliaHelper->deleteRule($indexName, $hit[AlgoliaHelper::ALGOLIA_API_OBJECT_ID], true);
                 }
 
                 $page++;

Helper/LandingPageHelper.php

@@ -59,7 +59,7 @@ public function __construct(
         parent::__construct($context);
     }
 
-    public function getLandingPage($pageId)
+    public function getLandingPage($pageId): LandingPage|null|false
     {
         if ($pageId !== null && $pageId !== $this->landingPage->getId()) {
             $this->landingPage->setStoreId($this->storeManager->getStore()->getId());

Helper/Logger.php

@@ -35,7 +35,7 @@ public function isEnable()
 
     public function getStoreName($storeId)
     {
-        if ($storeId === null) {
+        if ($storeId === null || !isset($this->stores[$storeId])) {
             return 'undefined store';
         }