From 0688b63d6bbe5cedf9e8d9172562fe406d2c8f3b Mon Sep 17 00:00:00 2001 From: avdata99 Date: Mon, 21 Aug 2023 13:45:53 -0300 Subject: [PATCH] Avoid CKAN 2.10 deprecation --- ckanext/hierarchy/groups_hierarchy/group/read.html | 2 +- ckanext/hierarchy/helpers.py | 13 ++++++++++--- ckanext/hierarchy/templates/organization/read.html | 2 +- 3 files changed, 12 insertions(+), 5 deletions(-) diff --git a/ckanext/hierarchy/groups_hierarchy/group/read.html b/ckanext/hierarchy/groups_hierarchy/group/read.html index e8e1fc10..f6b5d8ac 100644 --- a/ckanext/hierarchy/groups_hierarchy/group/read.html +++ b/ckanext/hierarchy/groups_hierarchy/group/read.html @@ -27,5 +27,5 @@ (_('Last Modified'), 'metadata_modified desc'), (_('Popular'), 'views_recent desc') if g.tracking_enabled else (false, false) ] %} - {% snippet 'snippets/search_form.html', form_id='group-datasets-search-form', type='dataset', query=q, sorting=sorting, sorting_selected=sort_by_selected, count=page.item_count, facets=facets, placeholder=_('Search datasets...'), show_empty=request.params, fields=fields, include_children_option=true %} + {% snippet 'snippets/search_form.html', form_id='group-datasets-search-form', type='dataset', query=q, sorting=sorting, sorting_selected=sort_by_selected, count=page.item_count, facets=facets, placeholder=_('Search datasets...'), show_empty=request.args, fields=fields, include_children_option=true %} {% endblock %} diff --git a/ckanext/hierarchy/helpers.py b/ckanext/hierarchy/helpers.py index dc5c09a2..ed6bd2bc 100644 --- a/ckanext/hierarchy/helpers.py +++ b/ckanext/hierarchy/helpers.py @@ -1,6 +1,6 @@ import ckan.plugins as p import ckan.model as model -from ckan.common import request, is_flask_request +from ckan.common import request def group_tree(organizations=[], type_='organization'): @@ -115,7 +115,14 @@ def get_allowable_parent_groups(group_id): def is_include_children_selected(): include_children_selected = False - if is_flask_request(): - if request.params.get('include_children'): + + if p.toolkit.check_ckan_version(min_version="2.10"): + is_flask = True + else: + from ckan.common import is_flask_request + is_flask = is_flask_request() + + if is_flask: + if request.args.get('include_children'): include_children_selected = True return include_children_selected diff --git a/ckanext/hierarchy/templates/organization/read.html b/ckanext/hierarchy/templates/organization/read.html index ca786d5f..c60d6d12 100644 --- a/ckanext/hierarchy/templates/organization/read.html +++ b/ckanext/hierarchy/templates/organization/read.html @@ -27,5 +27,5 @@ (_('Last Modified'), 'metadata_modified desc'), (_('Popular'), 'views_recent desc') if g.tracking_enabled else (false, false) ] %} - {% snippet 'snippets/search_form.html', form_id='organization-datasets-search-form', type='dataset', query=q, sorting=sorting, sorting_selected=sort_by_selected, count=page.item_count, facets=facets, placeholder=_('Search datasets...'), show_empty=request.params, fields=fields, include_children_option=true %} + {% snippet 'snippets/search_form.html', form_id='organization-datasets-search-form', type='dataset', query=q, sorting=sorting, sorting_selected=sort_by_selected, count=page.item_count, facets=facets, placeholder=_('Search datasets...'), show_empty=request.args, fields=fields, include_children_option=true %} {% endblock %}