Fix interdiffs when comparing revisions with stripped-slash filenames.
Review Request #6272 — Created Aug. 26, 2014 and submitted
The leading-slash stripping code strikes again. In this case, an initial
revision of a diff was uploaded prior to an upgrade, and a new version was
uploaded subsequent. The result is that the filenames don't match exactly
because one has leading slashes and the other doesn't.This change normalizes the filenames to always compare them without leading
slashes when figuring out which files match up with which.
Created two diff revisions and changed the FileDiffs in the database so that
the first one had leading slashes and the second didn't. Before this change, I
see two entries in the interdiff for that file. After, it shows the correct
interdiff.Tested all permutations of leading slash/no leading slash in interdiff
combinations.
- Testing Done:
-
Created two diff revisions and changed the FileDiffs in the database so that
the first one had leading slashes and the second didn't. Before this change, I see two entries in the interdiff for that file. After, it shows the correct interdiff. + + Tested all permutations of leading slash/no leading slash in interdiff
+ combinations. - Commit:
-
ee70b245ff10b9f174d4161981972f4f476b1742c38688b47cb03abf27aede1dd220537ffa0d642f
- Diff:
-
Revision 2 (+12 -3)
Tool: PEP8 Style Checker Processed Files: reviewboard/diffviewer/diffutils.py Tool: Pyflakes Processed Files: reviewboard/diffviewer/diffutils.py