Projet

Général

Profil

Development #8344

csv: permettre des recherches insensible à la casse

Ajouté par Serghei Mihai il y a plus de 8 ans. Mis à jour il y a environ 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
24 septembre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

avec un parametre ?insensitive ou du genre


Fichiers

Révisions associées

Révision d7e7654d (diff)
Ajouté par Serghei Mihai il y a plus de 8 ans

csv: handle case insensitive text search (#8344)

Révision 74260b60 (diff)
Ajouté par Serghei Mihai il y a plus de 8 ans

csv: remove test of string data filter (#8344)

Historique

#1

Mis à jour par Serghei Mihai il y a plus de 8 ans

Patch supplémentaire permettant de comparer le filter et le texte en unicode ainsi que la comparaison case-insensitive

#2

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.

#3

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.

#4

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
#5

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.

#6

Mis à jour par Serghei Mihai il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à En cours
#7

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 :)

#8

Mis à jour par Serghei Mihai il y a environ 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF