Closed Bug 986590 Opened 10 years ago Closed 10 years ago

Confusing error message when not finding reviewer

Categories

(bugzilla.mozilla.org :: Extensions, defect)

Production
defect
Not set
minor

Tracking

()

RESOLVED FIXED

People

(Reporter: mozilla, Unassigned)

Details

Attachments

(1 file)

When the provided reviewer is wrong, bugzilla always shows the message «You must provide a reviewer for review requests.» despite the field being filled.


Steps to reproduce:
- Create a new attachment and set review to ?
- Fill the with a wrong developer email, such as foo@example.com when it should have been foo@mozilla.com (I stumbled upon this using a «wrong email» I got from the hg log)
- Bugzilla simply answers «You must provide a reviewer for review requests.»… which you did provide.


Expected behavior:
- If there's no entry in the reviewer field «You must provide a reviewer for review requests.»
- If the reviewer field doesn't contain a @: «You must identify the reviewer with his email address.»
- If the email address is not found in bugzilla: «The reviewer foo@example.com is not a valid bugzilla user.»
This is an issue with the Review extension which is specific to BMO. Moving.
Assignee: ui → nobody
Component: User Interface → Extensions: Review
Product: Bugzilla → bugzilla.mozilla.org
QA Contact: default-qa
Version: unspecified → Production
Attached patch 986590_1.patchSplinter Review
Attachment #8395515 - Flags: review?(dkl)
Comment on attachment 8395515 [details] [diff] [review]
986590_1.patch

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

r=dkl
Attachment #8395515 - Flags: review?(dkl) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   3351341..f8a258d  4.2 -> 4.2
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Component: Extensions: Review → Extensions
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: