Closed
Bug 1097736
Opened 10 years ago
Closed 10 years ago
Can't assign reviewers to a review request
Categories
(MozReview Graveyard :: General, defect)
MozReview Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jgraham, Assigned: mconley)
Details
Attachments
(1 file, 1 obsolete file)
Trying to type in the reviewer names for a commit, I get a js error, but no dropdown: 15:17 < jgraham> TypeError: s is undefined 15:17 < jgraham> return r.indexOf(e) === 0 ? i.indexOf(e) === 0 ? r.localeCompare(i) : - 1 : i.indexOf(e) === 0 ? 1 : s.localeCompare(o) 15:18 < jgraham> It appears that s.data.fullname doesn't exist This is also broken in Chrome. AFAICT I can't create a review request at all until this works.
Comment 1•10 years ago
|
||
Thank you for filing and for taking the time to report the JS error. To help us reproduce, what string did you type in the box? Can you start a JS debugger and see what entry Review Board is barfing on?
Flags: needinfo?(james)
Reporter | ||
Comment 2•10 years ago
|
||
It doesn't really matter what I type in the box e.g. "gps" reproduces the above behaviour. The debugger output is in the summary. It comes from commits.min.whatever.js, line 173 of the prettified source.
Flags: needinfo?(james)
Assignee | ||
Comment 3•10 years ago
|
||
I think I know what's going on here. We've got code in commits.js that is doing comparisons on both usernames and fullnames in order to create the sorted autocomplete list. For profiles that are private, users that do not have staff / admin privileges to retrieve their email addresses or full names via the API. I believe we need to alter commits.js to take into account the possibility of fullnames not being available in the result that is returned from the server.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → mconley
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8521538 -
Flags: review?(gps)
Assignee | ||
Comment 5•10 years ago
|
||
/r/505 - Bug 1097736 - rbmozui: Can't assign reviewers to a review request. r=? Pull down this commit: hg pull review -r b72134811c6be795a5e53dd6f75dcaf816090fc7
Assignee | ||
Comment 6•10 years ago
|
||
https://reviewboard.mozilla.org/r/503/#review215 Hang on, I've got a better solution here.
Assignee | ||
Comment 7•10 years ago
|
||
/r/505 - Bug 1097736 - rbmozui: Can't assign reviewers to a review request. r=? Pull down this commit: hg pull review -r 824f15e433125ed4e07dccf2add386640947fdb7
Comment 8•10 years ago
|
||
https://reviewboard.mozilla.org/r/503/#review223 Ship It!
Attachment #8521538 -
Flags: review+
Assignee | ||
Comment 9•10 years ago
|
||
https://hg.mozilla.org/hgcustom/version-control-tools/rev/d104b9116891
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Attachment #8521538 -
Flags: review?(gps)
Assignee | ||
Comment 10•10 years ago
|
||
This fix has been pushed to production. Can you confirm the fix on your end, jgraham?
Flags: needinfo?(james)
Assignee | ||
Comment 12•9 years ago
|
||
Attachment #8521538 -
Attachment is obsolete: true
Attachment #8618609 -
Flags: review+
Assignee | ||
Comment 13•9 years ago
|
||
Updated•8 years ago
|
Product: Developer Services → MozReview
You need to log in
before you can comment on or make changes to this bug.
Description
•