From 344638f483a14023dc4aa22faffda0401e031bd9 Mon Sep 17 00:00:00 2001
From: Christian Hartmann <chris-hartmann@gmx.de>
Date: Wed, 22 Apr 2026 23:31:46 +0200
Subject: [PATCH] fix: only allow modifying sharing to form owners

Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
---
 .../SidebarTabs/SharingSearchDiv.vue          |  7 +++++-
 .../SidebarTabs/SharingShareDiv.vue           |  7 +++++-
 .../SidebarTabs/SharingSidebarTab.vue         | 23 ++++++++++++++-----
 3 files changed, 29 insertions(+), 8 deletions(-)

diff --git a/src/components/SidebarTabs/SharingSearchDiv.vue b/src/components/SidebarTabs/SharingSearchDiv.vue
index bf79fa991..eca3f687d 100644
--- a/src/components/SidebarTabs/SharingSearchDiv.vue
+++ b/src/components/SidebarTabs/SharingSearchDiv.vue
@@ -8,7 +8,7 @@
 		<NcSelectUsers
 			keep-open
 			:loading="showLoadingCircle"
-			:disabled="locked"
+			:disabled="locked || !isCurrentUserOwner"
 			:options="options"
 			:placeholder="t('forms', 'Search for user, group or team …')"
 			:aria-label-listbox="t('forms', 'Search for user, group or team …')"
@@ -46,6 +46,11 @@ export default {
 			type: Boolean,
 			required: true,
 		},
+
+		isCurrentUserOwner: {
+			type: Boolean,
+			required: true,
+		},
 	},
 
 	computed: {
diff --git a/src/components/SidebarTabs/SharingShareDiv.vue b/src/components/SidebarTabs/SharingShareDiv.vue
index 946f83216..5409ad65d 100644
--- a/src/components/SidebarTabs/SharingShareDiv.vue
+++ b/src/components/SidebarTabs/SharingShareDiv.vue
@@ -14,7 +14,7 @@
 			<span>{{ displayName }}</span>
 			<span>{{ displayNameAppendix }}</span>
 		</div>
-		<NcActions class="share-div__actions">
+		<NcActions class="share-div__actions" :disabled="!isCurrentUserOwner">
 			<NcActionCaption :name="t('forms', 'Permissions')" />
 			<NcActionCheckbox
 				:model-value="canEditForm"
@@ -80,6 +80,11 @@ export default {
 			type: Boolean,
 			required: true,
 		},
+
+		isCurrentUserOwner: {
+			type: Boolean,
+			required: true,
+		},
 	},
 
 	computed: {
diff --git a/src/components/SidebarTabs/SharingSidebarTab.vue b/src/components/SidebarTabs/SharingSidebarTab.vue
index ddc35a00b..592055fe7 100644
--- a/src/components/SidebarTabs/SharingSidebarTab.vue
+++ b/src/components/SidebarTabs/SharingSidebarTab.vue
@@ -20,6 +20,7 @@
 			:current-shares="form.shares"
 			:show-loading="isLoading"
 			:locked="locked"
+			:is-current-user-owner="isCurrentUserOwner"
 			@add-share="addShare" />
 
 		<!-- Public Link -->
@@ -31,7 +32,9 @@
 			</div>
 			<span class="share-div__desc">{{ t('forms', 'Share link') }}</span>
 			<NcActions>
-				<NcActionButton :disabled="locked" @click="addPublicLink">
+				<NcActionButton
+					:disabled="locked || !isCurrentUserOwner"
+					@click="addPublicLink">
 					<template #icon>
 						<IconPlus :size="20" />
 					</template>
@@ -80,7 +83,7 @@
 					</NcActionButton>
 					<NcActionButton
 						v-else
-						:disabled="locked"
+						:disabled="locked || !isCurrentUserOwner"
 						@click="makeEmbeddable(share)">
 						<template #icon>
 							<IconLinkBoxVariantOutline :size="20" />
@@ -88,7 +91,9 @@
 						<!-- TRANSLATORS: This means the link can be embedded into external websites -->
 						{{ t('forms', 'Convert to embeddable link') }}
 					</NcActionButton>
-					<NcActionButton :disabled="locked" @click="removeShare(share)">
+					<NcActionButton
+						:disabled="locked || !isCurrentUserOwner"
+						@click="removeShare(share)">
 						<template #icon>
 							<IconDelete :size="20" />
 						</template>
@@ -97,7 +102,7 @@
 					<NcActionButton
 						v-if="appConfig.allowPublicLink"
 						close-after-click
-						:disabled="locked"
+						:disabled="locked || !isCurrentUserOwner"
 						@click="addPublicLink">
 						<template #icon>
 							<IconPlus :size="20" />
@@ -160,7 +165,7 @@
 				<NcCheckboxRadioSwitch
 					id="share-switch__permit-all"
 					:checked="form.access.permitAllUsers"
-					:disabled="locked"
+					:disabled="locked || !isCurrentUserOwner"
 					type="switch"
 					@update:checked="onPermitAllUsersChange" />
 			</div>
@@ -176,7 +181,7 @@
 				<NcCheckboxRadioSwitch
 					id="share-switch__show-to-all"
 					:checked="form.access.showToAllUsers"
-					:disabled="locked"
+					:disabled="locked || !isCurrentUserOwner"
 					type="switch"
 					@update:checked="onShowToAllUsersChange" />
 			</div>
@@ -189,6 +194,7 @@
 				:key="'share-' + share.shareType + '-' + share.shareWith"
 				:share="share"
 				:locked="locked"
+				:is-current-user-owner="isCurrentUserOwner"
 				@remove-share="removeShare"
 				@update:share="updateShare" />
 		</TransitionGroup>
@@ -196,6 +202,7 @@
 </template>
 
 <script>
+import { getCurrentUser } from '@nextcloud/auth'
 import { generateOcsUrl } from '@nextcloud/router'
 import { loadState } from '@nextcloud/initial-state'
 import { showError } from '@nextcloud/dialogs'
@@ -274,6 +281,10 @@ export default {
 	},
 
 	computed: {
+		isCurrentUserOwner() {
+			return getCurrentUser().uid === this.form.ownerId
+		},
+
 		sortedShares() {
 			// Remove Link-Shares, which are handled separately, then sort
 			return this.form.shares