Bug #13841
csvdatasource: le traitement des filtres est incohérent
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
03 novembre 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
On a dit qu'on faisait un ET entre tous filtres.
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
Mis à jour par Frédéric Péters il y a plus de 7 ans
Dans le test, je trouverais plus clair d'écrire @'\n'.join(['int(id) <= 525', 'int(id) >= 511']), que les deux conditions apparaissent bien séparées.
Retirer le print.
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
- Fichier 0001-csvdatasource-fix-query-s-filters-processing-fixes-1.patch 0001-csvdatasource-fix-query-s-filters-processing-fixes-1.patch ajouté
Patch à jour par rapport à master.
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
- Fichier 0001-csvdatasource-fix-query-s-filters-processing-fixes-1.patch 0001-csvdatasource-fix-query-s-filters-processing-fixes-1.patch ajouté
Dsl, voilà qui est fait.
diff --git a/tests/test_csv_datasource.py b/tests/test_csv_datasource.py
index 22c73ef..9b63c20 100644
--- a/tests/test_csv_datasource.py
+++ b/tests/test_csv_datasource.py
@@ -559,9 +559,8 @@ def test_query_filter_multiline(app, setup):
'endpoint': 'query/query-1_/',
})
query = Query(slug='query-1_', resource=csvdata)
- query.filters = 'int(id) <= 525\nint(id) >= 511'
+ query.filters = '\n'.join(['int(id) <= 525', 'int(id) >= 511'])
query.save()
response = app.get(url)
- print response.json
assert response.json['err'] == 0
assert len(response.json['data']) == 2
J'ai aussi mis à jour le test qui ne passait plus avec le changement de signature de setup().
Mis à jour par Benjamin Dauvergne il y a plus de 7 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit e84d15c8ca58d38be49ac849d781d53a9fac7a0a.
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
csvdatasource: fix query's filters processing (fixes #13841)
All filters are implicitely ANDed together.