From 22bb8d65e96d5f1bd94aa7af753e91b83ffc66df Mon Sep 17 00:00:00 2001
From: grnd-alt <git@belakkaf.net>
Date: Thu, 19 Dec 2024 13:30:09 +0100
Subject: [PATCH] put copy logic into move dialog

Signed-off-by: grnd-alt <git@belakkaf.net>
---
 cypress/e2e/cardFeatures.js              |   4 +-
 src/App.vue                              |   3 -
 src/CardCloneDialog.vue                  | 120 -----------------------
 src/CardMoveDialog.vue                   |  30 ++++--
 src/components/cards/CardMenuEntries.vue |  16 +--
 5 files changed, 27 insertions(+), 146 deletions(-)
 delete mode 100644 src/CardCloneDialog.vue

diff --git a/cypress/e2e/cardFeatures.js b/cypress/e2e/cardFeatures.js
index 5d7d747e5..5d5115c3d 100644
--- a/cypress/e2e/cardFeatures.js
+++ b/cypress/e2e/cardFeatures.js
@@ -310,8 +310,8 @@ describe('Card', function () {
 				.find('.ProseMirror h1').contains('Hello world').should('be.visible')
 
 			cy.get('.app-sidebar-header .action-item__menutoggle').click()
-			cy.get('.v-popper__popper button:contains("Clone card")').click()
-			cy.get('.modal__content button:contains("Clone card")').click()
+			cy.get('.v-popper__popper button:contains("Move/copy card")').click()
+			cy.get('.modal__content button:contains("Copy card")').click()
 			cy.wait('@clone', { timeout: 7000 })
 			cy.get('.card:contains("Hello world")').should('have.length', 2)
 		})
diff --git a/src/App.vue b/src/App.vue
index 7f68bd18e..f43f6c516 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -26,7 +26,6 @@
 		</div>
 		<KeyboardShortcuts />
 		<CardMoveDialog />
-		<CardCloneDialog />
 	</NcContent>
 </template>
 
@@ -39,7 +38,6 @@ import { BoardApi } from './services/BoardApi.js'
 import { emit, subscribe } from '@nextcloud/event-bus'
 import { loadState } from '@nextcloud/initial-state'
 import CardMoveDialog from './CardMoveDialog.vue'
-import CardCloneDialog from './CardCloneDialog.vue'
 
 const boardApi = new BoardApi()
 
