Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
modules/locale/locale.install:734: update_variables_to_config('language.negotiation'
Thanks in advance for helping many hands to make light work!
See #2181257: [meta] Variables to config migration [d7] for instructions
Comment | File | Size | Author |
---|---|---|---|
#24 | interdiff-22-24.txt | 979 bytes | quietone |
#24 | 2130307-24.patch | 3.03 KB | quietone |
#22 | diff-21-22.txt | 779 bytes | quietone |
#22 | 2130307-22.patch | 3.09 KB | quietone |
#21 | 2130307-21.patch | 3.18 KB | quietone |
Comments
Comment #1
fastangel CreditAttribution: fastangel commentedworking on this.
Comment #2
fastangel CreditAttribution: fastangel commentedI think that we need to postponed this to D7. The variables in D8 are:
You can enable multiple system detection and order with a weight (See image attached). But In D6 we have only a variable (language_negotiation see https://api.drupal.org/api/drupal/includes%21locale.inc/function/locale_...). Then I think that we can't map :(.
What do you think?
Comment #3
fastangel CreditAttribution: fastangel commentedI talked with chx in irc and we move to postponed.
Comment #4
eliza411 CreditAttribution: eliza411 commentedMoving to the core queue to consolidate issues now that we're doing all the work there.
Comment #5
eliza411 CreditAttribution: eliza411 commentedMoving to the core queue to consolidate issues now that we're doing all the work there.
Comment #6
eliza411 CreditAttribution: eliza411 commentedComment #7
eliza411 CreditAttribution: eliza411 commentedComment #8
penyaskitoI guess this is still blocked on #2108599: Convert language_default to CMI and #2102477: Convert remainder of language negotiation settings to configuration system.
Comment #9
alanburke CreditAttribution: alanburke commentedBlocking issues now unblocked
Comment #10
michaellenahan CreditAttribution: michaellenahan commentedI wasn't able to test this locally, I got a wierd recursion error. I think that's probably just my setup though.
Comment #11
David Hernández CreditAttribution: David Hernández commentedJust applied the patch and run the test locally. Here's the output:
Comment #12
michaellenahan CreditAttribution: michaellenahan commentedThe last patch was not good. Here's another try.
I have a strange problem.
In the test runner I get a failure at line 50.
It turns out that url.source is set to path_prefix.
This is the literal value of url.source in /core/modules/language/config/language.negotiation.yml
I don't understand why this value isn't being set to boolean FALSE as a result of the test.
Comment #13
michaellenahan CreditAttribution: michaellenahan commentedI've removed the debugging line from the end.
Comment #14
michaellenahan CreditAttribution: michaellenahan commentedComment #15
David Hernández CreditAttribution: David Hernández commentedThere were a couple issues. One on the dump, where the second variable wasn't being created. The other one is the name of the language module (no longer locale) on the test. I've also fixed the test to use values more similar to the real environment.
Comment #16
benjy CreditAttribution: benjy commentedCommitted to 8.x. I fixed some indenting and moved the migration into the install folder on the way in.
Comment #18
jcost CreditAttribution: jcost commentedWill need to be submitted again to Core since moving from sandbox.
Comment #19
phenaproximaNeeds to be updated and merged into the parent issue.
Comment #20
phenaproximaUpdated.
Comment #21
quietone CreditAttribution: quietone commentedMoved to language module
Comment #22
quietone CreditAttribution: quietone commentedReroll. Since interdiff fails and the changes are minimal, I've included a diff
Comment #23
phenaproximaExcept for these two things, looks great.
This needs to be removed; config dependencies are not allowed in migration templates.
Nit: Need an extra line between the use statement and the doc comment.
Comment #24
quietone CreditAttribution: quietone commentedBoth fixed.
Comment #25
phenaproximaNoice. Pre-emptive RTBC assuming testbot passes.
Comment #26
webchickCommitted and pushed to 8.0.x. Thanks!
Comment #29
penyaskitoI'm not sure if this is enough, I think we need language_types config too. I will create another issue for that, now looking from d6 perspective but d7 should follow.
Comment #30
quietone CreditAttribution: quietone commented@penyaskito, language_types is at Variable to config: language.types [d7].
Comment #31
penyaskitoThis is not migrating language prefixes. Created #2625148: [PP-1] Migrate language prefixes and domains to language.negotiation [d7] for that.