Projet

Général

Profil

Bug #36988

API listant les démarches, quand backoffice-submission=on est passé, ne pas retourner toutes les démarches, même pour les admins.

Ajouté par Brice Mallet il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

En BO remontent les formulaires qui ont au moins un "Rôle pour la saisie backoffice" spécifié, si un agent tente la saisie alors qu'il n'a pas un rôle l'autorisant : erreur.

Exemple sur https://agents-meuse.test.entrouvert.org/

Fichiers

0001-api-do-not-advertise-all-forms-for-backoffice-submis.patch (2,53 ko) 0001-api-do-not-advertise-all-forms-for-backoffice-submis.patch Frédéric Péters, 16 octobre 2019 16:20
0001-api-do-not-advertise-all-forms-for-backoffice-submis.patch (2,06 ko) 0001-api-do-not-advertise-all-forms-for-backoffice-submis.patch Frédéric Péters, 17 octobre 2019 10:36

Révisions associées

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

api: do not advertise all forms for backoffice submission to admins (#36988)

Historique

#1

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

Tu peux filer toutes les infos d'accès y compris identifiant et mot de passe pour regarder ?

#2

Mis à jour par Brice Mallet il y a plus de 4 ans

J'ai été trop rapide dans mon analyse, je reprends :
  • "Contact" n'apparaît pas pour un compte n'ayant comme seul rôle "Agent (exemple : / xxx) : OK
  • "Contact" apparaît pour un compte ayant un rôle "Administrateur fonctionnel", sans avoir le rôle "Accueil" (exemple : / xxx) : KO
    Le problème est donc uniquement pour les administrateurs de démarches et non pour les agents traitants, ce qui du coup est bien moins gênant.
#3

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

  • Projet changé de Combo à w.c.s.
  • Sujet changé de Erreur sur tentative de saisie BO d'un formulaire, sans en avoir les droits à API listant les démarches, quand backoffice-submission=on est passé, ne pas retourner toutes les démarches, même pour les admins.

Oui tout est retourné pour les administrateurs; c'est le comportement de l'API côté w.c.s.; je viens d'écrire tout ça dans l'intitulé.

#4

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

  • Assigné à mis à Frédéric Péters
#5

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

Petite hésitation de dernière minute par rapport à welco mais d'un rapide test local c'est ok.

#6

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

A priori cette partie devient inutile :

        if backoffice_submission:
            list_all_forms = True

car en mode backoffice_submission on ne regarde plus list_all_forms.

#8

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

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit b876213f441bbdbad5543c4f57fdbb5dd0a85c9f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Wed Oct 16 16:17:35 2019 +0200

    api: do not advertise all forms for backoffice submission to admins (#36988)
#10

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

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

Formats disponibles : Atom PDF