diff options
| author | François Freitag <mail@franek.fr> | 2023-06-17 18:00:43 +0200 |
|---|---|---|
| committer | Mariusz Felisiak <felisiak.mariusz@gmail.com> | 2023-06-19 05:24:42 +0200 |
| commit | bcacc6321ae819965f478bfee2072a796801f298 (patch) | |
| tree | a032506dd7acc495623293c0cd0e7cdfecfa71eb /tests/postgres_tests/test_array.py | |
| parent | 0c5146523b614783b60ba9f9222769dff83c776c (diff) | |
Refs #34517 -- Restored skipping ImageFileField.update_dimension_fields without width/height fields.
This avoids reading the image size when the dimensions fields
(image_width, image_height) do not exist, as that operation may be
expensive.
Partially reverts ea53e7c09f1b8864c20c65976bbeaeab77abdaec, that dropped
the check for the dimension fields in update_dimension_fields(), because
the post_init signal was no longer registered without dimension fields.
However, another code path to that function exists: when the
ImageFileField is save()d, the name from the storage is setattr()ed on
the field, and ImageFileDescriptor calls update_dimension_fields()
because the image size might have changed. Keep bailing out early when
dimensions are unused.
Besides, computing the image dimensions causes to close() the file,
resulting in a backward-incompatible change. The test protects against
that change.
Diffstat (limited to 'tests/postgres_tests/test_array.py')
0 files changed, 0 insertions, 0 deletions
