Order translations alphabetically

Sounds like a valid use case.

Can we add a kernel test maybe to make sure they come out alphabetically.

Caught the same problem, thanx for saving time.)

I created a new branch against 11.x which first sorts the translations by their source string and then the context. The context is needed for equal source string with different context.

Before review, the test should be extended to also test the case for equal source strings but different context.

prudloff → changed the visibility of the branch 3350704-order-translations-alphabetically to hidden.

Removing the tests tag as https://git.drupalcode.org/issue/drupal-3350704/-/jobs/4589844 shows the coverage

See no issue with ordering these.

I think this might need a new database index - see note on the MR.

We need to add an index then to avoid the `using filesort` - thanks @tgauges

What kind of index needs to be added to "avoid the using filesort"?

I read a bit of MySQL ORDER BY Optimization but my experiments did not result in the avoidance of using filesort when running EXPLAIN SELECT s.* FROM locales_source s ORDER BY s.source ASC, s.context ASC;.

I tried the following on 10.11.11-MariaDB-ubu2204-log:

• Change the existing index (source(30), context) to (source(2048), context).
• Add a new index (source(2048)).
• Add a new index (context).

The largest context value in my database is 1452 queried with SELECT length(source) AS length FROM locales_source ORDER BY length DESC;.

Hello @tgauges first appreciate taking on this task!

Since this seems to be still in discussion moving to NW, discussions can still happen without being in review

@smustgrave sorry, my bad.

Since a usable index might not be possible I adjusted the merge request to only add the ORDER BY statements when multiple strings are loaded.

This too might not be acceptable. In that case, I propose a function API update so that callers can decide via parameter whether they want the translations sorted or not.

Another option might be to make the \Drupal\locale\StringDatabaseStorage::dbStringSelect function public so that the query can be adjusted and executed at will.