# HG changeset patch
# User Marcin Kuzminski <marcin@rhodecode.com>
# Date 2017-01-30 11:41:16
# Node ID b4f72747311d2f5c240aa6369856f58f03aeb738
# Parent  6a4c8be472a55b2443e24832c65e6cd07e953143

pr-model: don't use set to calculate commit ranges as they generate random order.

diff --git a/rhodecode/model/pull_request.py b/rhodecode/model/pull_request.py
--- a/rhodecode/model/pull_request.py
+++ b/rhodecode/model/pull_request.py
@@ -630,7 +630,7 @@ class PullRequestModel(BaseModel):
                 old=pull_request, new=None, changes=None)
 
         # re-compute commit ids
-        old_commit_ids = set(pull_request.revisions)
+        old_commit_ids = pull_request.revisions
         pre_load = ["author", "branch", "date", "message"]
         commit_ranges = target_repo.compare(
             target_commit.raw_id, source_commit.raw_id, source_repo, merge=True,
@@ -647,7 +647,7 @@ class PullRequestModel(BaseModel):
             commit.raw_id for commit in reversed(commit_ranges)]
         pull_request.updated_on = datetime.datetime.now()
         Session().add(pull_request)
-        new_commit_ids = set(pull_request.revisions)
+        new_commit_ids = pull_request.revisions
 
         changes = self._calculate_commit_id_changes(
             old_commit_ids, new_commit_ids)