ckanext/dcatdonl/ValuelistUpdater.py

@@ -27,7 +27,7 @@ _VALUELISTS = {
     'mdr_filetype_nal':                 'https://waardelijsten.dcat-ap-donl.nl/mdr_filetype_nal.json',
     'overheid_dataset_status':          'https://waardelijsten.dcat-ap-donl.nl/overheid_dataset_status.json',
     'overheid_frequency':               'https://waardelijsten.dcat-ap-donl.nl/overheid_frequency.json',
-    #'overheid_license':                 'https://waardelijsten.dcat-ap-donl.nl/overheid_license.json',
+    'overheid_license':                 'https://waardelijsten.dcat-ap-donl.nl/overheid_license.json',
     'overheid_openbaarheidsniveau':     'https://waardelijsten.dcat-ap-donl.nl/overheid_openbaarheidsniveau.json',
     'overheid_spatial_scheme':          'https://waardelijsten.dcat-ap-donl.nl/overheid_spatial_scheme.json',
     'overheid_spatial_gemeente':        'https://waardelijsten.dcat-ap-donl.nl/overheid_spatial_gemeente.json',
@@ -36,8 +36,6 @@ _VALUELISTS = {
     'overheid_spatial_waterschap':      'https://waardelijsten.dcat-ap-donl.nl/overheid_spatial_waterschap.json',
     'overheid_taxonomiebeleidsagenda':  'https://waardelijsten.dcat-ap-donl.nl/overheid_taxonomiebeleidsagenda.json'
 }
-# We skip overheid_license since it requires a full CKAN restart for changes in that file to
-# take effect.
 
 
 def update_valuelist(name, online_resource):

ckanext/dcatdonl/resources/overheid_dataset_status.json

@@ -13,14 +13,14 @@
             "en-UK": "Planned"
         }
     },
-    "http://data.overheid.nl/status/in onderzoek": {
+    "http://data.overheid.nl/status/in_onderzoek": {
         "code": "in onderzoek",
         "labels": {
             "nl-NL": "In onderzoek",
             "en-UK": "Being researched"
         }
     },
-    "http://data.overheid.nl/status/niet beschikbaar": {
+    "http://data.overheid.nl/status/niet_beschikbaar": {
         "code": "niet beschikbaar",
         "labels": {
             "nl-NL": "Niet schikbaar",

ckanext/dcatdonl/validator/uri_validator.py

@@ -12,11 +12,11 @@ def is_uri(key, data, errors, context):
     Checks if a given value is a valid URI. Will throw a ckan.plugins.toolkit.Invalid exception when
     it cannot.
 
-    :param key:
-    :param data:
-    :param errors:
-    :param context:
-    :return:
+    :param key: tuple
+    :param data: dict
+    :param errors: dict
+    :param context: dict
+    :return: tuple, dict, dict, dict
     """
     value = data[key]
 
@@ -26,12 +26,12 @@ def is_uri(key, data, errors, context):
     if isinstance(value, list):
         for uri in value:
             if not _valid_uri(uri):
-                errors[key].append('value ' + uri + 'is not a valid uri')
+                errors[key].append('value ' + uri + ' is not a valid uri')
 
         return key, data, errors, context
 
     if not _valid_uri(value):
-        errors[key].append('value ' + value + 'is not a valid uri')
+        errors[key].append('value ' + value + ' is not a valid uri')
 
     return key, data, errors, context
 
@@ -40,9 +40,9 @@ def _valid_uri(uri):
     """
     Validates a given uri
 
-    :param uri:
-    :return:
+    :param uri: string
+    :return: bool
     """
     parsed = urlparse(uri)
 
-    return all([parsed.scheme, parsed.netloc, parsed.path])
+    return all([parsed.scheme, parsed.netloc])