diff options
author | Chocobozzz <me@florianbigard.com> | 2020-03-20 14:43:12 +0100 |
---|---|---|
committer | Chocobozzz <me@florianbigard.com> | 2020-03-20 14:43:12 +0100 |
commit | 6cb556449064278bdcb1a54a7b980c2e7bc735f0 (patch) | |
tree | a3d72e90cfc6074a00cd04d5197dded12260b69e /client/src/app/videos/+video-watch/comment | |
parent | 5db4545cd2af2298ef6514dde2c36edc98ef5e33 (diff) | |
download | PeerTube-6cb556449064278bdcb1a54a7b980c2e7bc735f0.tar.gz PeerTube-6cb556449064278bdcb1a54a7b980c2e7bc735f0.tar.zst PeerTube-6cb556449064278bdcb1a54a7b980c2e7bc735f0.zip |
Fix add comment in threads with deleted comments
Diffstat (limited to 'client/src/app/videos/+video-watch/comment')
3 files changed, 9 insertions, 4 deletions
diff --git a/client/src/app/videos/+video-watch/comment/video-comment-add.component.ts b/client/src/app/videos/+video-watch/comment/video-comment-add.component.ts index 5784efcdf..0f7c19765 100644 --- a/client/src/app/videos/+video-watch/comment/video-comment-add.component.ts +++ b/client/src/app/videos/+video-watch/comment/video-comment-add.component.ts | |||
@@ -57,7 +57,7 @@ export class VideoCommentAddComponent extends FormReactive implements OnInit { | |||
57 | 57 | ||
58 | if (this.parentComment) { | 58 | if (this.parentComment) { |
59 | const mentions = this.parentComments | 59 | const mentions = this.parentComments |
60 | .filter(c => c.account.id !== this.user.account.id) // Don't add mention of ourselves | 60 | .filter(c => c.account && c.account.id !== this.user.account.id) // Don't add mention of ourselves |
61 | .map(c => '@' + c.by) | 61 | .map(c => '@' + c.by) |
62 | 62 | ||
63 | const mentionsSet = new Set(mentions) | 63 | const mentionsSet = new Set(mentions) |
diff --git a/client/src/app/videos/+video-watch/comment/video-comment.component.ts b/client/src/app/videos/+video-watch/comment/video-comment.component.ts index f7eca45fd..1313b6585 100644 --- a/client/src/app/videos/+video-watch/comment/video-comment.component.ts +++ b/client/src/app/videos/+video-watch/comment/video-comment.component.ts | |||
@@ -125,7 +125,12 @@ export class VideoCommentComponent implements OnInit, OnChanges { | |||
125 | const html = await this.markdownService.textMarkdownToHTML(this.comment.text, true) | 125 | const html = await this.markdownService.textMarkdownToHTML(this.comment.text, true) |
126 | this.sanitizedCommentHTML = await this.markdownService.processVideoTimestamps(html) | 126 | this.sanitizedCommentHTML = await this.markdownService.processVideoTimestamps(html) |
127 | this.newParentComments = this.parentComments.concat([ this.comment ]) | 127 | this.newParentComments = this.parentComments.concat([ this.comment ]) |
128 | this.commentAccount = new Account(this.comment.account) | 128 | |
129 | this.getUserIfNeeded(this.commentAccount) | 129 | if (this.comment.account) { |
130 | this.commentAccount = new Account(this.comment.account) | ||
131 | this.getUserIfNeeded(this.commentAccount) | ||
132 | } else { | ||
133 | this.comment.account = null | ||
134 | } | ||
130 | } | 135 | } |
131 | } | 136 | } |
diff --git a/client/src/app/videos/+video-watch/comment/video-comments.component.ts b/client/src/app/videos/+video-watch/comment/video-comments.component.ts index f2bb5c464..f1408effb 100644 --- a/client/src/app/videos/+video-watch/comment/video-comments.component.ts +++ b/client/src/app/videos/+video-watch/comment/video-comments.component.ts | |||
@@ -183,7 +183,7 @@ export class VideoCommentsComponent implements OnInit, OnChanges, OnDestroy { | |||
183 | // Mark the comment as deleted | 183 | // Mark the comment as deleted |
184 | this.softDeleteComment(commentToDelete) | 184 | this.softDeleteComment(commentToDelete) |
185 | 185 | ||
186 | if (this.highlightedThread.id === commentToDelete.id) this.highlightedThread = undefined | 186 | if (this.highlightedThread?.id === commentToDelete.id) this.highlightedThread = undefined |
187 | }, | 187 | }, |
188 | 188 | ||
189 | err => this.notifier.error(err.message) | 189 | err => this.notifier.error(err.message) |