Bug #16280
misc.ellipsize avec longueur limite < 5
Début:
11 mai 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Le code fait s[:length-5] et du coup pour une longueur limite < 5, le résultat n'est pas celui attendu.
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Nouveau à Rejeté
semble réglé depuis
commit 8cf415e6645ca1b5816e163ac199b84657ac91b4 (HEAD)
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Nov 12 11:39:43 2019 +0100
misc: add force_str(), to encode in Python 2 (#36515)
def ellipsize(s, length=30):
s = force_text(s, get_publisher().site_charset, errors='replace')
if s and len(s) >= length:
s = s[:length-5] + ' (...)'
return force_str(s)
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Rejeté à Nouveau
- Assigné à mis à Lauréline Guérin
n'importe quoi, je reprends
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Fichier 0001-misc-fix-ellipsize-for-mini-strings-16280.patch 0001-misc-fix-ellipsize-for-mini-strings-16280.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a presque 4 ans
Pour moi la longueur de test en supériorité stricte (ici 5) doit être égale à la longueur de la chaîne d'ellipse (ici 6), parce que là par exemple dans les tests :
('foo bar', 6, 'f (...)'), on remplace par une chaîne de même longueur, c'est dommage.
D'ailleurs, pourquoi ne pas compacter encore plus en utilisant le caractère Unicode prévu à cet effet, i.e. '…' (U+2026) ?
Edit: D'ailleurs (bis), il me semble que le fait que la chaîne a été tronquée serait plus intelligible sans l'espace avant les points de suspension entre parenthèses, non ?
Mis à jour par Lauréline Guérin il y a presque 4 ans
Mis à jour par Lauréline Guérin il y a presque 4 ans
Mis à jour par Paul Marillonnet il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Frédéric Péters il y a presque 4 ans
(moi j'aimais bien l'espace devant, mais vraiment pas important)
Mis à jour par Lauréline Guérin il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit bf52ee1ac7635673445c640ef8b58f4a9e4bcadd
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Fri Jun 19 15:24:52 2020 +0200
misc: fix ellipsize for mini strings (#16280)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: fix ellipsize for mini strings (#16280)