As part of making language support expand wider in Drupal core, we are moving language related code to their respective modules instead of locale/language module implementing it for them on their behalf. This mostly happened for path module in #1236680: Move path language settings from Locale to Path module and is further worked on for path and node modules in #1414314: Make node and path depend on language module only for language support, get rid of locale_language_name and #540294: Move node language settings from Locale to Node module.
The attached patch does the very simple code change to integrate language support in comment module. With this moving out of locale module and being only dependent on node settings, it will make comment support language without locale module being present, which is a major goal for Drupal 8.
The mentioned issues are a set of issues that are going to have their full effect when all land.
Comment | File | Size | Author |
---|---|---|---|
#3 | untangle-comment-from-locale-3.patch | 2.45 KB | Gábor Hojtsy |
untangle-comment-from-locale.patch | 2.45 KB | Gábor Hojtsy | |
Comments
Comment #2
tstoecklerI know this is how it was before, but can we wrap the first check in parens, please? I hate having to look up/remember operator precedence (== vs. &&).
Missing the semi-colon here, as you've probably noticed.
-24 days to next Drupal core point release.
Comment #3
Gábor HojtsyRight! Fixes attached. Thanks for the review.
Comment #4
tstoecklerWow, your patching speed is insane! :)
Anyway, this is RTBC (if it comes back green).
Comment #6
tstoeckler#3: untangle-comment-from-locale-3.patch queued for re-testing.
Comment #7
Tor Arne Thune CreditAttribution: Tor Arne Thune commentedOne of the test failures is caused by testHtmlEntitiesSample() that was just committed: #61456: Aggregator titles display quotes and other characters with HTML entity equivalents badly (write tests).
Comment #8
Gábor Hojtsy#3: untangle-comment-from-locale-3.patch queued for re-testing.
Comment #9
Gábor HojtsyTagging as current focus.
Comment #10
Tor Arne Thune CreditAttribution: Tor Arne Thune commentedI also manually tested this patch and the language inheritance of comments perform the same with or without the patch applied.
Comment #11
Dries CreditAttribution: Dries commentedI agree that locale.module should not have comment module specific code. Modules should be written with multi-lingual in mind.
Committed this patch to 8.x.
Comment #12
Gábor HojtsyLanded.
Comment #13
Gábor HojtsyTagging for the content handling leg of D8MI.