11 | | I don't even blame Pillow. I really think `get_image_dimensions` is bogus because it should never return `None`. Instead it should return a tuple with empty dimensions or error dimensions `(0, 0) or (1, 1)` because the calling function simply chokes when receiving a `None` (instead of dealing with `None` it *always* assumes it's a tuple, and *this* is a bug, either in the calling function or the called one) |
| 11 | I don't even blame Pillow. I really think `get_image_dimensions` is bogus because it should never return `None`. Instead it should return a tuple with empty dimensions or error dimensions `(0, 0) or (-1, -1)` because the calling function simply chokes when receiving a `None` (instead of dealing with `None` it *always* assumes it's a tuple, and *this* is a bug, either in the calling function or the called one) |
| 12 | |
| 13 | Forgot something obvious: the bug occurs only if the `ImageField` has a `width_field` and `height_field`. |