Standardized author name formatting and eliminated code duplication in Zenodo metadata processing.

scripts/auto-add-zenodo-entry.py

@@ -93,7 +93,7 @@ def complete_zenodo_data(zenodo_url):
             entry['description'] = remove_html_tags(metadata['description'])
         if 'creators' in metadata.keys():
             creators = metadata['creators']
-            entry['authors'] = [c['name'] for c in creators]
+            entry['authors'] = [reformat_author(c['name']) for c in creators]
         if 'license' in metadata.keys():
             entry['license'] = metadata['license']['id']
 
@@ -104,6 +104,25 @@ def complete_zenodo_data(zenodo_url):
 
     return entry
 
+def reformat_author(author_name):
+    """
+    Reformat author names from 'Lastname, Firstname' format to 'Firstname Lastname'.
+
+    Parameters
+    ----------
+    author_name : str
+        The author name in 'Lastname, Firstname' or other format.
+
+    Returns
+    -------
+    str
+        The reformatted author name in 'Firstname Lastname' format.
+    """
+    if ',' in author_name:
+        parts = author_name.split(',', 1)
+        return parts[1].strip() + ' ' + parts[0].strip()
+    return author_name
+
 
 if __name__ == "__main__":
     main()

scripts/generate_link_lists.py

@@ -223,7 +223,7 @@ def update_yaml_file(yaml_filename, use_github=True, use_zenodo=True):
                     entry['description'] = remove_html_tags(metadata['description'])
                 if 'creators' in metadata.keys():
                     creators = metadata['creators']
-                    entry['authors'] = ", ".join([c['name'] for c in creators])
+                    entry['authors'] = [format_author_name(c['name']) for c in creators]
                 if 'license' in metadata.keys():
                     entry['license'] = metadata['license']['id']
 
@@ -241,6 +241,25 @@ def update_yaml_file(yaml_filename, use_github=True, use_zenodo=True):
     # Write the modified content back to the YAML file
     write_yaml_file(yaml_filename, content)
 
+def format_author_name(author_name):
+    """
+    Reformat author names from "Lastname, Firstname" to "Firstname Lastname".
+
+    Parameters
+    ----------
+    author_name : str
+        Author name, potentially in the format "Lastname, Firstname".
+
+    Returns
+    -------
+    str
+        Author name in "Firstname Lastname" format.
+    """
+    if "," in author_name:
+        parts = author_name.split(",", 1)
+        return f"{parts[1].strip()} {parts[0].strip()}"
+    return author_name
+
 def clean_license(license):
     """
     Standardize license names.
@@ -478,56 +497,3 @@ def add_domains_to_dicts(dict_list):
     """Process list of dictionaries and add domains"""
     for item in dict_list:
         if 'url' in item:
-            item['domain'] = get_domains(item['url'])
-    return dict_list
-
-
-def complete_zenodo_data(zenodo_url):
-    """
-    Completes Zenodo data retrieval and structuring for inclusion in a YAML file.
-
-    Parameters
-    ----------
-    zenodo_url : str
-        The URL of the Zenodo record.
-
-    Returns
-    -------
-    entry : dict
-        A dictionary containing structured metadata and statistics
-        fetched from the Zenodo record.
-    """
-    zenodo_data = read_zenodo(zenodo_url)
-    entry = {}
-    urls = [zenodo_url]
-
-    if 'doi_url' in zenodo_data.keys():
-        doi_url = zenodo_data['doi_url']
-
-        # Add DOI URL to the URLs list if it's not already there
-        if doi_url not in urls:
-            urls.append(doi_url)
-    entry['url'] = urls
-
-    if 'metadata' in zenodo_data.keys():
-        metadata = zenodo_data['metadata']
-        # Update entry with Zenodo metadata and statistics
-        entry['name'] = metadata['title']
-        if 'publication_date' in metadata.keys():
-            entry['publication_date'] = metadata['publication_date']
-        if 'description' in metadata.keys():
-            entry['description'] = remove_html_tags(metadata['description'])
-        if 'creators' in metadata.keys():
-            creators = metadata['creators']
-            entry['authors'] = [c['name'] for c in creators]
-        if 'license' in metadata.keys():
-            entry['license'] = metadata['license']['id']
-
-    if 'stats' in zenodo_data.keys():
-        entry['num_downloads'] = zenodo_data['stats']['downloads']
-
-    return entry
-
-
-if __name__ == "__main__":
-    main()