Updated: Comment #0
Problem/Motivation
hook_language_types_info() allows to define additional language types to be used in the system. It also allow to specify predefined language negotiation settings for non-configurable language types through the fixed key. The name of they key is completely meaningless to almost everyone looking at the API docs for the first time. Additionally all the code managing language type configurability is extremely convoluted and is hard to follow even for the people that originally designed/wrote it.
Proposed resolution
- Rename the
fixedkey to something more self-documenting, such assettingsorconfig. - Clean-up the code that handles language type configurability.
Remaining tasks
- Agree on a solution
- Write a patch
- Fix failing tests
- Reviews
User interface changes
none
API changes
TBD
Comments
Comment #1
plachPostponed on #1862202: Objectify the language system.
Comment #2
plachComment #3
plachThe parent issue has been committed.
Comment #4
tim.plunkettComment #5
plachIt would really be better to have this sorted out before beta, at least the API change part.
Comment #6
plach... and we'll need a BC layer now.
Comment #7
plachComment #8
yesct commentedComment #24
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Thanks!
Comment #25
oily commented@smustgrave The 3 months has now elapsed. I am going to set this to 'Needs work', the work being to remove the @todo at lines 31- 32 from
core/modules/language/language.api.phpthen the issue should be closed?Comment #26
oily commentedComment #27
oily commentedComment #28
smustgrave commentedSorry no that does not appear to be the scope of this issue based on the summary.
Not sure where we landed that just removing the todo was the solution..
Removing the novice tag
Comment #29
oily commented@smustgrave Thanks, agreed.