Closed Bug 1263520 Opened 10 years ago Closed 10 years ago

Cannot set r+ back to r? directly

Categories

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

Production
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: xidorn, Assigned: dkl)

References

Details

Attachments

(1 file)

Steps to reproduce: 1. have a patch attached 2. make it r+ somehow (not sure whether it works if you just r+ yourself) 3. edit the details of the attachment and try to make it r? again Expected result: It is able to mark it r?(someone) again directly. Actual result: You have to set review flag to empty, and submit before you can set it to r?.
Priority: -- → P2
Assignee: nobody → dkl
Status: NEW → ASSIGNED
Attached patch 1263520_1.patchSplinter Review
Fix for my regression. Please validate.
Attachment #8740106 - Flags: review?(dylan)
Comment on attachment 8740106 [details] [diff] [review] 1263520_1.patch Review of attachment 8740106 [details] [diff] [review]: ----------------------------------------------------------------- r=dylan
Attachment #8740106 - Flags: review?(dylan) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git ea817b4..cc0c32b master -> master
Status: ASSIGNED → 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

Created:
Updated:
Size: