Projet

Général

Profil

Bug #80039

Calcul des demandes en attente de votre part et session_user dans les conditions de l'action

Ajouté par Valentin Deniaud il y a 9 mois. Mis à jour il y a 9 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
26 juillet 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Pour reproduire :
  • Avoir une demande qui arrive dans un statut où le rôle B peut effectuer une action seulement si {{ session_user_email == 'admin@localhost' }}
  • Accéder à la vue de traitement avec un compte qui dispose du rôle B et qui a l'email en question, la demande apparaît si on filtre sur « En attente de votre part »
  • Modifier la condition, genre {{ session_user_email == 'bla' }}
  • La demande n'apparaît plus, normal
  • Restaurer la condition
  • La demande ne réapparaît pas

Mon pdb me dit que c'est parce que la colonne actions_roles_array de la demande a été vidée par la modification de la condition.

Révisions associées

Révision 3a0957fa (diff)
Ajouté par Frédéric Péters il y a 9 mois

misc: do not consider session when computing action roles (#80039)

Historique

#2

Mis à jour par Frédéric Péters il y a 9 mois

En effet pour les tableaux de traitement les personnes qui ont visibilité (type "en attente") sont précalculées (ici dans "actions_roles_array"), ça fait que les conditions ne peuvent pas être effectives sur une condition qui va dépendre de la session (session_user_etc.).

La résolution ici va plutôt être de faire en sorte que ça ne fonctionne jamais, pour éviter les incompréhensions parce que ça a l'air de fonctionner puis un autre usager intervient, ou un cron, et ça ne fonctionne plus. Tout le temps ignorer session_user_etc. lors du calcul de actions_roles_array.

De la même manière une condition qui serait basée sur la date du jour amènerait le même type de comportement, avec variation selon le moment du dernier enregistrement de la demande, vs le moment de la consultation. Toutes ces situations ne pourront pas être évitées/détectées.

#3

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#4

Mis à jour par Robot Gitea il y a 9 mois

  • Statut changé de En cours à Solution proposée
#5

Mis à jour par Robot Gitea il y a 9 mois

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

Valentin Deniaud (vdeniaud) a approuvé une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Robot Gitea il y a 9 mois

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

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Transition automatique il y a 9 mois

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

Mis à jour par Transition automatique il y a 7 mois

Automatic expiration

Formats disponibles : Atom PDF