Merge pull request #111 from diverso-lab/edit_metadata

Edit metadata

Author: drorganvidez

app/modules/dataset/assets/js/stepper.js

@@ -219,6 +219,7 @@ function updateDropzoneStatus() {
 
 function get_summary() {
     const datasetTypeElement = document.querySelector('input[name="dataset_type"]:checked');
+    const isAnonymousUpload = datasetTypeElement?.value === "zenodo_anonymous";
     const datasetType = datasetTypeElement 
         ? document.querySelector(`label[for="${datasetTypeElement.id}"]`)?.innerText.trim() || "Not selected"
         : "Not selected";
@@ -257,6 +258,11 @@ function get_summary() {
     let summaryContent = `
         <h3>Summary</h3>
         <p><strong>Dataset Type:</strong><br>${datasetType}</p>
+        ${
+            isAnonymousUpload
+                ? '<div class="alert alert-warning py-3"><strong>Anonymous upload:</strong> enabled.</div>'
+                : ''
+        }
         <p><strong>Title:</strong> ${title}</p>
         <p><strong>Description:</strong> ${description}</p>
         <p><strong>Publication Type:</strong> ${publicationType}</p>
@@ -269,7 +275,9 @@ function get_summary() {
         <h4>Authors</h4>
     `;
 
-    if (authors.length > 0) {
+    if (isAnonymousUpload) {
+        summaryContent += '<p><strong>Anonymous:</strong> Enabled. Author details will be hidden in publication.</p>';
+    } else if (authors.length > 0) {
         summaryContent += '<ul>';
         authors.forEach((author, index) => {
             summaryContent += `

app/modules/dataset/routes.py

@@ -4,16 +4,29 @@
 import tempfile
 from datetime import datetime
 
-from flask import abort, current_app, jsonify, make_response, redirect, render_template, request, send_file, url_for
+from flask import (
+    abort,
+    current_app,
+    flash,
+    jsonify,
+    make_response,
+    redirect,
+    render_template,
+    request,
+    send_file,
+    url_for,
+)
 from flask_login import current_user, login_required
 
 from app.modules.apikeys.decorators import require_api_key
 from app.modules.dataset import dataset_bp
 from app.modules.dataset.decorators import is_dataset_owner
 from app.modules.dataset.forms import DataSetForm
-from app.modules.dataset.models import DataSet
+from app.modules.dataset.models import DataSet, PublicationType
 from app.modules.dataset.services import (
     AuthorService,
+    DatasetMetadataUpdateError,
+    DatasetMetadataValidationError,
     DataSetService,
     DOIMappingService,
     DSDownloadRecordService,
@@ -118,6 +131,47 @@ def create_dataset():
     return render_template("dataset/create_and_edit_dataset.html", form=form)
 
 
+@dataset_bp.route("/dataset/edit/<int:dataset_id>", methods=["GET", "POST"])
+@login_required
+def edit_metadata(dataset_id):
+    dataset = dataset_service.get_or_404(dataset_id)
+    form = DataSetForm()
+    if dataset.user_id != current_user.id:
+        abort(403)
+
+    if request.method == "POST":
+        is_ajax = request.headers.get("X-Requested-With") == "XMLHttpRequest"
+        try:
+            dataset_service.update_metadata_from_request(dataset, request.form, zenodo_service=zenodo_service)
+            if is_ajax:
+                return jsonify({"message": "Dataset updated successfully"}), 200
+            flash("Dataset updated successfully!", "success")
+        except DatasetMetadataValidationError as exc:
+            if is_ajax:
+                return jsonify({"message": str(exc)}), 400
+            flash(str(exc), "danger")
+            return redirect(url_for("dataset.edit_metadata", dataset_id=dataset_id))
+        except DatasetMetadataUpdateError as exc:
+            if is_ajax:
+                return jsonify({"message": f"Error updating metadata: {exc}"}), 400
+            flash(f"Error updating metadata: {exc}", "danger")
+        except Exception as exc:
+            logger.exception("[EDIT DATASET] Unexpected error updating dataset %s", dataset_id)
+            if is_ajax:
+                return jsonify({"message": f"Unexpected error updating metadata: {exc}"}), 400
+            flash(f"Unexpected error updating metadata: {exc}", "danger")
+
+        return redirect(url_for("dataset.list_dataset"))
+
+    return render_template(
+        "dataset/create_and_edit_dataset.html",
+        dataset=dataset,
+        is_edit=True,
+        form=form,
+        PublicationType=PublicationType,
+    )
+
+
 @dataset_bp.route("/datasets/list", methods=["GET"])
 @login_required
 def list_dataset():