This change introduces a bug, that results in non-validating Template field to be existing file unless I change Type field.

In D259#5085, @erik wrote:

When editing inline block in adm.console, button "Remove Link" is not available, and this makes problematic to input non-linked text after some link is added to empty editor.

In D259#5085, @erik wrote:

When editing inline block in adm.console, button "Remove Link" is not available, and this makes problematic to input non-linked text after some link is added to empty editor.

In D229#4319, @alex wrote:

And test plan still doesn't cover direct method usage of DateHelper class.

Also test plan doesn't have regression testing part, where all changed code is invoked and confirmed, that it works as before (places where you now do Status = ThemeStatus::ACTIVE instead of Enabled = 1.

All length checking code was changed and therefore test plan must include testing of all existing behavior (regression testing), not just the part that was fixed. I guess you can create field declaration with all possible validation options and just confirm, that:

• when array is passed in no validation error happens
• when invalid scalar value (maybe several values needs to be used to match each of validation options) is passed the validation error happens
• when valid scalar value (maybe several values needs to be used to match each of validation options) is passed no validation error happens