Skip to content

Commit 6570f87

Browse files
nfebesusnux
authored andcommitted
fix: No password set for new mail shares
Before the password is set on a new share, the computed property `hasUnsavedPassword` is used to check that `this.share.newPassword` is not undefined. Direct assignment without using Vue's `this.$set` makes it impossible for vue to detect that changes have happened on the share object. Hence the inreactivity. This worked initially most likely because `this.share.newPassword = await GeneratePassword()` was executed before the computed properties where evaluated. Resolves : #43919 Signed-off-by: fenn-cs <[email protected]>
1 parent 9f0028c commit 6570f87

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

apps/files_sharing/src/views/SharingDetailsTab.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -708,7 +708,7 @@ export default {
708708
709709
if (this.isNewShare) {
710710
if (this.isPasswordEnforced && this.isPublicShare) {
711-
this.share.newPassword = await GeneratePassword()
711+
this.$set(this.share, 'newPassword', await GeneratePassword())
712712
this.advancedSectionAccordionExpanded = true
713713
}
714714
/* Set default expiration dates if configured */

0 commit comments

Comments
 (0)