averello/nextcloud-server-mirror
1
0
Fork 0
mirror of https://github.com/nextcloud/server.git synced 2026-03-04 18:28:08 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: No password set for new mail shares

Browse Source 
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 : https://github.com/nextcloud/server/issues/43919

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
This commit is contained in:
fenn-cs
2024-03-14 00:43:10 +01:00
parent 078f51beeb
commit f9bef8f378
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
apps/files_sharing/src/views/SharingDetailsTab.vue
View File

@@ -752,7 +752,7 @@ export default {
if (this.isNewShare) {
if (this.isPasswordEnforced && this.isPublicShare) {
this.share.newPassword = await GeneratePassword()
this.$set(this.share, 'newPassword', await GeneratePassword())
this.advancedSectionAccordionExpanded = true
}
/* Set default expiration dates if configured */