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

edc_pharmacy/admin/stock/stock_admin.py

@@ -1,3 +1,5 @@
+import contextlib
+
 from django.contrib import admin
 from django.template.loader import render_to_string
 from django.urls import reverse
@@ -48,19 +50,19 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     show_form_tools = True
     show_history_label = True
-    autocomplete_fields = ["container"]
+    autocomplete_fields = ("container",)
 
     change_list_note = (
         "C=Confirmed, A=Allocated, T=Transferred to site, "
         "S=Confirmed at site, B=Stored at site, D=Dispensed"
     )
 
-    actions = [
+    actions = (
         print_labels,
         confirm_stock_from_queryset,
         go_to_add_repack_request_action,
         print_stock_report_action,
-    ]
+    )
 
     form = StockForm
 
@@ -184,29 +186,24 @@ class StockAdmin(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_display_links(self, request, list_display):
         display_links = super().get_list_display_links(request, list_display)
         if not request.user.userprofile.roles.filter(
             name__in=[PHARMACIST_ROLE, PHARMACY_SUPER_ROLE]
         ).exists():
-            try:
+            with contextlib.suppress(ValueError):
                 display_links.remove("formatted_code")
-            except ValueError:
-                pass
         return display_links
 
     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_fieldsets(self, request, obj=None):
         fieldsets = super().get_fieldsets(request, obj)
@@ -228,12 +225,7 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
                 mark_safe('<div style="color:red;">ERROR!</div>'),  # nosec B703, B308
             )
         except AllocationError:
-            return format_html(
-                "{}",
-                mark_safe(
-                    '<div style="color:red;">Allocation<BR>ERROR!</div>'
-                ),  # nosec B703, B308
-            )
+            return mark_safe('<div style="color:red;">Allocation<BR>ERROR!</div>')
         return obj.lot.assignment
 
     @admin.display(description="Lot #", ordering="lot__lot_no")
@@ -259,7 +251,7 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     @admin.display(description="C", boolean=True)
     def formatted_confirmation(self, obj):
         if obj:
-            return True if get_related_or_none(obj, "confirmation") else False
+            return bool(get_related_or_none(obj, "confirmation"))
         return None
 
     @admin.display(description="A", boolean=True)
@@ -269,7 +261,7 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
             and get_related_or_none(obj, "confirmation")
             and get_related_or_none(obj, "from_stock")
         ):
-            return True if get_related_or_none(obj, "allocation") else False
+            return bool(get_related_or_none(obj, "allocation"))
         return None
 
     @admin.display(description="T", boolean=True)
@@ -279,7 +271,7 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
             and get_related_or_none(obj, "confirmation")
             and get_related_or_none(obj, "from_stock")
         ):
-            return True if get_related_or_none(obj, "stocktransferitem") else False
+            return bool(get_related_or_none(obj, "stocktransferitem"))
         return None
 
     @admin.display(description="S", boolean=True)
@@ -289,7 +281,7 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
             and get_related_or_none(obj, "confirmation")
             and get_related_or_none(obj, "from_stock")
         ):
-            return True if get_related_or_none(obj, "confirmationatsiteitem") else False
+            return bool(get_related_or_none(obj, "confirmationatsiteitem"))
         return None
 
     @admin.display(description="B", boolean=True)
@@ -301,7 +293,7 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
             and get_related_or_none(obj, "confirmationatsiteitem")
             and not get_related_or_none(obj, "dispenseitem")
         ):
-            return True if get_related_or_none(obj, "storagebinitem") else False
+            return bool(get_related_or_none(obj, "storagebinitem"))
         return None
 
     @admin.display(description="D", boolean=True)
@@ -312,15 +304,12 @@ class StockAdmin(ModelAdminMixin, SimpleHistoryAdmin):
             and get_related_or_none(obj, "from_stock")
             and get_related_or_none(obj, "confirmationatsiteitem")
         ):
-            return True if get_related_or_none(obj, "dispenseitem") else False
+            return bool(get_related_or_none(obj, "dispenseitem"))
         return None
 
     @admin.display(description="Container", ordering="container__name")
     def container_str(self, obj):
-        return format_html(
-            "{}",
-            mark_safe("<BR>".join(str(obj.container).split(" "))),  # nosec B703, B308
-        )
+        return mark_safe("<BR>".join(str(obj.container).split(" ")))  # noqa: S308
 
     @admin.display(description="formulation", ordering="product__formulation__name")
     def formulation(self, obj):

edc_pharmacy/admin/stock/stock_proxy_admin.py

@@ -1,3 +1,5 @@
+import contextlib
+
 from django.contrib import admin
 from django_audit_fields import audit_fieldset_tuple
 
