Fix the display of legacy screenshots.

Review Request #7843 — Created Jan. 4, 2016 and submitted — Latest diff uploaded

Information

Review Board
release-2.5.x

Reviewers

Legacy screenshots were broken due to some assumptions made in the new
image diff code. They checked for an attribute to be non-null, and if
that was the case, it would activate diffs, which then required certain
state not available for screenshots. Instead, we just want to check that
the attribute has a truthy value.

This is a pretty simple change, and results in both screenshots and
image file attachment (with and without diffs) working correctly once
again.

Tested with legacy screenshots, and image file attachments (with different
diff ranges, and without diffs).

    Loading...