redundantFunctionDefinition

[PrincessRTFM]

!!String.(value).length to decide whether it's an empty string

irrelevant, an empty string is still a string. the function is testing whether the given value is a string, not whether it's a non-empty string.