Wrong PREFERRED LANGUAGE (code) on simplenews-subscriber-form

else { $form['language']['langcode'] = [ '#type' => 'select', /*WT+: form simplenews-subscriber-form get actual lang-code*/ //'#default_value' => $subscriber->language()->getId(), '#default_value' => $subscriber->langcode->value, /*<<<*/ '#options' => $language_options, '#required' => TRUE, ]; } } return $form; }

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Comment almost 2 years ago →
🇩🇪Germany marco.b
patch in #4 works great for me. Thanks.
Comment almost 2 years ago →
🇩🇪Germany marco.b
Comment almost 2 years ago →
🇬🇧United Kingdom adamps
Status changed to Needs work almost 2 years ago3:21pm 21 March 2023
Comment almost 2 years ago →
🇬🇧United Kingdom adamps
Thanks for the patch and review.

I think it would be better to use $subscriber->getLangcode() which should have the same result.
Issue was unassigned.
Status changed to Needs review over 1 year ago9:07am 2 April 2023
Comment over 1 year ago →
🇬🇧United Kingdom adamps
Comment over 1 year ago →
🇬🇧United Kingdom adamps
I don't use multilanguage on my sites so I can't test it. If someone can confirm the patch works then I will commit it. New release coming up soon, so this can be included if someone can test in the next few days.
Status changed to Closed: outdated over 1 year ago12:59pm 29 July 2023
Comment over 1 year ago →
🇬🇧United Kingdom adamps
The code that this issue relates to was deleted in 📌 Use "manage form display" for subscriber base fields Fixed . Instead the langcode field is provided using default code in Drupal Core. Hopefully that means the bug has gone away.
Comment 7 months ago →
🇺🇦Ukraine vlad.dancer Kyiv
I confirm that patch from #10 is needed for 3.x version and in 4.x there is no such problem.

Wrong PREFERRED LANGUAGE (code) on simplenews-subscriber-form

Problem/Motivation

Proposed resolution

Comments & Activities