The settings form is not loading the correct configuration value for it's default value.
Line 39 of src/Form/SettingForm.php should read:
'#default_value' => implode("\n", $config->get('elf_domains')),
It's missing the 'elf_' part of the configuration name.

CommentFileSizeAuthor
#2 2929456-SettingsForm-wrong-default.patch633 bytesdonaldp

Comments

donaldp created an issue. See original summary.

donaldp’s picture

donaldp commented
Issue summary: View changes
StatusFileSize
new 2929456-SettingsForm-wrong-default.patch633 bytes
ddrozdik’s picture

ddrozdik commented
Status: Active » Reviewed & tested by the community

Reviewed, looks good. Ready to be pushed.

  • ddrozdik committed be0a364 on 8.x-1.x authored by donaldp 
    Issue #2929456 by donaldp, ddrozdik: Settings Form loading wrong default...
ddrozdik’s picture

ddrozdik commented
Status: Reviewed & tested by the community » Fixed

@donaldp thank you. Your patch has been pushed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.