Merge branch 'main' into 228_filter_migration_iteration

Author: codingjoe

.github/workflows/ci.yml

@@ -46,12 +46,13 @@ jobs:
     strategy:
       matrix:
         python-version:
-          - "3.9"
           - "3.10"
+          - "3.11"
           - "3.12"
           - "3.13"
+          - "3.14"
         django-version:
-          - "4.2" # LTS
+          - "5.2" # LTS
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v6
@@ -71,11 +72,11 @@ jobs:
     strategy:
       matrix:
         python-version:
-          - "3.12"
+          - "3.10"
         django-version:
           # LTS gets tested on all OS
+          - "4.2"
           - "5.1"
-          - "5.2"
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v6

README.md

@@ -33,7 +33,11 @@ from pictures.models import PictureField
 
 class Profile(models.Model):
     title = models.CharField(max_length=255)
-    picture = PictureField(upload_to="avatars")
+    picture = PictureField(
+        upload_to="avatars", width_field="picture_width", height_field="picture_height"
+    )
+    picture_width = models.PositiveIntegerField(editable=False)
+    picture_height = models.PositiveIntegerField(editable=False)
 ```
 
 ```html
@@ -157,7 +161,11 @@ class Profile(models.Model):
     picture = PictureField(
         upload_to="avatars",
         aspect_ratios=[None, "1/1", "3/2", "16/9"],
+        width_field="picture_width",
+        height_field="picture_height",
     )
+    picture_width = models.PositiveIntegerField(editable=False)
+    picture_height = models.PositiveIntegerField(editable=False)
 ```
 
 ```html
@@ -235,10 +243,15 @@ class Profile(models.Model):
         validators=[
             MinSizeValidator(400, 300),  # At least 400x300 pixels
             MaxSizeValidator(4096, 4096),  # At most 4096x4096 pixels
-        ]
+        ],
+        width_field="picture_width",
+        height_field="picture_height",
     )
+    picture_width = models.PositiveIntegerField(editable=False)
+    picture_height = models.PositiveIntegerField(editable=False)
+
 
