Pipeline bugfixes- Fedora type Other, raise for permission error (#1493)

designsafe/apps/api/projects_v2/operations/project_publish_operations.py

@@ -378,6 +378,10 @@ class ProjectFileNotFound(Exception):
     """exception raised when attempting to copy a non-existent file for publication"""
 
 
+class PublicationDirectoryAlreadyExists(Exception):
+    """exception raised when attempting to publish into a directory that exists already"""
+
+
 def copy_publication_files(
     path_mapping: dict, project_id: str, version: Optional[int] = None
 ):
@@ -393,6 +397,13 @@ def copy_publication_files(
             pub_dirname = f"{project_id}v{version}"
 
         pub_root_dir = str(Path(f"{settings.DESIGNSAFE_PUBLISHED_PATH}") / pub_dirname)
+
+        # Prevent multiple attempts to publish the same project files.
+        if os.path.isdir(pub_root_dir):
+            raise PublicationDirectoryAlreadyExists(
+                f"Directory already exists: {pub_root_dir}"
+            )
+
         os.makedirs(pub_root_dir, exist_ok=True)
 
         for src_path in path_mapping:
@@ -427,6 +438,7 @@ def copy_publication_files(
     except PermissionError as exc:
         logger.error(exc)
         send_project_permissions_alert(project_id, version, str(exc))
+        raise exc
 
     finally:
         os.chmod("/corral-repl/tacc/NHERI/published", 0o555)

designsafe/apps/api/publications_v2/operations/fedora_graph_operations.py

@@ -317,7 +317,7 @@ def generate_manifest_other(project_id, version=1):
     """Generate the file manifest for an Other-type project"""
     fido_client = Fido()
     pub = Publication.objects.get(project_id=project_id)
-    uuid = pub.tree.nodes[0]["uuid"]
+    uuid = pub.tree["nodes"][0]["uuid"]
     file_tags = pub.value.get("fileTags", [])
 
     if version and version > 1: