Development #8344
csv: permettre des recherches insensible à la casse
0%
Description
avec un parametre ?insensitive
ou du genre
Fichiers
Révisions associées
csv: remove test of string data filter (#8344)
Historique
Mis à jour par Serghei Mihai il y a plus de 8 ans
- Fichier 0001-csv-handle-string-filter-criteria.patch 0001-csv-handle-string-filter-criteria.patch ajouté
- Fichier 0001-csv-handle-case-insensitive-text-search-8344.patch 0001-csv-handle-case-insensitive-text-search-8344.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Patch supplémentaire permettant de comparer le filter et le texte en unicode ainsi que la comparaison case-insensitive
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Je ne vois pas comment filter_criterai peut ne pas être de l'unicode, mais soit. Je convertirai filter_criterai en entrée, plutôt que dans le coeur de boucle, mais sinon Ack.
Ack pour le deuxième.
Mis à jour par Serghei Mihai il y a plus de 8 ans
Benjamin Dauvergne a écrit :
Je ne vois pas comment filter_criterai peut ne pas être de l'unicode, mais soit. Je convertirai filter_criterai en entrée, plutôt que dans le coeur de boucle, mais sinon Ack.
filter_criteria peut-être "string" quand get_data
est appelé directement, comme je le fais dans le test. Sinon, effectivement, il est renvoyé par Django en unicode.
Mis à jour par Serghei Mihai il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit d7e7654d065cf54bb0e687f1866a3e0c7a165c6b Author: Serghei Mihai <smihai@entrouvert.com> Date: Thu Sep 24 13:42:13 2015 +0200 csv: handle case insensitive text search (#8344) commit 3a0d3be3ffb89fc881205f77cc30e340fcc38ee4 Author: Serghei Mihai <smihai@entrouvert.com> Date: Thu Sep 24 13:25:23 2015 +0200 csv: handle string filter criteria
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
C'est le test qui est con alors s'il ne reproduit pas le contexte d'appel normal. Donc pas ack le 1, mais tu peux garder les tests en mettant juste "u" devant tes chaînes.
Mis à jour par Serghei Mihai il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à En cours
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Statut changé de En cours à Résolu (à déployer)
C'est vraiment pas important, fais des tests sur les encombrants plutôt :)
csv: handle case insensitive text search (#8344)