Improve validation

Author: kiblik

…s/0232_set_risk_acceptance_engagement.py …s/0245_set_risk_acceptance_engagement.pydojo/db_migrations/0232_set_risk_acceptance_engagement.py renamed to dojo/db_migrations/0245_set_risk_acceptance_engagement.py dojo/db_migrations/0232_set_risk_acceptance_engagement.py renamed to dojo/db_migrations/0245_set_risk_acceptance_engagement.py

@@ -7,7 +7,7 @@
 class Migration(migrations.Migration):
 
     dependencies = [
-        ('dojo', '0232_set_risk_acceptance_engagement'),
+        ('dojo', '0245_set_risk_acceptance_engagement'),
     ]
 
     operations = [

…0233_alter_risk_acceptance_engagement.py …0246_alter_risk_acceptance_engagement.pydojo/db_migrations/0233_alter_risk_acceptance_engagement.py renamed to dojo/db_migrations/0246_alter_risk_acceptance_engagement.py dojo/db_migrations/0233_alter_risk_acceptance_engagement.py renamed to dojo/db_migrations/0246_alter_risk_acceptance_engagement.py

@@ -3767,13 +3767,15 @@ def __str__(self):
 
     def clean(self):
         super().clean()
+        from dojo.risk_acceptance.helper import validate_findings_engagement
         if self.pk:
-            # Get all findings that do NOT belong to this engagement
-            problematic_findings = self.accepted_findings.exclude(test__engagement=self.engagement)
-            if problematic_findings.exists():
-                problematic_ids = list(problematic_findings.values_list("id", flat=True))
-                msg = f"Findings with IDs {problematic_ids} do not belong to engagement {self.engagement_id}."
-                raise ValidationError(msg)
+            validate_findings_engagement(self.engagement, self.accepted_findings.all())
+            # # Get all findings that do NOT belong to this engagement
+            # problematic_findings = self.accepted_findings.exclude(test__engagement=self.engagement)
+            # if problematic_findings.exists():
+            #     problematic_ids = list(problematic_findings.values_list("id", flat=True))
+            #     msg = f"Findings with IDs {problematic_ids} do not belong to engagement {self.engagement_id}."
+            #     raise ValidationError(msg)
 
     def filename(self):
         # logger.debug('path: "%s"', self.path)
@@ -3788,8 +3790,7 @@ def name_and_expiration_info(self):
     def get_breadcrumbs(self):
         bc = self.engagement.get_breadcrumbs()
         bc += [{"title": str(self),
-                "url": reverse("view_risk_acceptance", args=(
-                    self.engagement.product.id, self.id))}]
+                "url": reverse("view_risk_acceptance", args=(self.id, ))}]
         return bc
 
     @property