Remove deprecated code and warnings for embeded_* parameters and PIL drawer imports.

Author: bartTC

CHANGES.rst

@@ -2,22 +2,29 @@
 Changes
 =======
 
-Deprecation Warnings
-====================
+Change Log
+==========
 
-Removed in v9.0:
-----------------
+WIP (9.0)
+---------
 
-- Importing a PIL drawer from ``qrcode.image.styles.moduledrawers`` has been deprecated.
-  Update your code to import directly from the ``pil`` module instead:
+**Backwards incompatible changes:**
+
+- The ``width`` and ``height`` attributes was removed from the ``<svg>`` tag.
+  Instead, the ``viewBox`` attribute is now used for defining the dimensions.
+  Additionally, all SVG elements now utilize pixel units rather than millimeters,
+  which may cause rendering differences in browsers.
+
+- Importing a PIL drawer from ``qrcode.image.styles.moduledrawers`` is no longer
+  supported. Update your code to import directly from the ``pil`` module instead:
 
   .. code-block:: python
 
      from qrcode.image.styles.moduledrawers import SquareModuleDrawer  # Old
      from qrcode.image.styles.moduledrawers.pil import SquareModuleDrawer  # New
 
 - Calling ``QRCode.make_image`` or ``StyledPilImage`` with the arguments ``embeded_image``
-  or ``embeded_image_path`` have been deprecated due to typographical errors. Update
+  or ``embeded_image_path`` has been removed to typographical errors. Update
   your code to use the correct arguments ``embedded_image`` and ``embededd_image_path``:
 
   .. code-block:: python
@@ -29,20 +36,6 @@ Removed in v9.0:
      StyledPilImage(embeded_image=..., embeded_image_path=...)  # Old
      StyledPilImage(embedded_image=..., embedded_image_path=...)  # New
 
-- The ``width`` and ``height`` attributes will be removed from the ``<svg>`` tag.
-  Instead, the ``viewBox`` attribute is now used for defining the dimensions.
-  Additionally, all SVG elements now utilize pixel units rather than millimeters,
-  which may cause rendering differences in browsers.
-
-Change Log
-==========
-
-WIP (9.0)
----------
-
-- **Removed** ``width=.. height=...`` attributes from SVG tag, using viewBox instead. SVG elements now use pixel units instead of millimeters.
-
-
 WIP 8.x
 -------
 

pyproject.toml

