Merge pull request #6673 from LibreSign/refactor/extract-logic-to-choose-the-action

refactor: extract logic to choose the action

Author: vitormattos

src/helpers/SigningActionHelper.js

@@ -0,0 +1,36 @@
+/**
+ * SPDX-FileCopyrightText: 2026 LibreCode coop and LibreCode contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+/**
+ * Determine the primary action to execute based on current signing state
+ * @param {Object} signStore - Sign store instance
+ * @param {Object} signMethodsStore - SignMethods store instance
+ * @param {Boolean} needCreateSignature - Whether signature needs to be created
+ * @param {Boolean} needIdentificationDocuments - Whether identification documents are needed
+ * @returns {Object|null} - Action object with action name and callback or null if unable to sign
+ */
+export function getPrimarySigningAction(signStore, signMethodsStore, needCreateSignature, needIdentificationDocuments) {
+	if (signMethodsStore.needCertificate()) {
+		return { action: 'uploadCertificate' }
+	}
+
+	if (signMethodsStore.needCreatePassword()) {
+		return { action: 'createPassword' }
+	}
+
+	if (needCreateSignature) {
+		return { action: 'createSignature' }
+	}
+
+	if (needIdentificationDocuments) {
+		return { action: 'documents' }
+	}
+
+	if (signStore.errors.length > 0) {
+		return null
+	}
+
+	return { action: 'sign' }
+}

src/views/SignPDF/_partials/Sign.vue

@@ -175,6 +175,7 @@ import { useSignatureElementsStore } from '../../../store/signatureElements.js'
 import { useSignMethodsStore } from '../../../store/signMethods.js'
 import { useIdentificationDocumentStore } from '../../../store/identificationDocument.js'
 import { FILE_STATUS } from '../../../constants.js'
+import { getPrimarySigningAction } from '../../../helpers/SigningActionHelper.js'
 
 export default {
 	name: 'Sign',
@@ -255,19 +256,13 @@ export default {
 			return getCapabilities()?.libresign?.config?.['sign-elements']?.['can-create-signature'] === true
 		},
 		ableToSign() {
-			if (this.signMethodsStore.needCreatePassword()) {
-				return false
-			}
-			if (this.needCreateSignature) {
-				return false
-			}
-			if (this.needIdentificationDocuments) {
-				return false
-			}
-			if (this.signStore.errors.length > 0) {
-				return false
-			}
-			return true
+			const primaryAction = getPrimarySigningAction(
+				this.signStore,
+				this.signMethodsStore,
+				this.needCreateSignature,
+				this.needIdentificationDocuments
+			)
+			return primaryAction?.action === 'sign'
 		},
 		signRequestUuid() {
 			const doc = this.signStore.document || {}