Implements localized attributes #662

irevoire · 2024-07-30T16:47:03Z

Pull Request

Warning

The tests won't work since the Meilisearch implementation is broken currently in rc0, see the following PR meilisearch/meilisearch#4836
I did the test on the branch that implemented the change though.

Related issue

Fixes #660

What does this PR do?

Add a new search parameter named locales to the search method
Ensure the updateSettings route can accept the new localizedAttributes setting
Create new methods getLocalizedAttributes, updateLocalizedAttributes and resetLocalizedAttributes corresponding to the sub settings GET/PUT/DELETE routes
Add tests

src/Contracts/SearchQuery.php

src/Endpoints/Delegates/HandlesSettings.php

tests/Endpoints/SearchTest.php

src/Endpoints/Delegates/HandlesSettings.php

norkunas · 2024-08-01T03:47:06Z

Too many unrelated commits :)

irevoire · 2024-08-01T08:17:16Z

Oops yes good catch, I failed my rebase 😅

curquiza · 2024-08-01T09:09:54Z

Merging! The non-passing test is not related to this PR (already present on bump-meilisearch-v1.10)

668: Update version for the next release (v1.10.0) r=brunoocasali a=meili-bot _This PR is auto-generated._ The automated script updates the version of meilisearch-php to a new version: "v1.10.0" CHANGELOGS 👇 This version introduces features released on Meilisearch v1.10.0 🎉 Check out the changelog of [Meilisearch v1.10.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.10.0) for more information on the changes. ## 🚀 Enhancements - Add localized attributes settings (#662) `@/irevoire` ``` $client->index('INDEX_NAME')->updateLocalizedAttributes([ 'locales' => ['jpn'], 'attributePatterns' => ['*_ja'] ]); ``` - Add `locales` search parameter (#663) `@/irevoire` ```php $client->index('INDEX_NAME')->search('進撃の巨人', [ 'locales' => ['jpn'] ]); ``` - Add federation options when doing multi-search (#663) `@/irevoire` ```php $client->multiSearch([ (new SearchQuery()) ->setIndexUid('movies')) ->setQuery('batman') ->setLimit(5), (new SearchQuery()) ->setIndexUid('comics') ->setQuery('batman') ->setLimit(5), ], (new MultiSearchFederation()) ); ``` - Add capability to update documents by function (⚠️ experimental feature) (#664) `@/irevoire` ```php $function = ' if doc.id % context.modulo == 0 { doc.title = `kefir would read ${doc.title}`; }; doc.remove("comment"); doc.remove("genre"); '; $client-> index->updateDocumentsByFunction($function, ['context' => ['modulo' => 3]]); ``` ## ⚙️ Maintenance/misc - Update CI (#666 & #667) `@/norkunas` Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com>

irevoire added the Meilisearch bump Changes related to the Meilisearch bump version label Jul 30, 2024

irevoire requested a review from curquiza July 30, 2024 16:47