Move dossier field service to procedure presentation #2770
Conversation
fredZen
force-pushed
the
frederic/fix_2750-move_dossier_field_service_to_procedure_presentation
branch
2 times, most recently
from
44b6eac to
cc955a8
Compare
| end | ||
| end | ||
|
|
||
| def filtered_ids(dossiers, statut) |
There was a problem hiding this comment.
Je ne comprends pas trop statut, j'ai du mal à voir à quoi ça se rapporte
There was a problem hiding this comment.
Ce nom vient de là 801318b#diff-4d6ff7ee42566087f0d1aea8f4e4ec18R139
Je me suis retrouvé client du code ci-dessous quand j’ai écrit ça : 865988c#diff-4d6ff7ee42566087f0d1aea8f4e4ec18R76
Je n’ai pas trouvé le nom clair non plus (en général, state, status et leurs équivalents français sont des alertes qu‘un nom de variable est probablement trop générique).
Mais clarifier le nommage du statut n’a rien à voir avec cette PR => ailleurs ?
There was a problem hiding this comment.
Yes, ok pour une autre PR (d'ailleurs ça doit venir de moi ce "statut...)
fredZen
force-pushed
the
frederic/fix_2750-move_dossier_field_service_to_procedure_presentation
branch
from
cc955a8 to
0ce9c8a
Compare
|
Up, possible d‘avancer sur cette PR ? #1961 est bloquée derrière, et j‘aimerais bien travailler dessus, Camille me dit qu‘elle est attendue avec impatience. |
rather than in the view
Because they are pre-validated by rails on the model
fredZen
force-pushed
the
frederic/fix_2750-move_dossier_field_service_to_procedure_presentation
branch
from
0ce9c8a to
9086f99
Compare
fredZen
deleted the
frederic/fix_2750-move_dossier_field_service_to_procedure_presentation
branch
Fixes #2750
Le but haut niveau est de n‘utiliser que des données issues de
selfcomme noms de colonnes et comme order by. Comme ça, on peut être confiant qu’elles ont passé la validation de données, et donc qu‘elles ne vont pas taper dans des colonnes qu’on n‘avait pas prévues.