From 3041c58b6bd06159304fbdf9faef128f8c6cf3bd Mon Sep 17 00:00:00 2001
From: Dimitris Efstathiou <defstat@gmail.com>
Date: Mon, 4 Dec 2023 20:29:35 +0200
Subject: [PATCH] pkp/pkp-lib#7505 Load Jats when changes happen to publication

---
 .../contributors/ContributorsListPanel.vue    | 56 ++++++++---------
 .../jats/SubmissionJatsListPanel.vue          | 60 ++++++++-----------
 2 files changed, 54 insertions(+), 62 deletions(-)

diff --git a/src/components/ListPanel/contributors/ContributorsListPanel.vue b/src/components/ListPanel/contributors/ContributorsListPanel.vue
index 04d8ece04..22306837e 100644
--- a/src/components/ListPanel/contributors/ContributorsListPanel.vue
+++ b/src/components/ListPanel/contributors/ContributorsListPanel.vue
@@ -366,6 +366,8 @@ export default {
 				this.$emit('updated:contributors', newContributors);
 			}
 			this.closeFormModal();
+
+			this.getAndUpdatePublication();
 		},
 
 		/**
@@ -374,20 +376,7 @@ export default {
 		openPreviewModal() {
 			this.isLoading = true;
 
-			$.ajax({
-				url: this.publicationApiUrl,
-				type: 'GET',
-				context: this,
-				error: this.ajaxErrorCallback,
-				success(publication) {
-					this.$emit('updated:publication', publication);
-
-					this.isModalOpenedPreview = true;
-				},
-				complete(r) {
-					this.isLoading = false;
-				},
-			});
+			this.getAndUpdatePublication();
 		},
 
 		/**
@@ -445,6 +434,8 @@ export default {
 										},
 									);
 									this.$emit('updated:contributors', newContributors);
+									
+									this.getAndUpdatePublication();
 								},
 								complete(r) {
 									this.isLoading = false;
@@ -529,6 +520,8 @@ export default {
 		cancelOrdering() {
 			this.$emit('updated:contributors', this.itemsBeforeReordering);
 
+			this.getAndUpdatePublication();
+
 			this.itemsBeforeReordering = null;
 			this.isOrdering = false;
 		},
@@ -563,20 +556,7 @@ export default {
 				success(contributors) {
 					this.$emit('updated:contributors', contributors);
 
-					// Update the publication in the background so that
-					// any author strings are updated
-					$.ajax({
-						url: this.publicationApiUrl,
-						context: this,
-						type: 'GET',
-						success(publication) {
-							this.$emit('updated:publication', publication);
-						},
-						complete() {
-							this.isLoading = false;
-							this.isOrdering = false;
-						},
-					});
+					this.getAndUpdatePublication();
 				},
 				error: this.ajaxErrorCallback,
 				complete() {
@@ -651,6 +631,26 @@ export default {
 
 			this.$emit('updated:contributors', newItems);
 		},
+
+		/**
+		 * Update the publication in the background so that
+		 * any author strings are updated
+		 */
+		getAndUpdatePublication() {
+			$.ajax({
+				url: this.publicationApiUrl,
+				context: this,
+				type: 'GET',
+				success(publication) {
+					this.$emit('updated:publication', publication);
+				},
+				complete() {
+					this.isLoading = false;
+					this.isOrdering = false;
+				},
+			});
+		},
+		
 	},
 };
 </script>
diff --git a/src/components/ListPanel/jats/SubmissionJatsListPanel.vue b/src/components/ListPanel/jats/SubmissionJatsListPanel.vue
index a17443c2a..cc16dfd91 100644
--- a/src/components/ListPanel/jats/SubmissionJatsListPanel.vue
+++ b/src/components/ListPanel/jats/SubmissionJatsListPanel.vue
@@ -6,27 +6,28 @@
 					<pkp-header>
 						<h2>{{ title }}</h2>
 						<template #actions>
-							<div v-if="existingJatsFile">
-								<pkp-button
+							<div v-if="isDefaultContent">
+								<pkp-button 
 									v-if="
 										publication.status !== getConstant('STATUS_PUBLISHED') &&
 										canEditPublication
 									"
-									:disabled="isLoading"
-									:isWarnable="true"
-									@click="openDeleteModal"
+									ref="uploadXMLButton" @click="openFileBrowser"
 								>
