Skip to content

Commit 1158319

Browse files
authored
Merge pull request #56740 from nextcloud/fix/sharing-password-removal
fix(files_sharing): Allow removing password from link shares
2 parents 66d07f4 + f46f41e commit 1158319

File tree

5 files changed

+8
-8
lines changed

5 files changed

+8
-8
lines changed

apps/files_sharing/src/mixins/SharesMixin.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ export default {
179179
this.$set(this.share, 'newPassword', await GeneratePassword(true))
180180
} else {
181181
this.passwordProtectedState = false
182-
this.$delete(this.share, 'newPassword')
182+
this.$set(this.share, 'newPassword', '')
183183
}
184184
},
185185
},
@@ -339,7 +339,7 @@ export default {
339339

340340
if (propertyNames.includes('password')) {
341341
// reset password state after sync
342-
this.share.password = this.share.newPassword ?? ''
342+
this.share.password = this.share.newPassword || undefined
343343
this.$delete(this.share, 'newPassword')
344344

345345
// updates password expiration time after sync

dist/861-861.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/861-861.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/files_sharing-files_sharing_tab.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/files_sharing-files_sharing_tab.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)