Shouldn't this check against an empty string?