Problem/Motivation
There's an error on the configuration page when the "Additional host" setting has no hosts listed.
Also, the name is misleading since you'll need to include the host name of the site if you want the Tidy process to update it. It's more like "All host names to treat as being the same as this site"

Steps to reproduce
The default configuration doesn't include any hosts in the list, so this happens on install :(
If you've already added at least one host name Load a configuration with an empty aka_host setting.
Proposed resolution
The configuration page should not result in an error when aka_host is empty. There are two parts to update
- Include a 'web' host name in the default install (base_uri is 'https://web/', so it's a good example/hint)
- Fix the error on the setting page
User interface changes
Updated the text on the setting page and in the schema.yml.
Test
If you install Link Magician with the updated branch there will be an example in the default configuration. Remove the host 'web', save the configuration and reload.
| Comment | File | Size | Author |
|---|---|---|---|
| buildAkaHosts-undefined-array-key-0.png | 27.51 KB | jasongose |
Issue fork link_magician-3587509
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
- 3587509-error-on-configuration
changes, plain diff MR !15
- 3587509-error-fix
changes, plain diff MR !16
Comments
Comment #2
jasongose commentedComment #3
jasongose commentedComment #5
jasongose commentedComment #6
jasongose commentedComment #13
terracoders commentedChanging status to Reviewed & Tested
Comment #15
jasongose commentedComment #17
jasongose commented