Fix missing data for file attachment thumbnails for change descriptions.

Review Request #8232 — Created June 10, 2016 and submitted — Latest diff uploaded

Information

Review Board
release-2.5.x
d6213ed...

Reviewers

If a file attachment thumbnail needed to make use of the User currently
browsing the page, it could break when rendering change descriptions, as
the request variable wasn't being provided to the template context.
The implementation would only see an empty string. This was causing
problems with Power Pack's PDF thumbnail support.

We now simply ensure that the request is passed in as part of the
context.

Tested against a PDF thumbnail on a change description. I didn't see any
errors.

Had a customer test this as well, and he verified the errors went away on
his end.

    Loading...