Merge pull request #581 from Hafsa-Naeem/i10263-stable_3_5_0-fix

pkp/pkp-lib#10263 Relax editing metadata on published/posted materials

Author: Vitaliy-1

src/components/ListPanel/contributors/ContributorsListPanel.vue

@@ -12,10 +12,7 @@
 						<Spinner v-if="isLoading" />
 						<template #actions>
 							<PkpButton
-								v-if="
-									publication.status !== getConstant('STATUS_PUBLISHED') &&
-									canEditPublication
-								"
+								v-if="canEditPublication"
 								icon="Sort"
 								:is-active="isOrdering"
 								:disabled="isLoading"
@@ -39,11 +36,7 @@
 								{{ t('contributor.listPanel.preview') }}
 							</PkpButton>
 							<PkpButton
-								v-if="
-									!isOrdering &&
-									publication.status !== getConstant('STATUS_PUBLISHED') &&
-									canEditPublication
-								"
+								v-if="!isOrdering && canEditPublication"
 								:disabled="isLoading"
 								@click="openAddModal"
 							>
@@ -65,13 +58,7 @@
 						{{ localize(item.affiliation) }}
 					</div>
 				</template>
-				<template
-					v-if="
-						publication.status !== getConstant('STATUS_PUBLISHED') &&
-						canEditPublication
-					"
-					#item-actions="{item}"
-				>
+				<template v-if="canEditPublication" #item-actions="{item}">
 					<template v-if="isOrdering">
 						<Orderer
 							:item-id="item.id"

src/managers/GalleyManager/useGalleyManagerConfig.js

@@ -34,12 +34,6 @@ export const GalleyManagerConfiguration = {
 		Actions.GALLEY_EDIT,
 		Actions.GALLEY_SORT,
 	],
