Skip to content

Remove LocaleContextHolder; pass Locale everywhere - #13996

Draft
mkurz wants to merge 5 commits into
playframework:mainfrom
mkurz:locale_context_holder_removal_refactoring
Draft

mkurz wants to merge 5 commits into
playframework:mainfrom
mkurz:locale_context_holder_removal_refactoring

Conversation

@mkurz

@mkurz mkurz commented Jun 12, 2026

Copy link
Copy Markdown
Member

⛔ DO NOT MERGE YET. ⛔

This PR currently is just a proof that after

got merged we can totally remove LocaleContextHolder and instead pass Locale where needed (because now we control the binding code since we removed the Spring dependencies).

.bind(new Lang(Locale.FRENCH), TypedMap.empty(), Map("note" -> "foo").asJava)
myFormFr.errors("note").get(0).message() must beEqualTo("Größe muss zwischen 10 und 100 sein")
}
}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test only removed temporary. Needs to be roll backed before merge this PR.

@mkurz
mkurz force-pushed the locale_context_holder_removal_refactoring branch from 7fe3738 to aa94acb Compare June 12, 2026 14:08
@mkurz
mkurz force-pushed the locale_context_holder_removal_refactoring branch 2 times, most recently from dc76b46 to e08aed5 Compare June 17, 2026 12:27
@mkurz
mkurz force-pushed the locale_context_holder_removal_refactoring branch from e08aed5 to 7175d11 Compare September 23, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant