There's a fairly consistent pattern of using == '' or != '' throughout this PR.  Unless we're expecting non-scalar values that would have unintended effects (e.g. false or 0 returning empty) - I prefer the use of empty().