Merge branch 'main' into svg-compressed

Author: bartTC

.github/workflows/push.yml

@@ -8,7 +8,7 @@ jobs:
     strategy:
       max-parallel: 4
       matrix:
-        python-version: ["3.9", "3.10", "3.11", "3.12"]
+        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
 
     steps:
       - uses: actions/checkout@v4

CHANGES.rst

@@ -2,7 +2,42 @@
 Change log
 ==========
 
-8.0 ( 27 September 2024)
+WIP
+===
+
+- Added ``GappedCircleModuleDrawer`` (PIL) to render QR code modules as non-contiguous circles. (BenwestGate in `#373`_)
+- Removed the hardcoded 'id' argument from SVG elements. The fixed element ID caused conflicts when embedding multiple QR codes in a single document. (m000 in `#385`_)
+- Improved test coveraged (akx in `#315`_)
+- Fixed typos in code that used ``embeded`` instead of ``embedded``. For backwards compatibility, the misspelled parameter names are still accepted but now emit deprecation warnings. These deprecated parameter names will be removed in v9.0. (benjnicholls in `#349`_)
+- Migrate pyproject.toml to PEP 621-compliant [project] metadata format. (hroncok in `#399`_)
+- Allow execution as a Python module. (stefansjs in `#400`_)
+
+  ::
+
+    python -m qrcode --output qrcode.png "hello world"
+
+.. _#315: https://github.com/lincolnloop/python-qrcode/pull/315
+.. _#349: https://github.com/lincolnloop/python-qrcode/pull/349
+.. _#373: https://github.com/lincolnloop/python-qrcode/pull/373
+.. _#385: https://github.com/lincolnloop/python-qrcode/pull/385
+.. _#399: https://github.com/lincolnloop/python-qrcode/pull/399
+.. _#400: https://github.com/lincolnloop/python-qrcode/pull/400
+
+8.2 (01 May 2025)
+=================
+
+- Optimize QRColorMask apply_mask method for enhanced performance
+- Fix typos on StyledPilImage embeded_* parameters.
+  The old parameters with the typos are still accepted
+  for backward compatibility.
+
+
+8.1 (02 April 2025)
+====================
+
+- Added support for Python 3.13.
+
+8.0 (27 September 2024)
 ========================
 
 - Added support for Python 3.11 and 3.12.
@@ -15,10 +50,12 @@ Change log
 
 - Code quality and formatting utilises ruff_.
 
-- Removed ``typing_extensions`` as a dependency, as it's no longer required with
+- Removed ``typing_extensions`` as a dependency, as it's no longer required
+  with having Python 3.9+ as a requirement.
   having Python 3.9+ as a requirement.
 
-- Only allow high error correction rate (`qrcode.ERROR_CORRECT_H`) when generating
+- Only allow high error correction rate (`qrcode.ERROR_CORRECT_H`)
+  when generating
   QR codes with embedded images to ensure content is readable
 
 .. _Poetry: https://python-poetry.org

README.rst

@@ -211,7 +211,7 @@ circles by reducing this less than the default of ``Decimal(1)``.
 
 The ``StyledPilImage`` additionally accepts an optional ``color_mask``
 parameter to change the colors of the QR Code, and an optional
-``embeded_image_path`` to embed an image in the center of the code.
+``embedded_image_path`` to embed an image in the center of the code.
 
 Other color masks:
 
@@ -232,7 +232,7 @@ and an embedded image:
 
     img_1 = qr.make_image(image_factory=StyledPilImage, module_drawer=RoundedModuleDrawer())
     img_2 = qr.make_image(image_factory=StyledPilImage, color_mask=RadialGradiantColorMask())
-    img_3 = qr.make_image(image_factory=StyledPilImage, embeded_image_path="/path/to/image.png")
+    img_3 = qr.make_image(image_factory=StyledPilImage, embedded_image_path="/path/to/image.png")
 
 Examples
 ========

pyproject.toml

@@ -1,16 +1,16 @@
 [build-system]
-requires = ["poetry-core"]
+requires = ["poetry-core>=2"]
 build-backend = "poetry.core.masonry.api"
 
-[tool.poetry]
+[project]
 name = "qrcode"
-version = "8.0"
-packages = [{include = "qrcode"}]
+version = "8.2"
 description = "QR Code image generator"
