feat: Add possibility to clone cards when cloning a board

Signed-off-by: Max Bachhuber <[email protected]> Adjust BoardServiceTest to new dependencies Signed-off-by: Max Bachhuber <[email protected]> Add BoardCloneModal vue component to frontend. Adjust BoardApi and store to support clone options Signed-off-by: Max Bachhuber <[email protected]> Add license and credits Signed-off-by: Max Bachhuber <[email protected]> Fix PHP code style Signed-off-by: Max Bachhuber <[email protected]> Change default clone settings Signed-off-by: Max Bachhuber <[email protected]> Add accordion for advanced settings Signed-off-by: Max Bachhuber <[email protected]> Fix bug which caused board to be cloned when clicking out of the modal Signed-off-by: Max Bachhuber <[email protected]> Change wording of clone options Signed-off-by: Max Bachhuber <[email protected]> fix: Rebase failures Signed-off-by: Julius Härtl <[email protected]> update cloneBoards phpdoc make error message clear SPDX Header BoardCloneModal.vue Signed-off-by: grnd-alt <[email protected]>
nextcloud · Jan 2, 2025 · f2c30af · f2c30af
1 parent bdaf28e
commit f2c30af
Show file tree

Hide file tree

Showing 12 changed files with 406 additions and 108 deletions.
diff --git a/cypress/e2e/boardFeatures.js b/cypress/e2e/boardFeatures.js
@@ -5,6 +5,7 @@
 import { randUser } from '../utils/index.js'
 const user = randUser()
 const recipient = randUser()
+import { sampleBoard } from '../utils/sampleBoard'
 
 describe('Board', function() {
 
@@ -58,3 +59,73 @@ describe('Board', function() {
 			.should('be.visible')
 	})
 })
+
+describe('Board cloning', function() {
+	before(function() {
+		cy.createUser(user)
+	})
+
+	it('Clones a board without cards', function() {
+		const boardName = 'Clone board original'
+		const board = sampleBoard(boardName)
+		cy.createExampleBoard({ user, board }).then((board) => {
+			const boardId = board.id
+			cy.visit(`/apps/deck/board/${boardId}`)
+			cy.get('.app-navigation__list .app-navigation-entry:contains("' + boardName + '")')
+				.parent()
+				.find('button[aria-label="Actions"]')
+				.click()
+			cy.get('button:contains("Clone board")')
+				.click()
+
+			cy.get('.modal-container button:contains("Clone")')
+				.click()
+
+			cy.get('.app-navigation__list .app-navigation-entry:contains("' + boardName + '")')
+				.should('be.visible')
+
+			cy.get('.app-navigation__list .app-navigation-entry:contains("' + boardName + ' (copy)")')
+				.should('be.visible')
+
+			cy.get('.board-title h2').contains(boardName + ' (copy)')
+
+			cy.get('h3[aria-label="TestList"]')
+				.should('be.visible')
+		})
+	})
+
+	it('Clones a board with cards', function() {
+		const boardName = 'Clone with cards'
+		const board = sampleBoard(boardName)
+		cy.createExampleBoard({ user, board }).then((board) => {
+			const boardId = board.id
+			cy.visit(`/apps/deck/board/${boardId}`)
+			cy.get('.app-navigation__list .app-navigation-entry:contains("' + boardName + '")')
+				.parent()
+				.find('button[aria-label="Actions"]')
+				.click()
+			cy.get('button:contains("Clone board")')
+				.click()
+
+			cy.get('.checkbox-content__text:contains("Clone cards")')
+				.click()
+
+			cy.get('.modal-container button:contains("Clone")')
+				.click()
+
+			cy.get('.app-navigation__list .app-navigation-entry:contains("' + boardName + '")')
+				.should('be.visible')
+
+			cy.get('.app-navigation__list .app-navigation-entry:contains("' + boardName + ' (copy)")')
+				.should('be.visible')
+
+			cy.get('.board-title h2').contains(boardName + ' (copy)')
+
+			cy.get('h3[aria-label="TestList"]')
+				.should('be.visible')
+
+			cy.get('.card:contains("Hello world")')
+				.should('be.visible')
+		})
+	})
+})
diff --git a/docs/API.md b/docs/API.md
@@ -423,6 +423,25 @@ A 403 response might be returned if the users ability to create new boards has b
 
 ##### 200 Success
 
