Procédure : supprimer les anciennes colonnes `organisation` et `direction`

[Keirua]

HS19888

Deux champs de l'API /procedures/:id semblent ne plus correspondre à des données effectivement utilisées:

• organisation
• direction

Ces données sont nulles même si on affecte un service à la procédure. Après un peu de recherche, cela semble correspondre à des informations qui ne sont plus utilisées.

Organisation

J'ai l'impression que le seul usage de l'organisation est là :

app/models/procedure.rb
174:  def organisation_name
175:    service&.nom || organisation

… partout où c'est nécessaire, on utilise organisation_name et donc dans les faits, on utilise le service s'il est présent.

Il semble qu'il n'y ait pas de moyen d'éditer le champ organisation dans les vues (mais on l'accepte encore en paramètres d'entrées dans app/controllers/admin/procedures_controller.rb). Le seul moyen semble être via l'api pipedrive.

$ rg organisation app/views -i
app/views/admin/procedures/new_from_existing.html.haml
52:              = procedures.first.organisation_name

app/views/shared/dossiers/_identite_entreprise.html.haml
27:      %th.libelle Effectif de l'organisation :

app/views/users/dossiers/etablissement/_infos_entreprise.html.haml
28:    Effectif organisation :

direction

Pareil, rien dans les vues pour le spécifier, et cette fois ci aucune API pour lui donner une valeur

préconisation

• remplacer organisation par organisation_name dans l'API.
• supprimer direction de l'API, ou le remplacer par service.quelquechose (administrateur.nom ?)
• conserver la nomenclature actuelle dans l'API pour conserver la rétrocompatibilité
• supprimer la colonne organisation en base ? ou la migrer vers un service à créer pour l'occasion ?
• pareil pour direction
• nettoyer les controllers et les tests qui donnent l'impression que c'est utile.

Qu'en pensez-vous ?

[kemenaran]

Tout à fait exact : c'est de la vieille donnée, avant qu'on ait les champs "Service". Du coup le service&.nom || organisation sert juste à afficher cette vieille donnée.

Je crois que @Benjamin-Doberset avait envoyé un email aux admin des démarches sans service, pour leur demander d'en remplir un. Mais ça n'avait pas convaincu tout le monde.

J'imagine que des solutions possibles seraient de :

• Soit supprimer simplement la colonne (mais dans ce cas on perd le peu d'info qu'on a)
• Soit copier le contenu de la colonne dans un service (qui ne sera que partiellement complet)
• Soit intercaler une page à la connexion de l'admin ou de l'instructeur qui dit "Remplissez le service avant de continuer", comme ça on remplit les vieilles données