Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BO - Signalement Importés] Affectations clôturés mais signalement non-clôturé #1398

Closed
arnaudchouk opened this issue Jun 21, 2023 · 6 comments
Assignees
Labels
bug Quelque chose ne fonctionne pas v1.11.5
Milestone

Comments

@arnaudchouk
Copy link
Collaborator

arnaudchouk commented Jun 21, 2023

Territoire Isere - signalement 2021-6923
https://histologe.beta.gouv.fr/bo/signalements/803db3ea-4662-4f9c-91c3-35c66d63df4b
Indiqué comme en cours dans le listing

En tant que SA, quand j'ouvre le signalement tout à l'air normal et le bouton cloturer apparait
image

Par contre quand je me connecte à l aide d un compte test RT cree au sein du partenaire DDT, la page de signalement apparait grisée (cloturee) avec boutons reouvrir DDT/Tous
Pourtant toujours noté en cours dans le listing
image

@arnaudchouk arnaudchouk added the bug Quelque chose ne fonctionne pas label Jun 21, 2023
@emilschn emilschn self-assigned this Jul 6, 2023
@emilschn
Copy link
Collaborator

emilschn commented Jul 6, 2023

Ok, j'ai compris la description : le signalement est ouvert, mais tous les partenaires affectés l'ont cloturé.

DONC

Le souci ce serait que, lors de l'import, quand on a affecté puis cloturé toutes les affectations, on aurait aussi du clôturer le signalement au passage ? C'est bien ça ?
Autrement dit, si on a un motif de clôture lorsqu'on importe un signalement, on devrait le cloturer directement.
C'est bien ça ?

@emilschn emilschn changed the title [BO -Signalement Importés] Mauvais statut [BO - Signalement Importés] Affectations clôturés mais signalement non-clôturé Jul 6, 2023
@arnaudchouk
Copy link
Collaborator Author

c est ca - quand il y a motif de cloture >> signalement fermé

@emilschn
Copy link
Collaborator

emilschn commented Jul 6, 2023

Cette logique est déjà en place (et était déjà en place lors de l'import) :

  • Si il y a un statut de fermeture, on ferme
  • Sinon, si il y a un motif de cloture ou une date de cloture, on ferme aussi.

Je viens de regarder le fichier :

  • il n'y a pas de colonne statut
  • il y a des colonnes motif_cloture et date_cloture

Fonctionnement attendu
Sur les lignes où il y a un motif ou une date de cloture : les affectations ET le signalement sont fermés.

Fonctionnement inattendu
Sur les lignes où il n'y a rien dans ces colonnes (dont le signalement en question) :

  • le signalement n'est pas fermé : ça semble normal
  • les affectations sont fermées, comme tu le signales : ça semble anormal
    Je ne vois pas ce qui crée ce comportement, ce n'est pas prévu dans le code.

A priori, le signalement a été mis à jour lors de l'import (il existait déjà auparavant). Mais je ne pense pas que ça peut expliquer.

Bref, je sèche un peu pour l'instant @hmeneuvrier @sfinx13

@arnaudchouk
Copy link
Collaborator Author

un signalement ouvert peut etre cloture par tous les partenaires - ca ce n est pas anormal

de ce que je comprends si la ddt qui est RT est affectee et qu elle a cloture le signalement, le statut Partenaire affecté prend le dessus sur son affichage (page grisee car partenaire fermé)
alors qu a priori un compte RT et Partenaire à la fois devrait avoir la version RT toujours affichee (donc page accessible avec le bouton fermer affiché)

@emilschn
Copy link
Collaborator

emilschn commented Jul 7, 2023

Ok, donc ce que tu signales est une partie du souci, ça peut se corriger.

Ce que je signale et qui me surprend, c'est que pendant l'import, les affectations ont été immédiatement cloturées, alors que je ne vois pas spécialement de raison pour ça. Et ce n'est pas le seul signalement de cet import qui est dans ce cas-là.

Mais si on corrige ce que tu signales, les RT pourront cloturer le signalement, et ça permettra d'ajuster.
Je vais déjà voir pour corriger ça, puisque ça au moins, je comprends pourquoi ça le fait :)

@arnaudchouk
Copy link
Collaborator Author

j ai fouillé les mails - il n est pas impossible qu ils aient demandes à ce que les signalements soient importés sous le statut en cours, avec des partenaires affectes mais avec le statut cloturé

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Quelque chose ne fonctionne pas v1.11.5
Projects
None yet
Development

No branches or pull requests

5 participants