+### POST /boards/{boardId}/clone - Clone a board
+
+Creates a copy of the board.
+
+#### Request body
+
+| Parameter | Type   | Description                                          |
+| --------- | ------ | ---------------------------------------------------- |
+| withCards  | Bool   | Setting if the cards should be copied (Default: false) |
+| withAssignments  | Bool   | Setting if the card assignments should be cloned (Default: false) |
+| withLabels  | Bool   | Setting if the card labels should be cloned (Default: false) |
+| withDueDate  | Bool   | Setting if the card due dates should be cloned (Default: false) |
+| moveCardsToLeftStack  | Bool   | Setting if all cards should be moved to the most left column (useful for To-Do / Doing / Done boards) (Default: false) |
+| restoreArchivedCards  | Bool   | Setting if the archived cards should be unarchived (Default: false) |
+
+#### Response
+
+##### 200 Success
+
 ### DELETE /boards/{boardId}/acl/{aclId} - Delete an acl rule
 
 #### Response

diff --git a/lib/Controller/BoardApiController.php b/lib/Controller/BoardApiController.php
@@ -164,4 +164,13 @@ public function deleteAcl($aclId) {
 		$acl = $this->boardService->deleteAcl($aclId);
 		return new DataResponse($acl, HTTP::STATUS_OK);
 	}
+
+	/**
+	 * @NoAdminRequired
+	 */
+	public function clone(int $boardId, bool $withCards = false, bool $withAssignments = false, bool $withLabels = false, bool $withDueDate = false, bool $moveCardsToLeftStack = false, bool $restoreArchivedCards = false): DataResponse {
+		return new DataResponse(
+			$this->boardService->clone($boardId, $this->userId, $withCards, $withAssignments, $withLabels, $withDueDate, $moveCardsToLeftStack, $restoreArchivedCards)
+		);
+	}
 }
diff --git a/lib/Controller/BoardController.php b/lib/Controller/BoardController.php
@@ -135,11 +135,11 @@ public function deleteAcl($aclId) {
 
 	/**
 	 * @NoAdminRequired
-	 * @param $boardId
-	 * @return Board
 	 */
-	public function clone($boardId) {
-		return $this->boardService->clone($boardId, $this->userId);
+	public function clone(int $boardId, bool $withCards = false, bool $withAssignments = false, bool $withLabels = false, bool $withDueDate = false, bool $moveCardsToLeftStack = false, bool $restoreArchivedCards = false): DataResponse {
+		return new DataResponse(
+			$this->boardService->clone($boardId, $this->userId, $withCards, $withAssignments, $withLabels, $withDueDate, $moveCardsToLeftStack, $restoreArchivedCards)
+		);
 	}
 
 	/**

diff --git a/lib/Db/Label.php b/lib/Db/Label.php
@@ -7,6 +7,9 @@
 
 namespace OCA\Deck\Db;
 
+/**
+ * @method getTitle(): string
+ */
 class Label extends RelationalEntity {
 	protected $title;
 	protected $color;

diff --git a/lib/Service/AssignmentService.php b/lib/Service/AssignmentService.php
@@ -88,8 +88,6 @@ public function __construct(
 		$this->changeHelper = $changeHelper;
 		$this->activityManager = $activityManager;
 		$this->eventDispatcher = $eventDispatcher;
-
-		$this->assignmentServiceValidator->check(compact('userId'));
 		$this->currentUser = $userId;
 	}