Projet

Général

Profil

Development #58671

Ajouter un raccourci vers la page de conception du formulaire/du modèle de fiche depuis la vue traitement/fiches

Ajouté par Lauréline Guérin il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
17 novembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

dans #54935 on a ajouté un lien de la page de config d'un form/carddef vers la page de management
faire l'inverse: ajouter un lien depuis la page de management vers la page de config du form/carddef


Fichiers

Révisions associées

Révision 56b03ef7 (diff)
Ajouté par Lauréline Guérin il y a plus de 2 ans

management: add link to form/carddef page in sidebar (#58671)

Historique

#1

Mis à jour par Lauréline Guérin il y a plus de 2 ans

#3

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

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

Rouge.

#5

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

"Form page" ça va être un peu ambigu à traduire, on pourrait avoir un plus explicite "Form definition page" ?

Je verrais aussi un lien vers le workflow... parce que c'est souvent lui qu'on va voir quand ça dérape (mais je peux me tromper et ça peut être un autre ticket).

#7

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

Je vais être relou, je viens de relire inspect dans wcs/backoffice/management.py car on y affiche déjà ces liens, et on a les intitulés déjà traduits :
  • "View Form"
  • "View Card"
  • "View Workflow"

Je propose d'utiliser ça, donc.

Ensuite sur la gestion des droits, je ne connais pas encore assez bien la mécanique de filtrage is_global_accessible que je ne vois pas dans l'inspect, donc je vais rerererelire et te dire ça... plus tard :/

#8

Mis à jour par Lauréline Guérin il y a plus de 2 ans

dans wcs/admin/workflows:

def is_global_accessible():
    return get_publisher().get_backoffice_root().is_global_accessible('workflows')

wording modifié

j'ai simplifié la méthode has_user_access: plus de param user, on va chercher get_request().user (get_publisher().get_backoffice_root().is_global_accessible travaille déjà sur ce user)

(j'aurais bien posé has_user_access ailleurs que sur Workflow)

#9

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

Comme expliqué (en direct), sur le code de vérification de l'accès, je préfère avoir l’œil de Frédéric...

#10

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

+    print(resp)

(print égaré dans un test)

~~

+        if self.id in ('_default', '_carddef_default'):
+            return True

Je ne reprendrais pas ça, dans l'idée que ça va afficher pour tout le monde le lien "Voir le workflow" mais que les personnes sans accès du tout aux workflows ça va leur donner un accès interdit.

~~

+        if get_publisher().get_backoffice_root().is_global_accessible(self.formdef.backoffice_section):

alors que pour le lien workflow,

+        if self.formdef.workflow.has_user_access():

Ça fait pour moi que le lien "voir le formulaire" ne s'affichera qu'aux personnes qui ont un accès total à la page des formulaires, pas un accès restreint. (alors que pour les workflows c'est bien).

À suggérer de faire pareil une méthode has_user_access pour formdef je vois que ça existe déjà, is_managed_by(); pour le coup donc il y aurait moyen de l'utiliser ici + renommer le has_user_access pour rester sur un seul nom.

#12

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

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

Mis à jour par Lauréline Guérin il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 56b03ef7203a28d23e06eff5fa208f8dd0ea6639
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Thu Dec 9 16:11:53 2021 +0100

    management: add link to form/carddef page in sidebar (#58671)
#14

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

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

Mis à jour par Transition automatique il y a environ 2 ans

Automatic expiration

Formats disponibles : Atom PDF