🐛 archaeological files forms: fix preventive formset equipment service (refs #6173)

1 files changed, 8 insertions, 5 deletions

diff --git a/archaeological_files/forms.py b/archaeological_files/forms.py
index 4519fbd90..4c7feaa04 100644
--- a/archaeological_files/forms.py
+++ b/archaeological_files/forms.py
@@ -1017,14 +1017,14 @@ class PreventiveFileEquipmentServiceForm(PreventiveFileForm):
 
     def __init__(self, *args, **kwargs):
         super(PreventiveFileEquipmentServiceForm, self).__init__(*args, **kwargs)
-        price_agreement_id = None
+        self.price_agreement_id = None
         q = models.File.objects.filter(pk=self.file_id).values("price_agreement_id")
         if q.count():
-            price_agreement_id = q.all()[0]["price_agreement_id"]
+            self.price_agreement_id = q.all()[0]["price_agreement_id"]
         q = models.EquipmentServiceCost.objects.filter(
             available=True,
             parent__isnull=True,
-            price_agreement_id=price_agreement_id,
+            price_agreement_id=self.price_agreement_id,
             equipment_service_type__generic_equipment_type__txt_idx=self.type_filter,
         )
 
@@ -1052,9 +1052,12 @@ class PreventiveFileEquipmentServiceForm(PreventiveFileForm):
 
     def save(self, commit=True):
         item = super().save(commit=True)
-        if not item:
+        if not item or not self.price_agreement_id:
             return
-        for child in item.equipment_service_cost.equipment_service_type.children.all():
+        q = item.equipment_service_cost.equipment_service_type.children.filter(
+            price_agreement_id=self.price_agreement_id
+        )
+        for child in q.all():
             if not self._meta.model.objects.filter(
                 file_id=item.file_id, equipment_service_cost=child
             ).count():