From a07021fa8e8abcf5881cece65bbe0248338d16b6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Rub=C3=A9n=20D=C3=A1vila?= Date: Fri, 12 Feb 2016 21:03:07 -0500 Subject: Make more robust the test to see if a commit has been previously reverted. --- app/models/repository.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'app/models/repository.rb') diff --git a/app/models/repository.rb b/app/models/repository.rb index d98277fc257..68914dc0f8f 100644 --- a/app/models/repository.rb +++ b/app/models/repository.rb @@ -628,11 +628,11 @@ class Repository args = [commit.id, source_sha] args << { mainline: 1 } if commit.merge_commit? - return false unless diff_exists?(source_sha, commit.id) - revert_index = rugged.revert_commit(*args) + tree_id = revert_index.write_tree(rugged) return false if revert_index.conflicts? + return false unless diff_exists?(source_sha, tree_id) commit_with_hooks(user, target_branch) do |ref| committer = user_to_committer(user) @@ -640,14 +640,14 @@ class Repository message: commit.revert_message, author: committer, committer: committer, - tree: revert_index.write_tree(rugged), + tree: tree_id, parents: [rugged.lookup(source_sha)], update_ref: ref) end end - def diff_exists?(source_sha, target_sha) - rugged.diff(source_sha, target_sha).size.zero? + def diff_exists?(sha1, sha2) + rugged.diff(sha1, sha2).size > 0 end def merged_to_root_ref?(branch_name) -- cgit v1.2.3