0001-csvdatasource-remove-advanced-lookup-filters-13748.patch

Serghei Mihai, 13 novembre 2018 16:36

Voir les différences: en ligne côte à côte

Subject: [PATCH] csvdatasource: remove advanced lookup filters (#13748)

 passerelle/apps/csvdatasource/models.py | 31 +++++++++++-----------
 passerelle/apps/csvdatasource/views.py  | 31 +++++-----------------
 tests/test_csv_datasource.py            | 35 +------------------------
 3 files changed, 23 insertions(+), 74 deletions(-)

     from passerelle.utils.jsonresponse import APIError
     from passerelle.utils.api import endpoint
     import lookups
     identifier_re = re.compile(r"^[^\d\W]\w*\Z", re.UNICODE)
-...
                 for data in self.get_cached_rows(initial=False):
                     yield data
         def get_data(self, filters=None):
         def get_data(self, filters={}, case_insensitive=False):
             titles = [t.strip() for t in self.columns_keynames.split(',')]
             # validate filters (appropriate columns must exist)
             if filters:
                 for filter_key in filters.keys():
                     if not filter_key.split(lookups.DELIMITER)[0] in titles:
                         del filters[filter_key]
             for filter_key in filters.keys():
                 # allow 'q' filter
                 if filter_key == 'q':
                     continue
                 if filter_key not in titles:
                     del filters[filter_key]
             rows = self.get_cached_rows()
             data = []
             # build a generator of all filters
             def filters_generator(filters, titles):
             def filters_generator(filters, case_insensitive):
                 if not filters:
                     return
                 for key, value in filters.items():
                     try:
                         key, op = key.split(lookups.DELIMITER)
                     except (ValueError,):
                         op = 'eq'
                     yield lookups.get_lookup(op, key, value)
                     def operation(key, value, case_insensitive):
                         if key == 'q':
                             return lambda x: value.lower() in x['text'].lower()
                         if case_insensitive:
                             return lambda x: value.lower() == x[key].lower()
                         return lambda x: value == x[key]
                     yield operation(key, value, case_insensitive)
             # apply filters to data
             def super_filter(filters, data):
-...
                 return data
             data = list(super_filter(
                 filters_generator(filters, titles), rows
                 filters_generator(filters, case_insensitive), rows
             ))
             return data

         def _filters_builder(self, request):
             filters = {}
             obj = self.get_object()
             params = request.GET
             case_insensitive = 'case-insensitive' in params
             query = params.get('q', None)
             query = request.GET.get('q', None)
             if query:
                 if case_insensitive:
                     filters['text__icontains'] = query.lower()
                 else:
                     filters['text__contains'] = query
                 filters['q'] = query
             # builds filters according to csv file header
             for column_title in [t.strip() for t in obj.columns_keynames.split(',') if t]:
                 match = filter(
                     (lambda ct: lambda x: x.startswith(ct))(column_title), params.keys()
+                )
                 for key in match:
                     if case_insensitive:
                         filters[key + '__ieq'] = params[key].lower()
                     else:
                         filters[key] = params[key]
             if 'text' in filters:
                 if case_insensitive:
                     filters['text__ieq'] = filters['text'].lower()
                 else:
                     filters['text__eq'] = filters['text']
                 filters.pop('text')
                 if column_title in params.keys():
                     filters[column_title] = params[column_title]
             return filters
-...
         def get(self, request, *args, **kwargs):
             obj = self.get_object()
             filters = self._filters_builder(request)
             return {'data': obj.get_data(filters)}
             case_insensitive = 'case-insensitive' in request.GET
             return {'data': obj.get_data(filters, case_insensitive)}
     class NewQueryView(CreateView):

         assert result[0]['text'] == 'Eliot'
         assert len(result) == 1
     def test_advanced_filters(client, setup, filetype):
         csvdata, url = setup(filename=filetype, data=get_file_content(filetype))
         filters = {'id__gt':20, 'id__lt': 40}
         resp = client.get(url, filters)
         result = parse_response(resp)
         assert len(result) == 3
         for stuff in result:
             assert stuff['id'] in ('22', '36', '38')
     def test_advanced_filters_combo(client, setup, filetype):
         csvdata, url = setup(filename=filetype, data=get_file_content(filetype))
         filters = {
             'id__ge': '20',
             'id__lt': '40',
             'fam__gt': '234',
             'fam__le': '235',
             'fname__icontains': 'Sandra'
+        }
         resp = client.get(url, filters)
         result = parse_response(resp)
         assert len(result) == 1
         assert result[0]['id'] == '22'
         assert result[0]['lname'] == 'MARTIN'
     def test_unknown_operator(client, setup, filetype):
         csvdata, url = setup(filename=filetype, data=get_file_content(filetype))
         filters = {'id__whatever': '25', 'fname__icontains':'Eliot'}
         resp = client.get(url, filters)
         result = json.loads(resp.content)
         assert result['err'] == 1
         assert result['err_class'] == 'passerelle.apps.csvdatasource.lookups.InvalidOperatorError'
         assert result['err_desc'] == 'whatever is not a valid operator'
     def test_dialect(client, setup):
         csvdata, url = setup(data=data)
-...
+        }
         assert expected == csvdata.dialect_options
         filters = {'id__gt': '20', 'id__lt': '40', 'fname__icontains': 'Sandra'}
         filters = {'id': '22', 'fname': 'Sandra'}
         resp = client.get(url, filters)
         result = parse_response(resp)
         assert len(result) == 1
+    -

Projet

Général

Profil

Produits Entr'ouvert » Passerelle

0001-csvdatasource-remove-advanced-lookup-filters-13748.patch