Fix issue with no Require_lan set #1856 (#1892)

[clinton-hall]

Thanks @BradKollmyer

Description

A split of an empty string will return a list of [''] and return this for the assignment. So the default '' value will not set this as None but rather a single empty string.

Fixes #1856

Type of change

Please delete options that are not relevant.

• [X] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

User verified.

Test Configuration:

Checklist:

• [X] I have performed a self-review of my own code
• [X] New and existing unit tests pass locally with my changes

[clinton-hall]

@BradKollmyer I'm ready to merge this if you can confirm that this resolves the issue.

[BradKollmyer]

Changes look good.

[clinton-hall]

Thanks again for flagging this so quick... I was just looking at the settings today and one option is to set PRs to require approval before they get merged... the problem is I'm all on my own here and usually trying to do a quick tidy up during my lunch break!