missing `is not None`. Not strictly necessary, but it avoids ambiguity between an empty string and None in this case.