Fix closing issues bumping the review timestamp.

Review Request #2484 — Created July 17, 2011 and submitted

Information

Review Board

Reviewers

Fix closing issues bumping the review timestamp.

Closing open issues would bump the review timestamp, causing the reviews
to reorder confusingly. The logic for bumping the review timestamp made
the assumption that the review was not public (because the only time you'd
save a comment, previously, is when it was in draft form).

Now we only bump the review timestamp if we know it's not public. This
code has been moved to BaseComment so we only have to do it once, and it
also bumps the comment's own timestamp (which used to be modified as a
side-effect of saving the review).
Closed some issues on an older review. Saw that the review didn't shuffle around
on the page.
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Committed to master (d5009e2)