From e918e2497bd508a5d18ba4715f2d62316a4e4ddb Mon Sep 17 00:00:00 2001 From: Chris Hill-Scott Date: Tue, 1 Feb 2022 11:36:30 +0000 Subject: [PATCH] Rename mixin to clarify purpose `SortableMixin` no longer has anything to do with sorting, except that it inherits from the similarly-named `SortByNameMixin`. This commit: - renames it to describe its main purpose: doing equality (and identity) checks based on the `id` attribute - makes it no longer inherit from `SortByNameMixin`, so that the two can be used independently or combined --- app/broadcast_areas/models.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/app/broadcast_areas/models.py b/app/broadcast_areas/models.py index 48f30d8f7..d3663df48 100644 --- a/app/broadcast_areas/models.py +++ b/app/broadcast_areas/models.py @@ -14,7 +14,7 @@ from .populations import CITY_OF_LONDON from .repo import BroadcastAreasRepository, rtree_index -class SortableMixin(SortByNameMixin): +class IdEqualityMixin: def __repr__(self): return f'{self.__class__.__name__}(<{self.id}>)' @@ -75,7 +75,7 @@ class BaseBroadcastArea(ABC): return max(500, min(estimated_bleed, 5000)) -class BroadcastArea(BaseBroadcastArea, SortableMixin): +class BroadcastArea(BaseBroadcastArea, IdEqualityMixin, SortByNameMixin): def __init__(self, row): self.id, self.name, self._count_of_phones, self.library_id = row @@ -220,7 +220,7 @@ class CustomBroadcastAreas(SerialisedModelCollection): ) -class BroadcastAreaLibrary(SerialisedModelCollection, SortableMixin, GetItemByIdMixin): +class BroadcastAreaLibrary(SerialisedModelCollection, SortByNameMixin, IdEqualityMixin, GetItemByIdMixin): model = BroadcastArea