Fix: Container extension "meili_search" is not registered.

[pan93412]

Pull Request

Related issue

#370 (comment)

What does this PR do?

"meili_search" does not match the bundle name, "meilisearch," which causes some strange bugs, such as this issue.

I have completed a basic smoke test of PHP and YAML. I can confirm that the PHP configuration is now working with the Meilisearch bundle, while the old YAML configuration should function as it did before.

$ php bin/console meilisearch:import --update-settings
Importing for index App\Entity\Question
Indexed a batch of 70 / 70 App\Entity\Question entities into dev_questions index (70 indexed since start)
Setting "filterableAttributes" updated of "dev_questions".
Setting "sortableAttributes" updated of "dev_questions".
Done!

PR checklist

Please check if your PR fulfills the following requirements:

• Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
• Have you read the contributing guidelines?
• Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.73%. Comparing base (acf34e4) to head (5e19616).
Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##               main     #371   +/-   ##
=========================================
  Coverage     88.73%   88.73%           
  Complexity        1        1           
=========================================
  Files            20       20           
  Lines           914      914           
=========================================
  Hits            811      811           
  Misses          103      103           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[norkunas]

But isn't this a BC break for others?

Edit: hmm, strange, in my app I am using meilisearch also

[pan93412]

But isn't this a BC break for others?

Edit: hmm, strange, in my app I am using meilisearch also

https://github.com/search?q=path%3Aconfig%2Fpackages%2Fmeilisearch.php&type=code

Besides, there was basically nobody like me who successfully applied ConfigBuilder for meilisearch. Since YAML functions as it did before, I believe there are no breaking changes, as the old method is never functional.

[norkunas]

bors merge

[meili-bors]

Build succeeded:

• Code style
• integration-tests (PHP 7.4) (Symfony 5.4.*)
• integration-tests (PHP 8.1) (Symfony 6.4.*)
• integration-tests (PHP 8.2) (Symfony 6.4.*)
• integration-tests (PHP 8.2) (Symfony 7.0.*)
• integration-tests (PHP 8.2) (Symfony 7.1.*)
• integration-tests (PHP 8.3) (Symfony 6.4.*)
• integration-tests (PHP 8.3) (Symfony 7.0.*)
• integration-tests (PHP 8.3) (Symfony 7.1.*)
• integration-tests (PHP 8.4) (Symfony 6.4.*)
• integration-tests (PHP 8.4) (Symfony 7.0.*)
• integration-tests (PHP 8.4) (Symfony 7.1.*)
• integration-tests (PHP 8.4) (Symfony 7.2.*)

[norkunas]

Thank you @pan93412