@@ -15,7 +17,6 @@ from .stock_admin import StockAdmin
 
 @admin.register(StockProxy, site=edc_pharmacy_admin)
 class StockProxyAdmin(StockAdmin):
-
     fieldsets = (
         (
             "Stock item",
@@ -106,10 +107,8 @@ class StockProxyAdmin(StockAdmin):
         if not request.user.userprofile.roles.filter(
             name__in=[PHARMACIST_ROLE, PHARMACY_SUPER_ROLE]
         ).exists():
-            try:
+            with contextlib.suppress(ValueError):
                 display_links.remove("formatted_code")
-            except ValueError:
-                pass
         return display_links
 
     def get_view_only_site_ids_for_user(self, request) -> list[int]:

edc_pharmacy/admin/stock/stock_request_admin.py

@@ -3,7 +3,6 @@ from celery.states import SUCCESS
 from django.contrib import admin
 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 import audit_fieldset_tuple
@@ -26,7 +25,7 @@ class StatusListFilter(admin.SimpleListFilter):
     title = _("Status")
     parameter_name = "status"
 
-    def lookups(self, request, model_admin):
+    def lookups(self, request, model_admin):  # noqa: ARG002
         return (
             (PENDING, _("Pending")),
             (COMPLETE, _("Complete")),
@@ -34,7 +33,7 @@ class StatusListFilter(admin.SimpleListFilter):
             ("ZERO", _("Zero total")),
         )
 
-    def queryset(self, request, queryset):
+    def queryset(self, request, queryset):  # noqa: ARG002
         if self.value():
             if self.value() == PENDING:
                 return (
@@ -67,15 +66,15 @@ class StockRequestAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     list_per_page = 20
     ordering = ("-request_identifier",)
 
-    autocomplete_fields = ["container", "formulation", "location"]
+    autocomplete_fields = ("container", "formulation", "location")
     form = StockRequestForm
 
-    actions = [
+    actions = (
         prepare_stock_request_items_action,
         allocate_stock_to_subject,
         print_labels_from_stock_request_by_code,
         "delete_selected",
-    ]
+    )
 
     fieldsets = (
         (
@@ -194,10 +193,7 @@ class StockRequestAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     @admin.display(description="Container", ordering="container_name")
     def container_str(self, obj):
-        return format_html(
-            "{}",
-            mark_safe("<BR>".join(str(obj.container).split(" "))),  # nosec B703, B308
-        )
+        return mark_safe("<BR>".join(str(obj.container).split(" ")))  # noqa: S308
 
     @admin.display(description="Task")
     def task_status(self, obj):

edc_pharmacy/admin/stock/stock_request_item_admin.py

@@ -1,7 +1,6 @@
 from django.contrib import admin
 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_audit_fields import audit_fieldset_tuple
 from rangefilter.filters import DateRangeFilterBuilder
@@ -39,8 +38,8 @@ class StockRequestItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     show_cancel = True
     list_per_page = 20
     form = StockRequestItemForm
-    autocomplete_fields = ["rx"]
-    actions = [print_labels_from_stock_request_item, "delete_selected"]
+    autocomplete_fields = ("rx",)
+    actions = (print_labels_from_stock_request_item, "delete_selected")
 
     fieldsets = (
         (
@@ -98,27 +97,19 @@ class StockRequestItemAdmin(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="Product")
     def formulation(self, obj):
-        return format_html(
-            "{}",
-            mark_safe(
-                f"{obj.stock_request.formulation}<BR>{obj.stock_request.container}"
-            ),  # nosec B703, B308
-        )
+        return mark_safe(f"{obj.stock_request.formulation}<BR>{obj.stock_request.container}")  # noqa: S308
 
     @admin.display(description="Date", ordering="request_item_datetime")
     def item_date(self, obj):
@@ -147,13 +138,13 @@ class StockRequestItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     @admin.display(description="A", boolean=True)
     def allocated(self, obj):
         if obj:
-            return True if getattr(obj, "allocation", None) else False
+            return bool(getattr(obj, "allocation", None))
         return None
 
     @admin.display(description="T", boolean=True)
     def transferred(self, obj):
         if obj and getattr(obj, "allocation", None):
-            return True if obj.allocation.stock.stocktransferitem else False
+            return bool(obj.allocation.stock.stocktransferitem)
         return False
 
     @admin.display(description="Subject", ordering="appt_datetime")
@@ -211,7 +202,7 @@ class StockRequestItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
 
     def get_readonly_fields(self, request, obj=None):
         if obj:
-            return self.readonly_fields + ("stock_request",)
+            return tuple({*self.readonly_fields, "stock_request"})
         return self.readonly_fields
 
     def formfield_for_foreignkey(self, db_field, request, **kwargs):

edc_pharmacy/admin/stock/stock_transfer_admin.py

@@ -21,10 +21,10 @@ class ConfirmedAtSiteListFilter(SimpleListFilter):
     title = "Confirmed at site"
     parameter_name = "confirmed_at_site"
 
-    def lookups(self, request, model_admin):
+    def lookups(self, request, model_admin):  # noqa: ARG002
         return (YES, YES), (PARTIAL, "Partial"), (NO, NO)
 
-    def queryset(self, request, queryset):
+    def queryset(self, request, queryset):  # noqa: ARG002
         qs = None
         if self.value():
             if self.value() == YES:
@@ -60,10 +60,10 @@ class StockTransferAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     show_object_tools = True
     show_cancel = True
     list_per_page = 20
-    ordering = ["-transfer_identifier"]
+    ordering = ("-transfer_identifier",)
 
-    autocomplete_fields = ["from_location", "to_location"]
-    actions = [transfer_stock_action, print_transfer_stock_manifest_action]
+    autocomplete_fields = ("from_location", "to_location")
+    actions = (transfer_stock_action, print_transfer_stock_manifest_action)
 
     form = StockTransferForm
 

edc_pharmacy/admin/stock/stock_transfer_item_admin.py

@@ -19,10 +19,10 @@ class ConfirmedAtSiteFilter(SimpleListFilter):
     title = "Confirmed at site"
     parameter_name = "confirmed_at_site"
 
-    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():
             opts = dict(
@@ -53,7 +53,7 @@ class StockTransferItemAdmin(ModelAdminMixin, SimpleHistoryAdmin):
     show_cancel = True
     list_per_page = 20
 
-    autocomplete_fields = ["stock"]
+    autocomplete_fields = ("stock",)
 
     fieldsets = (
         (

edc_pharmacy/admin/stock/storage_bin_admin.py

@@ -27,7 +27,7 @@ class StorageBinAdmin(SiteModelAdminMixin, ModelAdminMixin, SimpleHistoryAdmin):
 
     form = StorageBinForm
 
-    actions = [add_to_storage_bin_action, move_to_storage_bin_action]
+    actions = (add_to_storage_bin_action, move_to_storage_bin_action)
 
     fieldsets = (
         (

edc_pharmacy/admin/stock/storage_bin_item_admin.py

@@ -23,8 +23,8 @@ class StorageBinItemAdmin(SiteModelAdminMixin, ModelAdminMixin, SimpleHistoryAdm
     show_cancel = True
     list_per_page = 20
 
-    autocomplete_fields = ["stock"]
-    actions = ["delete_selected"]
+    autocomplete_fields = ("stock",)
+    actions = ("delete_selected",)
 
     change_list_note = "Once an item is dispensed it is automatically removed from storage."
 

edc_pharmacy/form_validators/crf/study_medication_form_validator.py

@@ -2,14 +2,13 @@ from __future__ import annotations
 
 from datetime import datetime
 from typing import TYPE_CHECKING
-from zoneinfo import ZoneInfo
 
 from django.core.exceptions import ObjectDoesNotExist
 
 from edc_constants.constants import NO, YES
 from edc_crf.crf_form_validator import CrfFormValidator
 from edc_form_validators import INVALID_ERROR
-from edc_utils import formatted_datetime
+from edc_utils.text import formatted_datetime
 
 from ...utils import get_rx_model_cls, get_rxrefill_model_cls
 
@@ -28,20 +27,18 @@ class StudyMedicationFormValidator(CrfFormValidator):
         if self.related_visit.appointment.relative_next:
             next_appt_datetime = self.related_visit.appointment.relative_next.appt_datetime
 
-        if next_appt_datetime:
-            if (
-                self.refill_start_datetime
-                and self.refill_start_datetime.astimezone(ZoneInfo("UTC")) > next_appt_datetime
-            ):
-                local_dte = formatted_datetime(next_appt_datetime)
-                error_msg = (
-                    "Refill start date cannot be after next appointmnent date. "
-                    f"Next appointment date is {local_dte}."
-                )
+        if (
+            next_appt_datetime
+            and self.refill_start_datetime
+            and self.refill_start_datetime > next_appt_datetime
+        ):
+            local_dte = formatted_datetime(next_appt_datetime)
+            error_msg = (
+                "Refill start date cannot be after next appointmnent date. "
+                f"Next appointment date is {local_dte}."
+            )
 
-                self.raise_validation_error(
-                    {"refill_start_datetime": error_msg}, INVALID_ERROR
-                )
+            self.raise_validation_error({"refill_start_datetime": error_msg}, INVALID_ERROR)
 
         self.validate_refill_start_date_against_offschedule_date()
 
@@ -66,14 +63,14 @@ class StudyMedicationFormValidator(CrfFormValidator):
     @property
     def refill_start_datetime(self) -> datetime | None:
         if refill_start_datetime := self.cleaned_data.get("refill_start_datetime"):
-            return refill_start_datetime.astimezone(ZoneInfo("UTC"))
-        return refill_start_datetime
+            return refill_start_datetime
+        return None
 
     @property
     def refill_end_datetime(self) -> datetime | None:
         if refill_end_datetime := self.cleaned_data.get("refill_end_datetime"):
-            return refill_end_datetime.astimezone(ZoneInfo("UTC"))
-        return refill_end_datetime
+            return refill_end_datetime
+        return None
 
     @property
     def next_refill(self) -> RxRefill | None:
@@ -93,14 +90,15 @@ class StudyMedicationFormValidator(CrfFormValidator):
     @property
     def rx(self) -> Rx:
         try:
-            return get_rx_model_cls().objects.get(
+            obj = get_rx_model_cls().objects.get(
                 subject_identifier=self.subject_identifier,
                 medications__in=[self.medication],
             )
-        except ObjectDoesNotExist:
+        except ObjectDoesNotExist as e:
             self.raise_validation_error(
-                {"__all__": "Prescription does not exist"}, INVALID_ERROR
+                {"__all__": "Prescription does not exist"}, INVALID_ERROR, exc=e
             )
+        return obj
 
     @property
     def formulation(self) -> Formulation | None:
@@ -109,11 +107,13 @@ class StudyMedicationFormValidator(CrfFormValidator):
     @property
     def medication(self) -> Medication:
         if self.formulation:
-            return self.formulation.medication
-        self.raise_validation_error(
-            {"__all__": "Need the formulation to look up the prescription."},
-            INVALID_ERROR,
-        )
+            medication = self.formulation.medication
+        else:
+            self.raise_validation_error(
+                {"__all__": "Need the formulation to look up the prescription."},
+                INVALID_ERROR,
+            )
+        return medication
 
     @property
     def next_appointment(self) -> Appointment | None:

edc_pharmacy/forms/stock/confirmation_form.py

@@ -7,7 +7,7 @@ class ConfirmationForm(forms.ModelForm):
     class Meta:
         model = Confirmation
         fields = "__all__"
-        help_text = {"confirmation_identifier": "(read-only)"}
-        widgets = {
+        help_text = {"confirmation_identifier": "(read-only)"}  # noqa: RUF012
+        widgets = {  # noqa: RUF012
             "confirmation_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }

edc_pharmacy/forms/stock/dispense_form.py

@@ -7,7 +7,7 @@ class DispenseForm(forms.ModelForm):
     class Meta:
         model = Dispense
         fields = "__all__"
-        help_text = {"dispense_identifier": "(read-only)"}
-        widgets = {
+        help_text = {"dispense_identifier": "(read-only)"}  # noqa: RUF012
+        widgets = {  # noqa: RUF012
             "dispense_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }

edc_pharmacy/forms/stock/lot_form.py

@@ -4,14 +4,14 @@ from ...models import Lot
 
 
 class LotForm(forms.ModelForm):
-
     def clean(self):
         cleaned_data = super().clean()
-        if cleaned_data.get("product") and cleaned_data.get("assignment"):
-            if cleaned_data.get("product").assignment != cleaned_data.get("assignment"):
-                raise forms.ValidationError(
-                    {"assignment": "Assignment does not match product"}
-                )
+        if (
+            cleaned_data.get("product")
+            and cleaned_data.get("assignment")
+            and cleaned_data.get("product").assignment != cleaned_data.get("assignment")
+        ):
+            raise forms.ValidationError({"assignment": "Assignment does not match product"})
 
         return cleaned_data
 

edc_pharmacy/forms/stock/order_form.py

@@ -4,7 +4,6 @@ from ...models import Order, OrderItem
 
 
 class OrderForm(forms.ModelForm):
-
     def clean(self):
         cleaned_data = super().clean()
         if (
@@ -21,18 +20,17 @@ class OrderForm(forms.ModelForm):
     class Meta:
         model = Order
         fields = "__all__"
-        help_text = {"order_identifier": "(read-only)"}
-        widgets = {
+        help_text = {"order_identifier": "(read-only)"}  # noqa: RUF012
+        widgets = {  # noqa: RUF012
             "order_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }
 
 
 class OrderFormSuper(forms.ModelForm):
-
     class Meta:
         model = Order
         fields = "__all__"
-        help_text = {"order_identifier": "(read-only)"}
-        widgets = {
+        help_text = {"order_identifier": "(read-only)"}  # noqa: RUF012
+        widgets = {  # noqa: RUF012
             "order_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }

edc_pharmacy/forms/stock/product_form.py

@@ -4,7 +4,6 @@ from ...models import Product, Stock
 
 
 class ProductForm(forms.ModelForm):
-
     def clean(self):
         cleaned_data = super().clean()
         if (
@@ -17,7 +16,7 @@ class ProductForm(forms.ModelForm):
     class Meta:
         model = Product
         fields = "__all__"
-        help_text = {"product_identifier": "(read-only)"}
-        widgets = {
+        help_text = {"product_identifier": "(read-only)"}  # noqa: RUF012
+        widgets = {  # noqa: RUF012
             "product_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }

edc_pharmacy/forms/stock/receive_form.py

@@ -16,8 +16,8 @@ class ReceiveForm(forms.ModelForm):
     class Meta:
         model = Receive
         fields = "__all__"
-        help_text = {"receive_identifier": "(read-only)"}
-        widgets = {
+        help_text = {"receive_identifier": "(read-only)"}  # noqa: RUF012
+        widgets = {  # noqa: RUF012
             "receive_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }
 
@@ -26,7 +26,7 @@ class ReceiveFormSuper(forms.ModelForm):
     class Meta:
         model = Receive
         fields = "__all__"
-        help_text = {"receive_identifier": "(read-only)"}
-        widgets = {
+        help_text = {"receive_identifier": "(read-only)"}  # noqa: RUF012
+        widgets = {  # noqa: RUF012
             "receive_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }

edc_pharmacy/forms/stock/receive_item_form.py

@@ -7,7 +7,6 @@ from ...models import ReceiveItem
 
 
 class ReceiveItemForm(forms.ModelForm):
-
     def clean(self):
         cleaned_data = super().clean()
         if not self.instance.id:
@@ -18,12 +17,15 @@ class ReceiveItemForm(forms.ModelForm):
             if not cleaned_data.get("container"):
                 raise forms.ValidationError({"container": "This field is required"})
 
-        if cleaned_data.get("order_item") and cleaned_data.get("lot"):
-            if (
+        if (
+            cleaned_data.get("order_item")
+            and cleaned_data.get("lot")
+            and (
                 cleaned_data.get("order_item").product.assignment
                 != cleaned_data.get("lot").assignment
-            ):
-                raise forms.ValidationError({"lot": "Lot assignment does not match product"})
+            )
+        ):
+            raise forms.ValidationError({"lot": "Lot assignment does not match product"})
 
         # in unit_qty's
         if cleaned_data.get("qty"):

edc_pharmacy/forms/stock/repack_request_form.py

@@ -4,7 +4,6 @@ from ...models import RepackRequest
 
 
 class RepackRequestForm(forms.ModelForm):
-
     def clean(self):
         cleaned_data = super().clean()
         if cleaned_data.get("from_stock") and not getattr(
@@ -31,11 +30,14 @@ class RepackRequestForm(forms.ModelForm):
             > cleaned_data.get("from_stock").container.qty
         ):
             raise forms.ValidationError({"container": "Cannot pack into larger container."})
-        if cleaned_data.get("requested_qty") and self.instance.processed_qty:
-            if cleaned_data.get("requested_qty") < self.instance.processed_qty:
-                raise forms.ValidationError(
-                    {"requested_qty": "Cannot be less than the number of containers processed"}
-                )
+        if (
+            cleaned_data.get("requested_qty")
+            and self.instance.processed_qty
+            and cleaned_data.get("requested_qty") < self.instance.processed_qty
+        ):
+            raise forms.ValidationError(
+                {"requested_qty": "Cannot be less than the number of containers processed"}
+            )
         if (
             cleaned_data.get("requested_qty") * cleaned_data.get("container").qty
             > cleaned_data.get("from_stock").unit_qty
@@ -56,9 +58,9 @@ class RepackRequestForm(forms.ModelForm):
     class Meta:
         model = RepackRequest
         fields = "__all__"
-        help_text = {
+        help_text = {  # noqa: RUF012
             "repack_identifier": "(read-only)",
         }
-        widgets = {
+        widgets = {  # noqa: RUF012
             "repack_identifier": forms.TextInput(attrs={"readonly": "readonly"}),
         }