django-display-ids 0.1.2__py3-none-any.whl → 0.1.3__py3-none-any.whl

django_display_ids/contrib/rest_framework/views.py

@@ -5,6 +5,7 @@ from __future__ import annotations
 from typing import TYPE_CHECKING, Any
 
 from django_display_ids.conf import get_setting
+from django_display_ids.encoding import PREFIX_PATTERN
 from django_display_ids.exceptions import (
     InvalidIdentifierError,
     LookupError,
@@ -21,6 +22,8 @@ __all__ = [
     "DisplayIDLookupMixin",
 ]
 
+_NOT_SET: Any = object()
+
 
 def _get_drf_exceptions() -> tuple[type[Exception], type[Exception]]:
     """Lazily import DRF exceptions to avoid hard dependency."""
@@ -68,7 +71,7 @@ class DisplayIDLookupMixin:
 
     lookup_url_kwarg: str = "pk"
     lookup_strategies: tuple[StrategyName, ...] | None = None
-    display_id_prefix: str | None = None
+    display_id_prefix: str | None = _NOT_SET
     uuid_field: str | None = None
     slug_field: str | None = None
 
@@ -91,6 +94,24 @@ class DisplayIDLookupMixin:
             return self.lookup_strategies
         return get_setting("STRATEGIES")  # type: ignore[return-value]
 
+    def _get_display_id_prefix(self, model: type[models.Model]) -> str | None:
+        """Get the display ID prefix.
+
+        Returns the viewset's display_id_prefix if set (including None to
+        explicitly disable), otherwise falls back to the model's
+        display_id_prefix attribute.
+        """
+        if self.display_id_prefix is not _NOT_SET:
+            if self.display_id_prefix is not None and not PREFIX_PATTERN.match(
+                self.display_id_prefix
+            ):
+                raise ValueError(
+                    f"display_id_prefix must be 1-16 lowercase letters, "
+                    f"got: {self.display_id_prefix!r}"
+                )
+            return self.display_id_prefix
+        return getattr(model, "display_id_prefix", None)
+
     def get_queryset(self) -> Any:
         """Get the base queryset.
 
@@ -138,7 +159,7 @@ class DisplayIDLookupMixin:
                 model=model,
                 value=str(value),
                 strategies=self._get_strategies(),
-                prefix=self.display_id_prefix,
+                prefix=self._get_display_id_prefix(model),
                 uuid_field=self._get_uuid_field(),
                 slug_field=self._get_slug_field(),
                 queryset=queryset,

django_display_ids/models.py

@@ -7,7 +7,7 @@ from typing import ClassVar
 from django.db import models
 
 from .conf import get_setting
-from .encoding import encode_display_id
+from .encoding import PREFIX_PATTERN, encode_display_id
 
 __all__ = [
     "DisplayIDMixin",
@@ -92,6 +92,11 @@ class DisplayIDMixin(models.Model):
         if "display_id_prefix" in cls.__dict__:
             prefix = cls.__dict__["display_id_prefix"]
             if prefix is not None:
+                if not PREFIX_PATTERN.match(prefix):
+                    raise ValueError(
+                        f"{cls.__name__}.display_id_prefix must be 1-16 "
+                        f"lowercase letters, got: {prefix!r}"
+                    )
                 _register_prefix(prefix, cls.__name__)
 
     @classmethod

django_display_ids/views.py

@@ -7,6 +7,7 @@ from typing import TYPE_CHECKING, Any
 from django.http import Http404
 
 from .conf import get_setting
+from .encoding import PREFIX_PATTERN
 from .exceptions import (
     InvalidIdentifierError,
     LookupError,
@@ -23,6 +24,8 @@ __all__ = [
     "DisplayIDObjectMixin",
 ]
 
+_NOT_SET: Any = object()
+
 
 class DisplayIDObjectMixin:
     """Mixin for Django CBVs that resolves objects by display ID, UUID, or slug.
@@ -49,7 +52,7 @@ class DisplayIDObjectMixin:
     model: type[models.Model] | None = None
     lookup_param: str = "pk"
     lookup_strategies: tuple[StrategyName, ...] | None = None
-    display_id_prefix: str | None = None
+    display_id_prefix: str | None = _NOT_SET
     uuid_field: str | None = None
     slug_field: str | None = None
 
@@ -68,6 +71,26 @@ class DisplayIDObjectMixin:
             return self.lookup_strategies
         return get_setting("STRATEGIES")  # type: ignore[return-value]
 
+    def _get_display_id_prefix(self) -> str | None:
+        """Get the display ID prefix.
+
+        Returns the view's display_id_prefix if set (including None to
+        explicitly disable), otherwise falls back to the model's
+        display_id_prefix attribute.
+        """
+        if self.display_id_prefix is not _NOT_SET:
+            if self.display_id_prefix is not None and not PREFIX_PATTERN.match(
+                self.display_id_prefix
+            ):
+                raise ValueError(
+                    f"display_id_prefix must be 1-16 lowercase letters, "
+                    f"got: {self.display_id_prefix!r}"
+                )
+            return self.display_id_prefix
+        if self.model is not None:
+            return getattr(self.model, "display_id_prefix", None)
+        return None
+
     # These may be provided by parent classes
     kwargs: dict[str, Any]
 
@@ -114,7 +137,7 @@ class DisplayIDObjectMixin:
                 model=self.model,
                 value=str(value),
                 strategies=self._get_strategies(),
-                prefix=self.display_id_prefix,
+                prefix=self._get_display_id_prefix(),
                 uuid_field=self._get_uuid_field(),
                 slug_field=self._get_slug_field(),
                 queryset=qs,

{django_display_ids-0.1.2.dist-info → django_display_ids-0.1.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-display-ids
-Version: 0.1.2
+Version: 0.1.3
 Summary: Stripe-like prefixed IDs for Django. Works with existing UUIDs — no schema changes.
 Keywords: django,stripe,uuid,base62,prefixed-id,drf,shortuuid,nanoid,ulid
 License: ISC
@@ -17,7 +17,7 @@ Classifier: Programming Language :: Python :: 3.14
 Classifier: Typing :: Typed
 Requires-Dist: django>=4.2
 Requires-Python: >=3.12
-Project-URL: Source, https://joseph.is/django-display-ids
+Project-URL: Homepage, https://joseph.is/django-display-ids
 Description-Content-Type: text/markdown
 
 # django-display-ids
@@ -34,6 +34,8 @@ This library focuses on **lookup only** — it works with your existing UUID fie
 pip install django-display-ids
 ```
 
+No `INSTALLED_APPS` entry required — just import and use.
+
 ## Quick Start
 
 ```python
@@ -249,11 +251,13 @@ The `display_id` strategy requires a prefix. If no prefix is configured, the str
 |-----------|---------|-------------|
 | `lookup_param` / `lookup_url_kwarg` | `"pk"` | URL parameter name |
 | `lookup_strategies` | from settings | Strategies to try |
-| `display_id_prefix` | `None` | Expected prefix |
+| `display_id_prefix` | from model | Expected prefix (falls back to model's `display_id_prefix`) |
 | `uuid_field` | `"id"` | UUID field name on model |
 | `slug_field` | `"slug"` | Slug field name on model |
 
-### Django Settings
+### Django Settings (Optional)
+
+All settings have sensible defaults. Only add this if you need to override them:
 
 ```python
 # settings.py
@@ -298,10 +302,16 @@ Run tests:
 uv run pytest
 ```
 
+Run tests with coverage:
+
+```bash
+uv run pytest --cov=src/django_display_ids
+```
+
 Run tests across Python and Django versions:
 
 ```bash
-uvx nox -p
+uvx nox
 ```
 
 Lint and format:

{django_display_ids-0.1.2.dist-info → django_display_ids-0.1.3.dist-info}/RECORD

@@ -3,16 +3,16 @@ django_display_ids/admin.py,sha256=uRyPH3q5e9D5oMxs6PtCHq0syBXs--i1alRoe-EcIJo,2
 django_display_ids/conf.py,sha256=qTsCzKeNBdJpEVeEkx2kFeWHBFa_NZwV_tpt-UTyRR0,1132
 django_display_ids/contrib/__init__.py,sha256=sxGJK8Whb6cL7RqACqGRYrIZvaMwP3l6dYk3mIYWzDY,62
 django_display_ids/contrib/rest_framework/__init__.py,sha256=hBk-6m01T66J5bar3GKwApktOnHhDtK-gUpjghlKoJo,148
-django_display_ids/contrib/rest_framework/views.py,sha256=SuUDs4FIyEf8MI2-gTPly8RgL_OshvvRLHGUKXWFexc,5187
+django_display_ids/contrib/rest_framework/views.py,sha256=mu-twvdRbJedzSfLWVNn2BazFpAQzwRB5Eq3w2bxvvs,6056
 django_display_ids/encoding.py,sha256=csIwUZaQKSOLwRU6-DWGTNGvSxmroyK0Yt7TBCo0AFE,2945
 django_display_ids/exceptions.py,sha256=nmyRfpsqVvz226Zcu_QANwr8MudbfoX09mAgOCwuPuQ,3022
 django_display_ids/managers.py,sha256=EFvlQxsSFXeM8TVvV4NZKeKMC7QB3C0zYGgZ6bvSr4k,6884
-django_display_ids/models.py,sha256=bTuD6kMkuoMDH8P9wTgIvTktEnw1VAEFq-NuiTkP_K0,3891
+django_display_ids/models.py,sha256=_IXxaFlVw2MqobQUw4Cy4rB66LsTz6kiI5YpoSpnkCY,4156
 django_display_ids/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 django_display_ids/resolver.py,sha256=oCoA6jbGCFS8SMrkfD_oSSBQNrSxnxdooK5j933eA9Y,2494
 django_display_ids/strategies.py,sha256=Rq00-AW_FB8-K04u2oBK5J6kPiYgsE3TdYlLyK_zro0,4436
 django_display_ids/typing.py,sha256=2O3kT7XKkiE7WI9A5KkILPM-Zi7-zCy5gVvXQL_J2mI,478
-django_display_ids/views.py,sha256=mqkWPW8wIuYMeUd76n3ZA9oS4Fep7azmjHaOnu9s48U,4216
-django_display_ids-0.1.2.dist-info/WHEEL,sha256=XV0cjMrO7zXhVAIyyc8aFf1VjZ33Fen4IiJk5zFlC3g,80
-django_display_ids-0.1.2.dist-info/METADATA,sha256=foJtkEHg4lJ_hTDLOenjgKMfn4kHbHPITXQOGNic_RM,9366
-django_display_ids-0.1.2.dist-info/RECORD,,
+django_display_ids/views.py,sha256=sLsJm8Tpe3Qk1gOLcDzfpazxuaVqTCAdgVIXOONFnKQ,5096
+django_display_ids-0.1.3.dist-info/WHEEL,sha256=XV0cjMrO7zXhVAIyyc8aFf1VjZ33Fen4IiJk5zFlC3g,80
+django_display_ids-0.1.3.dist-info/METADATA,sha256=AjMU5ctNag1QVDINfx4gggW0VWfYkkIEE1brlokAUqo,9649
+django_display_ids-0.1.3.dist-info/RECORD,,