Fix the displayed line numbers in e-mails with filtered interdiffs.

Review Request #8246 — Created June 16, 2016 and submitted — Latest diff uploaded

Information

Review Board
release-2.5.x
6de893c...

Reviewers

The new code for calculating displayed line numbers in e-mails didn't
take into account that filtered interdiffs could result in equal chunks
without line numbers. It's been updated to check whether there are line
numbers in the appropriate side of the chunk before considering that
side valid.

Unit tests have been updated for these conditions.

Unit tests pass. Before, the new tests were failing with the same error
we saw in production with the same conditions.

    Loading...