Merge pull request #26058 from nextcloud/backport/25911/stable21

[stable21] Only clear share password model when actually saved
nextcloud · Mar 11, 2021 · 04dd7b2 · 04dd7b2
2 parents 1d0b210 + ec44315
commit 04dd7b2
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/apps/files_sharing/js/dist/files_sharing_tab.js b/apps/files_sharing/js/dist/files_sharing_tab.js
diff --git a/apps/files_sharing/js/dist/files_sharing_tab.js.map b/apps/files_sharing/js/dist/files_sharing_tab.js.map
diff --git a/apps/files_sharing/src/mixins/SharesMixin.js b/apps/files_sharing/src/mixins/SharesMixin.js
@@ -248,11 +248,14 @@ export default {
 					try {
 						await this.updateShare(this.share.id, properties)
 
+						if (propertyNames.indexOf('password') >= 0) {
+							// reset password state after sync
+							this.$delete(this.share, 'newPassword')
+						}
+
 						// clear any previous errors
 						this.$delete(this.errors, propertyNames[0])
 
-						// reset password state after sync
-						this.$delete(this.share, 'newPassword')
 					} catch ({ message }) {
 						if (message && message !== '') {
 							this.onSyncError(propertyNames[0], message)