Removed unused imports and minor code formatting

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,7 +216,7 @@ 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',
+    mapping_path = toolkit.config.get('ckanext.odsh.spatial.mapping',
                                       extension_path + '/resources/schleswig-holstein_geojson.csv')
 
     not_found = True
@@ -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)