diff options
| author | Piotr Zalewa <zaloon@gmail.com> | 2018-03-29 15:30:02 +0200 |
|---|---|---|
| committer | dklawren <dklawren@users.noreply.github.com> | 2018-03-29 15:30:02 +0200 |
| commit | 8f610ecde482b38120f56fb54a83974cc335120d (patch) | |
| tree | 63191c91274ba9dbb208852937e4c64eeae6582e /Bugzilla/CGI | |
| parent | 8376e3ae4d494f1f8ff14dc1d90a2ed3becc0189 (diff) | |
| download | bugzilla-8f610ecde482b38120f56fb54a83974cc335120d.tar.gz bugzilla-8f610ecde482b38120f56fb54a83974cc335120d.tar.xz | |
Bug 1441063: Fix the unaccepted revision comment (PhabBugz)
* PhabBugz: Fix the unaccepted revision comment
Summary:
Currently, we're sending the "User removed from revision" comment when
the Accept flag has been removed from a revision. This could happen in a
number of use cases:
1. The reviewer resigned from being a reviewer.
2. The reviewer has been removed from reviewers list.
3. Someone (author, reviewer) removed acceptance of the revision by changing
its status to "Needs Review".
This patch is sending a "flag is deactivated" or "reviewer removed from
revision" depending on the current status of the reviewer.
Test Plan:
Accept a revision.
Change the revision to "Needs Review".
Check the bug comment.
Accept the revision.
Remove the user from reviewers list.
Check the bug comment.
Reviewers: dkl
Bug #: 1441063
Differential Revision: https://phabricator.services.mozilla.com/D809
* Styling fixed as requested in review.
* Perl style fixes
Diffstat (limited to 'Bugzilla/CGI')
0 files changed, 0 insertions, 0 deletions