clinicedc 2.0.34__py3-none-any.whl → 2.0.36__py3-none-any.whl

edc_pharmacy/admin/prescription/rx_refill_admin.py

@@ -3,12 +3,12 @@ from django.contrib import admin
 from django.template.loader import render_to_string
 from django.urls import reverse
 from django.utils import timezone
-from django.utils.html import format_html
 from django.utils.safestring import mark_safe
 from django_audit_fields.admin import audit_fieldset_tuple
 
 from edc_appointment.models import Appointment
-from edc_utils import convert_php_dateformat, formatted_age
+from edc_utils.age import formatted_age
+from edc_utils.text import convert_php_dateformat
 
 from ...admin_site import edc_pharmacy_admin
 from ...forms import RxRefillForm
@@ -20,7 +20,7 @@ from ..model_admin_mixin import ModelAdminMixin
 class RxRefillAdmin(ModelAdminMixin, admin.ModelAdmin):
     show_object_tools = True
 
-    ordering: tuple[str, ...] = ("refill_start_datetime",)
+    ordering = ("refill_start_datetime",)
 
     autocomplete_fields = ("dosage_guideline", "formulation")
 
@@ -64,7 +64,7 @@ class RxRefillAdmin(ModelAdminMixin, admin.ModelAdmin):
         audit_fieldset_tuple,
     )
 
