Should there be an `else` to this condition, eg if `isinstance(value, basestring)`?