-	actionsRequiresUnpublishedState: [
-		Actions.GALLEY_ADD,
-		Actions.GALLEY_CHANGE_FILE,
-		Actions.GALLEY_DELETE,
-		Actions.GALLEY_SORT,
-	],
 };
 
 export function useGalleyManagerConfig() {
@@ -84,15 +78,6 @@ export function useGalleyManagerConfig() {
 	function getManagerConfig({submission, publication}) {
 		const permittedActions = GalleyManagerConfiguration.actions
 			.filter((action) => {
-				if (
-					publication.value.status === pkp.const.STATUS_PUBLISHED &&
-					GalleyManagerConfiguration.actionsRequiresUnpublishedState.includes(
-						action,
-					)
-				) {
-					return false;
-				}
-
 				return true;
 			})
 			.filter((action) => {

src/pages/workflow/WorkflowPageOJS.vue

@@ -27,6 +27,7 @@ import WorkflowPublicationVersionControl from './components/publication/Workflow
 import WorkflowChangeSubmissionLanguage from './components/publication/WorkflowChangeSubmissionLanguage.vue';
 import WorkflowPrimaryBasicMetadata from './components/primary/WorkflowPrimaryBasicMetadata.vue';
 import WorkflowSubmissionStatus from './components/primary/WorkflowSubmissionStatus.vue';
+import WorkflowPublicationEditWarning from './components/publication/WorkflowPublicationEditWarning.vue';
 import WorkflowPublicationEditDisabled from './components/publication/WorkflowPublicationEditDisabled.vue';
 
 import {useWorkflowConfigOJS as useWorkflowConfig} from './composables/useWorkflowConfig/useWorkflowConfigOJS';
@@ -57,6 +58,7 @@ const Components = markRaw({
 	WorkflowPublicationVersionControl,
 	WorkflowChangeSubmissionLanguage,
 	WorkflowSubmissionStatus,
+	WorkflowPublicationEditWarning,
 	WorkflowPublicationEditDisabled,
 });
 

src/pages/workflow/WorkflowPageOMP.vue

@@ -32,6 +32,7 @@ import WorkflowPublicationVersionControl from './components/publication/Workflow
 import WorkflowChangeSubmissionLanguage from './components/publication/WorkflowChangeSubmissionLanguage.vue';
 import WorkflowPrimaryBasicMetadata from './components/primary/WorkflowPrimaryBasicMetadata.vue';
 import WorkflowSubmissionStatus from './components/primary/WorkflowSubmissionStatus.vue';
+import WorkflowPublicationEditWarning from './components/publication/WorkflowPublicationEditWarning.vue';
 import WorkflowPublicationEditDisabled from './components/publication/WorkflowPublicationEditDisabled.vue';
 
 const props = defineProps({
@@ -60,6 +61,7 @@ const Components = markRaw({
 	WorkflowPublicationVersionControl,
 	WorkflowChangeSubmissionLanguage,
 	WorkflowSubmissionStatus,
+	WorkflowPublicationEditWarning,
 	WorkflowPublicationEditDisabled,
 	WorkflowMarketingForm,
 	WorkflowWorkTypeOMP,

src/pages/workflow/WorkflowPageOPS.vue

@@ -26,6 +26,7 @@ import WorkflowChangeSubmissionLanguage from './components/publication/WorkflowC
 import WorkflowPrimaryBasicMetadata from './components/primary/WorkflowPrimaryBasicMetadata.vue';
 import WorkflowPublicationRelationDropdownOPS from './components/publication/WorkflowPublicationRelationDropdownOPS.vue';
 import WorkflowSubmissionStatus from './components/primary/WorkflowSubmissionStatus.vue';
+import WorkflowPublicationEditWarning from './components/publication/WorkflowPublicationEditWarning.vue';
 import WorkflowPublicationEditDisabled from './components/publication/WorkflowPublicationEditDisabled.vue';
 
 const props = defineProps({
@@ -49,6 +50,7 @@ const Components = markRaw({
 	WorkflowPublicationVersionControl,
 	WorkflowChangeSubmissionLanguage,
 	WorkflowSubmissionStatus,
+	WorkflowPublicationEditWarning,
 	WorkflowPublicationEditDisabled,
 });
 

src/pages/workflow/components/publication/WorkflowPublicationEditWarning.vue

@@ -0,0 +1,13 @@
+<!-- file: WorkflowPublicationEditWarning.vue -->
+<template>
+	<div
+		class="bg -mx-5 -mt-5 bg-attention p-5 text-center text-base-normal text-on-dark"
+	>
+		{{ t('publication.editorEditWarning') }}
+	</div>
+</template>
+
+<script setup>
+import {useLocalize} from '@/composables/useLocalize';
+const {t} = useLocalize();
+</script>

src/pages/workflow/composables/useWorkflowConfig/workflowConfigEditorialOJS.js

@@ -712,7 +712,7 @@ export const PublicationConfig = {
 			const items = [];
 			if (selectedPublication.status === pkp.const.STATUS_PUBLISHED) {
 				items.push({
-					component: 'WorkflowPublicationEditDisabled',
+					component: 'WorkflowPublicationEditWarning',
 					props: {},
 				});
 			}
@@ -966,6 +966,7 @@ export const PublicationConfig = {
 					props: {
 						submission,
 						publication: selectedPublication,
+						canEdit: permissions.canEditPublication,
 					},
 				},
 			];

src/pages/workflow/composables/useWorkflowPermissions.js

@@ -49,11 +49,16 @@ export function useWorkflowPermissions({submission, selectedPublication}) {
 		});
 
 		canEditPublication = submission.value.canCurrentUserChangeMetadata;
-
-		if (
-			selectedPublication.value &&
-			selectedPublication.value?.status === pkp.const.STATUS_PUBLISHED
-		) {
+		const isAuthor = hasIntersection(activeStage.currentUserAssignedRoles, [
+			pkp.const.ROLE_ID_AUTHOR,
+		]);
+		// block authors from editing any version if there is a published or scheduled version
+		const hasBlockedPublication = submission.value.publications.some((p) =>
+			[pkp.const.STATUS_PUBLISHED, pkp.const.STATUS_SCHEDULED].includes(
+				p.status,
+			),
+		);
+		if (isAuthor && hasBlockedPublication) {
 			canEditPublication = false;
 		}