From 8396026968907af43002b6d779eb266342385f06 Mon Sep 17 00:00:00 2001
From: Peter Ringelmann <peter.ringelmann@nextcloud.com>
Date: Tue, 26 May 2026 11:04:20 +0200
Subject: [PATCH] refactor(settings): tighten auth-token dialog components

Signed-off-by: Peter Ringelmann <peter.ringelmann@nextcloud.com>
---
 apps/settings/src/components/AuthToken.vue    |   6 +-
 .../src/components/AuthTokenDeleteDialog.vue  | 164 +++++++-----------
 .../src/components/AuthTokenWipeDialog.vue    | 103 ++++-------
 3 files changed, 103 insertions(+), 170 deletions(-)

diff --git a/apps/settings/src/components/AuthToken.vue b/apps/settings/src/components/AuthToken.vue
index d94b9175930..f89e3c5a874 100644
--- a/apps/settings/src/components/AuthToken.vue
+++ b/apps/settings/src/components/AuthToken.vue
@@ -85,14 +85,12 @@
 		<AuthTokenDeleteDialog
 			v-if="deleteDialogOpen"
 			:token="token"
-			:open="deleteDialogOpen"
-			@update:open="deleteDialogOpen = $event"
+			:open.sync="deleteDialogOpen"
 			@confirm="confirmDelete" />
 		<AuthTokenWipeDialog
 			v-if="wipeDialogOpen"
 			:token="token"
-			:open="wipeDialogOpen"
-			@update:open="wipeDialogOpen = $event"
+			:open.sync="wipeDialogOpen"
 			@confirm="confirmWipe" />
 	</tr>
 </template>
diff --git a/apps/settings/src/components/AuthTokenDeleteDialog.vue b/apps/settings/src/components/AuthTokenDeleteDialog.vue
index 2627da0f36a..0085c3b2231 100644
--- a/apps/settings/src/components/AuthTokenDeleteDialog.vue
+++ b/apps/settings/src/components/AuthTokenDeleteDialog.vue
@@ -3,121 +3,83 @@
   - SPDX-License-Identifier: AGPL-3.0-or-later
 -->
 
-<template>
-	<NcDialog
-		:open="open"
-		:name="copy.title"
-		:buttons="buttons"
-		size="normal"
-		@update:open="onUpdateOpen">
-		<NcNoteCard v-if="wiping" type="error">
-			<p class="auth-token-delete-dialog__warning-headline">
-				<strong>{{ t('settings', 'Remote wipe has not started yet.') }}</strong>
-			</p>
-			<p>
-				{{ t('settings', 'Revoking now cancels the wipe. The device keeps its synced data.') }}
-			</p>
-		</NcNoteCard>
-		<p class="auth-token-delete-dialog__body">
-			{{ copy.body }}
-		</p>
-	</NcDialog>
-</template>
-
-<script lang="ts">
+<script setup lang="ts">
 import type { IDialogButton } from '@nextcloud/dialogs'
-import type { PropType } from 'vue'
 import type { IToken } from '../store/authtoken.ts'
 
 import { translate as t } from '@nextcloud/l10n'
-import { defineComponent } from 'vue'
+import { computed } from 'vue'
 import NcDialog from '@nextcloud/vue/components/NcDialog'
 import NcNoteCard from '@nextcloud/vue/components/NcNoteCard'
 import { TokenType } from '../store/authtoken.ts'
 