-authors = ["Lincoln Loop <[email protected]>"]
-license = "BSD"
-readme = ["README.rst", "CHANGES.rst"]
-homepage = "https://github.com/lincolnloop/python-qrcode"
+authors = [
+    { name = "Lincoln Loop", email = "[email protected]" },
+]
+license = { text = "BSD-3-Clause" }
+dynamic = [ "readme" ]
 keywords = ["qr", "denso-wave", "IEC18004"]
 classifiers = [
     "Development Status :: 5 - Production/Stable",
@@ -23,10 +23,36 @@ classifiers = [
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
     "Programming Language :: Python :: 3 :: Only",
     "Topic :: Multimedia :: Graphics",
     "Topic :: Software Development :: Libraries :: Python Modules",
 ]
+requires-python = "~=3.9"
+dependencies = [
+    "colorama; sys_platform == 'win32'",
+    "deprecation",
+]
+
+
+[project.optional-dependencies]
+pil = ["pillow >=9.1.0"]
+png = ["pypng"]
+all = ["pypng", "pillow >=9.1.0"]
+
+[project.urls]
+homepage = "https://github.com/lincolnloop/python-qrcode"
+repository = "https://github.com/lincolnloop/python-qrcode.git"
+documentation = "https://github.com/lincolnloop/python-qrcode#readme"
+changelog = "https://github.com/lincolnloop/python-qrcode/blob/main/CHANGES.rst"
+"Bug Tracker" = "https://github.com/lincolnloop/python-qrcode/issues"
+
+[project.scripts]
+qr = "qrcode.console_scripts:main"
+
+[tool.poetry]
+packages = [{include = "qrcode"}]
+readme = ["README.rst", "CHANGES.rst"]
 
 # There is no support for data files yet.
 # https://github.com/python-poetry/poetry/issues/9519
@@ -35,21 +61,6 @@ classifiers = [
 #     { destination = "share/man/man1", from = [ "doc/qr.1" ] },
 # ]
 
-[tool.poetry.scripts]
-qr = 'qrcode.console_scripts:main'
-
-
-[tool.poetry.dependencies]
-python = "^3.9"
-colorama = {version = "*", platform = "win32"}
-pypng = {version = "*", optional = true}
-pillow = {version = ">=9.1.0", optional = true}
-
-[tool.poetry.extras]
-pil = ["pillow"]
-png = ["pypng"]
-all = ["pypng","pillow"]
-
 [tool.poetry.group.dev.dependencies]
 pytest = {version = "*"}
 pytest-cov = {version = "*"}
@@ -70,3 +81,17 @@ date-format =" %%-d %%B %%Y"
 prereleaser.middle = [
     "qrcode.release.update_manpage"
 ]
+
+[tool.coverage.run]
+source = ["qrcode"]
+parallel = true
+
+[tool.coverage.report]
+exclude_lines = [
+    "@abc.abstractmethod",
+    "@overload",
+    "if (typing\\.)?TYPE_CHECKING:",
+    "pragma: no cover",
+    "raise NotImplementedError"
+]
+skip_covered = true

qrcode/__main__.py

@@ -0,0 +1,3 @@
+from .console_scripts import main
+
+main()

qrcode/console_scripts.py

@@ -9,7 +9,8 @@
 import optparse
 import os
 import sys
-from typing import Dict, Iterable, NoReturn, Optional, Set, Type
+from typing import NoReturn, Optional
+from collections.abc import Iterable
 from importlib import metadata
 
 import qrcode
@@ -145,7 +146,7 @@ def raise_error(msg: str) -> NoReturn:
         img.save(sys.stdout.buffer)
 
 
-def get_factory(module: str) -> Type[BaseImage]:
+def get_factory(module: str) -> type[BaseImage]:
     if "." not in module:
         raise ValueError("The image factory is not a full python path")
     module, name = module.rsplit(".", 1)
@@ -154,7 +155,7 @@ def get_factory(module: str) -> Type[BaseImage]:
 
 
 def get_drawer_help() -> str:
-    help: Dict[str, Set] = {}
+    help: dict[str, set] = {}
     for alias, module in default_factories.items():
         try:
             image = get_factory(module)

qrcode/image/base.py

@@ -1,22 +1,22 @@
 import abc
-from typing import TYPE_CHECKING, Any, Dict, Optional, Tuple, Type, Union
+from typing import TYPE_CHECKING, Any, Optional, Union
 
 from qrcode.image.styles.moduledrawers.base import QRModuleDrawer
 
 if TYPE_CHECKING:
     from qrcode.main import ActiveWithNeighbors, QRCode
 
 
-DrawerAliases = Dict[str, Tuple[Type[QRModuleDrawer], Dict[str, Any]]]
+DrawerAliases = dict[str, tuple[type[QRModuleDrawer], dict[str, Any]]]
 
 
-class BaseImage:
+class BaseImage(abc.ABC):
     """
     Base QRCode image output class.
     """
 
     kind: Optional[str] = None
-    allowed_kinds: Optional[Tuple[str]] = None
+    allowed_kinds: Optional[tuple[str]] = None
     needs_context = False
     needs_processing = False
     needs_drawrect = True
@@ -108,7 +108,7 @@ def is_eye(self, row: int, col: int):
 
 
 class BaseImageWithDrawer(BaseImage):
-    default_drawer_class: Type[QRModuleDrawer]
+    default_drawer_class: type[QRModuleDrawer]
     drawer_aliases: DrawerAliases = {}
 
     def get_default_module_drawer(self) -> QRModuleDrawer:

qrcode/image/styledpil.py

@@ -1,5 +1,12 @@
-import qrcode.image.base
+from __future__ import annotations
+
+import warnings
+from typing import overload
+
+import deprecation
 from PIL import Image
+
+import qrcode.image.base
 from qrcode.image.styles.colormasks import QRColorMask, SolidFillColorMask
 from qrcode.image.styles.moduledrawers import SquareModuleDrawer
 
@@ -29,7 +36,7 @@ class StyledPilImage(qrcode.image.base.BaseImageWithDrawer):
     data integrity A resampling filter can be specified (defaulting to
     PIL.Image.Resampling.LANCZOS) for resizing; see PIL.Image.resize() for possible
     options for this parameter.
-    The image size can be controlled by `embeded_image_ratio` which is a ratio
+    The image size can be controlled by `embedded_image_ratio` which is a ratio
     between 0 and 1 that's set in relation to the overall width of the QR code.
     """
 
@@ -41,14 +48,32 @@ class StyledPilImage(qrcode.image.base.BaseImageWithDrawer):
 
     def __init__(self, *args, **kwargs):
         self.color_mask = kwargs.get("color_mask", SolidFillColorMask())
-        embeded_image_path = kwargs.get("embeded_image_path", None)
-        self.embeded_image = kwargs.get("embeded_image", None)
-        self.embeded_image_ratio = kwargs.get("embeded_image_ratio", 0.25)
-        self.embeded_image_resample = kwargs.get(
-            "embeded_image_resample", Image.Resampling.LANCZOS
+
+        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", None)
+        )
+        self.embedded_image = kwargs.get(
+            "embedded_image", kwargs.get("embeded_image", None)
+        )
+        self.embedded_image_ratio = kwargs.get(
+            "embedded_image_ratio", kwargs.get("embeded_image_ratio", 0.25)
+        )
+        self.embedded_image_resample = kwargs.get(
+            "embedded_image_resample",
+            kwargs.get("embeded_image_resample", Image.Resampling.LANCZOS),
         )
-        if not self.embeded_image and embeded_image_path:
-            self.embeded_image = Image.open(embeded_image_path)
+        if not self.embedded_image and embedded_image_path:
+            self.embedded_image = Image.open(embedded_image_path)
 
         # the paint_color is the color the module drawer will use to draw upon
         # a canvas During the color mask process, pixels that are paint_color
@@ -59,12 +84,18 @@ def __init__(self, *args, **kwargs):
 
         super().__init__(*args, **kwargs)
 
+    @overload
+    def drawrect(self, row, col):
+        """
+        Not used.
+        """
+
     def new_image(self, **kwargs):
         mode = (
             "RGBA"
             if (
                 self.color_mask.has_transparency
-                or (self.embeded_image and "A" in self.embeded_image.getbands())
+                or (self.embedded_image and "A" in self.embedded_image.getbands())
             )
             else "RGB"
         )
@@ -79,23 +110,32 @@ def init_new_image(self):
 
     def process(self):
         self.color_mask.apply_mask(self._img)
-        if self.embeded_image:
-            self.draw_embeded_image()
-
+        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):
-        if not self.embeded_image:
+        return self.draw_embedded_image()
+
+    def draw_embedded_image(self):
+        if not self.embedded_image:
             return
         total_width, _ = self._img.size
         total_width = int(total_width)
-        logo_width_ish = int(total_width * self.embeded_image_ratio)
+        logo_width_ish = int(total_width * self.embedded_image_ratio)
         logo_offset = (
             int((int(total_width / 2) - int(logo_width_ish / 2)) / self.box_size)
             * self.box_size
         )  # round the offset to the nearest module
         logo_position = (logo_offset, logo_offset)
         logo_width = total_width - logo_offset * 2
-        region = self.embeded_image
-        region = region.resize((logo_width, logo_width), self.embeded_image_resample)
+        region = self.embedded_image
+        region = region.resize((logo_width, logo_width), self.embedded_image_resample)
         if "A" in region.getbands():
             self._img.alpha_composite(region, logo_position)
         else:
@@ -104,8 +144,7 @@ def draw_embeded_image(self):
     def save(self, stream, format=None, **kwargs):
         if format is None:
             format = kwargs.get("kind", self.kind)
-        if "kind" in kwargs:
-            del kwargs["kind"]
+        kwargs.pop("kind", None)
         self._img.save(stream, format=format, **kwargs)
 
     def __getattr__(self, name):