Add new cards to top or bottom

.github/workflows/lint-eslint.yml

@@ -28,7 +28,7 @@ jobs:
       src: ${{ steps.changes.outputs.src}}
 
     steps:
-      - uses: dorny/paths-filter@fbd0ab8f3e69293af611ebaee6363fc25e6d187d # v4.0.1
+      - uses: dorny/paths-filter@7b450fff21473bca461d4b92ce414b9d0420d706 # v4.0.2
         id: changes
         continue-on-error: true
         with:

.github/workflows/node.yml

@@ -28,7 +28,7 @@ jobs:
       src: ${{ steps.changes.outputs.src}}
 
     steps:
-      - uses: dorny/paths-filter@fbd0ab8f3e69293af611ebaee6363fc25e6d187d # v4.0.1
+      - uses: dorny/paths-filter@7b450fff21473bca461d4b92ce414b9d0420d706 # v4.0.2
         id: changes
         continue-on-error: true
         with:

.github/workflows/phpunit-mysql.yml

@@ -44,7 +44,7 @@ jobs:
       src: ${{ steps.changes.outputs.src}}
 
     steps:
-      - uses: dorny/paths-filter@fbd0ab8f3e69293af611ebaee6363fc25e6d187d # v4.0.1
+      - uses: dorny/paths-filter@7b450fff21473bca461d4b92ce414b9d0420d706 # v4.0.2
         id: changes
         continue-on-error: true
         with:

.github/workflows/phpunit-pgsql.yml

@@ -43,7 +43,7 @@ jobs:
       src: ${{ steps.changes.outputs.src }}
 
     steps:
-      - uses: dorny/paths-filter@fbd0ab8f3e69293af611ebaee6363fc25e6d187d # v4.0.1
+      - uses: dorny/paths-filter@7b450fff21473bca461d4b92ce414b9d0420d706 # v4.0.2
         id: changes
         continue-on-error: true
         with:

.github/workflows/phpunit-sqlite.yml

@@ -43,7 +43,7 @@ jobs:
       src: ${{ steps.changes.outputs.src}}
 
     steps:
-      - uses: dorny/paths-filter@fbd0ab8f3e69293af611ebaee6363fc25e6d187d # v4.0.1
+      - uses: dorny/paths-filter@7b450fff21473bca461d4b92ce414b9d0420d706 # v4.0.2
         id: changes
         continue-on-error: true
         with:

cypress/e2e/cardColor.js

@@ -46,7 +46,7 @@ describe('Card color', function () {
 
 		const newCardTitle = 'Card with color'
 
-		cy.get('.button-vue[aria-label*="Add card"]')
+		cy.get('[data-cy="action:add-card"]')
 			.first().click()
 		cy.get('.stack__card-add form input#new-stack-input-main')
 			.type(newCardTitle)

cypress/e2e/cardFeatures.js

@@ -25,6 +25,17 @@ const useModal = (useModal) => {
 	})
 }
 
