Skip to content

Commit

Permalink
Merge pull request #20469 from nextcloud/fix/sidebar/sharing/password…
Browse files Browse the repository at this point in the history
…-error-message

Properly display share error messages
  • Loading branch information
skjnldsv authored Apr 14, 2020
2 parents 09bb8ac + 47146a1 commit e4d378d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
2 changes: 1 addition & 1 deletion apps/files_sharing/js/dist/files_sharing_tab.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion apps/files_sharing/js/dist/files_sharing_tab.js.map

Large diffs are not rendered by default.

6 changes: 4 additions & 2 deletions apps/files_sharing/src/mixins/SharesMixin.js
Original file line number Diff line number Diff line change
Expand Up @@ -246,8 +246,10 @@ export default {

// reset password state after sync
this.$delete(this.share, 'newPassword')
} catch ({ property, message }) {
this.onSyncError(property, message)
} catch ({ message }) {
if (message && message !== '') {
this.onSyncError(property, message)
}
} finally {
this.saving = false
}
Expand Down

0 comments on commit e4d378d

Please sign in to comment.