Projet

Général

Profil

Development #62145

inspect d'une demande : afficher le nom du/des champ correspondant à une variable

Ajouté par Thomas Noël il y a environ 2 ans. Mis à jour il y a environ 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
24 février 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Dans l'inspect d'une demande, on a l'affichage des form_var_xxx. Pour savoir à quel champ ça correspond, ce n'est pas toujours évident.

On gagnerait souvent pas mal de temps à voir, à côté ou en dessous du form_var_xxx, l'indication du ou des champs correspondants dans le formulaire et/ou le workflow (donnée de traitement).

Quelque chose comme :

-------------------------------------------
form_var_date_rdv                24/02/2022
<i>Date du rendez-vous</i>
-------------------------------------------

Avec pourquoi pas un lien vers le/les champs.


Fichiers


Demandes liées

Lié à w.c.s. - Development #59897: Amélioration de la navigation sur les données de la page inspect des demandes/fichesFermé20 décembre 2021

Actions
Lié à w.c.s. - Bug #62998: Le lien dans l'inspect vers un champ d'un bloc de champ renvoie une 404Fermé21 mars 2022

Actions

Révisions associées

Révision 6a5b8ae9 (diff)
Ajouté par Lauréline Guérin il y a environ 2 ans

backoffice: display field label and url in formdata inspect page (#62145)

Historique

#1

Mis à jour par Marie Kuntz -> retour le 13 mai il y a environ 2 ans

  • Lié à Development #59897: Amélioration de la navigation sur les données de la page inspect des demandes/fiches ajouté
#2

Mis à jour par Stéphane Laget il y a environ 2 ans

Mais sans retour à la ligne si possible, avoir l'info à la suite, sur la même ligne

#3

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

  • Fichier 0001-backoffice-display-field-label-and-url-in-formdata-i.patch ajouté
  • Statut changé de Nouveau à Solution proposée
  • Assigné à mis à Lauréline Guérin
  • Patch proposed changé de Non à Oui
#4

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

Mais sans retour à la ligne si possible, avoir l'info à la suite, sur la même ligne

Je ne suis pas sûre que ce soit une bonne idée: le varname du champ (exemple form_var_date_rdv) est limité à 25% de la largeur disponible, avec ellipse si ça dépasse. On risque d'avoir le label du champ masqué et embarqué dans l'ellipse.

capture d'écran en pièce jointe, mais c'est plutôt moche ainsi

#6

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

  • Fichier 0001-backoffice-display-field-label-and-url-in-formdata-i.patch supprimé
#7

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

Perso pas enthousiaste (j'aurais dû l'écrire plus tôt), ça prend beaucoup de place; je serais éventuellement pour juste avoir après le nom de la variable un mini-lien (pas de super idée, https://fontawesome.com/v4/icon/eye ?) avec le libellé en attribut title.

form_var_adresse <a title="Adresse" href="...">&f06e;</a>

+ côté css, ul.form-inspector li code a { font-family: FontAwesome; }.

#8

Mis à jour par Pierre Cros il y a environ 2 ans

Pour répondre aux problèmes de place : mettre le lien vers le champ directement sur la variable ? Connaître le libellé ne me semble pas primordial sur cet écran (et on aurait l'info dès qu'on clique) (edit : j'ai écrit ça avant de comprendre la proposition de Fred, assez voisine)

#9

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

Oui, assez voisin, sur le lien intégral je n'ai pas proposé car ça va compliquer les copié·collés.

#10

Mis à jour par Pierre Cros il y a environ 2 ans

Mouaif, double-clic dans la cellule (copié) + clic molette (collé), ça fonctionnait toujours dans mon idée.

Mais effectivement si pour des raisons d'accessibilité ou autre il faut qu'un clic sur la cellule active le lien alors je préfère ta solution.

#11

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

Mouaif, double-clic dans la cellule (copié)

et envoyé sur la page du champ. (je rate un truc)

#14

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

C'était pas trop mon besoin (je voulais surtout voir le nom du champ) mais c'est bien quand même, peut-être mieux...! J'imagine que le "title" ça permet de passer la souris dessus pour voir le libellé ?

#15

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

J'imagine que le "title" ça permet de passer la souris dessus pour voir le libellé ?

Oui

#16

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

Sur :

if has_bo_formdef and v._field.id.startswith('bo'):

je pense que le "has_bo_formdef" ne sert à rien : s'il y a un champ "bo" à afficher dans l'inspect, c'est parce qu'il vient du formdef.workflow.backoffice_fields_formdef (dans l'inspect on n'affiche que les champs qui existent dans le formulaire et les données de traitement).

#17

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

OK j'ai enlevé la condition.
Mais si ça plante tu feras le hotfix ;)

#18

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

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

Promis :)

En réalité ma reflexion venait au départ du fait que tu faisais :

if has_bo_formdef and v._field.id.startswith('bo'):
   field_url = ...
elif access_to_formdef:
   field_url = ...

et je m'étais dit "ah mais si has_bo_formdef est faux et que c'est une variable qui commence par "bo" ça risque de tomber dans le second cas" pour finalement me dire que has_bo_formdef n'était pas nécessaire car sous-entendu dans le v._field.id.startswith('bo') et voilà j'ai validé et j'arrête de parler :)

#19

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 6a5b8ae9c0b0eef12edcc1af10023c592cd23262
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date:   Mon Mar 14 16:58:40 2022 +0100

    backoffice: display field label and url in formdata inspect page (#62145)
#20

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

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

Mis à jour par Marie Kuntz -> retour le 13 mai il y a environ 2 ans

  • Lié à Bug #62998: Le lien dans l'inspect vers un champ d'un bloc de champ renvoie une 404 ajouté
#22

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

Automatic expiration

Formats disponibles : Atom PDF