Fix conflicts that could occur with move flag anchors.

Review Request #8796 — Created March 3, 2017 and submitted

Information

Review Board
release-2.5.x
836a7c2...

Reviewers

The move flags had anchors set to jump to the other side of a move when
clicked. These were using pure line numbers, which could cause conflicts
when referencing a line that another move block in the same file started
at. For example, a move flag in the left-hand side of the diff viewer
that referenced line 100 in the right-hand side could jump to the wrong
flag if the left-hand side also had a move flag starting at its own line
100.

Fixing this is a simple matter of namespacing the anchors so that it
specifically references the correct side of the diff.

Tested that comment flags were working correctly and not conflicting with
each other.

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