From 5b406b0172118317c0c89823e93fabce57848698 Mon Sep 17 00:00:00 2001
From: konrad <k@knt.li>
Date: Sun, 31 Oct 2021 13:16:28 +0000
Subject: [PATCH] feat: allow openid users to export their data without a
 password (#918)

Co-authored-by: kolaente <k@knt.li>
Reviewed-on: https://kolaente.dev/vikunja/frontend/pulls/918
Reviewed-by: dpschen <dpschen@noreply.kolaente.de>
Co-authored-by: konrad <k@knt.li>
Co-committed-by: konrad <k@knt.li>
---
 src/views/user/DataExportDownload.vue  | 49 ++++++++++++++----------
 src/views/user/settings/DataExport.vue | 53 +++++++++++++++-----------
 2 files changed, 58 insertions(+), 44 deletions(-)

diff --git a/src/views/user/DataExportDownload.vue b/src/views/user/DataExportDownload.vue
index c8db65419..4c02f7fc8 100644
--- a/src/views/user/DataExportDownload.vue
+++ b/src/views/user/DataExportDownload.vue
@@ -1,27 +1,29 @@
 <template>
 	<div class="content">
 		<h1>{{ $t('user.export.downloadTitle') }}</h1>
-		<p>{{ $t('user.export.descriptionPasswordRequired') }}</p>
-		<div class="field">
-			<label class="label" for="currentPasswordDataExport">
-				{{ $t('user.settings.currentPassword') }}
-			</label>
-			<div class="control">
-				<input
-					class="input"
-					:class="{'is-danger': errPasswordRequired}"
-					id="currentPasswordDataExport"
-					:placeholder="$t('user.settings.currentPasswordPlaceholder')"
-					type="password"
-					v-model="password"
-					@keyup="() => errPasswordRequired = password === ''"
-					ref="passwordInput"
-				/>
+		<template v-if="isLocalUser">
+			<p>{{ $t('user.export.descriptionPasswordRequired') }}</p>
+			<div class="field">
+				<label class="label" for="currentPasswordDataExport">
+					{{ $t('user.settings.currentPassword') }}
+				</label>
+				<div class="control">
+					<input
+						class="input"
+						:class="{'is-danger': errPasswordRequired}"
+						id="currentPasswordDataExport"
+						:placeholder="$t('user.settings.currentPasswordPlaceholder')"
+						type="password"
+						v-model="password"
+						@keyup="() => errPasswordRequired = password === ''"
+						ref="passwordInput"
+					/>
+				</div>
+				<p class="help is-danger" v-if="errPasswordRequired">
+					{{ $t('user.deletion.passwordRequired') }}
+				</p>
 			</div>
-			<p class="help is-danger" v-if="errPasswordRequired">
-				{{ $t('user.deletion.passwordRequired') }}
-			</p>
-		</div>
+		</template>
 
 		<x-button
 			v-focus
@@ -48,9 +50,14 @@ export default {
 	created() {
 		this.dataExportService = new DataExportService()
 	},
+	computed: {
+		isLocalUser() {
+			return this.$store.state.auth.info?.isLocalUser
+		},
+	},
 	methods: {
 		download() {
-			if (this.password === '') {
+			if (this.password === '' && this.isLocalUser) {
 				this.errPasswordRequired = true
 				this.$refs.passwordInput.focus()
 				return
diff --git a/src/views/user/settings/DataExport.vue b/src/views/user/settings/DataExport.vue
index 03236d607..18b00699d 100644
--- a/src/views/user/settings/DataExport.vue
+++ b/src/views/user/settings/DataExport.vue
@@ -3,29 +3,31 @@
 		<p>
 			{{ $t('user.export.description') }}
 		</p>
-		<p>
-			{{ $t('user.export.descriptionPasswordRequired') }}
-		</p>
-		<div class="field">
-			<label class="label" for="currentPasswordDataExport">
-				{{ $t('user.settings.currentPassword') }}
-			</label>
-			<div class="control">
-				<input
-					class="input"
-					:class="{'is-danger': errPasswordRequired}"
-					id="currentPasswordDataExport"
-					:placeholder="$t('user.settings.currentPasswordPlaceholder')"
-					type="password"
-					v-model="password"
-					@keyup="() => errPasswordRequired = password === ''"
-					ref="passwordInput"
-				/>
-			</div>
-			<p class="help is-danger" v-if="errPasswordRequired">
-				{{ $t('user.deletion.passwordRequired') }}
+		<template v-if="isLocalUser">
+			<p>
+				{{ $t('user.export.descriptionPasswordRequired') }}
 			</p>
-		</div>
+			<div class="field">
+				<label class="label" for="currentPasswordDataExport">
+					{{ $t('user.settings.currentPassword') }}
+				</label>
+				<div class="control">
+					<input
+						class="input"
+						:class="{'is-danger': errPasswordRequired}"
+						id="currentPasswordDataExport"
+						:placeholder="$t('user.settings.currentPasswordPlaceholder')"
+						type="password"
+						v-model="password"
+						@keyup="() => errPasswordRequired = password === ''"
+						ref="passwordInput"
+					/>
+				</div>
+				<p class="help is-danger" v-if="errPasswordRequired">
+					{{ $t('user.deletion.passwordRequired') }}
+				</p>
+			</div>
+		</template>
 
 		<x-button
 			:loading="dataExportService.loading"
@@ -48,12 +50,17 @@ export default {
 			errPasswordRequired: false,
 		}
 	},
+	computed: {
+		isLocalUser() {
+			return this.$store.state.auth.info?.isLocalUser
+		},
+	},
 	mounted() {
 		this.setTitle(`${this.$t('user.export.title')} - ${this.$t('user.settings.title')}`)
 	},
 	methods: {
 		async requestDataExport() {
-			if (this.password === '') {
+			if (this.password === '' && this.isLocalUser) {
 				this.errPasswordRequired = true
 				this.$refs.passwordInput.focus()
 				return