@@ -47,7 +45,6 @@ export default {
 	name: 'App',
 	components: {
 		CardMoveDialog,
-		CardCloneDialog,
 		AppNavigation,
 		NcModal,
 		NcContent,
diff --git a/src/CardCloneDialog.vue b/src/CardCloneDialog.vue
deleted file mode 100644
index 821ddd824..000000000
--- a/src/CardCloneDialog.vue
+++ /dev/null
@@ -1,120 +0,0 @@
-<!--
-  - SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
-  - SPDX-License-Identifier: AGPL-3.0-or-later
--->
-<template>
-	<NcModal v-if="modalShow" :title="t('deck', 'Clone card')" @close="modalShow = false">
-		<div class="modal__content">
-			<h3>{{ t('deck', 'Clone card to another board') }}</h3>
-			<NcSelect v-model="selectedBoard"
-				:input-label="t('deck', 'Select a board')"
-				:placeholder="t('deck', 'Select a board')"
-				:options="activeBoards"
-				:max-height="100"
-				label="title"
-				@option:selected="loadStacksFromBoard" />
-			<NcSelect v-model="selectedStack"
-				:disabled="stacksFromBoard.length === 0"
-				:placeholder="stacksFromBoard.length === 0 ? t('deck', 'No lists available') : t('deck', 'Select a list')"
-				:input-label="t('deck', 'Select a list')"
-				:options="stacksFromBoard"
-				:max-height="100"
-				label="title" />
-
-			<button :disabled="!isBoardAndStackChoosen" class="primary" @click="cloneCard">
-				{{ t('deck', 'Clone card') }}
-			</button>
-			<button @click="modalShow = false">
-				{{ t('deck', 'Cancel') }}
-			</button>
-		</div>
-	</NcModal>
-</template>
-
-<script>
-import { NcModal, NcSelect } from '@nextcloud/vue'
-import { generateUrl } from '@nextcloud/router'
-import axios from '@nextcloud/axios'
-import { subscribe, unsubscribe } from '@nextcloud/event-bus'
-import { mapGetters, mapState } from 'vuex'
-
-export default {
-	name: 'CardCloneDialog',
-	components: { NcModal, NcSelect },
-	data() {
-		return {
-			card: null,
-			modalShow: false,
-			selectedBoard: '',
-			selectedStack: '',
-			stacksFromBoard: [],
-		}
-	},
-	computed: {
-		...mapGetters([
-			'boards',
-			'stackById',
-			'boardById',
-		]),
-		...mapState({
-			currentBoard: state => state.currentBoard,
-		}),
-		activeBoards() {
-			return this.$store.getters.boards.filter((item) => item.deletedAt === 0 && item.archived === false)
-		},
-		isBoardAndStackChoosen() {
-			return !(this.selectedBoard === '' || this.selectedStack === '')
-		},
-	},
-	mounted() {
-		subscribe('deck:card:show-clone-dialog', this.openModal)
-	},
-	destroyed() {
-		unsubscribe('deck:card:show-clone-dialog', this.openModal)
-	},
-	methods: {
-		openModal(card) {
-			this.card = card
-			this.selectedStack = this.stackById(this.card.stackId)
-			this.selectedBoard = this.boardById(this.selectedStack.boardId)
-			this.loadStacksFromBoard(this.selectedBoard)
-			this.modalShow = true
-		},
-		async loadStacksFromBoard(board) {
-			try {
-				const url = generateUrl('/apps/deck/stacks/' + board.id)
-				const response = await axios.get(url)
-				this.stacksFromBoard = response.data
-			} catch (err) {
-				return err
-			}
-		},
-		async cloneCard() {
-			this.$store.dispatch('cloneCard', { cardId: this.card.id, targetStackId: this.selectedStack.id })
-			this.modalShow = false
-		},
-	},
-}
-</script>
-
-<style lang="scss" scoped>
-.modal__content {
-	min-width: 250px;
-	min-height: 120px;
-	margin: 20px 20px 100px 20px;
-
-	h3 {
-		font-weight: bold;
-		text-align: center;
-	}
-
-	.select {
-		margin-bottom: 12px;
-	}
-}
-
-.modal__content button {
-	float: right;
-	margin-top: 50px;
-}
-</style>
diff --git a/src/CardMoveDialog.vue b/src/CardMoveDialog.vue
index 0593be4da..aa0389b99 100644
--- a/src/CardMoveDialog.vue
+++ b/src/CardMoveDialog.vue
@@ -5,7 +5,7 @@
 <template>
 	<NcModal v-if="modalShow" :title="t('deck', 'Move card to another board')" @close="modalShow = false">
 		<div class="modal__content">
-			<h3>{{ t('deck', 'Move card to another board') }}</h3>
+			<h3>{{ t('deck', 'Move/copy card to another board') }}</h3>
 			<NcSelect v-model="selectedBoard"
 				:input-label="t('deck', 'Select a board')"
 				:placeholder="t('deck', 'Select a board')"
@@ -21,25 +21,29 @@
 				:max-height="100"
 				label="title" />
 
-			<button :disabled="!isBoardAndStackChoosen" class="primary" @click="moveCard">
+			<NcButton :disabled="!isBoardAndStackChoosen" type="primary" @click="moveCard">
 				{{ t('deck', 'Move card') }}
-			</button>
-			<button @click="modalShow = false">
+			</NcButton>
+			<NcButton :disabled="!isBoardAndStackChoosen" type="primary" @click="cloneCard">
+				{{ t('deck', 'Copy card') }}
+			</NcButton>
+			<NcButton @click="modalShow = false">
 				{{ t('deck', 'Cancel') }}
-			</button>
+			</NcButton>
 		</div>
 	</NcModal>
 </template>
 
 <script>
-import { NcModal, NcSelect } from '@nextcloud/vue'
+import { NcModal, NcSelect, NcButton } from '@nextcloud/vue'
 import { generateUrl } from '@nextcloud/router'
 import axios from '@nextcloud/axios'
 import { subscribe, unsubscribe } from '@nextcloud/event-bus'
+import { mapGetters } from 'vuex'
 
 export default {
 	name: 'CardMoveDialog',
-	components: { NcModal, NcSelect },
+	components: { NcModal, NcSelect, NcButton },
 	data() {
 		return {
 			card: null,
@@ -50,6 +54,7 @@ export default {
 		}
 	},
 	computed: {
+		...mapGetters(['stackById', 'boardById']),
 		activeBoards() {
 			return this.$store.getters.boards.filter((item) => item.deletedAt === 0 && item.archived === false)
 		},
@@ -66,6 +71,9 @@ export default {
 	methods: {
 		openModal(card) {
 			this.card = card
+			this.selectedStack = this.stackById(this.card.stackId)
+			this.selectedBoard = this.boardById(this.selectedStack.boardId)
+			this.loadStacksFromBoard(this.selectedBoard)
 			this.modalShow = true
 		},
 		async loadStacksFromBoard(board) {
@@ -86,6 +94,10 @@ export default {
 			}
 			this.modalShow = false
 		},
+		async cloneCard() {
+			this.$store.dispatch('cloneCard', { cardId: this.card.id, targetStackId: this.selectedStack.id })
+			this.modalShow = false
+		},
 	},
 }
 </script>
@@ -106,6 +118,10 @@ export default {
 	}
 }
 
+button{
+	margin-left: 6px;
+}
+
 .modal__content button {
 	float: right;
 	margin-top: 50px;
diff --git a/src/components/cards/CardMenuEntries.vue b/src/components/cards/CardMenuEntries.vue
index 8f4401a78..af5fc6944 100644
--- a/src/components/cards/CardMenuEntries.vue
+++ b/src/components/cards/CardMenuEntries.vue
@@ -31,15 +31,7 @@
 			icon="icon-external"
 			:close-after-click="true"
 			@click="openCardMoveDialog">
-			{{ t('deck', 'Move card') }}
-		</NcActionButton>
-		<NcActionButton v-if="canEdit"
-			:close-after-click="true"
-			@click="openCardCloneDialog">
-			<template #icon>
-				<CloneIcon :size="20" decorative />
-			</template>
-			{{ t('deck', 'Clone card') }}
+			{{ t('deck', 'Move/copy card') }}
 		</NcActionButton>
 		<NcActionButton v-for="action in cardActions"
 			:key="action.label"
@@ -66,7 +58,6 @@
 import { NcActionButton } from '@nextcloud/vue'
 import { mapGetters, mapState } from 'vuex'
 import ArchiveIcon from 'vue-material-design-icons/Archive.vue'
-import CloneIcon from 'vue-material-design-icons/ContentCopy.vue'
 import CardBulletedIcon from 'vue-material-design-icons/CardBulleted.vue'
 import { generateUrl } from '@nextcloud/router'
 import { getCurrentUser } from '@nextcloud/auth'
@@ -77,7 +68,7 @@ import { emit } from '@nextcloud/event-bus'
 
 export default {
 	name: 'CardMenuEntries',
-	components: { CloneIcon, NcActionButton, ArchiveIcon, CardBulletedIcon },
+	components: { NcActionButton, ArchiveIcon, CardBulletedIcon },
 	props: {
 		card: {
 			type: Object,
@@ -177,9 +168,6 @@ export default {
 		openCardMoveDialog() {
 			emit('deck:card:show-move-dialog', this.card)
 		},
-		openCardCloneDialog() {
-			emit('deck:card:show-clone-dialog', this.card)
-		},
 	},
 }
 </script>