diff --git a/ckanext/odsh/validation.py b/ckanext/odsh/validation.py index f57355077335e2b1391c97e69e082b1c5824bfc2..2131bfa4947c96af61e82c6b1f09cbe12fa7fe99 100644 --- a/ckanext/odsh/validation.py +++ b/ckanext/odsh/validation.py @@ -2,23 +2,15 @@ import logging import csv import re -import urllib.request -import urllib.error -import urllib.parse import json import ckan.logic as logic - -from itertools import count -from dateutil.parser import parse - import ckan.plugins.toolkit as toolkit import ckan.model as model +import pkg_resources +from dateutil.parser import parse from ckan.lib.navl.dictization_functions import Missing - from ckanext.odsh.helpers import get_package_dict from ckanext.odsh.helpers import odsh_resource_formats -import ckan.plugins.toolkit as tk -import pkg_resources _ = toolkit._ @@ -76,14 +68,14 @@ def validate_extras(key, data, errors, context): harvesting = ('ignore_auth' in context) and ( context['ignore_auth'] == True) owner_org = data[('owner_org',)] - lenient_with = tk.config.get('ckanext.odsh.lenient_with', '') + lenient_with = toolkit.config.get('ckanext.odsh.lenient_with', '') is_optional_temporal_start = toolkit.asbool( - tk.config.get('ckanext.odsh.is_optional_temporal_start', False) + toolkit.config.get('ckanext.odsh.is_optional_temporal_start', False) ) or (harvesting and (owner_org in lenient_with)) require_at_least_one_category = toolkit.asbool( - tk.config.get('ckanext.odsh.require_at_least_one_category', False) + toolkit.config.get('ckanext.odsh.require_at_least_one_category', False) ) validate_extra_groups( data=data, @@ -195,7 +187,7 @@ def known_spatial_uri(key, data, errors, context): _copy_spatial_uri_temp_to_extras(data) value = _extract_value(data, 'spatial_uri') require_spatial_uri = toolkit.asbool( - tk.config.get('ckanext.odsh.require_spatial_uri', False) + toolkit.config.get('ckanext.odsh.require_spatial_uri', False) ) error_message_spatial_uri_empty = 'spatial_uri: empty not allowed' @@ -212,7 +204,7 @@ def known_spatial_uri(key, data, errors, context): has_old_uri = old_uri != None and len(old_uri) > 0 if not poly: poly = pkg.extras.get('spatial', None) - if (not poly) and require_spatial_uri: + if not poly and require_spatial_uri: raise toolkit.Invalid(error_message_spatial_uri_empty) # if has_old_uri and require_spatial_uri: # raise toolkit.Invalid(error_message_spatial_uri_empty) @@ -224,8 +216,8 @@ def known_spatial_uri(key, data, errors, context): return extension_path = pkg_resources.resource_filename('ckanext.odsh', '') - mapping_path = tk.config.get('ckanext.odsh.spatial.mapping', - extension_path + '/resources/schleswig-holstein_geojson.csv') + mapping_path = toolkit.config.get('ckanext.odsh.spatial.mapping', + extension_path + '/resources/schleswig-holstein_geojson.csv') not_found = True spatial_text = str() @@ -262,7 +254,8 @@ def _copy_spatial_uri_temp_to_extras(data): from the user interface to extras ''' extras_data = data.get(('__extras',)) - spatial_uri = extras_data.get('spatial_uri_temp') or extras_data.get('spatial_url_temp') + spatial_uri = extras_data.get( + 'spatial_uri_temp') or extras_data.get('spatial_url_temp') if _extract_value(data, 'spatial_uri') is None: next_index = next_extra_index(data)