Dana Keeler (she/her) (use needinfo) [:keeler] (on leave) Reporter
	Description • 9 years ago

STR:
* Add a patch to a restricted bug
* While doing so, ask for review from someone not in the groups the bug is restricted to (and not cc'd on the bug)

At this point, bugzilla complains that you haven't specified a reviewer.

* Finish attaching the patch without asking for review
* Click on "details" for the patch, attempt to ask for review

Now bugzilla will complain that that person isn't in the group or cc'd on the bug.

* Go back to the bug and manually cc the reviewer
* Go back to the details page and ask for review

Now bugzilla stops complaining.

I would expect this to work more like this:
* Add a patch to a restricted bug
* While doing so, ask for review. Bugzilla automatically cc's the person (like I believe it does with other bugs).

	:glob ✱ Assignee
	Comment 2 • 9 years ago

- trigger the requestee checks to throw an error instead of silently dropping the requestee

	Dylan Hardison [:dylan] (he/him)
	Comment 3 • 9 years ago

Comment on attachment 8579190 [details] [diff] [review]
1125987_1.patch

Review of attachment 8579190 [details] [diff] [review]:
-----------------------------------------------------------------

r=dylan

this is still less evil than monkey-patching ->can()...

	:glob ✱ Assignee
	Comment 4 • 9 years ago

To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   31e3b4e..9a07ea2  master -> master