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

edc_randomization/randomizer.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import contextlib
 import warnings
 from datetime import datetime
 from pathlib import Path
@@ -8,6 +9,7 @@ from typing import TYPE_CHECKING, Any
 from django.apps import apps as django_apps
 from django.conf import settings
 from django.core.exceptions import ObjectDoesNotExist, ValidationError
+from django.db.models import Q
 
 from edc_registration.utils import get_registered_subject_model_cls
 
@@ -25,15 +27,15 @@ if TYPE_CHECKING:
     from edc_registration.models import RegisteredSubject
 
 
-class InvalidAssignmentDescriptionMap(Exception):
+class InvalidAssignmentDescriptionMap(Exception):  # noqa: N818
     pass
 
 
-class RandomizationListFileNotFound(Exception):
+class RandomizationListFileNotFound(Exception):  # noqa: N818
     pass
 
 
-class RandomizationListNotLoaded(Exception):
+class RandomizationListNotLoaded(Exception):  # noqa: N818
     pass
 
 
@@ -41,7 +43,7 @@ class RandomizationError(Exception):
     pass
 
 
-class AlreadyRandomized(ValidationError):
+class AlreadyRandomized(ValidationError):  # noqa: N818
     pass
 
 
@@ -113,10 +115,10 @@ class Randomizer:
         subject_identifier: str | None = None,
         identifier_attr: str | None = None,
         identifier_object_name: str | None = None,
-        report_datetime: datetime = None,
-        site: Any = None,
-        user: str = None,
-        **kwargs,
+        report_datetime: datetime | None = None,
+        site: Any | None = None,
+        user: str | None = None,
+        **kwargs,  # noqa: ARG002
     ):
         self._model_obj = None
         self._registration_obj = None
@@ -208,8 +210,10 @@ class Randomizer:
     @property
     def sid(self):
         """Returns the SID."""
-        if self.model_obj.sid is None:
-            raise RandomizationError(f"SID cannot be None. See {self.model_obj}.")
+        if not self.model_obj.sid:
+            raise RandomizationError(
+                f"SID cannot be None. See {self.model_obj}. Got {self.model_obj.sid}"
+            )
         return self.model_obj.sid
 
     @property
@@ -233,7 +237,7 @@ class Randomizer:
         if not self._model_obj:
             try:
                 obj = self.model_cls().objects.get(**self.identifier_opts)