-									{{ t('common.delete') }}
+									{{ t('common.upload') }}
 								</pkp-button>
 							</div>
 							<div v-else>
-								<pkp-button 
+								<pkp-button
 									v-if="
 										publication.status !== getConstant('STATUS_PUBLISHED') &&
 										canEditPublication
 									"
-									ref="uploadXMLButton" @click="openFileBrowser">
-									{{ t('common.upload') }}
+									:disabled="isLoading"
+									:isWarnable="true"
+									@click="openDeleteModal"
+								>
+									{{ t('common.delete') }}
 								</pkp-button>
 							</div>
 							<pkp-button ref="downloadJatsXMLButton" @click="downloadJatsXML">
@@ -44,14 +45,14 @@
 						</pre>
 					</div>
 					<div>
-						<div v-if="existingJatsFile" class="filePanel__hasData">
-							<div class="filePanel__fileContentFooter">
-								<span>{{ lastModifiedAtActual }}</span>
+						<div v-if="isDefaultContent" class="filePanel__hasData">
+							<div class="filePanel__defaultContentFooter">
+								<span>{{ t('publication.jats.autoCreatedMessage') }}</span>
 							</div>
 						</div>
 						<div v-else>
-							<div class="filePanel__defaultContentFooter">
-								<span>{{ t('publication.jats.autoCreatedMessage') }}</span>
+							<div class="filePanel__fileContentFooter">
+								<span>{{ lastModifiedAtActual }}</span>
 							</div>
 						</div>
 					</div>
@@ -161,18 +162,11 @@ export default {
 		this.fetchWorkingJatsFile();
 	},
 	computed: {
-		defaultJatsContent() {
-			return this.workingJatsProps['defaultJatsContent'];
-		},
-		existingJatsFile() {
-			return this.workingJatsProps['existingJatsFile'];
+		isDefaultContent() {
+			return this.workingJatsProps['isDefaultContent'];
 		},
 		workingJatsContent() {
-			if (this.existingJatsFile) {
-				return this.existingJatsFile['jatsContent'];
-			} else {
-				return this.defaultJatsContent;
-			}
+			return this.workingJatsProps['jatsContent'];
 		},
 		/**
 		 * URL to the API endpoint for the current publication
@@ -187,15 +181,15 @@ export default {
 		lastModifiedAtActual() {
 			var actual = '';
 			
-			if (this.existingJatsFile) {
+			if (!this.isDefaultContent) {
 				actual = this.i18nLastModifiedAt.replace(
 					'{$modificationDate}',
-					this.existingJatsFile.updatedAt,
+					this.workingJatsProps.updatedAt,
 				);
 
 				actual = actual.replace(
 					'{$username}',
-					this.existingJatsFile.uploaderUserName,
+					this.workingJatsProps.uploaderUserName,
 				);
 			}
 
@@ -298,8 +292,8 @@ export default {
 		 */
 		downloadJatsXML() {
 			// Ensure existingJatsFile.url is available and is a string containing the URL
-			if (this.existingJatsFile && this.existingJatsFile.url) {
-				const downloadUrl = this.existingJatsFile.url;
+			if (!this.workingJatsProps.isDefaultContent) {
+				const downloadUrl = this.workingJatsProps.url;
 
 				// Create a link element
 				const link = document.createElement('a');
@@ -315,8 +309,8 @@ export default {
 
 				// Remove the link from the DOM
 				document.body.removeChild(link);
-			} else if (this.defaultJatsContent) {
-				const jatsContent = this.defaultJatsContent;
+			} else {
+				const jatsContent = this.workingJatsProps.jatsContent;
 				const blob = new Blob([jatsContent], { type: 'application/xml' });
 				const blobUrl = URL.createObjectURL(blob);
 
@@ -330,8 +324,6 @@ export default {
 				// Clean up the URL object
 				URL.revokeObjectURL(blobUrl);
 				document.body.removeChild(link);
-			} else {
-				console.error('Download URL is not available.');
 			}
 		},
 	},
@@ -342,7 +334,7 @@ export default {
 			} 
 
 			if (newValue != null && newValue[0] != null) {
-				if (newValue[0].hasOwnProperty('existingJatsFile')) {
+				if (newValue[0].hasOwnProperty('jatsContent')) {
 					this.workingJatsProps = newValue[0];
 					this.hasLoadedContent = true;
 				}