Caractère exécutoire ne vaut pas approbation

rik · rik · commit 3bc0dd77c6af · 2025-06-04T19:05:54.000+02:00
Ce sont deux impacts bien distincts, il ne faut pas les mélanger. fix #1333
diff --git a/django/core/models.py b/django/core/models.py
@@ -62,7 +62,6 @@ class EventImpact(StrEnum):
     TypeDocument.CC: {
         "Délibération de prescription du conseil municipal": EventImpact.EN_COURS,
         "Approbation du préfet": EventImpact.APPROUVE,
-        "Caractère exécutoire": EventImpact.APPROUVE,
         "Retrait de l'annulation totale": EventImpact.APPROUVE,
         "Abandon": EventImpact.ABANDON,
         "Retrait de la délibération de prescription": EventImpact.ABANDON,
@@ -74,7 +73,6 @@ class EventImpact(StrEnum):
         "Délibération de l'établissement public qui prescrit": EventImpact.EN_COURS,
         "Retrait de la délibération d'approbation": EventImpact.EN_COURS,
         "Délibération d'approbation": EventImpact.APPROUVE,
-        "Caractère exécutoire": EventImpact.APPROUVE,
         "Retrait de l'annulation totale": EventImpact.APPROUVE,
         "Abandon": EventImpact.ABANDON,
         "Retrait de la délibération de prescription": EventImpact.ABANDON,
@@ -85,15 +83,13 @@ class EventImpact(StrEnum):
     TypeDocument.SD: {
         "Délibération de l'établissement public qui prescrit": EventImpact.EN_COURS,
         "Délibération d'approbation": EventImpact.APPROUVE,
-        "Caractère exécutoire": EventImpact.APPROUVE,
         "Abandon": EventImpact.ABANDON,
         "Annulation TA totale": EventImpact.ANNULE,
         "Annulation TA": EventImpact.ANNULE,
         "Caducité": EventImpact.CADUC,
     },
     TypeDocument.PLU: {
         "Délibération de prescription du conseil municipal ou communautaire": EventImpact.EN_COURS,
-        "Caractère exécutoire": EventImpact.APPROUVE,
         "Retrait de l'annulation totale": EventImpact.APPROUVE,
         "Délibération d'approbation du municipal ou communautaire": EventImpact.APPROUVE,
         "Délibération d'approbation du conseil municipal ou communautaire": EventImpact.APPROUVE,
@@ -108,7 +104,6 @@ class EventImpact(StrEnum):
     },
     TypeDocument.POS: {
         "Délibération de prescription du conseil municipal ou communautaire": EventImpact.EN_COURS,
-        "Caractère exécutoire": EventImpact.APPROUVE,
         "Délibération d'approbation du municipal ou communautaire": EventImpact.APPROUVE,
         "Délibération d'approbation du conseil municipal ou communautaire": EventImpact.APPROUVE,
         "Délibération d'approbation": EventImpact.APPROUVE,
diff --git a/django/core/tests/test_models.py b/django/core/tests/test_models.py
@@ -57,7 +57,7 @@ def test_date_approbation_retourne_plus_recent_event_approbation(
             collectivite_porteuse=commune,
         )
         procedure.event_set.create(
-            type="Caractère exécutoire",
+            type="Délibération d'approbation du conseil municipal ou communautaire",
             date_evenement_string="2022-12-01",
         )
         procedure.event_set.create(
@@ -92,7 +92,7 @@ def test_date_approbation_ignore_event_pas_approbation(
             date_evenement_string="2023-12-01",
         )
         procedure.event_set.create(
-            type="Caractère exécutoire",
+            type="Délibération d'approbation du conseil municipal ou communautaire",
             date_evenement_string="2024-12-01",
         )
         procedure.event_set.create(
@@ -119,7 +119,7 @@ def test_date_approbation_ignore_event_approbation_pas_valid(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
         procedure.event_set.create(
-            type="Caractère exécutoire",
+            type="Délibération d'approbation",
             date_evenement_string="2024-12-01",
             is_valid=False,
         )
@@ -152,7 +152,7 @@ def test_date_approbation_ignore_event_apres(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
         procedure.event_set.create(
-            type="Caractère exécutoire",
+            type="Délibération d'approbation du conseil municipal ou communautaire",
             date_evenement_string="2022-12-01",
         )
         procedure.event_set.create(
@@ -343,7 +343,7 @@ def test_principale_opposable(
         procedure = Procedure.objects.create(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
-        event = procedure.event_set.create(type="Caractère exécutoire")
+        event = procedure.event_set.create(type="Délibération d'approbation")
 
         assert event.impact == EventImpact.APPROUVE
         with django_assert_num_queries(1):
@@ -376,7 +376,7 @@ def test_ignore_event_apres(
             date_evenement_string="2024-12-03",
         )
         event_approbation = procedure.event_set.create(
-            type="Caractère exécutoire", date_evenement_string="2024-12-05"
+            type="Délibération d'approbation", date_evenement_string="2024-12-05"
         )
 
         assert event_prescription.impact == EventImpact.EN_COURS
@@ -427,7 +427,9 @@ def test_principale_ignore_event_invalide(
         procedure = Procedure.objects.create(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
-        event = procedure.event_set.create(type="Caractère exécutoire", is_valid=False)
+        event = procedure.event_set.create(
+            type="Délibération d'approbation", is_valid=False
+        )
 
         assert event.impact is None
         with django_assert_num_queries(1):
@@ -464,7 +466,7 @@ def test_secondaire_non_opposable(
             doc_type=TypeDocument.PLUI,
             collectivite_porteuse=commune,
         )
-        procedure_secondaire.event_set.create(type="Caractère exécutoire")
+        procedure_secondaire.event_set.create(type="Délibération d'approbation")
 
         with django_assert_num_queries(1):
             procedure_with_events = Procedure.objects.with_events().get(
@@ -551,14 +553,14 @@ def test_plus_recente_opposable(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
         procedure_opposable.event_set.create(
-            type="Caractère exécutoire", date_evenement_string="2024-12-01"
+            type="Délibération d'approbation", date_evenement_string="2024-12-01"
         )
 
         procedure_precedente = commune.procedures.create(
             doc_type=TypeDocument.PLU, collectivite_porteuse=commune
         )
         procedure_precedente.event_set.create(
-            type="Caractère exécutoire", date_evenement_string="2023-12-01"
+            type="Délibération d'approbation", date_evenement_string="2023-12-01"
         )
 
         with django_assert_num_queries(2):
@@ -584,14 +586,14 @@ def test_plans_et_schemas_opposable(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
         plan_opposable.event_set.create(
-            type="Caractère exécutoire", date_evenement_string="2024-01-02"
+            type="Délibération d'approbation", date_evenement_string="2024-01-02"
         )
 
         schema_opposable = commune.procedures.create(
             doc_type=TypeDocument.SCOT, collectivite_porteuse=commune
         )
         schema_opposable.event_set.create(
-            type="Caractère exécutoire", date_evenement_string="2024-01-01"
+            type="Délibération d'approbation", date_evenement_string="2024-01-01"
         )
 
         with django_assert_num_queries(2):
@@ -616,7 +618,7 @@ def test_opposable_sans_prescription(
         procedure_opposable = commune.procedures.create(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
-        procedure_opposable.event_set.create(type="Caractère exécutoire")
+        procedure_opposable.event_set.create(type="Délibération d'approbation")
 
         with django_assert_num_queries(2):
             commune = Commune.objects.with_procedures_principales().get()
@@ -661,7 +663,7 @@ def test_abrogation_non_opposable(
             type="Abrogation",
             collectivite_porteuse=commune,
         )
-        procedure_approuvee.event_set.create(type="Caractère exécutoire")
+        procedure_approuvee.event_set.create(type="Délibération d'approbation")
 
         with django_assert_num_queries(2):
             commune = Commune.objects.with_procedures_principales().get()
@@ -686,7 +688,7 @@ def test_ignore_procedures_secondaires(
             doc_type=TypeDocument.PLUI,
             collectivite_porteuse=commune,
         )
-        procedure_secondaire.event_set.create(type="Caractère exécutoire")
+        procedure_secondaire.event_set.create(type="Délibération d'approbation")
 
         with django_assert_num_queries(2):
             commune = Commune.objects.with_procedures_principales().get()
@@ -708,12 +710,12 @@ def test_ignore_procedures_archivees(
         procedure_supprimee = commune.procedures.create(
             soft_delete=True, collectivite_porteuse=commune
         )
-        procedure_supprimee.event_set.create(type="Caractère exécutoire")
+        procedure_supprimee.event_set.create(type="Délibération d'approbation")
 
         procedure_doublon = commune.procedures.create(
             doublon_cache_de=procedure_reelle, collectivite_porteuse=commune
         )
-        procedure_doublon.event_set.create(type="Caractère exécutoire")
+        procedure_doublon.event_set.create(type="Délibération d'approbation")
 
         with django_assert_num_queries(2):
             commune = Commune.objects.with_procedures_principales().get()
@@ -736,14 +738,14 @@ def test_ignore_event_apres(
             doc_type=TypeDocument.PLUI, collectivite_porteuse=commune
         )
         procedure_opposable_fevrier.event_set.create(
-            type="Caractère exécutoire", date_evenement_string="2024-02-01"
+            type="Délibération d'approbation", date_evenement_string="2024-02-01"
         )
 
         procedure_opposable_janvier = commune.procedures.create(
             doc_type=TypeDocument.PLU, collectivite_porteuse=commune
         )
         procedure_opposable_janvier.event_set.create(
-            type="Caractère exécutoire", date_evenement_string="2024-01-01"
+            type="Délibération d'approbation", date_evenement_string="2024-01-01"
         )
 
         with django_assert_num_queries(1):

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ def test_date_approbation_retourne_plus_recent_event_approbation(`
`57`	`57`	`collectivite_porteuse=commune,`
`58`	`58`	`)`
`59`	`59`	`procedure.event_set.create(`
`60`		`- type="Caractère exécutoire",`
	`60`	`+ type="Délibération d'approbation du conseil municipal ou communautaire",`
`61`	`61`	`date_evenement_string="2022-12-01",`
`62`	`62`	`)`
`63`	`63`	`procedure.event_set.create(`
`@@ -92,7 +92,7 @@ def test_date_approbation_ignore_event_pas_approbation(`
`92`	`92`	`date_evenement_string="2023-12-01",`
`93`	`93`	`)`
`94`	`94`	`procedure.event_set.create(`
`95`		`- type="Caractère exécutoire",`
	`95`	`+ type="Délibération d'approbation du conseil municipal ou communautaire",`
`96`	`96`	`date_evenement_string="2024-12-01",`
`97`	`97`	`)`
`98`	`98`	`procedure.event_set.create(`
`@@ -119,7 +119,7 @@ def test_date_approbation_ignore_event_approbation_pas_valid(`
`119`	`119`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`120`	`120`	`)`
`121`	`121`	`procedure.event_set.create(`
`122`		`- type="Caractère exécutoire",`
	`122`	`+ type="Délibération d'approbation",`
`123`	`123`	`date_evenement_string="2024-12-01",`
`124`	`124`	`is_valid=False,`
`125`	`125`	`)`
`@@ -152,7 +152,7 @@ def test_date_approbation_ignore_event_apres(`
`152`	`152`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`153`	`153`	`)`
`154`	`154`	`procedure.event_set.create(`
`155`		`- type="Caractère exécutoire",`
	`155`	`+ type="Délibération d'approbation du conseil municipal ou communautaire",`
`156`	`156`	`date_evenement_string="2022-12-01",`
`157`	`157`	`)`
`158`	`158`	`procedure.event_set.create(`
`@@ -343,7 +343,7 @@ def test_principale_opposable(`
`343`	`343`	`procedure = Procedure.objects.create(`
`344`	`344`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`345`	`345`	`)`
`346`		`- event = procedure.event_set.create(type="Caractère exécutoire")`
	`346`	`+ event = procedure.event_set.create(type="Délibération d'approbation")`
`347`	`347`
`348`	`348`	`assert event.impact == EventImpact.APPROUVE`
`349`	`349`	`with django_assert_num_queries(1):`
`@@ -376,7 +376,7 @@ def test_ignore_event_apres(`
`376`	`376`	`date_evenement_string="2024-12-03",`
`377`	`377`	`)`
`378`	`378`	`event_approbation = procedure.event_set.create(`
`379`		`- type="Caractère exécutoire", date_evenement_string="2024-12-05"`
	`379`	`+ type="Délibération d'approbation", date_evenement_string="2024-12-05"`
`380`	`380`	`)`
`381`	`381`
`382`	`382`	`assert event_prescription.impact == EventImpact.EN_COURS`
`@@ -427,7 +427,9 @@ def test_principale_ignore_event_invalide(`
`427`	`427`	`procedure = Procedure.objects.create(`
`428`	`428`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`429`	`429`	`)`
`430`		`- event = procedure.event_set.create(type="Caractère exécutoire", is_valid=False)`
	`430`	`+ event = procedure.event_set.create(`
	`431`	`+ type="Délibération d'approbation", is_valid=False`
	`432`	`+ )`
`431`	`433`
`432`	`434`	`assert event.impact is None`
`433`	`435`	`with django_assert_num_queries(1):`
`@@ -464,7 +466,7 @@ def test_secondaire_non_opposable(`
`464`	`466`	`doc_type=TypeDocument.PLUI,`
`465`	`467`	`collectivite_porteuse=commune,`
`466`	`468`	`)`
`467`		`- procedure_secondaire.event_set.create(type="Caractère exécutoire")`
	`469`	`+ procedure_secondaire.event_set.create(type="Délibération d'approbation")`
`468`	`470`
`469`	`471`	`with django_assert_num_queries(1):`
`470`	`472`	`procedure_with_events = Procedure.objects.with_events().get(`
`@@ -551,14 +553,14 @@ def test_plus_recente_opposable(`
`551`	`553`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`552`	`554`	`)`
`553`	`555`	`procedure_opposable.event_set.create(`
`554`		`- type="Caractère exécutoire", date_evenement_string="2024-12-01"`
	`556`	`+ type="Délibération d'approbation", date_evenement_string="2024-12-01"`
`555`	`557`	`)`
`556`	`558`
`557`	`559`	`procedure_precedente = commune.procedures.create(`
`558`	`560`	`doc_type=TypeDocument.PLU, collectivite_porteuse=commune`
`559`	`561`	`)`
`560`	`562`	`procedure_precedente.event_set.create(`
`561`		`- type="Caractère exécutoire", date_evenement_string="2023-12-01"`
	`563`	`+ type="Délibération d'approbation", date_evenement_string="2023-12-01"`
`562`	`564`	`)`
`563`	`565`
`564`	`566`	`with django_assert_num_queries(2):`
`@@ -584,14 +586,14 @@ def test_plans_et_schemas_opposable(`
`584`	`586`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`585`	`587`	`)`
`586`	`588`	`plan_opposable.event_set.create(`
`587`		`- type="Caractère exécutoire", date_evenement_string="2024-01-02"`
	`589`	`+ type="Délibération d'approbation", date_evenement_string="2024-01-02"`
`588`	`590`	`)`
`589`	`591`
`590`	`592`	`schema_opposable = commune.procedures.create(`
`591`	`593`	`doc_type=TypeDocument.SCOT, collectivite_porteuse=commune`
`592`	`594`	`)`
`593`	`595`	`schema_opposable.event_set.create(`
`594`		`- type="Caractère exécutoire", date_evenement_string="2024-01-01"`
	`596`	`+ type="Délibération d'approbation", date_evenement_string="2024-01-01"`
`595`	`597`	`)`
`596`	`598`
`597`	`599`	`with django_assert_num_queries(2):`
`@@ -616,7 +618,7 @@ def test_opposable_sans_prescription(`
`616`	`618`	`procedure_opposable = commune.procedures.create(`
`617`	`619`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`618`	`620`	`)`
`619`		`- procedure_opposable.event_set.create(type="Caractère exécutoire")`
	`621`	`+ procedure_opposable.event_set.create(type="Délibération d'approbation")`
`620`	`622`
`621`	`623`	`with django_assert_num_queries(2):`
`622`	`624`	`commune = Commune.objects.with_procedures_principales().get()`
`@@ -661,7 +663,7 @@ def test_abrogation_non_opposable(`
`661`	`663`	`type="Abrogation",`
`662`	`664`	`collectivite_porteuse=commune,`
`663`	`665`	`)`
`664`		`- procedure_approuvee.event_set.create(type="Caractère exécutoire")`
	`666`	`+ procedure_approuvee.event_set.create(type="Délibération d'approbation")`
`665`	`667`
`666`	`668`	`with django_assert_num_queries(2):`
`667`	`669`	`commune = Commune.objects.with_procedures_principales().get()`
`@@ -686,7 +688,7 @@ def test_ignore_procedures_secondaires(`
`686`	`688`	`doc_type=TypeDocument.PLUI,`
`687`	`689`	`collectivite_porteuse=commune,`
`688`	`690`	`)`
`689`		`- procedure_secondaire.event_set.create(type="Caractère exécutoire")`
	`691`	`+ procedure_secondaire.event_set.create(type="Délibération d'approbation")`
`690`	`692`
`691`	`693`	`with django_assert_num_queries(2):`
`692`	`694`	`commune = Commune.objects.with_procedures_principales().get()`
`@@ -708,12 +710,12 @@ def test_ignore_procedures_archivees(`
`708`	`710`	`procedure_supprimee = commune.procedures.create(`
`709`	`711`	`soft_delete=True, collectivite_porteuse=commune`
`710`	`712`	`)`
`711`		`- procedure_supprimee.event_set.create(type="Caractère exécutoire")`
	`713`	`+ procedure_supprimee.event_set.create(type="Délibération d'approbation")`
`712`	`714`
`713`	`715`	`procedure_doublon = commune.procedures.create(`
`714`	`716`	`doublon_cache_de=procedure_reelle, collectivite_porteuse=commune`
`715`	`717`	`)`
`716`		`- procedure_doublon.event_set.create(type="Caractère exécutoire")`
	`718`	`+ procedure_doublon.event_set.create(type="Délibération d'approbation")`
`717`	`719`
`718`	`720`	`with django_assert_num_queries(2):`
`719`	`721`	`commune = Commune.objects.with_procedures_principales().get()`
`@@ -736,14 +738,14 @@ def test_ignore_event_apres(`
`736`	`738`	`doc_type=TypeDocument.PLUI, collectivite_porteuse=commune`
`737`	`739`	`)`
`738`	`740`	`procedure_opposable_fevrier.event_set.create(`
`739`		`- type="Caractère exécutoire", date_evenement_string="2024-02-01"`
	`741`	`+ type="Délibération d'approbation", date_evenement_string="2024-02-01"`
`740`	`742`	`)`
`741`	`743`
`742`	`744`	`procedure_opposable_janvier = commune.procedures.create(`
`743`	`745`	`doc_type=TypeDocument.PLU, collectivite_porteuse=commune`
`744`	`746`	`)`
`745`	`747`	`procedure_opposable_janvier.event_set.create(`
`746`		`- type="Caractère exécutoire", date_evenement_string="2024-01-01"`
	`748`	`+ type="Délibération d'approbation", date_evenement_string="2024-01-01"`
`747`	`749`	`)`
`748`	`750`
`749`	`751`	`with django_assert_num_queries(1):`