Your way is longer, and we know any instance of the "Instance" model has a "logo" attribute so calling "hasattr" seems redundant. I'm inclined to leave it as-is.