Merge pull request #50524 from nextcloud/fix/destination-drop-check

nextcloud · Jan 30, 2025 · af8189f · af8189f
2 parents 16921d7 + e7d9985
commit af8189f
Show file tree

Hide file tree

Showing 9 changed files with 16 additions and 11 deletions.
diff --git a/apps/files_sharing/src/components/NewFileRequestDialog.vue b/apps/files_sharing/src/components/NewFileRequestDialog.vue
@@ -218,10 +218,9 @@ export default defineComponent({
 	methods: {
 		onPageNext() {
 			const form = this.$refs.form as HTMLFormElement
-			if (!form.checkValidity()) {
-				form.reportValidity()
-				return
-			}
+
+			// Reset custom validity
+			form.querySelectorAll('input').forEach(input => input.setCustomValidity(''))
 
 			// custom destination validation
 			// cannot share root
@@ -232,6 +231,12 @@ export default defineComponent({
 				return
 			}
 
+			// If the form is not valid, show the error message
+			if (!form.checkValidity()) {
+				form.reportValidity()
+				return
+			}
+
 			if (this.currentStep === STEP.FIRST) {
 				this.currentStep = STEP.SECOND
 				return

diff --git a/dist/1191-1191.js.map b/dist/1191-1191.js.map
diff --git a/dist/1191-1191.js.map.license b/dist/1191-1191.js.map.license
diff --git a/dist/1191-1191.js → dist/9835-9835.js b/dist/1191-1191.js → dist/9835-9835.js
diff --git a/dist/1191-1191.js.license → dist/9835-9835.js.license b/dist/1191-1191.js.license → dist/9835-9835.js.license
diff --git a/dist/9835-9835.js.map b/dist/9835-9835.js.map
diff --git a/dist/9835-9835.js.map.license b/dist/9835-9835.js.map.license
@@ -0,0 +1 @@
+9835-9835.js.license
diff --git a/dist/files_sharing-init.js b/dist/files_sharing-init.js
diff --git a/dist/files_sharing-init.js.map b/dist/files_sharing-init.js.map