Développement #60056
Essayer d'afficher les espaces insécables dans l'inspect
Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
23 December 2021
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
Description
Dans #60002#note-7, on remarque qu'on fait l'effort de traduire les messages d'erreur renvoyés par chrono mais dans l'inspect ça donne des «\xa0...\xa0» et c'est dommage.
Mais peut-être qu'en dehors de ce contexte « message d'erreur en français » on a envie de distinguer les espaces insécables, auquel cas ce ticket peut s'avérer complexe.
Files
Associated revisions
History
Updated by Frédéric Péters almost 3 years ago
Pour reprendre de #60002#note-7,
['Il est nécessaire de passer un paramètre «\xa0agendas\xa0» ou «\xa0subscribed\xa0».'] (list)
i.e. ça concerne l'affichage d'éléments de type liste (ce qui amène python à un repr() de chacun de ses éléments) et le repr() d'une chaine et bien il échappe l'espace insécable et autres.
Sans tout chambouler et après avoir testé différentes choses, je serais pour juste viser le cas précis ici, quelque chose comme :
r += htmltext('<li>%s
') % (k, k)
- r = htmltext(' <div class="value"><span>%s</span>') % ellipsize(safe(v), 10000)
if isinstance(v, list):
+ # custom behaviour for lists
+ r = htmltext('<div class="value"><span>[')
def custom_repr(var):
+ if isinstance(var, str):
+ var = var.replace('\xa0', ' ') # avoid escaped non breaking space
+ return repr(var)
+ r = ', '.join(custom_repr(x) for x in v)
r = htmltext(']</span>')
else:
+ r += htmltext(' <div class="value"><span>%s</span>') % ellipsize(safe(v), 10000)
Updated by Frédéric Péters almost 3 years ago
- File 0001-backoffice-avoid-escaping-of-non-breaking-spaces-in-.patch 0001-backoffice-avoid-escaping-of-non-breaking-spaces-in-.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Thomas Noël (congés → 5 décembre) almost 3 years ago
- Status changed from Solution proposée to Solution validée
Updated by Frédéric Péters almost 3 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 1530f58646c5f0d3e402e585e47b5b7cd78e4ed4 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Dec 24 12:32:22 2021 +0100 backoffice: avoid escaping of non-breaking spaces in inspect page (#60056)
Updated by Frédéric Péters almost 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
backoffice: avoid escaping of non-breaking spaces in inspect page (#60056)