Projet

Général

Profil

Bug #55633

actions de masse sur des demandes avec une fonction redéfinie au niveau de la demande

Ajouté par Frédéric Péters il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
16 juillet 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Avec l'autorisation de plusieurs rôles pour une seule fonction via l'action liaison fonction/rôle (#55019) la prise en compte de ces fonctions pour déterminer si un bouton d'action de masse doit apparaitre ou non est cassée.

Fichiers

0001-backoffice-consider-multiple-roles-attribution-in-ma.patch (2,76 ko) 0001-backoffice-consider-multiple-roles-attribution-in-ma.patch Frédéric Péters, 16 juillet 2021 14:32
0001-backoffice-consider-multiple-roles-attribution-in-ma.patch (2,75 ko) 0001-backoffice-consider-multiple-roles-attribution-in-ma.patch Frédéric Péters, 16 juillet 2021 15:25

Révisions associées

Révision a18639ae (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

backoffice: consider multiple roles attribution in mass action buttons (#55633)

Historique

#2

Mis à jour par Frédéric Péters il y a presque 3 ans

  • Statut changé de Nouveau à En cours
#3

Mis à jour par Frédéric Péters il y a presque 3 ans

C'était passé inaperçu parce que 1/ les tests créaient toujours des formdata.workflow_roles = {'clé': 'valeur'}, pas de liste, 2/ il n'y avait de toute façon pas de vérification que le bon attribut était posé sur l'HTML.

J'ai modifié pour enregistrer les workflow_roles avec la bonne forme, uniquement sur ce test, le faire au-delà a révélé #55638.

Et j'ai traité le nouveau format pour poser les bons attributs.

#4

Mis à jour par Thomas Noël il y a presque 3 ans

Peut-être un petit trou dans la raquette sur :

                    if isinstance(function_value, str):
                        # single role, defined at formdef level
                        function_values = {function_value}
                    elif function_value:
                        # list of roles, defined at formdata level
                        function_values = set(function_value or [])

si function_value ne déclench aucune des conditions if/elif, alors function_values n'existera pas ou gardera la valeur précédemment calculée. Vu le (function_value or []) dans le elif, je pense qu'il peut être remplacé par un simple else:

#5

Mis à jour par Frédéric Péters il y a presque 3 ans

Ah oui le else était l'intention et puis j'ai fait le or et puis j'ai été distrait…

#6

Mis à jour par Thomas Noël il y a presque 3 ans

  • Statut changé de Solution proposée à Solution validée

Frédéric Péters a écrit :

j'ai été distrait…

COMME TOUJOURS.

#7

Mis à jour par Frédéric Péters il y a presque 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit a18639ae6d434829a1c3d3c613c2c1bc3cd77d6f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Jul 16 13:54:07 2021 +0200

    backoffice: consider multiple roles attribution in mass action buttons (#55633)
#8

Mis à jour par Frédéric Péters il y a presque 3 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF