From 4dba92d72a539f56d81d7653305ca338763d391b Mon Sep 17 00:00:00 2001
From: Thomas Lehmann <t.lehmann@strato.de>
Date: Thu, 7 Nov 2024 19:47:13 +0100
Subject: [PATCH] refactor(files_sharing): dissolve component SharingEntryLink
 into SharingTab

According to #48925 screen designs we've to split this into

1. Link (creation) and
2. List (of created share links)

Intermediate step.

Refs: #48925
---
 apps/files_sharing/src/views/SharingTab.vue | 53 ++++++++++++++++++---
 1 file changed, 46 insertions(+), 7 deletions(-)

diff --git a/apps/files_sharing/src/views/SharingTab.vue b/apps/files_sharing/src/views/SharingTab.vue
index 3f9b32a041609..113a4d4ba1cba 100644
--- a/apps/files_sharing/src/views/SharingTab.vue
+++ b/apps/files_sharing/src/views/SharingTab.vue
@@ -59,13 +59,12 @@
 			<hr>
 			<h3>External shares</h3>
 
-			<!-- link shares list -->
-			<SharingLinkList v-if="!loading"
-				ref="linkShareList"
+			<!-- ***** DISSOLVED OUT FROM ShareLinkList ***** -->
+			<SharingEntryLink v-if="!hasLinkShares && canReshare"
 				:can-reshare="canReshare"
 				:file-info="fileInfo"
-				:shares="linkShares"
-				@open-sharing-details="toggleShareDetailsView" />
+				@add:share="addShare" />
+			<!-- ***** DISSOLVED OUT FROM ShareLinkList ***** -->
 
 			<!-- TODO: component must either be configurable or diffentiated into two -->
 			<!-- add new email/federated share input -->
@@ -77,7 +76,22 @@
 				:shares="shares"
 				@open-sharing-details="toggleShareDetailsView" />
 
-			<!-- projects -->
+			<!-- ***** DISSOLVED OUT FROM ShareLinkList ***** -->
+			<template v-if="hasShares">
+				<!-- using shares[index] to work with .sync -->
+				<SharingEntryLink v-for="(share, index) in shares"
+					:key="share.id"
+					:index="shares.length > 1 ? index + 1 : null"
+					:can-reshare="canReshare"
+					:share.sync="shares[index]"
+					:file-info="fileInfo"
+					@add:share="addShare(...arguments)"
+					@update:share="awaitForShare(...arguments)"
+					@remove:share="removeShare"
+					@open-sharing-details="openSharingDetails(share)" />
+			</template>
+			<!-- ***** DISSOLVED OUT FROM ShareLinkList ***** -->
+
 			<CollectionList v-if="projectsEnabled && fileInfo"
 				:id="`${fileInfo.id}`"
 				type="file"
@@ -131,11 +145,15 @@ import SharingInherited from './SharingInherited.vue'
 import SharingLinkList from './SharingLinkList.vue'
 import SharingList from './SharingList.vue'
 import SharingDetailsTab from './SharingDetailsTab.vue'
+import { getCapabilities } from '@nextcloud/capabilities'
+import SharingEntryLink from '../components/SharingEntryLink.vue'
+import ShareDetails from '../mixins/ShareDetails.js'
 
 export default {
 	name: 'SharingTab',
 
 	components: {
+		SharingEntryLink,
 		NcAvatar,
 		CollectionList,
 		SharingEntryInternal,
@@ -147,7 +165,7 @@ export default {
 		SharingDetailsTab,
 	},
 
-	mixins: [ShareTypes],
+	mixins: [ShareTypes, ShareDetails],
 
 	data() {
 		return {
@@ -170,6 +188,7 @@ export default {
 			showSharingDetailsView: false,
 			shareDetailsData: {},
 			returnFocusElement: null,
+			canLinkShare: getCapabilities().files_sharing.public.enabled,
 		}
 	},
 
@@ -187,6 +206,26 @@ export default {
 			return !!(this.fileInfo.permissions & OC.PERMISSION_SHARE)
 				|| !!(this.reshare && this.reshare.hasSharePermission && this.config.isResharingAllowed)
 		},
+
+		/**
+		 * Do we have link shares?
+		 * Using this to still show the `new link share`
+		 * button regardless of mail shares
+		 *
+		 * @return {Array}
+		 */
+		hasLinkShares() {
+			return this.shares.filter(share => share.type === this.SHARE_TYPES.SHARE_TYPE_LINK).length > 0
+		},
+
+		/**
+		 * Do we have any link or email shares?
+		 *
+		 * @return {boolean}
+		 */
+		hasShares() {
+			return this.shares.length > 0
+		},
 	},
 
 	methods: {