clinicedc 2.0.7__py3-none-any.whl → 2.0.8__py3-none-any.whl

edc_notification/notification/model_notification.py

@@ -17,10 +17,10 @@ class ModelNotification(Notification):
 
     model: str | None = None  # label_lower format
 
-    model_operations = [CREATE, UPDATE, DELETE]
+    model_operations: tuple[str] = (CREATE, UPDATE, DELETE)
 
-    create_fields = ["created"]
-    update_fields = ["modified"]
+    create_fields: tuple[str] = ("created",)
+    update_fields: tuple[str] = ("modified",)
 
     email_body_template: str = (
         "\n\nDo not reply to this email\n\n"
@@ -177,30 +177,3 @@ class ModelNotification(Notification):
         )
 
         return opts
-
-    # @property
-    # def test_template_options(self) -> dict:
-    #     class Site:
-    #         domain = "gaborone.example.com"
-    #         name = "gaborone"
-    #         id = 99
-    #
-    #     class Meta:
-    #         label_lower = self.model
-    #
-    #     class DummyHistory:
-    #         def __init__(self, objects):
-    #             self._objects = objects
-    #
-    #         def all(self):
-    #             return self._objects
-    #
-    #     class DummyInstance:
-    #         id = 99
-    #         subject_identifier = "123456910"
-    #         site = Site()
-    #         _meta = Meta()
-    #
-    #     instance = DummyInstance()
-    #     instance.history = DummyHistory(objects=[instance])
-    #     return dict(instance=instance)

edc_notification/notification/new_model_notification.py

@@ -3,4 +3,4 @@ from .model_notification import ModelNotification
 
 
 class NewModelNotification(ModelNotification):
-    model_operations = [CREATE]
+    model_operations = (CREATE,)

edc_notification/notification/notification.py

