Development #27984
csvdatasource: filtre 'q' insensible à la case par défaut
Début:
14 novembre 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Pour ne plus à avoir à passer le paramètre "case-insensitive" dans les appels
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans
- Fichier 0001-csvdatasource-make-q-filter-case-insensitive-by-defa.patch 0001-csvdatasource-make-q-filter-case-insensitive-by-defa.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans
- Lié à Development #26188: csvdatasource: recherche insensible aux accents ajouté
Mis à jour par Frédéric Péters il y a plus de 5 ans
Je laisserais certains tests passant case-insensitive, pour être sûr que ça ne plante pas dans cette situation.
Tant qu'à toucher cette ligne,
filters = ["query['q'].lower() in text.lower()"]je la modifierais pour ne pas avoir à faire query['q'].lower() une fois par ligne. ("%s in text.lower()" % repr(request.GET['q'].lower()), genre, pas testé).
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans
- Fichier 0001-csvdatasource-make-q-filter-case-insensitive-by-defa.patch 0001-csvdatasource-make-q-filter-case-insensitive-by-defa.patch ajouté
Ok.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Bon, j'ai du mater le mauvais patch, ok.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8213169a3ecc936d217dbb3821e33e6f7c6743c9
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Wed Nov 14 11:11:42 2018 +0100
csvdatasource: make 'q' filter case-insensitive by default (#27984)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
csvdatasource: make 'q' filter case-insensitive by default (#27984)