Prevent useless comment fetching for screenshots and file attachments.

Review Request #3479 — Created Nov. 7, 2012 and submitted — Latest diff uploaded

Information

Review Board
release-1.6.x

Reviewers

Prevent useless comment fetching for screenshots and file attachments.

Screenshots and file attachments without comments weren't having their
_comments variable set, which was causing us to do a fetch later on
for each screenshot and file attachment. We now make sure to always
set _comments in review_detail, preventing these extra, useless fetches.
Tested on some review requests with many active and inactive screenshots
and file attachments. Saw that the extra queries went away and page load
time decreased.