-    list_display: tuple[str, ...] = (
+    list_display = (
         "subject_identifier",
         "dashboard",
         "duration",
@@ -78,7 +78,7 @@ class RxRefillAdmin(ModelAdminMixin, admin.ModelAdmin):
         "verified_datetime",
     )
 
-    list_filter: tuple[str, ...] = (
+    list_filter = (
         "active",
         "refill_start_datetime",
         "refill_end_datetime",
@@ -88,7 +88,7 @@ class RxRefillAdmin(ModelAdminMixin, admin.ModelAdmin):
         "site",
     )
 
-    search_fields: tuple[str, ...] = (
+    search_fields = (
         "id",
         "site__id",
         "rx__id",
@@ -115,14 +115,8 @@ class RxRefillAdmin(ModelAdminMixin, admin.ModelAdmin):
             else "???"
         )
         context = dict(
-            refill_start_date=format_html(
-                "{}",
-                mark_safe(" ".join(refill_start_date)),  # nosec B703, B308
-            ),
-            refill_end_date=format_html(
-                "{}",
-                mark_safe(" ".join(refill_end_date)),  # nosec B703, B308
-            ),
+            refill_start_date=mark_safe(" ".join(refill_start_date)),  # nosec B703, B308  # noqa: S308
+            refill_end_date=mark_safe(" ".join(refill_end_date)),  # nosec B703, B308  # noqa: S308
             number_of_days=obj.number_of_days,
         )
         return render_to_string("edc_pharmacy/duration.html", context=context)
@@ -177,7 +171,7 @@ class RxRefillInlineAdmin(admin.StackedInline):
 
     model = RxRefill
 
-    fields: tuple[str, ...] = (
+    fields = (
         "dosage_guideline",
         "formulation",
         "refill_start_datetime",
@@ -188,8 +182,8 @@ class RxRefillInlineAdmin(admin.StackedInline):
         "frequency_units",
     )
 
-    search_fields: tuple[str, ...] = "dosage_guideline__medication__name"
+    search_fields = ("dosage_guideline__medication__name",)
 
-    ordering: tuple[str, ...] = ("refill_start_datetime",)
+    ordering = ("refill_start_datetime",)
 
     extra = 0

edc_pharmacy/admin/remove_fields_for_blinded_users.py

@@ -5,7 +5,7 @@ from edc_randomization.blinding import user_is_blinded_from_request
 from .list_filters import AssignmentListFilter
 
 
-def remove_fields_for_blinded_users(request: WSGIRequest, fields: tuple) -> tuple:
+def remove_fields_for_blinded_users(request: WSGIRequest, fields: tuple) -> tuple[str, ...]:
     """You need to secure custom SimpleListFilters yourself"""
     if user_is_blinded_from_request(request):
         fields = list(fields)

edc_pharmacy/admin/reports/stock_availability_admin.py

@@ -66,8 +66,8 @@ class StockAvailabilityModelAdmin(
     queryset_filter: dict | None = None
     qa_report_list_display_insert_pos = 3
     include_note_column = False
-    ordering = ["relative_days"]
-    list_display = [
+    ordering = ("relative_days",)
+    list_display = (
         "dashboard",
         "subject",
         "site",
@@ -77,17 +77,17 @@ class StockAvailabilityModelAdmin(
         "formatted_codes",
         "formatted_bins",
         "last_updated",
-    ]
+    )
 
-    list_filter = [
+    list_filter = (
         HasCodesListFilter,
         ("appt_date", DateRangeFilterBuilder()),
         ("relative_days", NumericRangeFilterBuilder()),
         "visit_code",
         "site_id",
-    ]
+    )
 
-    search_fields = ["subject_identifier"]
+    search_fields = ("subject_identifier",)
 
     def get_queryset(self, request) -> QuerySet:
         update_report(self, request)
@@ -99,13 +99,17 @@ class StockAvailabilityModelAdmin(
     @admin.display(description="Codes", ordering="codes")
     def formatted_codes(self, obj):
         if obj.codes:
-            return format_html(f'<span style="font-family:courier;">{obj.codes}</span>')
+            return format_html(
+                '<span style="font-family:courier;">{codes}</span>', codes=obj.codes
+            )
         return None
 
     @admin.display(description="Bins", ordering="bins")
     def formatted_bins(self, obj):
         if obj.codes:
-            return format_html(f'<span style="font-family:courier;">{obj.bins}</span>')
+            return format_html(
+                '<span style="font-family:courier;">{bins}</span>', bins=obj.bins
+            )
         return None
 
     @admin.display(description="subject", ordering="subject_identifier")

edc_pharmacy/admin/stock/allocation_admin.py

@@ -22,10 +22,10 @@ class TransferredFilter(SimpleListFilter):
     title = "Transferred"
     parameter_name = "transferred"
 
-    def lookups(self, request, model_admin):
+    def lookups(self, request, model_admin):  # noqa: ARG002
         return (YES, YES), (NO, NO)
 
-    def queryset(self, request, queryset):
+    def queryset(self, request, queryset):  # noqa: ARG002
         qs = None
         if self.value():
             if self.value() == YES:
@@ -39,10 +39,10 @@ class DispensedFilter(SimpleListFilter):
     title = "Dispensed"
     parameter_name = "dispensed"
 
-    def lookups(self, request, model_admin):
+    def lookups(self, request, model_admin):  # noqa: ARG002
         return (YES, YES), (NO, NO)
 
-    def queryset(self, request, queryset):
+    def queryset(self, request, queryset):  # noqa: ARG002
         qs = None
         if self.value():
             if self.value() == YES:
@@ -56,10 +56,10 @@ class HasStockFilter(SimpleListFilter):
     title = "Orphaned"
     parameter_name = "orphaned"
 
-    def lookups(self, request, model_admin):
+    def lookups(self, request, model_admin):  # noqa: ARG002
         return (YES, YES), (NO, NO)
 
-    def queryset(self, request, queryset):
+    def queryset(self, request, queryset):  # noqa: ARG002
         qs = None
         if self.value():
             if self.value() == YES:
@@ -71,7 +71,6 @@ class HasStockFilter(SimpleListFilter):
 
 @admin.register(Allocation, site=edc_pharmacy_admin)
 class AllocationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
-
     change_list_title = "Pharmacy: Allocations"
     change_form_title = "Pharmacy: Allocation"
     history_list_display = ()
@@ -79,7 +78,7 @@ class AllocationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     show_cancel = True
     list_per_page = 20
 
-    actions = ["delete_selected"]
+    actions = ("delete_selected",)
 
     ordering = (
         "registered_subject__subject_identifier",
@@ -145,18 +144,15 @@ class AllocationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     def get_list_display(self, request):
         fields = super().get_list_display(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_list_filter(self, request):
         fields = super().get_list_filter(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_search_fields(self, request):
         fields = super().get_search_fields(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     @admin.display(description="ALLOCATION #", ordering="allocation_identifier")
     def identifier(self, obj):
@@ -168,12 +164,12 @@ class AllocationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     @admin.display(description="T", boolean=True)
     def transferred(self, obj):
-        return True if obj.stock.stocktransferitem else False
+        return bool(obj.stock.stocktransferitem)
 
     @admin.display(description="D", boolean=True)
     def dispensed(self, obj):
         if obj:
-            return True if get_related_or_none(obj.stock, "dispenseitem") else False
+            return bool(get_related_or_none(obj.stock, "dispenseitem"))
         return None
 
     @admin.display(description="Product", ordering="stock__product")
@@ -207,7 +203,7 @@ class AllocationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         return render_to_string("edc_pharmacy/stock/items_as_link.html", context=context)
 
     @admin.display(description="Subject #", ordering="registered_subject__subject_identifier")
-    def dashboard(self, obj=None, label=None):
+    def dashboard(self, obj=None, label=None):  # noqa: ARG002
         context = {}
         try:
             url = reverse(

edc_pharmacy/admin/stock/allocation_proxy_admin.py

@@ -9,9 +9,8 @@ from .allocation_admin import AllocationAdmin
 
 @admin.register(AllocationProxy, site=edc_pharmacy_admin)
 class AllocationProxyAdmin(AllocationAdmin):
-
     @admin.display(description="Assignment")
-    def assignment(self, obj):
+    def assignment(self, obj):  # noqa: ARG002
         return None
 
     @admin.display(description="Stock #")

edc_pharmacy/admin/stock/confirmation_admin.py

@@ -16,7 +16,6 @@ from edc_utils.date import to_local
 
 @admin.register(Confirmation, site=edc_pharmacy_admin)
 class ConfirmationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
-
     change_list_title = "Pharmacy: Confirmed stock"
     change_form_title = "Pharmacy: Confirmed stock"
     history_list_display = ()
@@ -26,7 +25,7 @@ class ConfirmationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     form = ConfirmationForm
 
-    actions = ["delete_selected"]
+    actions = ("delete_selected",)
 
     fieldsets = (
         (
@@ -56,18 +55,15 @@ class ConfirmationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     def get_list_display(self, request):
         fields = super().get_list_display(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_list_filter(self, request):
         fields = super().get_list_filter(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_search_fields(self, request):
         fields = super().get_search_fields(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     @admin.display(description="CONFIRMED #", ordering="confirmation_identifier")
     def identifier(self, obj):

edc_pharmacy/admin/stock/confirmation_at_site_item_admin.py

@@ -22,7 +22,7 @@ class ConfirmationAtSiteItemAdmin(SiteModelAdminMixin, ModelAdminMixin, SimpleHi
     show_cancel = True
     list_per_page = 20
 
-    actions = ["delete_selected"]
+    actions = ("delete_selected",)
 
     ordering = ("-transfer_confirmation_item_identifier",)
 

edc_pharmacy/admin/stock/container_admin.py

@@ -68,7 +68,7 @@ class ContainerAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         "may_dispense_as",
         "created",
     )
-    radio_fields = {"container_type": admin.VERTICAL, "units": admin.VERTICAL}
+    radio_fields = {"container_type": admin.VERTICAL, "units": admin.VERTICAL}  # noqa: RUF012
     search_fields = (
         "name",
         "display_name",
@@ -105,7 +105,7 @@ class ContainerAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     def may_dispense(self, obj):
         return obj.may_dispense_as
 
-    def get_readonly_fields(self, request, obj=None):
+    def get_readonly_fields(self, request, obj=None):  # noqa: ARG002
         if obj:
-            return self.readonly_fields + ("name",)
+            return tuple({*self.readonly_fields, "name"})
         return self.readonly_fields

edc_pharmacy/admin/stock/location_admin.py

@@ -40,12 +40,9 @@ class LocationAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         ),
     )
 
-    search_fields = ["id", "name", "contact_name"]
+    search_fields = ("id", "name", "contact_name")
 
-    def get_readonly_fields(self, request, obj=None):
+    def get_readonly_fields(self, request, obj=None):  # noqa: ARG002
         if obj:
-            return self.readonly_fields + (
-                "name",
-                "display_name",
-            )
+            return tuple({*self.readonly_fields, "name", "display_name"})
         return self.readonly_fields

edc_pharmacy/admin/stock/lot_admin.py

@@ -65,7 +65,7 @@ class LotAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         audit_fieldset_tuple,
     )
 
-    list_filter: tuple[str, ...] = (
+    list_filter = (
         "lot_no",
         "expiration_date",
         "product",
@@ -75,7 +75,7 @@ class LotAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         "modified",
     )
 
-    list_display: tuple[str, ...] = (
+    list_display = (
         "lot_no",
         "expiration_date",
         "product",
@@ -84,13 +84,13 @@ class LotAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         "created",
         "modified",
     )
-    radio_fields: tuple[str, ...] = {"assignment": admin.VERTICAL}
+    radio_fields = {"assignment": admin.VERTICAL}  # noqa: RUF012
 
-    search_fields: tuple[str, ...] = ("lot_no",)
+    search_fields = ("lot_no",)
 
-    ordering: tuple[str, ...] = ("-expiration_date",)
+    ordering = ("-expiration_date",)
 
-    def get_readonly_fields(self, request, obj=None):
+    def get_readonly_fields(self, request, obj=None):  # noqa: ARG002
         # if obj:
         #     return self.readonly_fields + ("lot_no", "product", "assignment")
         return self.readonly_fields
@@ -114,15 +114,12 @@ class LotAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     def get_list_display(self, request):
         fields = super().get_list_display(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_list_filter(self, request):
         fields = super().get_list_filter(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_search_fields(self, request):
         fields = super().get_search_fields(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)

edc_pharmacy/admin/stock/order_admin.py

@@ -54,7 +54,7 @@ class OrderAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         "modified",
     )
     list_filter = ("sent", "status", "order_datetime")
-    radio_fields = {"status": admin.VERTICAL}
+    radio_fields = {"status": admin.VERTICAL}  # noqa: RUF012
     search_fields = ("id", "order_identifier", "title")
     readonly_fields = ("order_identifier", "sent")
 
@@ -95,10 +95,7 @@ class OrderAdmin(ModelAdminMixin, SimpleHistoryAdmin):
             "edc_pharmacy/stock/items_as_link.html", context=context
         )
         rendered = [add_order_items_button, order_items_link]
-        return format_html(
-            "{}",
-            mark_safe("<BR>".join([r for r in rendered if r])),  # nosec B703, B308
-        )
+        return mark_safe("<BR>".join([r for r in rendered if r]))  # noqa: S308
 
     @admin.display(description="Order date", ordering="order_datetime")
     def order_date(self, obj):

edc_pharmacy/admin/stock/order_item_admin.py

@@ -3,7 +3,6 @@ from decimal import Decimal
 from django.contrib import admin, messages
 from django.template.loader import render_to_string
 from django.urls import reverse
-from django.utils.html import format_html
 from django.utils.safestring import mark_safe
 from django.utils.translation import gettext as _
 from django_audit_fields.admin import audit_fieldset_tuple
@@ -32,8 +31,8 @@ class OrderItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     form = OrderItemForm
     ordering = ("-order_item_identifier",)
-    autocomplete_fields = ["product", "container"]
-    actions = [delete_order_items_action]
+    autocomplete_fields = ("product", "container")
+    actions = (delete_order_items_action,)
 
     fieldsets = (
         (
@@ -66,7 +65,7 @@ class OrderItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         OrderItemStatusListFilter,
         ProductAssignmentListFilter,
     )
-    radio_fields = {"status": admin.VERTICAL}
+    radio_fields = {"status": admin.VERTICAL}  # noqa: RUF012
     search_fields = (
         "id",
         "order__id",
@@ -79,33 +78,29 @@ class OrderItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     def get_list_display(self, request):
         fields = super().get_list_display(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_list_filter(self, request):
         fields = super().get_list_filter(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_search_fields(self, request):
         fields = super().get_search_fields(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
-    def get_readonly_fields(self, request, obj=None):
+    def get_readonly_fields(self, request, obj=None):  # noqa: ARG002
         if obj:
-            return self.readonly_fields + ("order",)
+            return tuple({*self.readonly_fields, "order"})
         return self.readonly_fields
 
     def get_queryset(self, request):
         queryset = super().get_queryset(request)
         msg = _("All order items have been received")
-        if not message_in_queue(request, msg):
-            if (
-                queryset.values("unit_qty").filter(unit_qty=0).count()
-                == queryset.model.objects.values("unit_qty").all().count()
-            ):
-                messages.add_message(request, messages.INFO, msg)
+        if not message_in_queue(request, msg) and (
+            queryset.values("unit_qty").filter(unit_qty=0).count()
+            == queryset.model.objects.values("unit_qty").all().count()
+        ):
+            messages.add_message(request, messages.INFO, msg)
         return queryset
 
     @admin.display(description="ORDER ITEM #", ordering="-order_item_identifier")
@@ -185,10 +180,7 @@ class OrderItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
             received_items_link,
         ]
         renders = [r for r in renders if r]
-        return format_html(
-            "{}",
-            mark_safe("<BR>".join(renders)),  # nosec B703, B308
-        )
+        return mark_safe("<BR>".join(renders))  # noqa: S308
 
     @staticmethod
     def get_receive_obj(obj: OrderItem) -> Receive | None:

edc_pharmacy/admin/stock/product_admin.py

@@ -55,28 +55,25 @@ class ProductAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     )
     ordering = ("product_identifier",)
     readonly_fields = ("product_identifier", "name")
-    radio_fields = {"assignment": admin.VERTICAL}
+    radio_fields = {"assignment": admin.VERTICAL}  # noqa: RUF012
 
     @admin.display(description="PRODUCT #", ordering="product_identifier")
     def identifier(self, obj):
         return obj.product_identifier.split("-")[0]
 
-    def get_readonly_fields(self, request, obj=None):
+    def get_readonly_fields(self, request, obj=None):  # noqa: ARG002
         if obj:
-            return self.readonly_fields + ("formulation", "assignment")
+            return tuple({*self.readonly_fields, "formulation", "assignment"})
         return self.readonly_fields
 
     def get_list_display(self, request):
         fields = super().get_list_display(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_list_filter(self, request):
         fields = super().get_list_filter(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_search_fields(self, request):
         fields = super().get_search_fields(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)

edc_pharmacy/admin/stock/receive_admin.py

@@ -25,8 +25,8 @@ class ReceiveAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     form = ReceiveForm
     ordering = ("-receive_identifier",)
-    actions = [print_labels_from_receive, confirm_received_stock_action]
-    autocomplete_fields = ["supplier"]
+    actions = (print_labels_from_receive, confirm_received_stock_action)
+    autocomplete_fields = ("supplier",)
 
     fieldsets = (
         (

edc_pharmacy/admin/stock/receive_item_admin.py

@@ -27,7 +27,7 @@ class ReceiveItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     include_audit_fields_in_list_display = False
     ordering = ("-receive_item_identifier",)
 
-    actions = [delete_receive_items_action, print_labels_from_receive_item]
+    actions = (delete_receive_items_action, print_labels_from_receive_item)
 
     fieldsets = (
         (
@@ -143,9 +143,9 @@ class ReceiveItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     def identifier(self, obj):
         return obj.receive_item_identifier
 
-    def get_readonly_fields(self, request, obj=None):
+    def get_readonly_fields(self, request, obj=None):  # noqa: ARG002
         if obj:
-            return self.readonly_fields + ("receive",)
+            return tuple({*self.readonly_fields, "receive"})
         return self.readonly_fields
 
     def formfield_for_foreignkey(self, db_field, request, **kwargs):

edc_pharmacy/admin/stock/repack_request_admin.py

@@ -44,13 +44,13 @@ class RequestRepackAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     ordering = ("-repack_identifier",)
 
-    autocomplete_fields = ["from_stock", "container"]
+    autocomplete_fields = ("from_stock", "container")
     form = RepackRequestForm
-    actions = [
+    actions = (
         process_repack_request_action,
         print_labels_from_repack_request,
         confirm_repacked_stock_action,
-    ]
+    )
 
     change_list_note = render_to_string(
         "edc_pharmacy/stock/instructions/repack_instructions.html"
@@ -106,18 +106,15 @@ class RequestRepackAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     def get_list_display(self, request):
         fields = super().get_list_display(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_list_filter(self, request):
         fields = super().get_list_filter(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     def get_search_fields(self, request):
         fields = super().get_search_fields(request)
-        fields = remove_fields_for_blinded_users(request, fields)
-        return fields
+        return remove_fields_for_blinded_users(request, fields)
 
     @admin.display(description="Repack date", ordering="repack_datetime")
     def repack_date(self, obj):
@@ -168,7 +165,7 @@ class RequestRepackAdmin(ModelAdminMixin, SimpleHistoryAdmin):
         result = get_task_result(obj)
         return getattr(result, "status", None)
 
-    def get_readonly_fields(self, request, obj=None):
+    def get_readonly_fields(self, request, obj=None):  # noqa: ARG002
         if obj and (obj.processed_qty or Decimal(0)) > Decimal(0):
             f = [
                 "repack_identifier",

edc_pharmacy/admin/stock/stock_adjustment_admin.py

@@ -22,7 +22,7 @@ class StockAdjustmentAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     show_form_tools = True
     show_history_label = True
-    autocomplete_fields = ["stock"]
+    autocomplete_fields = ("stock",)
 
     # form = StockAdjustmentForm