feat(federation): unassign labels

Signed-off-by: grnd-alt <git@belakkaf.net>

Author: grnd-alt

appinfo/routes.php

@@ -142,6 +142,7 @@
 		['name' => 'card_ocs#create', 'url' => '/api/v{apiVersion}/cards', 'verb' => 'POST'],
 		['name' => 'card_ocs#update', 'url' => '/api/v{apiVersion}/cards/{cardId}', 'verb' => 'PUT'],
 		['name' => 'card_ocs#assignLabel', 'url' => '/api/v{apiVersion}/cards/{cardId}/label/{labelId}', 'verb' => 'POST'],
+		['name' => 'card_ocs#removeLabel', 'url' => '/api/v{apiVersion}/cards/{cardId}/label/{labelId}', 'verb' => 'DELETE'],
 
 		['name' => 'stack_ocs#create', 'url' => '/api/v{apiVersion}/stacks', 'verb' => 'POST'],
 		['name' => 'stack_ocs#delete', 'url' => '/api/v{apiVersion}/stacks/{stackId}/{boardId}', 'verb' => 'DELETE', 'defaults' => ['boardId' => null]],

lib/Controller/CardOcsController.php

@@ -77,6 +77,20 @@ public function assignLabel(?int $boardId, int $cardId, int $labelId): DataRespo
 		return new DataResponse($this->cardService->assignLabel($cardId, $labelId));
 	}
 
+	#[NoAdminRequired]
+	#[PublicPage]
+	#[NoCSRFRequired]
+	public function removeLabel(?int $boardId, int $cardId, int $labelId): DataResponse {
+		if ($boardId) {
+			$board = $this->boardService->find($boardId, false);
+			if ($board->getExternalId()) {
+				return new DataResponse($this->externalBoardService->removeLabelOnRemote($board, $cardId, $labelId));
+			}
+		}
+
+		return new DataResponse($this->cardService->removeLabel($cardId, $labelId));
+	}
+
 	#[NoAdminRequired]
 	#[PublicPage]
 	#[NoCSRFRequired]

lib/Service/ExternalBoardService.php

@@ -147,6 +147,18 @@ public function assignLabelOnRemote(Board $localBoard, int $cardId, int $labelId
 		return $this->proxy->getOcsData($resp);
 	}
 
+	public function removeLabelOnRemote(Board $localBoard, int $cardId, int $labelId): array {
+		$this->configService->ensureFederationEnabled();
+		$this->permissionService->checkPermission($this->boardMapper, $localBoard->getId(), Acl::PERMISSION_EDIT, $this->userId, false, false);
+		$shareToken = $localBoard->getShareToken();
+		$ownerCloudId = $this->cloudIdManager->resolveCloudId($localBoard->getOwner());
+		$url = $ownerCloudId->getRemote() . '/ocs/v2.php/apps/deck/api/v1.0/cards/' . $cardId . '/label/' . $labelId;
+		$resp = $this->proxy->delete($ownerCloudId->getId(), $shareToken, $url, [
+			'boardId' => $localBoard->getExternalId(),
+		]);
+		return $this->proxy->getOcsData($resp);
+	}
+
 	public function createStackOnRemote(
 		Board $localBoard,
 		string $title,

src/services/CardApi.js

@@ -204,7 +204,7 @@ export class CardApi {
 		return axios.post(this.ocsUrl(`/cards/${data.card.id}/label/${data.labelId}`), { boardId: data.boardId ? data.boardId : null })
 			.then(
 				(response) => {
-					return Promise.resolve(response.data)
+					return Promise.resolve(response.data.ocs.data)
 				},
 				(err) => {
 					return Promise.reject(err)
@@ -216,10 +216,14 @@ export class CardApi {
 	}
 
 	removeLabelFromCard(data) {
-		return axios.delete(this.url(`/cards/${data.card.id}/label/${data.labelId}`))
+		return axios.delete(this.ocsUrl(`/cards/${data.card.id}/label/${data.labelId}`), {
+			data: {
+				boardId: data.boardId ? data.boardId : null,
+			},
+		})
 			.then(
 				(response) => {
-					return Promise.resolve(response.data)
+					return Promise.resolve(response.data.ocs.data)
 				},
 				(err) => {
 					return Promise.reject(err)

src/store/card.js

@@ -355,6 +355,7 @@ export default function cardModuleFactory() {
 				commit('updateCardProperty', { property: 'labels', card: data.card })
 			},
 			async removeLabel({ commit }, data) {
+				data.boardId = this.state.currentBoard.id
 				await apiClient.removeLabelFromCard(data)
 				commit('updateCardProperty', { property: 'labels', card: data.card })
 			},