[BO - Signalement] Photos non rattachées aux désordres

[hmeneuvrier]

Ticket

#2437

Description

Suite aux modifications de File.setDesordreSlug() dans ce commit (757f670#diff-efb705be4240485a34a0a7cca55154c6f3c2e2c8e0f7706b504b260ec3f5d9f6), les photos des signalements n'étaient plus rattachées à leur désordre depuis le 4 avril

Changements apportés

• Correction de File.setDesordreSlug() pour chercher aussi dans les slugs de catégorie (et création de Signalement.getDesordreCategorieSlugs())
• Passage du signalement à FileFactory->createFromFileArray() dans SignalementDraftFileMessageHandler.php
• Ajout d'assertions dans SignalementDraftFileMessageHandlerTest.php
• Création d'une migration pour corriger l'existant

Pré-requis

Tests

• Faire un signalement avec des photos dans des désordres, et vérifier qu'elles sont bien rattachées aux désordres dans le BO et dans la BDD aussi
• En base, mettre NULL pour certains desordreSlug de signalement créés après le 4 avril, jouer la migration et s'assurer qu'elle remplit bien le desordreSlug
• [ ]

[emilschn]

J'ai passé un desordre_slug à NULL.
La migration s'est bien jouée, mais elle a attribué desordres_batiment_isolation_photos_upload (suffixe photos_upload qui ne me semble pas correct).
Je ne pense pas que ça fonctionne comme ça ? D'ailleurs la liaison est mal faite dans la fiche du signalement, du coup.

[hmeneuvrier]

J'ai passé un desordre_slug à NULL. La migration s'est bien jouée, mais elle a attribué desordres_batiment_isolation_photos_upload (suffixe photos_upload qui ne me semble pas correct). Je ne pense pas que ça fonctionne comme ça ? D'ailleurs la liaison est mal faite dans la fiche du signalement, du coup.

Bien vu, j'ai seulement fait un return str_replace('_details_photos_upload', '', $key); il faut que je prenne en compte que ça peut-petre seuelement _photos_upload

Merci

[numew]

Je confirme le soucis sur la migration remonté par emilien, le reste est ok

[hmeneuvrier]

@emilschn @numew migration réparée

[sonarcloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[emilschn]

@emilschn @numew migration réparée

Je suis parti du signalement-draft 00000000-0000-0000-2023-locataire001
J'ai exécuté la migration
Ca m'a mis desordres_batiment_proprete pour un des file.
Mais dans la fiche du BO, ça laisse le slug, et ça n'affiche pas le terme en français.

[numew]

Ok

[hmeneuvrier]

Je suis parti du signalement-draft 00000000-0000-0000-2023-locataire001
J'ai exécuté la migration
Ca m'a mis desordres_batiment_proprete pour un des file.
Mais dans la fiche du BO, ça laisse le slug, et ça n'affiche pas le terme en français.

Je crois que le payload de ce draft est faux car il n'y a pas de désordre desordres_batiment_proprete dans le payload

[emilschn]

Retour lié à un autre ticket. C'est ok pour celui-ci du coup !