MozReview: Return `FileDiff` list sorted by default model sorting (pk) Bug 1248008 r?smacleod draft
authorPiotr Zalewa <pzalewa@mozilla.com>
Fri, 06 Jan 2017 12:59:31 +0100
changeset 270 98c105ed468da05e8df4252cbdc45672c72ca388
parent 253 08f493196e8c29c20d4e08b51a8373091077bb06
child 271 561b51d223040c662e620f590049b8cb7a0faf46
child 272 7a49e9319bd53c294b946ba4524a90798d03d95b
push idunknown
push userunknown
push dateunknown
reviewerssmacleod
bugs1248008
MozReview: Return `FileDiff` list sorted by default model sorting (pk) Bug 1248008 r?smacleod MozReview is adding a commit message `FileDiff` which has to be displayed as the first one from the set. Sorting by dest_filename would be causing issues when paging is involved. MozReview-Commit-ID: 4iQs3hBbfSC
reviewboard/reviewboard/diffviewer/diffutils.py
--- a/reviewboard/reviewboard/diffviewer/diffutils.py
+++ b/reviewboard/reviewboard/diffviewer/diffutils.py
@@ -399,20 +399,22 @@ def get_diff_files(diffset, filediff=Non
 
         if force_interdiff:
             f['force_interdiff_revision'] = interdiffset.revision
 
         files.append(f)
 
     log_timer.done()
 
-    if len(files) == 1:
-        return files
-    else:
-        return get_sorted_filediffs(files, key=lambda f: f['filediff'])
+    # MozReview is adding a commit message filediff which has to be the first
+    # from the set. Sorting by dest_filename would cause issues when paging
+    # is involved. Since the filediff's are already in a logical order when
+    # sorted by id (which they should be here) removing this sort shouldn't
+    # make UX worse.
+    return files
 
 
 def populate_diff_chunks(files, enable_syntax_highlighting=True,
                          request=None):
     """Populates a list of diff files with chunk data.
 
     This accepts a list of files (generated by get_diff_files) and generates
     diff chunk data for each file in the list. The chunk data is stored in