Development #56387
cacher l'option "inclure les demandes pour lesquelles l'usager n'est pas l'auteur" et revenir à l'interrogation de l'ancienne API
0%
Description
Parce que #56385 et Development #56363: Cellule "demande de l'usager" remonte d'autres demandes et qu'on est jeudi, je pense qu'il pourrait être opportun de modifier WcsCurrentFormsCellForm pour ne pas afficher le champ include_forms_user_can_access et WcsCurrentFormsCell pour que son get_api_url revienne à la version d'avant #56012, qui appelle /api/user/forms.
Qu'on se donne ensuite le temps de correctement reprendre toutes les situations et les paramètres et les API.
(alternativement du gros revert et la migration pour virer l'option etc. mais ça m'a l'air plus lourd)
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus de 2 ans
Frédéric Péters a écrit :
ne pas afficher le champ include_forms_user_can_access et WcsCurrentFormsCell pour que son get_api_url revienne à la version d'avant #56012, qui appelle /api/user/forms.
Pas si simple, en fait le patch fautif ajoute un paramètre ?status qui semble-t-il n'existe pas et du coup vire le code de filtrage des résultats qui existait avant : je ne pense pas qu'il y ait de demi-mesure entre tenter un fix qui consisterait à remettre du code de filtrage manuel, et le gros revert. Je vais partir sur le gros revert.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Fichier 0001-Revert-wcs-option-to-get-forms-user-can-access-in-cu.patch 0001-Revert-wcs-option-to-get-forms-user-can-access-in-cu.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 2 ans
Pas si simple, en fait le patch fautif ajoute un paramètre ?status qui semble-t-il n'existe pas et du coup vire le code de filtrage des résultats qui existait avant : je ne pense pas qu'il y ait de demi-mesure entre tenter un fix qui consisterait à remettre du code de filtrage manuel, et le gros revert. Je vais partir sur le gros revert.
Je reste à penser que ça aurait été jouable le patch qui rétablirait get_api_url et les deux get_cell_extra_context concernés comme avant (+ du pytest.skip sur les tests etc. et oui je me rends compte qu'il peut s'en cacher des choses là-dedans).
Mais donc go là-dessus mais mentionner ce ticket dans le message de commit et il manque dans le patch la migration qui retire la colonne.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution proposée à En cours
Valentin Deniaud a écrit :
Pas si simple, en fait le patch fautif ajoute un paramètre ?status qui semble-t-il n'existe pas
Mauvaise analyse de ma part, ce paramètre existe et fonctionne, le problème n'est pas là. Il faudrait comprendre le bug avant de revert.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de En cours à Solution proposée
Pas réussi à reproduire en local mais pour me rassurer j'ai tapé des protojson chez le client, l'ancienne API renvoie bien ce qu'il faut et pas la nouvelle, donc le revert va corriger le bug.
Frédéric Péters a écrit :
Mais donc go là-dessus mais mentionner ce ticket dans le message de commit
Fait sur la branche
et il manque dans le patch la migration qui retire la colonne.
Il me semble qu'elle y est ou on ne parle pas de la même chose ? (create mode 100644 combo/apps/wcs/migrations/0036_remove_wcscurrentformscell_include_forms_user_can_access.py)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
J'ai du passer à côté lors de ma première lecture; go ainsi.
Mis à jour par Valentin Deniaud il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 98b8e5aa63b0939ff1ed20dfd69fd1130541d332
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Aug 26 10:53:00 2021 +0200
Revert "wcs: option to get forms user can access in current form cell (#56012)" (#56387)
This reverts commit 299429195ca6eea235eb59b996cc000847c15dbf and f8aae6b1970d498a3f9ee86706915b5924234d3b.Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Revert "wcs: option to get forms user can access in current form cell (#56012)" (#56387)
This reverts commit 299429195ca6eea235eb59b996cc000847c15dbf and f8aae6b1970d498a3f9ee86706915b5924234d3b.