Revert "Fix references to cgit id="

This reverts commit 5c55e7a7d962e3143b3a7e28b0098d0e7139d75c. It
appears that some repositories have problems with this change.
Fix up local site permission denied tests.

The url resolvers changed such that we need to catch ViewDoesNotExist in
addition to NoReverseMatch. This also adds a div that we use for some style
fixes in rbcommons.
Fix links to file attachment review UIs.

Using file attachments with local sites was creating broken links. The issue
here was that it was using pk instead of display_id.
Fix the other file attachment link.

I fixed the one for the "Review" command but not the thumbnail overlay. Trivial
fix.
Fix should_update_issue_status to avoid uncaught exceptions.

Every so often, I see a traceback from our servers coming from
BaseCommentResource.should_update_issue_status. The issue here is that
issue_status isn't in the kwargs, but our issue_string_to_status will raise an
exception if the passed-in value is None. This changes it to always return False
in the case that issue_status isn't in kwargs, and fixes up a little bit of
formatting.

Reviewed at http://reviews.reviewboard.org/r/4025/
Merge branch 'release-1.6.x' into release-1.7.x