[ASP] Fix wrong department code

[celine-m-s]

La ville de Saint-Denis était reliée au mauvais département donc les fiches salariés revenaient en erreur.

[rsebille]

postgres@127.0.0.1:itou=# SELECT * FROM cities_city WHERE name ILIKE 'SAINT-DENIS';
  id   |    name     |      slug       | department |     post_codes      | code_insee |                       coords                       | edition_mode | normalized_name 
-------+-------------+-----------------+------------+---------------------+------------+----------------------------------------------------+--------------+-----------------
  3908 | Saint-Denis | saint-denis-11  | 11         | {11310}             | 11339      | 0101000020E6100000910F7A36ABBE014004560E2DB2AD4540 | AUTO         | saint denis
 11156 | Saint-Denis | saint-denis-30  | 30         | {30500}             | 30247      | 0101000020E61000003D0AD7A370FD10402E90A0F8311E4640 | AUTO         | saint denis
 34940 | Saint-Denis | saint-denis-974 | 974        | {97400,97417,97490} | 97411      | 0101000020E61000000000007E9CBA4B4048D27F969DE234C0 | MANUAL       | saint denis
 34602 | Saint-Denis | saint-denis-93  | 93         | {93200,93210}       | 93066      | 0101000020E6100000772D211FF4EC024066F7E461A1764840 | AUTO         | saint denis

postgres@127.0.0.1:itou=# SELECT * FROM asp_commune WHERE name = 'SAINT-DENIS'; 
  id   | start_date |  end_date  | code  |    name     |          created_at           | city_id | normalized_name 
-------+------------+------------+-------+-------------+-------------------------------+---------+-----------------
  8145 | 1900-01-01 | 1999-12-31 | 11339 | SAINT-DENIS | 2023-10-02 08:17:37.220209+00 |  [NULL] | saint denis
  8146 | 2000-01-01 | [NULL]     | 11339 | SAINT-DENIS | 2023-10-02 08:17:37.221774+00 |  [NULL] | saint denis
 24509 | 2000-01-01 | [NULL]     | 30247 | SAINT-DENIS | 2023-10-02 08:18:05.227375+00 |  [NULL] | saint denis
 63605 | 1900-01-01 | 1967-12-31 | 75066 | SAINT-DENIS | 2023-10-02 08:19:23.19301+00  |  [NULL] | saint denis
 74509 | 2000-01-01 | [NULL]     | 89342 | SAINT-DENIS | 2023-10-02 08:19:42.753408+00 |  [NULL] | saint denis
 75526 | 2000-01-01 | [NULL]     | 93066 | SAINT-DENIS | 2023-10-02 08:19:44.585166+00 |  [NULL] | saint denis
 24508 | 1900-01-01 | 1999-12-31 | 30247 | SAINT-DENIS | 2023-10-02 08:18:05.225963+00 |  [NULL] | saint denis
 74508 | 1900-01-01 | 1999-12-31 | 89342 | SAINT-DENIS | 2023-10-02 08:19:42.751779+00 |  [NULL] | saint denis
 75525 | 1968-01-01 | 1999-12-31 | 93066 | SAINT-DENIS | 2023-10-02 08:19:44.583567+00 |  [NULL] | saint denis
 76198 | 1900-01-01 | 2018-02-28 | 97811 | SAINT-DENIS | 2023-10-02 08:19:45.79943+00  |  [NULL] | saint denis

postgres@127.0.0.1:itou=# SELECT * FROM asp_commune WHERE code = '97411';
  id   | start_date |  end_date  | code  |           name           |          created_at           | city_id |     normalized_name      
-------+------------+------------+-------+--------------------------+-------------------------------+---------+--------------------------
 76147 | 1943-01-01 | [NULL]     | 97411 | SAINT-DENIS/STE-CLOTILDE | 2023-10-02 08:19:45.707652+00 |  [NULL] | saint denis/ste clotilde
 76146 | 1900-01-01 | 1942-12-31 | 97411 | SAINT-DENIS/STE-CLOTILDE | 2023-10-02 08:19:45.705743+00 |  [NULL] | saint denis/ste clotilde

postgres@127.0.0.1:itou=# SELECT COUNT(*) FROM users_jobseekerprofile WHERE birth_place_id = 76198;
 count 
-------
    92

Franchement je serais presque tenté de dire qu'on ne devrais juste pas utiliser Commune(pk=76198), donc on pourrais la supprimer de la DB mais elle serais sûrement recréée lors du prochain import de référentiel donc on retomberais dans le problème :/.
Donc peut-être un booléen ignore/hidden et un manager qui .filter(ignore=False) ? 🤷

[celine-m-s]

Merci pour les infos. Je comprends mieux.
J'ajoute un commit avec le manager pour retirer l'option dans les formulaires mais je garde mon pansement pour l'historique.

[rsebille]

Merci pour les infos. Je comprends mieux. J'ajoute un commit avec le manager pour retirer l'option dans les formulaires mais je garde mon pansement pour l'historique.

Pour l'historique je pense que tu peux remplacer ton pansement par une migration de donnée, comme ça en plus on sera sûr d'avoir le même l'état partout sans avoir de modification manuelle :).

[rsebille]

Merci 🙏, et à voir pour l'histoire du .delete() mais si on pouvais s'en passer je pense que ça serais mieux.