@@ -214,7 +214,7 @@ class Notification:
 
     def send_email(
         self,
-        email_to: list[str],
+        email_to: list[str] | None = None,
         fail_silently: bool | None = None,
         email_body_template: str | None = None,
         **kwargs,

edc_notification/notification/updated_model_notification.py

@@ -3,9 +3,9 @@ from .model_notification import ModelNotification
 
 
 class UpdatedModelNotification(ModelNotification):
-    model_operations = [UPDATE]
+    model_operations = (UPDATE,)
 
-    update_fields = ["modified"]
+    update_fields = ("modified",)
 
     email_subject_template = (
         "*UPDATE* {test_subject_line}{protocol_name}: "

edc_offstudy/auths.py

@@ -8,10 +8,15 @@ from edc_auth.site_auths import site_auths
 
 from .auth_objects import OFFSTUDY, OFFSTUDY_SUPER, OFFSTUDY_VIEW, codenames
 
-site_auths.add_group(*codenames, name=OFFSTUDY, no_delete=True)
-site_auths.add_group(*codenames, name=OFFSTUDY_SUPER)
-site_auths.add_group(*codenames, name=OFFSTUDY_VIEW, view_only=True)
-site_auths.update_role(OFFSTUDY, name=CLINICIAN_ROLE)
-site_auths.update_role(OFFSTUDY, name=NURSE_ROLE)
-site_auths.update_role(OFFSTUDY_SUPER, name=CLINICIAN_SUPER_ROLE)
-site_auths.update_role(OFFSTUDY_VIEW, name=AUDITOR_ROLE)
+
+def update_site_auths():
+    site_auths.add_group(*codenames, name=OFFSTUDY, no_delete=True)
+    site_auths.add_group(*codenames, name=OFFSTUDY_SUPER)
+    site_auths.add_group(*codenames, name=OFFSTUDY_VIEW, view_only=True)
+    site_auths.update_role(OFFSTUDY, name=CLINICIAN_ROLE)
+    site_auths.update_role(OFFSTUDY, name=NURSE_ROLE)
+    site_auths.update_role(OFFSTUDY_SUPER, name=CLINICIAN_SUPER_ROLE)
+    site_auths.update_role(OFFSTUDY_VIEW, name=AUDITOR_ROLE)
+
+
+update_site_auths()

edc_pdutils/df_exporters/csv_model_exporter.py

@@ -30,6 +30,7 @@ class CsvModelExporter:
         **kwargs,
     ):
         self.model = model or queryset.model._meta.label_lower
+        self.export_folder = export_folder
         self.df_maker = self.df_maker_cls(
             model=model,
             queryset=queryset,
@@ -46,8 +47,10 @@ class CsvModelExporter:
 
     def to_csv(self):
         dataframe = self.df_maker.dataframe
-        return self.csv_exporter.to_csv(dataframe=dataframe)
+        return self.csv_exporter.to_csv(dataframe=dataframe, export_folder=self.export_folder)
 
     def to_stata(self):
         dataframe = self.df_maker.dataframe
-        return self.csv_exporter.to_stata(dataframe=dataframe)
+        return self.csv_exporter.to_stata(
+            dataframe=dataframe, export_folder=self.export_folder
+        )

edc_pharmacy/auths.py

@@ -18,21 +18,25 @@ from .auth_objects import (
     prescriber_codenames,
 )
 
-site_auths.add_post_update_func(
-    "edc_pharmacy", remove_default_model_permissions_from_edc_permissions
-)
-site_auths.add_custom_permissions_tuples(
-    model="edc_pharmacy.edcpermissions", codename_tuples=navbar_tuples
-)
 
-site_auths.add_group(*pharmacy_codenames, name=PHARMACY_VIEW, view_only=True)
-site_auths.add_group(*prescriber_codenames, name=PHARMACY_PRESCRIBER, no_delete=True)
-site_auths.add_group(*pharmacy_site_codenames, name=PHARMACY_SITE, no_delete=False)
-site_auths.add_group(*pharmacy_codenames, name=PHARMACY, no_delete=False)
+def update_site_auths() -> None:
+    site_auths.add_post_update_func(
+        "edc_pharmacy", remove_default_model_permissions_from_edc_permissions
+    )
+    site_auths.add_custom_permissions_tuples(
+        model="edc_pharmacy.edcpermissions", codename_tuples=navbar_tuples
+    )
+
+    site_auths.add_group(*pharmacy_codenames, name=PHARMACY_VIEW, view_only=True)
+    site_auths.add_group(*prescriber_codenames, name=PHARMACY_PRESCRIBER, no_delete=True)
+    site_auths.add_group(*pharmacy_site_codenames, name=PHARMACY_SITE, no_delete=False)
+    site_auths.add_group(*pharmacy_codenames, name=PHARMACY, no_delete=False)
+
+    site_auths.add_role(PHARMACY_VIEW, name=PHARMACY_AUDITOR_ROLE)
+    site_auths.add_role(PHARMACY_PRESCRIBER, name=PHARMACY_PRESCRIBER_ROLE)
+    site_auths.add_role(PHARMACY_SITE, PYLABELS, name=SITE_PHARMACIST_ROLE)
+    site_auths.add_role(PHARMACY, PYLABELS, name=PHARMACIST_ROLE)
+    site_auths.add_role(PHARMACY, name=PHARMACY_SUPER_ROLE)
 
 
-site_auths.add_role(PHARMACY_VIEW, name=PHARMACY_AUDITOR_ROLE)
-site_auths.add_role(PHARMACY_PRESCRIBER, name=PHARMACY_PRESCRIBER_ROLE)
-site_auths.add_role(PHARMACY_SITE, PYLABELS, name=SITE_PHARMACIST_ROLE)
-site_auths.add_role(PHARMACY, PYLABELS, name=PHARMACIST_ROLE)
-site_auths.add_role(PHARMACY, name=PHARMACY_SUPER_ROLE)
+update_site_auths()

edc_pharmacy/models/medication/formulation.py

@@ -61,18 +61,17 @@ class Formulation(BaseUuidModel):
     def save(self, *args, **kwargs):
         self.description = self.get_description()
         if not self.imp:
-            self.imp_description = None
+            self.imp_description = ""
         else:
             self.imp_description = (
                 self.imp_description if self.imp_description else self.description
             )
         super().save(*args, **kwargs)
 
-    def get_description(self):
+    def get_description(self) -> str:
         return (
             f"{self.medication} {round_half_away_from_zero(self.strength, 0)}"
             f"{self.get_units_display()} "
-            # f"{self.get_formulation_type_display()} "
             f"{self.get_route_display()}"
         )
 
@@ -80,7 +79,6 @@ class Formulation(BaseUuidModel):
         return (
             f"{self.medication} {round_half_away_from_zero(self.strength, 0)}"
             f"{self.get_units_display()} "
-            # f"{self.get_formulation_type_display()} "
         )
 
     def get_description_with_assignment(self, assignment: Assignment) -> str:
@@ -103,9 +101,9 @@ class Formulation(BaseUuidModel):
     class Meta(BaseUuidModel.Meta):
         verbose_name = "Formulation"
         verbose_name_plural = "Formulations"
-        constraints = [
+        constraints = (
             UniqueConstraint(
                 fields=["medication", "strength", "units", "formulation_type"],
                 name="%(app_label)s_%(class)s_med_stren_uniq",
-            )
-        ]
+            ),
+        )

edc_pharmacy/prescribe/create_prescription.py

@@ -30,11 +30,11 @@ def create_prescription(
     for medication_name in medication_names:
         try:
             obj = medication_model_cls.objects.get(name__iexact=medication_name)
-        except ObjectDoesNotExist:
+        except ObjectDoesNotExist as e:
             raise PrescriptionError(
                 "Unable to create prescription. Medication does not exist. "
                 f"Got {medication_name}"
-            )
+            ) from e
         else:
             medications.append(obj)
     try:
@@ -51,7 +51,7 @@ def create_prescription(
         try:
             rx = rx_model_cls.objects.create(**opts)
         except ObjectDoesNotExist as e:
-            raise CommandError(f"Site does not exists. site_id={site_id}. Got {e}")
+            raise CommandError(f"Site does not exists. site_id={site_id}. Got {e}") from e
         for obj in medications:
             rx.medications.add(obj)
     else:

edc_pharmacy/utils/confirm_stock.py

@@ -15,7 +15,7 @@ def confirm_stock(
     stock_codes: list[str],
     fk_attr: str | None = None,
     confirmed_by: str | None = None,
-    user_created: str = None,
+    user_created: str | None = None,
 ) -> tuple[list[str], list[str], list[str]]:
     """Confirm stock instances given a list of stock codes
     and a request/receive pk.

edc_pharmacy/utils/confirm_stock_at_site.py

@@ -26,7 +26,7 @@ def confirm_stock_at_site(
     stock_transfer: StockTransfer,
     stock_codes: list[str],
     location: UUID,
-    request: WSGIRequest = None,
+    request: WSGIRequest | None = None,
 ) -> tuple[list[str], list[str], list[str]]:
     """Confirm stock instances given a list of stock codes
     and a request/receive pk.

edc_pharmacy/views/confirmation_at_site_view.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import contextlib
 import uuid
 from uuid import uuid4
 
@@ -97,13 +98,12 @@ class ConfirmationAtSiteView(
         return min(unconfirmed_count, 12)
 
     def get_stock_codes(self, stock_transfer):
-        stock_codes = [
+        return [
             code
             for code in stock_transfer.stocktransferitem_set.values_list(
                 "stock__code", flat=True
             ).all()
         ]
-        return stock_codes
 
     def get_unconfirmed_count(self, stock_transfer) -> int:
         return (
@@ -120,10 +120,8 @@ class ConfirmationAtSiteView(
     def site(self) -> Site | None:
         obj = None
         if self.kwargs.get("site_id"):
-            try:
+            with contextlib.suppress(ObjectDoesNotExist):
                 obj = Site.objects.get(id=self.kwargs.get("site_id"))
-            except ObjectDoesNotExist:
-                pass
         return obj
 
     @property
@@ -169,14 +167,13 @@ class ConfirmationAtSiteView(
     def confirmation_at_site_changelist_url(self) -> str:
         if self.confirmation_at_site:
             url = reverse("edc_pharmacy_admin:edc_pharmacy_confirmationatsite_changelist")
-            url = f"{url}?q={self.confirmation_at_site.transfer_confirmation_identifier}"
-            return url
+            return f"{url}?q={self.confirmation_at_site.transfer_confirmation_identifier}"
         return "/"
 
     def get_stock_transfer(
         self,
         stock_transfer_identifier: str,
-        suppress_msg: bool = None,
+        suppress_msg: bool | None = None,
     ) -> StockTransfer | None:
         stock_transfer = None
         try:
@@ -198,7 +195,7 @@ class ConfirmationAtSiteView(
                 )
         return stock_transfer
 
-    def post(self, request, *args, **kwargs) -> HttpResponseRedirect:
+    def post(self, request, *args, **kwargs) -> HttpResponseRedirect:  # noqa: ARG002
         # cancel
         if request.POST.get("cancel") and request.POST.get("cancel") == "cancel":
             url = reverse("edc_pharmacy:home_url")

edc_prn/admin_site.py

@@ -0,0 +1,5 @@
+from edc_model_admin.admin_site import EdcAdminSite
+
+from .apps import AppConfig
+
+edc_prn_admin = EdcAdminSite(name="edc_prn_admin", app_label=AppConfig.name)

edc_prn/prn.py

@@ -58,19 +58,18 @@ class Prn:
         try:
             return django_apps.get_model(self.model)
         except LookupError as e:
-            raise PrnError(f"{e}. See {self!r}")
+            raise PrnError(f"{e}. See {self!r}") from e
 
-    def get_show_on_dashboard(self, subject_identifier=None, **kwargs):
+    def get_show_on_dashboard(self, subject_identifier=None, **kwargs):  # noqa: ARG002
         count = 0
-        if self.show_on_dashboard:
-            if subject_identifier:
-                opts = dict(subject_identifier=subject_identifier)
-                try:
-                    count = self.model_cls.objects.filter(**opts).count()
-                except FieldError:
-                    opts = self.get_query_opts(subject_identifier)
-                    count = self.model_cls.objects.filter(**opts).count()
-        return True if count and self.show_on_dashboard else False
+        if self.show_on_dashboard and subject_identifier:
+            opts = dict(subject_identifier=subject_identifier)
+            try:
+                count = self.model_cls.objects.filter(**opts).count()
+            except FieldError:
+                opts = self.get_query_opts(subject_identifier)
+                count = self.model_cls.objects.filter(**opts).count()
+        return count and self.show_on_dashboard
 
     def get_query_opts(self, subject_identifier):
         """Returns alternative query opts to search on

edc_prn/urls.py

@@ -0,0 +1,11 @@
+from django.urls import path
+from django.views.generic.base import RedirectView
+
+from .admin_site import edc_prn_admin
+
+app_name = "edc_prn"
+
+urlpatterns = [
+    path("admin/", edc_prn_admin.urls),
+    path("", RedirectView.as_view(url="admin/"), name="home_url"),
+]

edc_protocol_incident/action_items.py

@@ -9,8 +9,8 @@ from .constants import (
 
 
 class ProtocolDeviationViolationAction(ActionWithNotification):
-    reference_model = None  # "my_app.protocolincident"
-    admin_site_name = None  # "my_app_admin"
+    reference_model = "edc_protocol_incident.protocoldeviationviolation"
+    admin_site_name = "edc_protocol_incident_admin"
 
     name = PROTOCOL_DEVIATION_VIOLATION_ACTION
     display_name = "Submit Protocol Deviation / Violation Report"
@@ -25,8 +25,8 @@ class ProtocolDeviationViolationAction(ActionWithNotification):
 
 
 class ProtocolIncidentAction(ActionWithNotification):
-    reference_model = None  # "my_app.protocolincident"
-    admin_site_name = None  # "my_app"
+    reference_model = "edc_protocol_incident.protocolincident"
+    admin_site_name = "edc_protocol_incident_admin"
 
     name = PROTOCOL_INCIDENT_ACTION
     display_name = "Submit Protocol Incident Report"

edc_protocol_incident/auths.py

@@ -14,24 +14,31 @@ from .auth_objects import (
 )
 from .constants import PROTOCOL_DEVIATION_VIOLATION, PROTOCOL_INCIDENT
 
-incident_type = getattr(settings, "EDC_PROTOCOL_VIOLATION_TYPE", PROTOCOL_DEVIATION_VIOLATION)
-
-site_auths.add_group(*protocol_violation_codenames, name=PROTOCOL_VIOLATION)
-site_auths.add_group(*protocol_violation_view_codenames, name=PROTOCOL_VIOLATION_VIEW)
-site_auths.add_group(*protocol_incident_codenames, name=PROTOCOL_INCIDENT)
-site_auths.add_group(*protocol_incident_view_codenames, name=PROTOCOL_INCIDENT_VIEW)
-
-if incident_type == PROTOCOL_DEVIATION_VIOLATION:
-    site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_ROLE)
-    site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_SUPER_ROLE)
-    site_auths.update_role(PROTOCOL_VIOLATION_VIEW, name=AUDITOR_ROLE)
-elif incident_type == PROTOCOL_INCIDENT:
-    site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_ROLE)
-    site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_SUPER_ROLE)
-    site_auths.update_role(PROTOCOL_INCIDENT_VIEW, name=AUDITOR_ROLE)
-else:
-    raise ValueError(
-        "Invalid value for settings.EDC_PROTOCOL_VIOLATION_TYPE. "
-        f"Expected `{PROTOCOL_INCIDENT}` or `{PROTOCOL_DEVIATION_VIOLATION}`. "
-        f"Got {incident_type}."
+
+def update_site_auths() -> None:
+    incident_type = getattr(
+        settings, "EDC_PROTOCOL_VIOLATION_TYPE", PROTOCOL_DEVIATION_VIOLATION
     )
+
+    site_auths.add_group(*protocol_violation_codenames, name=PROTOCOL_VIOLATION)
+    site_auths.add_group(*protocol_violation_view_codenames, name=PROTOCOL_VIOLATION_VIEW)
+    site_auths.add_group(*protocol_incident_codenames, name=PROTOCOL_INCIDENT)
+    site_auths.add_group(*protocol_incident_view_codenames, name=PROTOCOL_INCIDENT_VIEW)
+
+    if incident_type == PROTOCOL_DEVIATION_VIOLATION:
+        site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_ROLE)
+        site_auths.update_role(PROTOCOL_VIOLATION, name=CLINICIAN_SUPER_ROLE)
+        site_auths.update_role(PROTOCOL_VIOLATION_VIEW, name=AUDITOR_ROLE)
+    elif incident_type == PROTOCOL_INCIDENT:
+        site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_ROLE)
+        site_auths.update_role(PROTOCOL_INCIDENT, name=CLINICIAN_SUPER_ROLE)
+        site_auths.update_role(PROTOCOL_INCIDENT_VIEW, name=AUDITOR_ROLE)
+    else:
+        raise ValueError(
+            "Invalid value for settings.EDC_PROTOCOL_VIOLATION_TYPE. "
+            f"Expected `{PROTOCOL_INCIDENT}` or `{PROTOCOL_DEVIATION_VIOLATION}`. "
+            f"Got {incident_type}."
+        )
+
+
+update_site_auths()

edc_pylabels/auths.py

@@ -2,4 +2,9 @@ from edc_auth.site_auths import site_auths
 
 from .auth_objects import PYLABELS, codenames
 
-site_auths.add_group(*codenames, name=PYLABELS, no_delete=False)
+
+def update_site_auths() -> None:
+    site_auths.add_group(*codenames, name=PYLABELS, no_delete=False)
+
+
+update_site_auths()

edc_qareports/auths.py

@@ -9,12 +9,16 @@ from .auth_objects import (
     qa_reports_codenames,
 )
 
-# groups
-site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS)
-site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS_AUDIT, view_only=True)
 
+def update_site_auths() -> None:
+    # groups
+    site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS)
+    site_auths.add_group(*qa_reports_codenames, name=QA_REPORTS_AUDIT, view_only=True)
 
-# roles
-site_auths.add_role(QA_REPORTS, name=QA_REPORTS_ROLE)
-site_auths.add_role(QA_REPORTS, name=QA_REPORTS_SUPER_ROLE)
-site_auths.add_role(QA_REPORTS_AUDIT, name=QA_REPORTS_AUDIT_ROLE)
+    # roles
+    site_auths.add_role(QA_REPORTS, name=QA_REPORTS_ROLE)
+    site_auths.add_role(QA_REPORTS, name=QA_REPORTS_SUPER_ROLE)
+    site_auths.add_role(QA_REPORTS_AUDIT, name=QA_REPORTS_AUDIT_ROLE)
+
+
+update_site_auths()

edc_randomization/admin.py

@@ -1,3 +1,5 @@
+import contextlib
+
 from django.conf import settings
 from django.contrib import admin
 from django.contrib.admin.sites import AlreadyRegistered
@@ -65,7 +67,8 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
 
     def get_readonly_fields(self, request, obj=None):
         readonly_fields = super().get_readonly_fields(request, obj=obj)
-        readonly_fields += (
+        readonly_fields = (
+            *readonly_fields,
             "subject_identifier",
             "sid",
             "site_name",
@@ -75,7 +78,8 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
             "allocated_datetime",
             "allocated_site",
             "randomizer_name",
-        ) + audit_fields
+            *audit_fields,
+        )
         return tuple(set(readonly_fields))
 
     def get_queryset(self, request):
@@ -90,7 +94,7 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
         return qs
 
     def get_list_display(self, request) -> tuple[str, ...]:
-        list_display = [
+        fields = (
             "sid",
             "assignment",
             "site_name",
@@ -98,22 +102,24 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
             "allocated_datetime",
             "allocated_site",
             "randomizer_name",
-        ]
+        )
+        if flds := site_randomizers.get_by_model(
+            self.model._meta.label_lower
+        ).get_extra_list_display():
+            fields = list(fields)
+            for pos, fname in flds:
+                fields.insert(pos, fname)
+            fields = tuple(fields)
         if user_is_blinded(request.user.username) or (
             not user_is_blinded(request.user.username)
             and RANDO_UNBLINDED not in [g.name for g in request.user.groups.all()]
         ):
-            list_display.remove("assignment")
-        if flds := site_randomizers.get_by_model(
-            self.model._meta.label_lower
-        ).get_extra_list_display():
-            for pos, fldname in flds:
-                list_display.insert(pos, fldname)
-        return tuple(list_display)
+            fields = tuple([fname for fname in fields if fname != "assignment"])
+        return fields
 
     @staticmethod
     def get_fieldnames(request) -> tuple[str, ...]:
-        fields = [
+        fields = (
             "subject_identifier",
             "sid",
             "assignment",
@@ -122,33 +128,35 @@ class RandomizationListModelAdmin(TemplatesModelAdminMixin, admin.ModelAdmin):
             "allocated_datetime",
             "allocated_site",
             "randomizer_name",
-        ]
+        )
         if user_is_blinded(request.user.username) or (
             not user_is_blinded(request.user.username)
             and RANDO_UNBLINDED not in [g.name for g in request.user.groups.all()]
         ):
-            fields.remove("assignment")
-        return tuple(fields)
+            fields = tuple([fname for fname in fields if fname != "assignment"])
+        return fields
 
     def get_list_filter(self, request) -> tuple[str, ...]:
-        list_filter = [
+        fields = (
             "assignment",
             "allocated_datetime",
             "allocated_site",
             "site_name",
             "randomizer_name",
-        ]
+        )
         if flds := site_randomizers.get_by_model(
             self.model._meta.label_lower
         ).get_extra_list_filter():
-            for pos, fldname in flds:
-                list_filter.insert(pos, fldname)
+            fields = list(fields)
+            for pos, fname in flds:
+                fields.insert(pos, fname)
+            fields = tuple(fields)
         if user_is_blinded(request.user.username) or (
             not user_is_blinded(request.user.username)
             and RANDO_UNBLINDED not in [g.name for g in request.user.groups.all()]
         ):
-            list_filter.remove("assignment")
-        return tuple(list_filter)
+            fields = tuple([fname for fname in fields if fname != "assignment"])
+        return fields
 
 
 def register_admin():
@@ -156,10 +164,8 @@ def register_admin():
     for randomizer_cls in site_randomizers._registry.values():
         model = randomizer_cls.model_cls()
         admin_cls = type(f"{model.__name__}ModelAdmin", (RandomizationListModelAdmin,), {})
-        try:
+        with contextlib.suppress(AlreadyRegistered):
             edc_randomization_admin.register(model, admin_cls)
-        except AlreadyRegistered:
-            pass
 
 
 register_admin()

edc_randomization/auths.py

@@ -9,11 +9,16 @@ from .auth_objects import (
     update_rando_group_permissions,
 )
 
-site_auths.add_post_update_func(
-    "edc_randomization", remove_default_model_permissions_from_edc_permissions
-)
 
-site_auths.add_group(get_rando_permissions_codenames, name=RANDO_BLINDED, view_only=True)
-site_auths.add_group(get_rando_permissions_codenames, name=RANDO_UNBLINDED, view_only=True)
-site_auths.add_post_update_func("edc_randomization", update_rando_group_permissions)
-site_auths.add_post_update_func("edc_randomization", make_randomizationlist_view_only)
+def update_site_auths() -> None:
+    site_auths.add_post_update_func(
+        "edc_randomization", remove_default_model_permissions_from_edc_permissions
+    )
+
+    site_auths.add_group(get_rando_permissions_codenames, name=RANDO_BLINDED, view_only=True)
+    site_auths.add_group(get_rando_permissions_codenames, name=RANDO_UNBLINDED, view_only=True)
+    site_auths.add_post_update_func("edc_randomization", update_rando_group_permissions)
+    site_auths.add_post_update_func("edc_randomization", make_randomizationlist_view_only)
+
+
+update_site_auths()