Projet

Général

Profil

Development #14384

le filtre statut "en attente d'une action" devrait prendre en compte le caractère final du statut

Ajouté par Thomas Noël il y a plus de 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
19 décembre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Le filtre "ouvert" affiche toute les demandes dont le statut n'est pas final.

Le filtre "en attente d'une action"" affiche les demandes dont le statut a une action attendue par l'agent... même si le statut est final.

Il peut donc y avoir des cas où on aura des demandes "en attente d'une action" qui ne sont pas "ouverte" ... c'est pas logique.


Fichiers


Demandes liées

Lié à w.c.s. - Bug #14392: sql : opérateur Contains vs liste videFermé20 décembre 2016

Actions

Révisions associées

Révision fdf9921f (diff)
Ajouté par Frédéric Péters il y a plus de 7 ans

backoffice: don't include endpoint status in "waiting" listings (#14384)

Révision 4c53eddc (diff)
Ajouté par Frédéric Péters il y a plus de 7 ans

forms: always limit actionable items to open items (#14384)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 7 ans

J'ai hésité à revoir get_listing_items() pour avoir un code différent en mode SQL (où on doit pouvoir combiner le tout en une seule requête) mais maintenir deux chemins différents ça va être trop lourd à maintenir.

#2

Mis à jour par Thomas Noël il y a plus de 7 ans

Ok, c'est vrai que c'est peut-être pas optimal, mais bon.

Sinon juste pour la forme, j'aurai vu le if selected_filter == 'waiting': identé, i.e. inclu en fin du bloc else: juste au-dessus.

#4

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de Nouveau à Résolu (à déployer)
commit fdf9921f8ea3af07bd0bad4bef64c7b21d5e9078
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Mon Dec 19 17:26:05 2016 +0100

    backoffice: don't include endpoint status in "waiting" listings (#14384)
#5

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de Résolu (à déployer) à En cours

J'ai oublié d'appliquer ça aussi pour le badge (x sur n) qui apparait sur la page de sélection du formulaire; et du coup je pense que je vais refaire le patch différemment (remettre le code d'avant patch ici, et modifier get_actionable_ids pour prendre en compte le caractère final).

#7

Mis à jour par Frédéric Péters il y a plus de 7 ans

Pas encore ok;

en local

Exception:
  type = '<class 'psycopg2.ProgrammingError'>', value = 'ERREUR:  erreur de syntaxe sur ou prs de  ) 
LINE 1: ...47ce1', 'fc2b18010fd2479595732163ea15ceef'] AND status IN ()
                                                                      ^

#8

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Lié à Bug #14392: sql : opérateur Contains vs liste vide ajouté
#9

Mis à jour par Frédéric Péters il y a plus de 7 ans

Voilà, bug sql traité dans son ticket dédié.

#10

Mis à jour par Thomas Noël il y a plus de 7 ans

Ack

#11

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 4c53eddc2d22a65fd8334ad22c7346709b5db707
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Dec 20 08:19:28 2016 +0100

    forms: always limit actionable items to open items (#14384)
#12

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Formats disponibles : Atom PDF