-Use `None` to limit only one dimension: `MaxSizeValidator(2048, None)` limits only width.
+# Use `None` to limit only one dimension: `MaxSizeValidator(2048, None)` limits only width.
 ```
 
 > [!IMPORTANT]

pictures/models.py

@@ -6,6 +6,7 @@
 import math
 from fractions import Fraction
 from pathlib import Path
+from types import NotImplementedType
 
 from django.core import checks
 from django.core.files.base import ContentFile
@@ -105,7 +106,7 @@ def name(self) -> str:
     def path(self) -> Path:
         return Path(self.storage.path(self.name))
 
-    def process(self, image) -> Image:
+    def process(self, image) -> Image.Image:
         image = ImageOps.exif_transpose(image)  # crates a copy
         height = self.height or self.width / Fraction(*image.size)
         size = math.floor(self.width), math.floor(height)
@@ -130,7 +131,7 @@ def delete(self):
 
 
 class PictureFieldFile(ImageFieldFile):
-    def __xor__(self, other) -> tuple[set[Picture], set[Picture]]:
+    def __xor__(self, other) -> tuple[set[Picture], set[Picture]] | NotImplementedType:
         """Return the new and obsolete :class:`Picture` instances."""
         if not isinstance(other, PictureFieldFile):
             return NotImplemented
@@ -192,7 +193,7 @@ def height(self):
         return self._get_image_dimensions()[1]
 
     @property
-    def aspect_ratios(self) -> {Fraction | None: {str: {int: Picture}}}:
+    def aspect_ratios(self) -> dict[Fraction | None, dict[str, dict[int, Picture]]]:
         self._require_file()
         return self.get_picture_files(
             file_name=self.name,
@@ -210,7 +211,7 @@ def get_picture_files(
         img_height: int,
         storage: Storage,
         field: PictureField,
-    ) -> {Fraction | None: {str: {int: Picture}}}:
+    ) -> dict[Fraction | None, dict[str, dict[int, Picture]]]:
         PictureClass = import_string(conf.get_settings().PICTURE_CLASS)
         return {
             ratio: {
@@ -291,7 +292,7 @@ def _check_aspect_ratios(self):
         return errors
 
     def _check_width_height_field(self):
-        if None in self.aspect_ratios and not (self.width_field and self.height_field):
+        if not (self.width_field and self.height_field):
             return [
                 checks.Warning(
                     "width_field and height_field attributes are missing",

pictures/utils.py

@@ -61,7 +61,7 @@ def sizes(
 
 
 def source_set(
-    size: (int, int), *, ratio: str | Fraction | None, max_width: int, cols: int
+    size: tuple[int, int], *, ratio: str | Fraction | None, max_width: int, cols: int
 ) -> set:
     ratio = Fraction(ratio) if ratio else None
     img_width, img_height = size
@@ -82,8 +82,8 @@ def placeholder(width: int, height: int, alt):
     hue = random.randint(0, 360)  # NoQA S311
     img = Image.new("RGB", (width, height), color=f"hsl({hue}, 40%, 80%)")
     draw = ImageDraw.Draw(img)
-    draw.line(((0, 0, width, height)), width=3, fill=f"hsl({hue}, 60%, 20%)")
-    draw.line(((0, height, width, 0)), width=3, fill=f"hsl({hue}, 60%, 20%)")
+    draw.line((0, 0, width, height), width=3, fill=f"hsl({hue}, 60%, 20%)")
+    draw.line((0, height, width, 0), width=3, fill=f"hsl({hue}, 60%, 20%)")
     draw.rectangle(
         (width / 4, height / 4, width * 3 / 4, height * 3 / 4),
         fill=f"hsl({hue}, 40%, 80%)",

pyproject.toml

@@ -23,17 +23,17 @@ classifiers = [
   "Topic :: Software Development",
   "Programming Language :: Python :: 3",
   "Programming Language :: Python :: 3 :: Only",
-  "Programming Language :: Python :: 3.9",
   "Programming Language :: Python :: 3.10",
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
   "Programming Language :: Python :: 3.13",
+  "Programming Language :: Python :: 3.14",
   "Framework :: Django",
   "Framework :: Django :: 4.2",
   "Framework :: Django :: 5.1",
   "Framework :: Django :: 5.2",
 ]
-requires-python = ">=3.9"
+requires-python = ">=3.10"
 dependencies = ["django>=4.2.0", "pillow>=11.3.0"]
 
 [project.optional-dependencies]
@@ -44,7 +44,7 @@ test = [
   "redis",
 ]
 dramatiq = [
-  "dramatiq<2",
+  "dramatiq",
   "django-dramatiq",
 ]
 celery = [

tests/test_models.py

@@ -825,9 +825,19 @@ def test_check_aspect_ratios(self):
         assert errors
         assert errors[0].id == "fields.E100"
 
-    def test_check_width_height_field(self):
-        assert not PictureField(aspect_ratios=["3/2"])._check_width_height_field()
-        with override_field_aspect_ratios(Profile.picture.field, [None]):
+    @pytest.mark.parametrize(
+        "aspect_ratios",
+        [
+            ("3/2",),
+            (None,),
+            (
+                "3/2",
+                None,
+            ),
+        ],
+    )
+    def test_check_width_height_field(self, aspect_ratios):
+        with override_field_aspect_ratios(Profile.picture.field, aspect_ratios):
             errors = Profile.picture.field._check_width_height_field()
         assert errors
         assert errors[0].id == "fields.E101"

tests/test_utils.py

@@ -145,14 +145,14 @@ def test_placeholder():
     assert img.height == 1200
 
 
-class TestPicture(Picture):
+class SamplePicture(Picture):
     @property
     def url(self):
         return f"/media/{self.parent_name}"
 
 
 def test_reconstruct(image_upload_file):
-    picture = TestPicture(
+    picture = SamplePicture(
         image_upload_file.name,
         "WEBP",
         "16/9",
@@ -164,7 +164,7 @@ def test_reconstruct(image_upload_file):
     assert isinstance(reconstructed, Storage)
 
     assert utils.reconstruct(
-        "tests.test_utils.TestPicture",
+        "tests.test_utils.SamplePicture",
         [],
         {
             "parent_name": "test.jpg",
@@ -173,7 +173,7 @@ def test_reconstruct(image_upload_file):
             "storage": default_storage,
             "width": 100,
         },
-    ) == TestPicture(
+    ) == SamplePicture(
         "test.jpg",
         "JPEG",
         "16/9",

tests/testapp/settings.py

@@ -151,7 +151,6 @@
 DRAMATIQ_BROKER = {
     "BROKER": os.getenv("DRAMATIQ_BROKER", "dramatiq.brokers.redis.RedisBroker"),
     "MIDDLEWARE": [
-        "dramatiq.middleware.Prometheus",
         "dramatiq.middleware.AgeLimit",
         "dramatiq.middleware.TimeLimit",
         "dramatiq.middleware.Callbacks",