-            except ObjectDoesNotExist:
+            except ObjectDoesNotExist as e:
                 opts = dict(site_name=self.site.name, **self.extra_model_obj_options)
                 self._model_obj = (
                     self.model_cls()
@@ -245,7 +249,7 @@ class Randomizer:
                     fld_str = ", ".join([f"{k}=`{v}`" for k, v in opts.items()])
                     raise AllocationError(
                         f"Randomization failed. No additional SIDs available for {fld_str}."
-                    )
+                    ) from e
             else:
                 raise AlreadyRandomized(
                     f"{self.identifier_object_name.title()} already randomized. "
@@ -282,7 +286,7 @@ class Randomizer:
         Called by `registration_obj`.
         """
         return self.get_registration_model_cls().objects.get(
-            sid__isnull=True, **self.identifier_opts
+            (Q(sid__isnull=True) | Q(sid="")), **self.identifier_opts
         )
 
     @property
@@ -298,14 +302,14 @@ class Randomizer:
         if not self._registration_obj:
             try:
                 self._registration_obj = self.get_unallocated_registration_obj()
-            except ObjectDoesNotExist:
+            except ObjectDoesNotExist as e:
                 try:
                     obj = self.get_registration_model_cls().objects.get(**self.identifier_opts)
-                except ObjectDoesNotExist:
+                except ObjectDoesNotExist as e:
                     raise RandomizationError(
                         f"{self.identifier_object_name.title()} does not exist. "
                         f"Got {getattr(self, self.identifier_attr)}"
-                    )
+                    ) from e
                 else:
                     raise AlreadyRandomized(
                         f"{self.identifier_object_name.title()} already randomized. "
@@ -313,7 +317,7 @@ class Randomizer:
                         f"Got {getattr(obj, self.identifier_attr)} "
                         f"SID={obj.sid}",
                         code=self.get_registration_model_cls()._meta.label_lower,
-                    )
+                    ) from e
         return self._registration_obj
 
     @property
@@ -347,7 +351,7 @@ class Randomizer:
                 "Randomization list file not found. "
                 f"Got `{cls.get_randomizationlist_path()}`. See Randomizer {cls.name}."
             )
-        try:
+        with contextlib.suppress(RandomizationListAlreadyImported):
             result = cls.importer_cls(
                 assignment_map=cls.assignment_map,
                 randomizationlist_path=cls.get_randomizationlist_path(),
@@ -356,8 +360,6 @@ class Randomizer:
                 extra_csv_fieldnames=cls.extra_csv_fieldnames,
                 **kwargs,
             ).import_list(**kwargs)
-        except RandomizationListAlreadyImported:
-            pass
         return result
 
     @classmethod

edc_randomization/utils.py

@@ -1,7 +1,8 @@
 from __future__ import annotations
 
 import csv
-import os
+import sys
+from pathlib import Path
 from typing import Any
 
 from django.conf import settings
@@ -20,13 +21,13 @@ class RandomizationListExporterError(Exception):
     pass
 
 
-class SubjectNotRandomization(Exception):
+class SubjectNotRandomization(Exception):  # noqa: N818
     pass
 
 
 def get_assignment_for_subject(
     subject_identifier: str,
-    randomizer_name: str = None,
+    randomizer_name: str,
     identifier_fld: str | None = None,
 ) -> str:
     """Returns the assignment for a randomized subject.
@@ -42,7 +43,7 @@ def get_assignment_for_subject(
 
 def get_assignment_description_for_subject(
     subject_identifier: str,
-    randomizer_name: str = None,
+    randomizer_name: str,
     identifier_fld: str | None = None,
 ) -> str:
     """Returns the assignment description for a randomized subject.
@@ -60,7 +61,7 @@ def get_assignment_description_for_subject(
 
 def get_object_for_subject(
     subject_identifier: str,
-    randomizer_name: str = None,
+    randomizer_name: str,
     identifier_fld: str | None = None,
     label: str | None = None,
 ) -> Any:
@@ -81,11 +82,11 @@ def get_object_for_subject(
     }
     try:
         obj = randomizer_cls.model_cls().objects.get(**opts)
-    except ObjectDoesNotExist:
+    except ObjectDoesNotExist as e:
         raise SubjectNotRandomization(
             f"{label.title()} not randomized. See Randomizer `{randomizer_name}`. "
             f"Got {identifier_fld}=`{subject_identifier}`."
-        )
+        ) from e
     return obj
 
 
@@ -118,7 +119,7 @@ def generate_fake_randomization_list(
 
     # get site ID and write the file
     site_id = sites.get_by_attr("name", site_name)
-    with open(filename, "a+", newline="") as f:
+    with Path(filename).open("a+", newline="") as f:
         writer = csv.DictWriter(f, fieldnames=["sid", "assignment", "site_name", "country"])
         if write_header:
             writer.writeheader()
@@ -133,7 +134,7 @@ def generate_fake_randomization_list(
                 )
             )
 
-    print(f"(*) Added {slots} slots for {site_name}.")
+    sys.stdout.write(f"(*) Added {slots} slots for {site_name}.\n")
 
 
 def export_randomization_list(
@@ -143,20 +144,20 @@ def export_randomization_list(
 
     try:
         user = get_user_model().objects.get(username=username)
-    except ObjectDoesNotExist:
-        raise RandomizationListExporterError(f"User `{username}` does not exist")
+    except ObjectDoesNotExist as e:
+        raise RandomizationListExporterError(f"User `{username}` does not exist") from e
     if not user.has_perm(randomizer_cls.model_cls()._meta.label_lower.replace(".", ".view_")):
         raise RandomizationListExporterError(
             f"User `{username}` does not have "
             f"permission to view '{randomizer_cls.model_cls()._meta.label_lower}'"
         )
-    path = path or settings.EXPORT_FOLDER
+    path = Path(path or settings.EXPORT_FOLDER)
     timestamp = timezone.now().strftime("%Y%m%d%H%M")
-    filename = os.path.expanduser(
+    filename = Path(
         f"~/{settings.APP_NAME}_{randomizer_cls.name}_"
         f"randomizationlist_exported_{timestamp}.csv"
-    )
-    filename = os.path.join(path, filename)
+    ).expanduser()
+    filename = path / filename
 
     df = (
         read_frame(randomizer_cls.model_cls().objects.all(), verbose=False)
@@ -172,5 +173,5 @@ def export_randomization_list(
         sep="|",
     )
     df.to_csv(**opts)
-    print(filename)
+    sys.stdout.write(f"{filename!s}\n")
     return filename

edc_refusal/auths.py

@@ -3,5 +3,10 @@ from edc_screening.auth_objects import SCREENING, SCREENING_SUPER
 
 from .auth_objects import codenames
 
-site_auths.update_group(*codenames, name=SCREENING, no_delete=True)
-site_auths.update_group(*codenames, name=SCREENING_SUPER)
+
+def update_site_auths() -> None:
+    site_auths.update_group(*codenames, name=SCREENING, no_delete=True)
+    site_auths.update_group(*codenames, name=SCREENING_SUPER)
+
+
+update_site_auths()

edc_refusal/model_mixins.py

@@ -33,7 +33,7 @@ class SubjectRefusalModelMixin(models.Model):
 
     @staticmethod
     def get_search_slug_fields():
-        return ["screening_identifier"]
+        return ("screening_identifier",)
 
     class Meta:
         abstract = True

edc_registration/auths.py

@@ -4,27 +4,32 @@ from edc_auth.constants import PII, PII_VIEW
 from edc_auth.site_auths import site_auths
 from edc_export.constants import EXPORT
 
-if django_apps.is_installed("edc_export"):
-    site_auths.update_group("edc_registration.export_registeredsubject", name=EXPORT)
-site_auths.update_group(
-    "edc_registration.display_dob",
-    "edc_registration.display_firstname",
-    "edc_registration.display_identity",
-    "edc_registration.display_initials",
-    "edc_registration.display_lastname",
-    "edc_registration.view_historicalregisteredsubject",
-    "edc_registration.view_registeredsubject",
-    name=PII,
-)
 
-site_auths.update_group(
-    "edc_registration.display_dob",
-    "edc_registration.display_firstname",
-    "edc_registration.display_identity",
-    "edc_registration.display_initials",
-    "edc_registration.display_lastname",
-    "edc_registration.view_historicalregisteredsubject",
-    "edc_registration.view_registeredsubject",
-    name=PII_VIEW,
-)
-site_auths.add_pii_model("edc_registration.registeredsubject")
+def update_site_auths() -> None:
+    if django_apps.is_installed("edc_export"):
+        site_auths.update_group("edc_registration.export_registeredsubject", name=EXPORT)
+    site_auths.update_group(
+        "edc_registration.display_dob",
+        "edc_registration.display_firstname",
+        "edc_registration.display_identity",
+        "edc_registration.display_initials",
+        "edc_registration.display_lastname",
+        "edc_registration.view_historicalregisteredsubject",
+        "edc_registration.view_registeredsubject",
+        name=PII,
+    )
+
+    site_auths.update_group(
+        "edc_registration.display_dob",
+        "edc_registration.display_firstname",
+        "edc_registration.display_identity",
+        "edc_registration.display_initials",
+        "edc_registration.display_lastname",
+        "edc_registration.view_historicalregisteredsubject",
+        "edc_registration.view_registeredsubject",
+        name=PII_VIEW,
+    )
+    site_auths.add_pii_model("edc_registration.registeredsubject")
+
+
+update_site_auths()

edc_registration/model_mixins/updates_or_creates_registered_subject_model_mixin.py

@@ -5,7 +5,9 @@ from typing import TYPE_CHECKING
 
 from django.core.exceptions import ObjectDoesNotExist
 from django.db import models
+from django.db.models import CharField, TextField
 
+from edc_constants.constants import NULL_STRING
 from edc_model import DEFAULT_BASE_FIELDS
 
 from ..utils import get_registered_subject_model_cls
@@ -99,13 +101,20 @@ class UpdatesOrCreatesRegistrationModelMixin(models.Model):
         """
         registration_options = {}
         rs = self.registration_model()
-        for k, v in self.__dict__.items():
-            if k not in DEFAULT_BASE_FIELDS + ["_state"]:
+        for fname, value in self.__dict__.items():
+            if fname not in (*DEFAULT_BASE_FIELDS, "_state"):
                 try:
-                    getattr(rs, k)
-                    registration_options.update({k: v})
+                    getattr(rs, fname)
                 except AttributeError:
                     pass
+                else:
+                    value = (  # noqa: PLW2901
+                        NULL_STRING
+                        if value is None
+                        and isinstance(rs._meta.get_field(fname), (CharField, TextField))
+                        else value
+                    )
+                    registration_options.update({fname: value})
         registration_identifier = registration_options.get("registration_identifier")
         if registration_identifier:
             registration_options["registration_identifier"] = self.to_string(

edc_registration/models/registered_subject.py

@@ -143,7 +143,7 @@ class RegisteredSubject(UniqueSubjectIdentifierModelMixin, SiteModelMixin, BaseU
 
     def save(self, *args, **kwargs):
         if self.identity:
-            self.additional_key = None
+            self.additional_key = ""
         self.set_uuid_as_subject_identifier_if_none()
         self.raise_on_duplicate("subject_identifier")
         self.raise_on_duplicate("identity")

edc_reportable/utils/get_reference_range_collection.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import contextlib
 from typing import TYPE_CHECKING
 
 from django.core.exceptions import ObjectDoesNotExist
@@ -21,10 +22,8 @@ def get_reference_range_collection(obj) -> ReferenceRangeCollection:
             name = obj.requisition.panel_object.reference_range_collection_name
         except AttributeError:
             name = obj.panel_object.reference_range_collection_name
-        try:
+        with contextlib.suppress(ObjectDoesNotExist):
             reference_range_collection = reference_range_colllection_model_cls().objects.get(
                 name=name
             )
-        except ObjectDoesNotExist:
-            pass
     return reference_range_collection

edc_reportable/utils/load_data.py

@@ -60,7 +60,7 @@ def load_reference_ranges(
     collection_name: str,
     normal_data: dict[str, list[Formula]],
     grading_data: dict[str, list[Formula]],
-    reportable_grades: list[int],
+    reportable_grades: list[int] | None = None,
     reportable_grades_exceptions: dict[str, list[int]] | None = None,
     keep_existing: bool | None = None,
     create_missing_normal: bool | None = None,

edc_review_dashboard/auths.py

@@ -10,23 +10,28 @@ from edc_auth.utils import remove_default_model_permissions_from_edc_permissions
 
 REVIEW = "REVIEW"
 
-site_auths.add_post_update_func(
-    "edc_review_dashboard", remove_default_model_permissions_from_edc_permissions
-)
 
-site_auths.add_custom_permissions_tuples(
-    model="edc_review_dashboard.edcpermissions",
-    codename_tuples=[
-        (
-            "edc_review_dashboard.view_subject_review_listboard",
-            "Can access subject review listboard",
-        )
-    ],
-)
-site_auths.add_group("edc_review_dashboard.view_subject_review_listboard", name=REVIEW)
+def update_site_auths():
+    site_auths.add_post_update_func(
+        "edc_review_dashboard", remove_default_model_permissions_from_edc_permissions
+    )
+
+    site_auths.add_custom_permissions_tuples(
+        model="edc_review_dashboard.edcpermissions",
+        codename_tuples=[
+            (
+                "edc_review_dashboard.view_subject_review_listboard",
+                "Can access subject review listboard",
+            )
+        ],
+    )
+    site_auths.add_group("edc_review_dashboard.view_subject_review_listboard", name=REVIEW)
+
+    site_auths.update_role(REVIEW, name=AUDITOR_ROLE)
+    site_auths.update_role(REVIEW, name=TMG_ROLE)
+    site_auths.update_role(REVIEW, name=CLINICIAN_ROLE)
+    site_auths.update_role(REVIEW, name=CLINICIAN_SUPER_ROLE)
+    site_auths.update_role(REVIEW, name=NURSE_ROLE)
+
 
-site_auths.update_role(REVIEW, name=AUDITOR_ROLE)
-site_auths.update_role(REVIEW, name=TMG_ROLE)
-site_auths.update_role(REVIEW, name=CLINICIAN_ROLE)
-site_auths.update_role(REVIEW, name=CLINICIAN_SUPER_ROLE)
-site_auths.update_role(REVIEW, name=NURSE_ROLE)
+update_site_auths()

edc_screening/age_evaluator.py

@@ -8,11 +8,11 @@ from edc_reportable.exceptions import ValueBoundryError
 
 class AgeEvaluator(ReportableAgeEvaluator):
     def __init__(self, **kwargs) -> None:
-        self.reasons_ineligible: str | None = None
+        self.reasons_ineligible: str = ""
         super().__init__(**kwargs)
 
     def eligible(self, age: int | None = None) -> bool:
-        self.reasons_ineligible = None
+        self.reasons_ineligible = ""
         eligible = False
         if age:
             try:
@@ -26,7 +26,7 @@ class AgeEvaluator(ReportableAgeEvaluator):
         return eligible
 
     def in_bounds_or_raise(self, age: int = None, **kwargs):
-        self.reasons_ineligible = None
+        self.reasons_ineligible = ""
         dob = localtime(timezone.now() - relativedelta(years=age)).date()
         age_units = "years"
         report_datetime = localtime(timezone.now())

edc_screening/auths.py

@@ -9,38 +9,43 @@ from edc_auth.utils import remove_default_model_permissions_from_edc_permissions
 
 from .auth_objects import SCREENING, SCREENING_ROLE, SCREENING_SUPER, SCREENING_VIEW
 
-site_auths.add_post_update_func(
-    "edc_screening", remove_default_model_permissions_from_edc_permissions
-)
 
-site_auths.add_custom_permissions_tuples(
-    model="edc_screening.edcpermissions",
-    codename_tuples=(
-        ("edc_screening.view_screening_listboard", "Can access Screening listboard"),
-        ("edc_screening.nav_screening_section", "Can access screening section"),
-    ),
-)
+def update_site_auths() -> None:
+    site_auths.add_post_update_func(
+        "edc_screening", remove_default_model_permissions_from_edc_permissions
+    )
 
-site_auths.add_group(
-    "edc_screening.view_screening_listboard",
-    "edc_screening.nav_screening_section",
-    name=SCREENING,
-)
+    site_auths.add_custom_permissions_tuples(
+        model="edc_screening.edcpermissions",
+        codename_tuples=(
+            ("edc_screening.view_screening_listboard", "Can access Screening listboard"),
+            ("edc_screening.nav_screening_section", "Can access screening section"),
+        ),
+    )
 
-site_auths.add_group(
-    "edc_screening.view_screening_listboard",
-    "edc_screening.nav_screening_section",
-    name=SCREENING_SUPER,
-)
+    site_auths.add_group(
+        "edc_screening.view_screening_listboard",
+        "edc_screening.nav_screening_section",
+        name=SCREENING,
+    )
+
+    site_auths.add_group(
+        "edc_screening.view_screening_listboard",
+        "edc_screening.nav_screening_section",
+        name=SCREENING_SUPER,
+    )
+
+    site_auths.add_group(
+        "edc_screening.view_screening_listboard",
+        "edc_screening.nav_screening_section",
+        name=SCREENING_VIEW,
+    )
+
+    site_auths.add_role(SCREENING, name=SCREENING_ROLE)
+    site_auths.update_role(SCREENING, name=CLINICIAN_ROLE)
+    site_auths.update_role(SCREENING, name=NURSE_ROLE)
+    site_auths.update_role(SCREENING_SUPER, name=CLINICIAN_SUPER_ROLE)
+    site_auths.update_role(SCREENING_VIEW, name=AUDITOR_ROLE)
 
-site_auths.add_group(
-    "edc_screening.view_screening_listboard",
-    "edc_screening.nav_screening_section",
-    name=SCREENING_VIEW,
-)
 
-site_auths.add_role(SCREENING, name=SCREENING_ROLE)
-site_auths.update_role(SCREENING, name=CLINICIAN_ROLE)
-site_auths.update_role(SCREENING, name=NURSE_ROLE)
-site_auths.update_role(SCREENING_SUPER, name=CLINICIAN_SUPER_ROLE)
-site_auths.update_role(SCREENING_VIEW, name=AUDITOR_ROLE)
+update_site_auths()

edc_screening/eligibility.py

@@ -45,7 +45,7 @@ class Eligibility:
         self.eligible = all([v for v in self.criteria.values()])
 
         if self.eligible:
-            self.reasons_ineligible = None
+            self.reasons_ineligible = ""
         else:
             self.reasons_ineligible = {k: v for k, v in self.criteria.items() if not v}
             for k, v in self.criteria.items():

edc_screening/gender_evaluator.py

@@ -6,7 +6,7 @@ class GenderEvaluator:
 
     def __init__(self, gender=None, **kwargs) -> None:  # noqa
         self.eligible = False
-        self.reasons_ineligible = None
+        self.reasons_ineligible = ""
         if gender in self.eligible_gender:
             self.eligible = True
         else:

edc_screening/model_mixins/eligibility_model_mixin.py

@@ -45,8 +45,6 @@ class EligibilityModelMixin(EligibilityFieldsModelMixin, models.Model):
         """
         # if self.eligibility_cls:
         self.eligibility_cls(model_obj=self)
-        # self.eligible = eligibility_obj.is_eligible
-        # self.reasons_ineligible = eligibility_obj.reasons_ineligible
         if not self.id:
             self.screening_identifier = self.identifier_cls().identifier
         if self.eligible:

edc_screening/model_mixins/screening_methods_model_mixin.py

@@ -16,7 +16,7 @@ class ScreeningMethodsModeMixin(models.Model):
 
     @staticmethod
     def get_search_slug_fields():
-        return ["screening_identifier", "subject_identifier", "reference"]
+        return "screening_identifier", "subject_identifier", "reference"
 
     @property
     def estimated_dob(self: SubjectScreeningModelStub) -> date:

edc_screening/screening_eligibility.py

@@ -174,7 +174,7 @@ class ScreeningEligibility:
         setattr(
             self.model_obj,
             self.reasons_ineligible_fld_name,
-            "|".join(self.reasons_ineligible.values()) or None,
+            "|".join(self.reasons_ineligible.values()) or "",
         )
         self.set_eligible_model_field()
         self.set_fld_attrs_on_model()
@@ -229,9 +229,7 @@ class ScreeningEligibility:
         }
 
     def formatted_reasons_ineligible(self) -> str:
-        str_values = "<BR>".join(
-            [x for x in self.reasons_ineligible.values() if x is not None]
-        )
+        str_values = "<BR>".join([x for x in self.reasons_ineligible.values() if x])
         return format_html(
             "{}",
             mark_safe(str_values),  # nosec B703 B308

edc_screening/utils.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import contextlib
 import re
 from typing import TYPE_CHECKING, Any
 
@@ -33,14 +34,15 @@ def get_subject_screening_model_cls() -> Any:
     return django_apps.get_model(get_subject_screening_model())
 
 
-def format_reasons_ineligible(*str_values: str, delimiter=None) -> str:
+def format_reasons_ineligible(*str_values: str | None, delimiter: str | None = None) -> str:
     reasons = None
     delimiter = delimiter or "|"
-    str_values = tuple(x for x in str_values if x is not None)
+    str_values = str_values or []
+    str_values = tuple(x for x in str_values if x)
     if str_values:
         reasons = format_html(
             "{}",
-            mark_safe(delimiter.join(str_values)),  # nosec B703 B308
+            mark_safe(delimiter.join(str_values)),  # noqa: S308
         )
     return reasons
 
@@ -73,10 +75,10 @@ def get_subject_screening_or_raise(
             )
     except ObjectDoesNotExist as e:
         if is_modelform:
-            raise forms.ValidationError("Not allowed. Screening form not found.")
+            raise forms.ValidationError("Not allowed. Screening form not found.") from e
         raise ObjectDoesNotExist(
             f"{e} screening_identifier={screening_identifier}. Perhaps catch this in the form."
-        )
+        ) from e
     return subject_screening
 
 
@@ -101,10 +103,8 @@ def is_eligible_or_raise(
     )
 
     url_name = url_name or "screening_listboard_url"
-    try:
+    with contextlib.suppress(InvalidDashboardUrlName):
         url_name = url_names.get(url_name)
-    except InvalidDashboardUrlName:
-        pass
 
     if not subject_screening.eligible:
         try:
@@ -128,7 +128,7 @@ def is_eligible_or_raise(
         else:
             msg = format_html(
                 'Not allowed. Subject is not eligible. See subject <A href="{}">{}</A>',
-                mark_safe(url),  # nosec B308 B703
+                mark_safe(url),  # noqa: S308
                 subject_screening.screening_identifier,
             )
         raise forms.ValidationError(msg)