-export default defineComponent({
-	name: 'AuthTokenDeleteDialog',
+const props = defineProps<{
+	/** The token being revoked */
+	token: IToken
+	/** Whether the dialog is open */
+	open: boolean
+}>()
 
-	components: {
-		NcDialog,
-		NcNoteCard,
-	},
+const emit = defineEmits<{
+	'update:open': [open: boolean]
+	confirm: []
+}>()
 
-	props: {
-		token: {
-			type: Object as PropType<IToken>,
-			required: true,
-		},
+const wiping = computed(() => props.token.type === TokenType.WIPING_TOKEN)
 
-		open: {
-			type: Boolean,
-			required: true,
-		},
-	},
-
-	emits: {
-		'update:open': (open: boolean) => typeof open === 'boolean',
-		confirm: () => true,
-	},
-
-	computed: {
-		wiping(): boolean {
-			return this.token.type === TokenType.WIPING_TOKEN
-		},
-
-		copy(): { title: string, body: string, action: string } {
-			if (this.wiping) {
-				return {
-					title: t('settings', 'Revoke and cancel pending wipe?'),
-					body: t('settings', 'Only continue if you no longer need the device to be wiped.'),
-					action: t('settings', 'Revoke and cancel wipe'),
-				}
-			}
-			return {
-				title: t('settings', 'Revoke app password?'),
-				body: t('settings', 'The app or device will lose access on its next sync. This cannot be undone.'),
-				action: t('settings', 'Revoke'),
-			}
-		},
-
-		buttons(): IDialogButton[] {
-			return [
-				{
-					label: t('settings', 'Cancel'),
-					variant: 'tertiary',
-					callback: () => {
-						this.$emit('update:open', false)
-					},
-				},
-				{
-					label: this.copy.action,
-					variant: 'error',
-					callback: () => {
-						this.$emit('confirm')
-						this.$emit('update:open', false)
-					},
-				},
-			]
-		},
-	},
-
-	methods: {
-		t,
-		onUpdateOpen(value: boolean): void {
-			this.$emit('update:open', value)
-		},
-	},
+const messages = computed(() => {
+	if (wiping.value) {
+		return {
+			title: t('settings', 'Revoke and cancel pending wipe?'),
+			body: t('settings', 'Only continue if you no longer need the device to be wiped.'),
+			action: t('settings', 'Revoke and cancel wipe'),
+		}
+	}
+	return {
+		title: t('settings', 'Revoke app password?'),
+		body: t('settings', 'The app or device will lose access on its next sync. This cannot be undone.'),
+		action: t('settings', 'Revoke'),
+	}
 })
+
+const buttons = computed<IDialogButton[]>(() => [
+	{
+		label: t('settings', 'Cancel'),
+		variant: 'tertiary',
+		callback: () => emit('update:open', false),
+	},
+	{
+		label: messages.value.action,
+		variant: 'error',
+		callback: () => {
+			emit('confirm')
+			emit('update:open', false)
+		},
+	},
+])
 </script>
 
-<style lang="scss" scoped>
-.auth-token-delete-dialog {
-	&__warning-headline {
-		margin-block-end: calc(var(--default-grid-baseline) / 2);
-	}
+<template>
+	<NcDialog
+		:open="open"
+		:name="messages.title"
+		:buttons="buttons"
+		size="normal"
+		@update:open="emit('update:open', $event)">
+		<NcNoteCard
+			v-if="wiping"
+			:heading="t('settings', 'Remote wipe has not started yet.')"
+			type="error">
+			{{ t('settings', 'Revoking now cancels the wipe. The device keeps its synced data.') }}
+		</NcNoteCard>
+		<p class="auth-token-delete-dialog__body">
+			{{ messages.body }}
+		</p>
+	</NcDialog>
+</template>
 
-	&__body {
-		margin-block-start: calc(var(--default-grid-baseline) * 2);
-	}
+<style lang="scss" scoped>
+.auth-token-delete-dialog__body {
+	margin-block-start: calc(var(--default-grid-baseline) * 2);
 }
 </style>
diff --git a/apps/settings/src/components/AuthTokenWipeDialog.vue b/apps/settings/src/components/AuthTokenWipeDialog.vue
index 39f60bf9999..559e3acda8e 100644
--- a/apps/settings/src/components/AuthTokenWipeDialog.vue
+++ b/apps/settings/src/components/AuthTokenWipeDialog.vue
@@ -3,79 +3,52 @@
   - SPDX-License-Identifier: AGPL-3.0-or-later
 -->
 
+<script setup lang="ts">
+import type { IDialogButton } from '@nextcloud/dialogs'
+import type { IToken } from '../store/authtoken.ts'
+
+import { translate as t } from '@nextcloud/l10n'
+import { computed } from 'vue'
+import NcDialog from '@nextcloud/vue/components/NcDialog'
+
+defineProps<{
+	/** The token to wipe. Kept for prop-shape parity with AuthTokenDeleteDialog. */
+	token: IToken
+	/** Whether the dialog is open */
+	open: boolean
+}>()
+
+const emit = defineEmits<{
+	'update:open': [open: boolean]
+	confirm: []
+}>()
+
+const buttons = computed<IDialogButton[]>(() => [
+	{
+		label: t('settings', 'Cancel'),
+		variant: 'tertiary',
+		callback: () => emit('update:open', false),
+	},
+	{
+		label: t('settings', 'Wipe device'),
+		variant: 'error',
+		callback: () => {
+			emit('confirm')
+			emit('update:open', false)
+		},
+	},
+])
+</script>
+
 <template>
 	<NcDialog
 		:open="open"
 		:name="t('settings', 'Confirm wipe')"
 		:buttons="buttons"
 		size="normal"
-		@update:open="onUpdateOpen">
+		@update:open="emit('update:open', $event)">
 		<p>
 			{{ t('settings', 'Do you really want to wipe your data from this device?') }}
 		</p>
 	</NcDialog>
 </template>
-
-<script lang="ts">
-import type { IDialogButton } from '@nextcloud/dialogs'
-import type { PropType } from 'vue'
-import type { IToken } from '../store/authtoken.ts'
-
-import { translate as t } from '@nextcloud/l10n'
-import { defineComponent } from 'vue'
-import NcDialog from '@nextcloud/vue/components/NcDialog'
-
-export default defineComponent({
-	name: 'AuthTokenWipeDialog',
-
-	components: {
-		NcDialog,
-	},
-
-	props: {
-		token: {
-			type: Object as PropType<IToken>,
-			required: true,
-		},
-
-		open: {
-			type: Boolean,
-			required: true,
-		},
-	},
-
-	emits: {
-		'update:open': (open: boolean) => typeof open === 'boolean',
-		confirm: () => true,
-	},
-
-	computed: {
-		buttons(): IDialogButton[] {
-			return [
-				{
-					label: t('settings', 'Cancel'),
-					variant: 'tertiary',
-					callback: () => {
-						this.$emit('update:open', false)
-					},
-				},
-				{
-					label: t('settings', 'Wipe device'),
-					variant: 'error',
-					callback: () => {
-						this.$emit('confirm')
-						this.$emit('update:open', false)
-					},
-				},
-			]
-		},
-	},
-
-	methods: {
-		t,
-		onUpdateOpen(value: boolean): void {
-			this.$emit('update:open', value)
-		},
-	},
-})
-</script>