I guess it doesn't matter too much, since the end result of each error showing up one at a time will be the same, but it seems weird to set this error only to have it potentially shadowed by the TargetField check below. Maybe just change this `err =` to a `return`?