From 93719f1c50b23f5564befec2dae4dc7865920ac3 Mon Sep 17 00:00:00 2001
From: Vitor Mattos <vitor@php.rio>
Date: Thu, 16 Jan 2025 12:00:09 -0300
Subject: [PATCH] fix: handle settings after backend upgrade

Signed-off-by: Vitor Mattos <vitor@php.rio>
---
 lib/Controller/FileController.php      | 6 ++++++
 src/views/CreatePassword.vue           | 4 ++--
 src/views/ReadCertificate.vue          | 4 ++--
 src/views/ResetPassword.vue            | 4 ++--
 src/views/Settings/CollectMetadata.vue | 3 ++-
 src/views/Settings/Validation.vue      | 9 ++++++---
 6 files changed, 20 insertions(+), 10 deletions(-)

diff --git a/lib/Controller/FileController.php b/lib/Controller/FileController.php
index 0efb5d608a..a30522e869 100644
--- a/lib/Controller/FileController.php
+++ b/lib/Controller/FileController.php
@@ -35,6 +35,7 @@
 use OCP\Files\File;
 use OCP\Files\Node;
 use OCP\Files\NotFoundException;
+use OCP\IAppConfig;
 use OCP\IL10N;
 use OCP\IPreview;
 use OCP\IRequest;
@@ -64,6 +65,7 @@ public function __construct(
 		private RequestSignatureService $requestSignatureService,
 		private AccountService $accountService,
 		private IPreview $preview,
+		private IAppConfig $appConfig,
 		private IMimeIconProvider $mimeIconProvider,
 		private FileService $fileService,
 		private ValidateHelper $validateHelper,
@@ -178,6 +180,10 @@ public function validateBinary(): DataResponse {
 	#[ApiRoute(verb: 'GET', url: '/api/{apiVersion}/file/validate/', requirements: ['apiVersion' => '(v1)'])]
 	public function validate(?string $type = null, $identifier = null): DataResponse {
 		try {
+			$isValidationUrlPrivate = (bool)$this->appConfig->getValueBool(Application::APP_ID, 'make_validation_url_private', false);
+			if ($isValidationUrlPrivate) {
+				throw new LibresignException($this->l10n->t('You are not logged in. Please log in.'));
+			}
 			if ($type === 'Uuid' && !empty($identifier)) {
 				try {
 					$this->fileService
diff --git a/src/views/CreatePassword.vue b/src/views/CreatePassword.vue
index b71b65f24f..1abedde842 100644
--- a/src/views/CreatePassword.vue
+++ b/src/views/CreatePassword.vue
@@ -16,8 +16,8 @@
 		<template #actions>
 			<NcButton :disabled="hasLoading"
 				native-type="submit"
-				@click="send()"
-				type="primary">
+				type="primary"
+				@click="send()">
 				<template #icon>
 					<NcLoadingIcon v-if="hasLoading" :size="20" />
 				</template>
diff --git a/src/views/ReadCertificate.vue b/src/views/ReadCertificate.vue
index 1430a93c89..48221489b5 100644
--- a/src/views/ReadCertificate.vue
+++ b/src/views/ReadCertificate.vue
@@ -85,8 +85,8 @@
 		<template v-if="Object.keys(certificateData).length === 0" #actions>
 			<NcButton :disabled="hasLoading"
 				native-type="submit"
-				@click="send()"
-				type="primary">
+				type="primary"
+				@click="send()">
 				<template #icon>
 					<NcLoadingIcon v-if="hasLoading" :size="20" />
 				</template>
diff --git a/src/views/ResetPassword.vue b/src/views/ResetPassword.vue
index b2962d71da..978edd5c9a 100644
--- a/src/views/ResetPassword.vue
+++ b/src/views/ResetPassword.vue
@@ -22,8 +22,8 @@
 			<NcButton :disabled="!canSave"
 				:class="hasLoading ? 'btn-load loading primary btn-confirm' : 'primary btn-confirm'"
 				native-type="submit"
-				@click="send()"
-				type="primary">
+				type="primary"
+				@click="send()">
 				<template #icon>
 					<NcLoadingIcon v-if="hasLoading" :size="20" />
 				</template>
diff --git a/src/views/Settings/CollectMetadata.vue b/src/views/Settings/CollectMetadata.vue
index 51de819547..12310d317b 100644
--- a/src/views/Settings/CollectMetadata.vue
+++ b/src/views/Settings/CollectMetadata.vue
@@ -38,7 +38,8 @@ export default {
 	methods: {
 		async getData() {
 			const responseCollectMetadata = await axios.get(generateOcsUrl('/apps/provisioning_api/api/v1/config/apps/libresign/collect_metadata'))
-			this.collectMetadataEnabled = !!responseCollectMetadata.data.ocs.data.data
+			const value = responseCollectMetadata?.data?.ocs?.data?.data
+			this.collectMetadataEnabled = value === true || value === 'true'
 		},
 		saveCollectMetadata() {
 			OCP.AppConfig.setValue('libresign', 'collect_metadata', this.collectMetadataEnabled ? 1 : 0)
diff --git a/src/views/Settings/Validation.vue b/src/views/Settings/Validation.vue
index a395a1407c..2a4316d846 100644
--- a/src/views/Settings/Validation.vue
+++ b/src/views/Settings/Validation.vue
@@ -96,19 +96,22 @@ export default {
 			const response = await axios.get(
 				generateOcsUrl('/apps/provisioning_api/api/v1/config/apps/libresign/make_validation_url_private'),
 			)
-			this.makeValidationUrlPrivate = response.data.ocs.data.data === true
+			const value = response?.data?.ocs?.data.data
+			this.makeValidationUrlPrivate = value === true || value === 'true'
 		},
 		async getAddFooterData() {
 			const response = await axios.get(
 				generateOcsUrl('/apps/provisioning_api/api/v1/config/apps/libresign/add_footer'),
 			)
-			this.addFooter = response.data.ocs.data.data !== '0'
+			const value = response?.data?.ocs?.data.data
+			this.addFooter = value === true || value === 'true'
 		},
 		async getWriteQrcodeOnFooter() {
 			const response = await axios.get(
 				generateOcsUrl('/apps/provisioning_api/api/v1/config/apps/libresign/write_qrcode_on_footer'),
 			)
-			this.writeQrcodeOnFooter = response.data.ocs.data.data !== '0'
+			const value = response?.data?.ocs?.data.data
+			this.writeQrcodeOnFooter = value === true || value === 'true'
 		},
 		async getValidationUrlData() {
 			const response = await axios.get(