Filter existing reviewers when accessing mentionedUsers
When calculating the list of mentioned users we want to filter
out existing reviewers and CC(either added to the change already
or added manually in the reply dialog by the user).
Currently filtering was done when subscribing which means updates
to the reviewer/CC list is not reflected in the filter.
The filter was also called before the reviewerList.accounts was
populated making the filter currently obsolete.
Release-Notes: skip
Google-bug-id: b/236921879
Change-Id: Ie4acc90efa1a779cfe4c5165876a084538a3039e
diff --git a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts
index 95805fa..e89b418 100644
--- a/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts
+++ b/polygerrit-ui/app/elements/change/gr-reply-dialog/gr-reply-dialog.ts
@@ -259,7 +259,10 @@
account?: AccountInfo;
get ccs() {
- return [...this._ccs, ...this.mentionedUsers];
+ return [
+ ...this._ccs,
+ ...this.mentionedUsers.filter(v => !this.isAlreadyReviewerOrCC(v)),
+ ];
}
/**
@@ -648,7 +651,7 @@
this,
() => this.getCommentsModel().mentionedUsersInDrafts$,
x => {
- this.mentionedUsers = x.filter(v => !this.isAlreadyReviewerOrCC(v));
+ this.mentionedUsers = x;
this.reviewersMutated =
this.reviewersMutated || this.mentionedUsers.length > 0;
}
