• 
      

    Attach parent diffs with renamed and modified files

    Review Request #7178 — Created April 7, 2015 and submitted — Latest diff uploaded

    Information

    Review Board
    release-2.0.x
    423060d...

    Reviewers

    Previously, if a review request was uploaded with a parent diff it was
    possible that parent diffs would not be attached to FileDiffs in the
    case where the parent diff for a file was the result of a rename and a
    modify. This would result in the underlying SCM not being able to
    locate the parent revision and generate the original file for the
    diffviewer.

    Now instead of attaching parent diffs to FileDiffs based on the
    original file names in the parent diff, we consider the destination
    file names. This results in parent diffs being correctly attached when
    they contain a renamed and modified file.

    Ran unit tests.

    Uploaded a review request with a parent diff that had a modified and
    renamed file. The diff was visible in the diffviewer when uploaded
    with this patch applied; it was not visible when it was uploaded
    without the patch applied.