cleanse data group/org name for datagov
cleanse data group/org name for datagov


Former-commit-id: 864798958ce17be8aab36096ea9ed641e6415467

--- a/documents/datagov-export-groups.py
+++ b/documents/datagov-export-groups.py
@@ -39,41 +39,43 @@
     groups = {}
     for doc in docsdb.view('app/datasetGroups'):
             group_name = doc.key
-            pkg_name = filter(lambda x: x in '0123456789abcdefghijklmnopqrstuvwxyz-_',
-                              doc.value.replace("http://data.gov.au/dataset/", '').replace('/', '')[:100]);
-            if group_name in groups.keys():
-                groups[group_name] = list(set(groups[group_name] + [pkg_name]))
-            else:
-                groups[group_name] = [pkg_name]
+            if group_name != "Not specified":
+                pkg_name = filter(lambda x: x in '0123456789abcdefghijklmnopqrstuvwxyz-_',
+                                  doc.value.replace("http://data.gov.au/dataset/", '').replace('/', '')[:100]);
+                if group_name in groups.keys():
+                    groups[group_name] = list(set(groups[group_name] + [pkg_name]))
+                else:
+                    groups[group_name] = [pkg_name]
 
     # add dataset to group(s)
     for group_name in groups.keys():
-        group_url = name_munge(group_name[:100])
-        print group_name
-        print groups[group_name]
-        try:
-            # Update the group details
-            group_entity = ckan.group_entity_get(group_url)
-            print "group "+group_name+" exists"
-            if 'packages' in group_entity.keys():
-                group_entity['packages'] = list(set(group_entity['packages'] + groups[group_name]))
-            else:
-                group_entity['packages'] = groups[group_name]
-            ckan.group_entity_put(group_entity)
-        except CkanApiError, e:
-            if ckan.last_status == 404:
-                print "group "+group_name+" does not exist, creating"
-                group_entity = {
-                    'name': group_url,
-                    'title': group_name,
-                    'description': group_name,
-                    'packages': groups[group_name]
-                }
-                #print group_entity
-                ckan.group_register_post(group_entity)
-            elif ckan.last_status == 409:
-                print "group already exists"
-            else:
-                raise LoaderError('Unexpected status %s adding to group under \'%s\': %r' % (
-                    ckan.last_status, pkg_name, e.args))
+        if group_name != "Not specified":
+            group_url = name_munge(group_name[:100])
+            print group_name
+            print groups[group_name]
+            try:
+                # Update the group details
+                group_entity = ckan.group_entity_get(group_url)
+                print "group "+group_name+" exists"
+                if 'packages' in group_entity.keys():
+                    group_entity['packages'] = list(set(group_entity['packages'] + groups[group_name]))
+                else:
+                    group_entity['packages'] = groups[group_name]
+                ckan.group_entity_put(group_entity)
+            except CkanApiError, e:
+                if ckan.last_status == 404:
+                    print "group "+group_name+" does not exist, creating"
+                    group_entity = {
+                        'name': group_url,
+                        'title': group_name,
+                        'description': group_name,
+                        'packages': groups[group_name]
+                    }
+                    #print group_entity
+                    ckan.group_register_post(group_entity)
+                elif ckan.last_status == 409:
+                    print "group already exists"
+                else:
+                    raise LoaderError('Unexpected status %s adding to group under \'%s\': %r' % (
+                        ckan.last_status, pkg_name, e.args))
 

--- a/documents/datagov-export.py
+++ b/documents/datagov-export.py
@@ -133,15 +133,26 @@
                               doc.value['url'].replace("http://data.gov.au/dataset/", '').replace('/', '')[:100]);
             print pkg_name
             #add to or create organization using direct API
-            org_name = name_munge(doc.value['metadata']["Agency"][:100])
+            agency = doc.value['metadata']["Agency"]
+            if agency == "APS":
+                agency = "Australian Public Service Commission"
+            if agency == "Shared Services, Treasury Directorate":
+                agency = "Shared Services Procurement, Treasury Directorate"
+            if agency == "Treasury - Shared Services":
+                agency = "Shared Services Procurement, Treasury Directorate"
+            if agency == "Territory and Municipal Services (TAMS)":
+                agency = "Territory and Municipal Services Directorate"
+            if agency == "State Library of NSW":
+                agency = "State Library of New South Wales"
+            org_name = name_munge(agency[:100])
             if org_name not in orgs_list:
                 orgs_list = ckandirect.action.organization_list()['result']
                 #print orgs_list
                 if org_name not in orgs_list:
                     try:
                         print "org not found, creating " + org_name
-                        ckandirect.action.organization_create(name=org_name, title=doc.value['metadata']["Agency"],
-                                                              description=doc.value['metadata']["Agency"])
+                        ckandirect.action.organization_create(name=org_name, title=agency,
+                                                              description=agency)
                         orgs_list.append(org_name)
                     except ckanapi.ValidationError, e:
                         print e
@@ -156,6 +167,7 @@
             org_id = orgs_ids[org_name]
             print "org id is "+org_id
             tags = []
+            creator = doc.value['metadata']["DCTERMS.Creator"]
             if doc.value['agencyID'] == "AGIMO":
                 if len(doc.value['metadata']["Keywords / Tags"]) > 0:
                     if hasattr(doc.value['metadata']["Keywords / Tags"], '__iter__'):
@@ -170,15 +182,14 @@
                     'title': doc.value['metadata']['DCTERMS.Title'],
                     'url': doc.value['metadata']['DCTERMS.Source.URI'],
                     'tags': tags, #tags are mandatory?
-                    'author': doc.value['metadata']["DCTERMS.Creator"],
-                    'maintainer': doc.value['metadata']["DCTERMS.Creator"],
+                    'author': creator,
+                    'maintainer': creator,
                     'licence_id': get_licence_id(doc.value['metadata']['DCTERMS.License']),
                     'notes': html2text.html2text(doc.value['metadata']['Description']),
                     'owner_org': org_id
                     #todo add missing key values like jurasdiction
                 }
-            if doc.value['agencyID'] == "qld":
-                package_entity = doc.value['metadata']
+
 
             try:
                 #print package_entity
@@ -191,6 +202,7 @@
                     raise LoaderError('Unexpected status %s checking for package under \'%s\': %r' % (
                         ckan.last_status, pkg_name, e.args))
             pkg = ckan.package_entity_get(pkg_name)
+
 
             # add resources (downloadable data files)
             if 'Download' in doc.value['metadata'].keys():