Labels: Type-Defect Priority-Medium
New issue 1281 by nheijermans: Adding comments in diff viewer can delete or
overwrite existing comments.
*NOTE: Do not post confidential information in this bug report.*
What version are you running?
Review Board 1.0.1
What's the URL of the page containing the problem?
Any diff viewer page.
What steps will reproduce the problem?
1. Add a new review comment to a line, then save it.
2. Add another review comment on the same line as in step 1 by clicking on
either the left or right line number (the tooltip containing the contents
of your previous comments will disappear when you're at the right area).
Then, save the comment.
3. Delete the comment you just added.
What is the expected output? What do you see instead?
There are two things that I would expect, neither of which happen.
First, I would expect that selecting a line number would pop up a text
dialog with the existing comment for that line, not a dialog for adding a
new comment (although this may not be happening due to the ability to
comment on multiple lines).
Second, I would expect both comments to appear in the database, or the
first of the two comments if I go back and delete the second one I added.
However, if I stop after step 2, the database contains only the second
comment. If I complete step 3, the database contains no comments for the
relevant line, while the UI still contains the original comment.
What operating system are you using? What browser?
Linux (Ubuntu 9.04) with Firefox 3.5.2.
Please provide any additional information below.
This bug has been an issue for some new users of review board, who thought
it was possible to add multiple comments on a line using the steps 1 and 2
above. The bug is more likely to occur on diffs where line numbers go into
the thousands, because it's more likely that the user will click on a line
number instead of the comment bubble.
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
You received this message because you are subscribed to the Google Groups
To post to this group, send email to email@example.com
To unsubscribe from this group, send email to
For more options, visit this group at