We don't usually disallow `null` but allow `""`... the zero check is correct, but is the `null` check?