Donne systématiquement un code INSEE à une collectivité

- Renomme le champ en base de données pour mieux indiquer qu'il est spécial et ajoute une propriété qui dérive le code INSEE de l'id.
- Lance les migrations dans le flot de déploiement Scalingo

fix #1313

Author: rik

django/Procfile

@@ -1 +1,2 @@
 web: gunicorn core.wsgi --worker-class=gevent --log-file=-
+postdeploy: bin/post_deploy

django/bin/post_deploy

@@ -0,0 +1,3 @@
+#!/bin/sh
+
+./manage.py migrate

django/core/migrations/0002_rename_code_insee_collectivite_code_insee_unique.py

@@ -0,0 +1,17 @@
+# Generated by Django 5.1.5 on 2025-04-24 09:44
+# ruff: noqa: RUF012
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("core", "0001_initial"),
+    ]
+
+    operations = [
+        migrations.RenameField(
+            model_name="collectivite",
+            old_name="code_insee",
+            new_name="code_insee_unique",
+        ),
+    ]

django/core/models.py

@@ -210,7 +210,7 @@ class Procedure(models.Model):
     type = models.CharField(blank=True, null=True)  # noqa: DJ001
     numero = models.CharField(blank=True, null=True)  # noqa: DJ001
     collectivite_porteuse = models.ForeignKey(
-        "Collectivite", models.DO_NOTHING, to_field="code_insee"
+        "Collectivite", models.DO_NOTHING, to_field="code_insee_unique"
     )
     created_at = models.DateTimeField(db_default=models.functions.Now())
     doublon_cache_de = models.OneToOneField(
@@ -317,7 +317,7 @@ def __str__(self) -> str:
 
 class Collectivite(models.Model):
     id = models.CharField(primary_key=True)  # Au format code_type
-    code_insee = models.CharField(  # noqa: DJ001
+    code_insee_unique = models.CharField(  # noqa: DJ001
         unique=True,
         null=True,
         db_comment="Peut-être vide pour une COMD ayant le même code que sa commune parente",
@@ -336,6 +336,10 @@ class Collectivite(models.Model):
     def __str__(self) -> str:
         return self.nom
 
+    @property
+    def code_insee(self) -> str:
+        return self.id.split("_")[0]
+
 
 class CommuneQuerySet(models.QuerySet):
     def with_procedures_principales(self, avant: date | None = None) -> Self:

django/core/tests/test_models.py

@@ -20,12 +20,17 @@ def create_commune() -> Commune:
     departement = region.departements.create()
     return Commune.objects.create(
         id="12345_COM",
-        code_insee="12345",
+        code_insee_unique="12345",
         type="COM",
         departement=departement,
     )
 
 
+class TestCollectivite:
+    def test_code_insee(self) -> None:
+        assert Commune(id="12345_COM").code_insee == "12345"
+
+
 def test_tous_document_types_ont_event_impact() -> None:
     assert list(TypeDocument) == list(EVENT_IMPACT_BY_DOC_TYPE.keys())
 
@@ -196,7 +201,7 @@ def test_plui(
         procedure.perimetre.add(commune)
         procedure.perimetre.create(
             id="12346_COM",
-            code_insee="12346",
+            code_insee_unique="12346",
             type="COM",
             departement=commune.departement,
         )
@@ -228,7 +233,7 @@ def test_pluih(
         procedure.perimetre.add(commune)
         procedure.perimetre.create(
             id="12346_COM",
-            code_insee="12346",
+            code_insee_unique="12346",
             type="COM",
             departement=commune.departement,
         )
@@ -260,7 +265,7 @@ def test_pluim(
         procedure.perimetre.add(commune)
         procedure.perimetre.create(
             id="12346_COM",
-            code_insee="12346",
+            code_insee_unique="12346",
             type="COM",
             departement=commune.departement,
         )
@@ -300,7 +305,7 @@ def test_pluihm(
         procedure.perimetre.add(commune)
         procedure.perimetre.create(
             id="12346_COM",
-            code_insee="12346",
+            code_insee_unique="12346",
             type="COM",
             departement=commune.departement,
         )

django/core/tests/test_views.py

@@ -14,7 +14,7 @@ def create_commune_et_procedure(
     departement = region.departements.create(code_insee=code_insee[:2])
     commune = Commune.objects.create(
         id=f"{code_insee}_{type_collectivite}",
-        code_insee=code_insee,
+        code_insee_unique=code_insee,
         type=type_collectivite,
         departement=departement,
     )