- 🇴🇲Oman omlx
As a workaround I installed Search API module & enabled Database Search as backend, then in processors page under index page I enable Ignore characters and add
\p{M}
to the list.
\p{M}
- Unicode property classes matches any diacritic mark.I hope this helps till this bug fixed
- 🇫🇷France andypost
It will affect coming help topics module which is using search index (core one) to search help pages and its translations
- 🇬🇧United Kingdom catch
This needs an issue summary update - there are at least two proposed resolutions.
- 🇨🇦Canada Charlie ChX Negyesi 🍁Canada
I have been working on this, researching etc and I believe the best we can do is to ignore the current implementation, snapshot what intl does into a simple mapping: https://gist.github.com/chx/ce1d45398996bbadcaf0bd65a61f5902#file-remove... the gist also contains the generator script.
doesn't need intl in core at all. It's less than ten kilobytes and as fast as it gets. It handles the first 8192 (0x2000) characters https://jrgraphix.net/r/Unicode/ which includes most non-Hiragana/Katakana scripts. Does 827 replacements:
- last update
over 1 year ago Custom Commands Failed - @charlie-chx-negyesi opened merge request.
- last update
over 1 year ago Custom Commands Failed - 🇨🇦Canada Charlie ChX Negyesi 🍁Canada
I pushed this solution , reading the most important new class is much more pleasant in the [blob viewer](https://git.drupalcode.org/project/drupal/-/blob/9f5588d947f03e9eb079d46...) the diff viewer breaks the map line and it's a bit long then.
- last update
over 1 year ago Custom Commands Failed - Status changed to Needs review
over 1 year ago 5:40pm 22 July 2023 - Status changed to Needs work
over 1 year ago 12:53am 23 July 2023 - 🇺🇸United States smustgrave
For the CC failure.
Could the change record be expanded to include the new service remove_diacritics and how it might be used? Also the addition of a new module that's deprecated?
- last update
over 1 year ago Custom Commands Failed - last update
over 1 year ago Custom Commands Failed - last update
over 1 year ago 29,834 pass, 26 fail - last update
over 1 year ago 29,844 pass, 17 fail - last update
over 1 year ago Custom Commands Failed - last update
over 1 year ago 29,883 pass - Status changed to Needs review
over 1 year ago 12:37pm 23 July 2023 - 🇨🇦Canada Charlie ChX Negyesi 🍁Canada
Updated the change notice too and we are green.
- last update
over 1 year ago 29,883 pass - 🇨🇦Canada Charlie ChX Negyesi 🍁Canada
The fun continues in https://www.drupal.org/project/remove_diacritics → it at least provides the necessity functionality.
I guess this should be closed again (since no one worked on it for three years but me and I won't again) but I do not have the permissions to do so.
- Status changed to Needs work
over 1 year ago 11:52am 8 August 2023 The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- First commit to issue fork.
31:35 28:34 Running30:56 27:50 Running- Status changed to Needs review
about 1 year ago 5:39am 11 September 2023 - Status changed to Needs work
about 1 year ago 1:28pm 21 September 2023 - 🇺🇸United States smustgrave
Will need test coverage for sure.
Also will also need framework manager sign off to add a new module believe.
After all that a change record will be needed.