Merge pull request #9 from LukeTully/master

Corrected duplicate resource name count
frictionlessdata · Jul 18, 2018 · c87e07d · c87e07d
2 parents ccb52d3 + d67741e
commit c87e07d
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/ckan_datapackage_tools/converter.py b/ckan_datapackage_tools/converter.py
@@ -79,8 +79,9 @@ def dataset_to_datapackage(dataset_dict):
     names = {}
     for resource in dp.get('resources', []):
         if resource['name'] in names.keys():
-            resource['name'] = resource['name'] + str(names[resource['name']])
-            names[resource['name']] += 1
+            old_resource_name = resource['name']
+            resource['name'] = resource['name'] + str(names[old_resource_name])
+            names[old_resource_name] += 1
         else:
             names[resource['name']] = 0
 
@@ -131,7 +132,7 @@ def _datapackage_resource_to_ckan_resource(resource):
     elif resource.inline:
         resource_dict['data'] = resource.source
     else:
-        raise NotImplemented('Multipart resources not yet supported')
+        raise NotImplementedError('Multipart resources not yet supported')
 
     if resource.descriptor.get('description'):
         resource_dict['description'] = resource.descriptor['description']