Django - Ignore les event invalides pour la date d'approbation

La queryset `events` est utilisée par `annotate(date_approbation)` mais n'incluait pas le filtre `is_valid`. C'est maintenant le cas.

Et on peut donc enlever ce filtre du `annotate(dernier_event_impactant)`.

Author: rik

django/core/models.py

@@ -125,7 +125,7 @@ class EventImpact(StrEnum):
 
 class ProcedureQuerySet(models.QuerySet):
     def with_events(self, *, avant: date | None = None) -> Self:
-        events = Event.objects.all()
+        events = Event.objects.filter(is_valid=True)
         if avant:
             events = events.filter(date_evenement_string__lt=str(avant))
 
@@ -142,7 +142,6 @@ def with_events(self, *, avant: date | None = None) -> Self:
                 then=models.Subquery(
                     events.filter(
                         procedure=models.OuterRef("pk"),
-                        is_valid=True,
                         type__in=event_impact_by_event_type.keys(),
                     ).values("type")[:1]
                 ),

django/core/tests/test_models.py

@@ -110,6 +110,25 @@ def test_date_approbation_ignore_event_pas_approbation(
 
             assert procedure_with_events.date_approbation == "2024-12-01"
 
+    @pytest.mark.django_db
+    def test_date_approbation_ignore_event_approbation_pas_valid(
+        self, django_assert_num_queries: DjangoAssertNumQueries
+    ) -> None:
+        commune = create_commune()
+        procedure = Procedure.objects.create(
+            doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
+        )
+        procedure.event_set.create(
+            type="Caractère exécutoire",
+            date_evenement_string="2024-12-01",
+            is_valid=False,
+        )
+
+        with django_assert_num_queries(1):
+            procedure_with_events = Procedure.objects.with_events().get(id=procedure.id)
+
+            assert procedure_with_events.date_approbation == "0000-00-00"
+
     @pytest.mark.django_db
     def test_date_approbation_quand_event_approbation_manquant(
         self, django_assert_num_queries: DjangoAssertNumQueries