• 
      

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

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

    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.

    reviewbot
    1. Tool: Pyflakes
      Processed Files:
          reviewboard/reviews/templatetags/tests.py
          reviewboard/diffviewer/tests.py
          reviewboard/diffviewer/diffutils.py
      
      
      
      Tool: PEP8 Style Checker
      Processed Files:
          reviewboard/reviews/templatetags/tests.py
          reviewboard/diffviewer/tests.py
          reviewboard/diffviewer/diffutils.py
      
      
    2. 
        
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-2.5.x (c2e033e)