Skip to content

[stable32] fix(Util): getScripts also need to reorder core translations - #59641

Merged
susnux merged 2 commits into
stable32from
backport/59518/stable32
Apr 15, 2026
Merged

[stable32] fix(Util): getScripts also need to reorder core translations#59641
susnux merged 2 commits into
stable32from
backport/59518/stable32

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #59518

Currently `core-common` and `core-main` are prepanded to the scripts,
as they provide the global state. But the script ordering logic does not
know about this and might sort core translations after they are used.
Meaning we need to register core translations as soon as the global
scope is initialized (so that `OC.L10N.register` is available).
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@backportbot
backportbotBot requested a review from CarlSchwanApril 15, 2026 06:53
@backportbot
backportbotBot requested a review from a team as a code ownerApril 15, 2026 06:53
@backportbot
backportbotBot requested review from ArtificialOwl, icewind1991, provokateurin and susnux and removed request for a teamApril 15, 2026 06:53
@backportbotbackportbotBot added this to the Nextcloud 32.0.9 milestone Apr 15, 2026
Comment threadlib/public/Util.php Outdated
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnux merged commit 4deb89a into stable32Apr 15, 2026
235 of 246 checks passed
@susnux
susnux deleted the backport/59518/stable32 branch April 15, 2026 16:11
@nextcloud-botnextcloud-bot mentioned this pull request Apr 20, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugjavascript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@susnux@provokateurin