@@ -31,7 +31,6 @@ classifiers = [
 requires-python = "~=3.9"
 dependencies = [
   "colorama; sys_platform == 'win32'",
-  "deprecation",
 ]
 
 

qrcode/image/styledpil.py

@@ -1,9 +1,7 @@
 from __future__ import annotations
 
-import warnings
 from typing import overload
 
-import deprecation
 from PIL import Image
 
 import qrcode.image.base
@@ -49,26 +47,12 @@ class StyledPilImage(qrcode.image.base.BaseImageWithDrawer):
     def __init__(self, *args, **kwargs):
         self.color_mask = kwargs.get("color_mask", SolidFillColorMask())
 
-        if kwargs.get("embeded_image_path") or kwargs.get("embeded_image"):
-            warnings.warn(
-                "The 'embeded_*' parameters are deprecated. Use 'embedded_image_path' "
-                "or 'embedded_image' instead. The 'embeded_*' parameters will be "
-                "removed in v9.0.",
-                category=DeprecationWarning,
-                stacklevel=2,
-            )
-
         # allow embeded_ parameters with typos for backwards compatibility
-        embedded_image_path = kwargs.get(
-            "embedded_image_path", kwargs.get("embeded_image_path")
-        )
-        self.embedded_image = kwargs.get("embedded_image", kwargs.get("embeded_image"))
-        self.embedded_image_ratio = kwargs.get(
-            "embedded_image_ratio", kwargs.get("embeded_image_ratio", 0.25)
-        )
+        embedded_image_path = kwargs.get("embedded_image_path")
+        self.embedded_image = kwargs.get("embedded_image")
+        self.embedded_image_ratio = kwargs.get("embedded_image_ratio", 0.25)
         self.embedded_image_resample = kwargs.get(
-            "embedded_image_resample",
-            kwargs.get("embeded_image_resample", Image.Resampling.LANCZOS),
+            "embedded_image_resample", Image.Resampling.LANCZOS
         )
         if not self.embedded_image and embedded_image_path:
             self.embedded_image = Image.open(embedded_image_path)
@@ -111,15 +95,6 @@ def process(self):
         if self.embedded_image:
             self.draw_embedded_image()
 
-    @deprecation.deprecated(
-        deprecated_in="9.0",
-        removed_in="8.3",
-        current_version="8.2",
-        details="Use draw_embedded_image() instead",
-    )
-    def draw_embeded_image(self):
-        return self.draw_embedded_image()
-
     def draw_embedded_image(self):
         if not self.embedded_image:
             return

qrcode/image/styles/moduledrawers/__init__.py

@@ -1,46 +0,0 @@
-"""
-Module for lazy importing of PIL drawers with a deprecation warning.
-
-Currently, importing a PIL drawer from this module is allowed for backwards
-compatibility but will raise a DeprecationWarning.
-
-This will be removed in v9.0.
-"""
-
-import warnings
-
-from qrcode.constants import PIL_AVAILABLE
-
-
-def __getattr__(name):
-    """Lazy import with deprecation warning for PIL drawers."""
-    # List of PIL drawer names that should trigger deprecation warnings
-    pil_drawers = {
-        "CircleModuleDrawer",
-        "GappedCircleModuleDrawer",
-        "GappedSquareModuleDrawer",
-        "HorizontalBarsDrawer",
-        "RoundedModuleDrawer",
-        "SquareModuleDrawer",
-        "VerticalBarsDrawer",
-    }
-
-    if name in pil_drawers:
-        # Only render a warning if PIL is actually installed. Otherwise it would
-        # raise an ImportError directly, which is fine.
-        if PIL_AVAILABLE:
-            warnings.warn(
-                f"Importing '{name}' directly from this module is deprecated."
-                f"Please use 'from qrcode.image.styles.moduledrawers.pil import {name}' "
-                f"instead. This backwards compatibility import will be removed in v9.0.",
-                DeprecationWarning,
-                stacklevel=2,
-            )
-
-        # Import and return the drawer from the pil module
-        from . import pil  # noqa: PLC0415
-
-        return getattr(pil, name)
-
-    # For any other attribute, raise AttributeError
-    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")

qrcode/main.py

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
 import sys
-import warnings
 from bisect import bisect_left
 from typing import Generic, Literal, NamedTuple, Optional, TypeVar, cast, overload
 
@@ -335,22 +334,8 @@ def make_image(self, image_factory=None, **kwargs):
 
         If the data has not been compiled yet, make it first.
         """
-        # Raise a warning that 'embeded' is still used
-        if kwargs.get("embeded_image_path") or kwargs.get("embeded_image"):
-            warnings.warn(
-                "The 'embeded_*' parameters are deprecated. Use 'embedded_image_path' "
-                "or 'embedded_image' instead. The 'embeded_*' parameters will be "
-                "removed in v9.0.",
-                category=DeprecationWarning,
-                stacklevel=2,
-            )
-
-        # allow embeded_ parameters with typos for backwards compatibility
         if (
-            kwargs.get("embedded_image_path")
-            or kwargs.get("embedded_image")
-            or kwargs.get("embeded_image_path")
-            or kwargs.get("embeded_image")
+            kwargs.get("embedded_image_path") or kwargs.get("embedded_image")
         ) and self.error_correction != constants.ERROR_CORRECT_H:
             raise ValueError(
                 "Error correction level must be ERROR_CORRECT_H if an embedded image is provided"