+const addCardsAtTop = (enabled) => {
+	return cy.request({
+		method: 'POST',
+		url: `${Cypress.env('baseUrl')}/ocs/v2.php/apps/deck/api/v1.0/config/stackAddCardAtTop?format=json`,
+		auth,
+		body: { value: enabled },
+	}).then((response) => {
+		expect(response.status).to.eq(200)
+	})
+}
+
 describe('Card', function () {
 	let boardId
 	before(function () {
@@ -52,7 +63,7 @@ describe('Card', function () {
 		cy.get('.board .stack').eq(0).within(() => {
 			cy.get('.card:contains("Hello world")').should('be.visible')
 
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.first().click()
 
 			cy.get('.stack__card-add form input#new-stack-input-main')
@@ -63,13 +74,78 @@ describe('Card', function () {
 		})
 	})
 
+	describe('New card position', function() {
+		afterEach(function() {
+			addCardsAtTop(false)
+		})
+
+		it('Adds new cards to the configured side of the list', function() {
+			const bottomCardTitle = 'Card added at bottom'
+			const firstTopCardTitle = 'First card added at top'
+			const secondTopCardTitle = 'Second card added at top'
+			const finalBottomCardTitle = 'Card added at bottom again'
+
+			addCardsAtTop(false)
+			cy.intercept({ method: 'POST', url: '**/ocs/v2.php/apps/deck/api/v1.0/cards' }).as('createCard')
+			cy.intercept({ method: 'PUT', url: '**/ocs/v2.php/apps/deck/api/v1.0/cards/*/reorder' }).as('reorderCard')
+			cy.intercept({ method: 'POST', url: '**/ocs/v2.php/apps/deck/api/v1.0/config/stackAddCardAtTop' }).as('setCardPosition')
+			cy.visit(`/apps/deck/#/board/${boardId}`)
+
+			cy.get('.board .stack').eq(0).within(() => {
+				cy.get('[data-cy="action:add-card"]').click()
+				cy.get('.stack__card-add input[type="text"]').type(bottomCardTitle)
+				cy.get('.stack__card-add input[type="submit"]').click()
+				cy.wait('@createCard')
+				cy.get('.card').last().should('contain', bottomCardTitle)
+			})
+
+			cy.get('[data-cy="navigation:settings"]').click()
+			cy.get('[data-cy="setting:add-card-at-top"] input[role="switch"]').check({ force: true })
+			cy.wait('@setCardPosition')
+			cy.visit(`/apps/deck/#/board/${boardId}`)
+
+			for (const title of [firstTopCardTitle, secondTopCardTitle]) {
+				cy.get('.board .stack').eq(0).within(() => {
+					cy.get('[data-cy="action:add-card"]').click()
+					cy.get('.stack__card-add input[type="text"]').type(title)
+					cy.get('.stack__card-add input[type="submit"]').click()
+					cy.wait('@createCard')
+				})
+			}
+
+			cy.get('.board .stack').eq(0).within(() => {
+				cy.get('.card').eq(0).should('contain', secondTopCardTitle)
+				cy.get('.card').eq(1).should('contain', firstTopCardTitle)
+			})
+			cy.get('@reorderCard.all').should('have.length', 0)
+
+			cy.reload()
+			cy.get('.board .stack').eq(0).within(() => {
+				cy.get('.card').eq(0).should('contain', secondTopCardTitle)
+				cy.get('.card').eq(1).should('contain', firstTopCardTitle)
+			})
+
+			cy.get('[data-cy="navigation:settings"]').click()
+			cy.get('[data-cy="setting:add-card-at-top"] input[role="switch"]').uncheck({ force: true })
+			cy.wait('@setCardPosition')
+			cy.visit(`/apps/deck/#/board/${boardId}`)
+			cy.get('.board .stack').eq(0).within(() => {
+				cy.get('[data-cy="action:add-card"]').click()
+				cy.get('.stack__card-add input[type="text"]').type(finalBottomCardTitle)
+				cy.get('.stack__card-add input[type="submit"]').click()
+				cy.wait('@createCard')
+				cy.get('.card').last().should('contain', finalBottomCardTitle)
+			})
+		})
+	})
+
 	it('Create card from overview', function () {
 		cy.visit(`/apps/deck/#/`)
 		const newCardTitle = 'Test create from overview'
 		cy.intercept({ method: 'POST', url: '**/ocs/v2.php/apps/deck/api/v1.0/cards' }).as('save')
 		cy.intercept({ method: 'GET', url: '**/apps/deck/boards/*' }).as('getBoard')
 
-		cy.get('.button-vue[aria-label*="Add card"]')
+		cy.get('[data-cy="action:add-card"]')
 			.first().click()
 
 		// Somehow this avoids the electron crash
@@ -98,7 +174,7 @@ describe('Card', function () {
 		cy.visit(`/apps/deck/#/board/${boardId}`)
 		const absoluteUrl = `https://example.com`
 		cy.get('.board .stack').eq(0).within(() => {
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.first().click()
 
 			cy.get('.stack__card-add form input#new-stack-input-main')
@@ -120,7 +196,7 @@ describe('Card', function () {
 		const absoluteUrl = `https://example.com`
 		const plainTitle = 'New title'
 		cy.get('.board .stack').eq(0).within(() => {
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.first().click()
 
 			cy.get('.stack__card-add form input#new-stack-input-main')
@@ -238,7 +314,9 @@ describe('Card', function () {
 			cy.visit(`/apps/deck/board/${boardId}`)
 			cy.reload()
 			cy.get('.board .stack').eq(0).within(() => {
-				cy.get(`.card:contains("${newCardTitle}")`).should('be.visible')
+				cy.get(`.card:contains("${newCardTitle}")`)
+					.scrollIntoView({ block: 'center' })
+					.should('be.visible')
 			})
 		})
 	})
@@ -260,7 +338,7 @@ describe('Card', function () {
 		it('Set a due date', function () {
 			const newCardTitle = 'Card with a due date'
 
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.first().click()
 			cy.get('.stack__card-add form input#new-stack-input-main')
 				.type(newCardTitle)
@@ -296,7 +374,7 @@ describe('Card', function () {
 		it('Add a label', function () {
 			const newCardTitle = 'Card with labels'
 
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.first().click()
 			cy.get('.stack__card-add form input#new-stack-input-main')
 				.type(newCardTitle)
@@ -316,7 +394,9 @@ describe('Card', function () {
 			cy.get('.vs__selected .tag:contains("Action needed")')
 				.parent().find('button').click()
 
-			cy.get(`.card:contains("${newCardTitle}")`).find('.labels li:contains("Later")')
+			cy.get(`.card:contains("${newCardTitle}")`)
+				.scrollIntoView({ block: 'center' })
+				.find('.labels li:contains("Later")')
 				.should('be.visible')
 			cy.get(`.card:contains("${newCardTitle}")`).find('.labels li:contains("Action needed")')
 				.should('not.exist')

cypress/e2e/sharingFeatures.js

@@ -32,7 +32,7 @@ describe('Board', function() {
 			cy.login(recipient)
 			cy.visit(`/apps/deck/#/board/${boardId}`)
 			cy.get('.board-title').contains(board.title)
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.should('not.exist')
 		})
 	})
@@ -50,7 +50,7 @@ describe('Board', function() {
 			cy.login(recipient)
 			cy.visit(`/apps/deck/#/board/${boardId}`)
 			cy.get('.board-title').contains(board.title)
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.should('not.exist')
 		})
 	})
@@ -71,7 +71,7 @@ describe('Board', function() {
 			cy.login(recipient)
 			cy.visit(`/apps/deck/#/board/${boardId}`)
 			cy.get('.board-title').contains(board.title)
-			cy.get('.button-vue[aria-label*="Add card"]')
+			cy.get('[data-cy="action:add-card"]')
 				.first().click()
 		})
 	})

l10n/fr.js

@@ -212,6 +212,7 @@ OC.L10N.register(
     "Mark card as completed/not completed" : "Marquer la carte (non) terminée",
     "Open card menu" : "Ouvrir le menu de la carte",
     "Currently present people" : "Personnes actuellement présentes",
+    "Search cards" : "Rechercher des cartes",
     "Loading board" : "Chargement du tableau…",
     "Board not found" : "Tableau non trouvé",
     "Create a new list to add cards to this board" : "Créer une nouvelle liste pour ajouter des cartes à ce tableau",
@@ -258,6 +259,7 @@ OC.L10N.register(
     "title and color value must be provided" : "Le titre et la couleur doivent être renseignés",
     "Edit" : "Modifier",
     "Add a new tag" : "Ajouter une nouvelle étiquette",
+    "Search boards" : "Rechercher des tableaux",
     "Board name" : "Nom du tableau",
     "Members" : "Membres",
     "Assign to users/groups/team" : "Attribuer à des utilisateurs / groupes / équipes",

l10n/fr.json

@@ -210,6 +210,7 @@
     "Mark card as completed/not completed" : "Marquer la carte (non) terminée",
     "Open card menu" : "Ouvrir le menu de la carte",
     "Currently present people" : "Personnes actuellement présentes",
+    "Search cards" : "Rechercher des cartes",
     "Loading board" : "Chargement du tableau…",
     "Board not found" : "Tableau non trouvé",
     "Create a new list to add cards to this board" : "Créer une nouvelle liste pour ajouter des cartes à ce tableau",
@@ -256,6 +257,7 @@
     "title and color value must be provided" : "Le titre et la couleur doivent être renseignés",
     "Edit" : "Modifier",
     "Add a new tag" : "Ajouter une nouvelle étiquette",
+    "Search boards" : "Rechercher des tableaux",
     "Board name" : "Nom du tableau",
     "Members" : "Membres",
     "Assign to users/groups/team" : "Attribuer à des utilisateurs / groupes / équipes",

l10n/uk.js

@@ -315,16 +315,16 @@ OC.L10N.register(
     "Choose attachment" : "Вибрати вкладення",
     "Select Date" : "Вкажіть дату",
     "Later today – {timeLocale}" : "Пізніше сьогодні - {timeLocale}",
-    "Set due date for later today" : "Встановіть термін сплати на сьогоднішній день",
+    "Set due date for later today" : "Встановіть термін завершення на пізніше сьогодні",
     "Tomorrow – {timeLocale}" : "Завтра - {timeLocale}",
     "Set due date for tomorrow" : "Встановіть дедлайн на завтра",
     "This weekend – {timeLocale}" : "На цих вихідних - {timeLocale}",
     "Set due date for this weekend" : "Встановіть термін виконання на ці вихідні",
     "Next week – {timeLocale}" : "Наступного тижня - {timeLocale}",
     "Set due date for next week" : "Встановіть термін виконання на наступний тиждень",
-    "Assign a due date to this card…" : "Призначте цій картці термін оплати...",
+    "Assign a due date to this card…" : "Призначте цій картці термін завершення...",
     "Set a due date" : "Встановити дату завершення",
-    "Add due date" : "Додати термін сплати",
+    "Add due date" : "Додати термін завершення",
     "Choose a date" : "Виберіть дату",
     "Remove due date" : "Вилучити кінцеву дату",
     "Due at:" : "На часі:",
@@ -382,7 +382,7 @@ OC.L10N.register(
     "Delete board" : "Вилучити дошку",
     "Leave board" : "Залишити дошку",
     "Importing board..." : "Імпортування дошки...",
-    "Board imported successfully" : "Плату успішно імпортовано",
+    "Board imported successfully" : "Дошку успішно імпортовано",
     "Import board" : "Імпорт дошки",
     "Clone {boardTitle}" : "Копіювати {boardTitle}",
     "Clone cards" : "Копіювати картки",

l10n/uk.json

@@ -313,16 +313,16 @@
     "Choose attachment" : "Вибрати вкладення",
     "Select Date" : "Вкажіть дату",
     "Later today – {timeLocale}" : "Пізніше сьогодні - {timeLocale}",
-    "Set due date for later today" : "Встановіть термін сплати на сьогоднішній день",
+    "Set due date for later today" : "Встановіть термін завершення на пізніше сьогодні",
     "Tomorrow – {timeLocale}" : "Завтра - {timeLocale}",
     "Set due date for tomorrow" : "Встановіть дедлайн на завтра",
     "This weekend – {timeLocale}" : "На цих вихідних - {timeLocale}",
     "Set due date for this weekend" : "Встановіть термін виконання на ці вихідні",
     "Next week – {timeLocale}" : "Наступного тижня - {timeLocale}",
     "Set due date for next week" : "Встановіть термін виконання на наступний тиждень",
-    "Assign a due date to this card…" : "Призначте цій картці термін оплати...",
+    "Assign a due date to this card…" : "Призначте цій картці термін завершення...",
     "Set a due date" : "Встановити дату завершення",
-    "Add due date" : "Додати термін сплати",
+    "Add due date" : "Додати термін завершення",
     "Choose a date" : "Виберіть дату",
     "Remove due date" : "Вилучити кінцеву дату",
     "Due at:" : "На часі:",
@@ -380,7 +380,7 @@
     "Delete board" : "Вилучити дошку",
     "Leave board" : "Залишити дошку",
     "Importing board..." : "Імпортування дошки...",
-    "Board imported successfully" : "Плату успішно імпортовано",
+    "Board imported successfully" : "Дошку успішно імпортовано",
     "Import board" : "Імпорт дошки",
     "Clone {boardTitle}" : "Копіювати {boardTitle}",
     "Clone cards" : "Копіювати картки",

lib/Controller/CardOcsController.php

@@ -21,6 +21,12 @@
 use OCP\IRequest;
 
 class CardOcsController extends OCSController {
+	/**
+	 * Sentinel order used when the client does not request a specific position.
+	 * Cards created with this order are appended to the end of the stack.
+	 */
+	private const DEFAULT_ORDER = 999;
+
 	public function __construct(
 		string $appName,
 		IRequest $request,
@@ -36,7 +42,7 @@ public function __construct(
 
 	#[NoAdminRequired]
 	#[PublicPage]
-	public function create(string $title, int $stackId, ?int $boardId = null, ?string $type = 'plain', ?string $owner = null, ?int $order = 999, ?string $description = '', $duedate = null, $startdate = null, ?array $labels = [], ?array $users = [], ?string $color = null) {
+	public function create(string $title, int $stackId, ?int $boardId = null, ?string $type = 'plain', ?string $owner = null, ?int $order = self::DEFAULT_ORDER, ?string $description = '', $duedate = null, $startdate = null, ?array $labels = [], ?array $users = [], ?string $color = null) {
 		if ($boardId) {
 			$board = $this->boardService->find($boardId, false);
 			if ($board->getExternalId()) {
@@ -48,7 +54,8 @@ public function create(string $title, int $stackId, ?int $boardId = null, ?strin
 		if (!$owner) {
 			$owner = $this->userId;
 		}
-		$card = $this->cardService->create($title, $stackId, $type, $order, $owner, $description, $duedate, $startdate, $color);
+		// An explicit order means the client wants the card at that position, so shift the surrounding cards
+		$card = $this->cardService->create($title, $stackId, $type, $order, $owner, $description, $duedate, $startdate, $color, insertAtPosition: $order !== self::DEFAULT_ORDER);
 
 		// foreach ($labels as $label) {
 		// 	$this->assignLabel($card->getId(), $label);