- Issue created by @tr
- 🇺🇸United States tr Cascadia
Currently the test complains about:
Array to string conversion /builds/project/ip2country/tests/src/Kernel/OptionsProvider/OptionsProviderTest.php:59
Let me revert my first attempt at a fix (8c155dee469c7be6b1) to figure out what exactly the initial problem was.
- Merge request !30Revert "Cast output of getProvider() because core now returns TranslatableMarkup." → (Merged) created by tr
- 🇺🇸United States tr Cascadia
(Just for the record)
The error is (truncated, because there are over 200 countries):--- Expected +++ Actual @@ @@ Array ( - 'AF' => 'Afghanistan' - 'AL' => 'Albania' - 'DZ' => 'Algeria' + 'AF' => Drupal\Core\StringTranslation\TranslatableMarkup Object (...) + 'AL' => Drupal\Core\StringTranslation\TranslatableMarkup Object (...) + 'DZ' => Drupal\Core\StringTranslation\TranslatableMarkup Object (...)
So in short the core country.manager service is now returning markup objects instead of the country names.
- 🇺🇸United States tr Cascadia
That fixed the markup issues, and exposed a core change made in 📌 Update the countries list to latest Unicode CLDR version 44 Fixed which changed the spelling of one country name (Turkey =>Türkiye) and added a new country (Sark).
- Status changed to Fixed
5 months ago 12:34am 10 September 2024 - 🇺🇸United States tr Cascadia
That fixed the OptionsProvider tests in D10 and D11. Merged.
Automatically closed - issue fixed for 2 weeks with no activity.