Fix replying to file attachment comments when the file attachment is removed.

Commenting on a file attachment, then removing the file attachment, then
replying to the comment was broken. This was due to looking up the
associated review_request only using the relationship for active file
attachments. We now check the inactive_review_request relationship too.
This is something that screenshot comments already do, but that fix
never made it to file attachments.

A unit test was added for this.
Show Users' fullname in review emails

When displaying the name in review emails, show the users' fullname if they have
a first name setup. Otherwise show their username. This change is made in both
html and text emails.

Testing done:
Manual. Tested with email setting on and off.

Reviewed at http://reviews.reviewboard.org/r/4104/
Fixes bug 2965
Fix overzealous file existance caching.

File existence check caching had a flaw where, if a file didn't exist,
we'd cache that it didn't exist, but if it was later pushed and then did
exist, we'd still think it didn't.

We now only cache when it exists. If it doesn't, we don't cache, and
will check again the next time get_file_exists is called.
Merge branch 'release-1.7.x'