Development #7925
ajouter concerned_roles et is_endpoint aux vues SQL
0%
Description
Pour de l'affichage mélangé des demandes, ce sera pratique et performant de passer par la vue wcs_all_forms mais il lui manque deux indications : les rôles concernés par la demande et l'info comme quoi la demande est en attente ou pas (statut dans endpoint_status).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 9 ans
- Fichier 0001-sql-add-concerned_roles-and-is_endpoint-columns-to-v.patch 0001-sql-add-concerned_roles-and-is_endpoint-columns-to-v.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 9 ans
- Fichier 0001-sql-add-concerned_roles-and-is_at_endpoint-columns-t.patch 0001-sql-add-concerned_roles-and-is_at_endpoint-columns-t.patch ajouté
Changement de libellé de colonne, is_endpoint → is_at_endpoint.
Mis à jour par Thomas Noël il y a presque 9 ans
Ça me semble bien ok.
Juste sur la fin, lors du store d'un workflow, j'aurais remplacé le if old_endpoints != set([x.id for x in self.get_endpoint_status()]):
effectué pour chaque formdef par un if update_endpoints:
, le booléen update_endpoints
étant calculé au début du store (car finalement le old_endpoints on ne s'en sert jamais).
Mis à jour par Thomas Noël il y a presque 9 ans
(et en fait, j'ai tatassé ça, parce que le # instruct all formdefs to update their security rules
il n'est plus exact)
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Fichier 0001-sql-add-concerned_roles-and-is_at_endpoint-columns-t.patch 0001-sql-add-concerned_roles-and-is_at_endpoint-columns-t.patch ajouté
Voilà le patch revu.
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit fb32cc5f38a5136112d0433e0eecedae7b6997eb Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Jul 23 18:11:20 2015 +0200 sql: add concerned_roles and is_at_endpoint columns to views (#7925)
sql: add concerned_roles and is_at_endpoint columns to views (#7925)