Is there an advantage to this way of checking the type? I would have used ``isinstance(field, image_fields)`` to ensure that it will work for subclasses of ``ImageField`` as well. Do you think that makes sense?