summaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorZ.J. van de Weg <zegerjan@gitlab.com>2016-08-03 13:37:39 +0200
committerZ.J. van de Weg <zegerjan@gitlab.com>2016-08-11 21:29:57 +0200
commitb497b0ce3fc3c1882639f9c7d55f7991ce41f15d (patch)
tree353668ef819c38e3e9a208530a36501b4c524313 /app
parent826862d48ef80ddd849b9e3cb05ef37ba7be41e9 (diff)
downloadgitlab-ce-b497b0ce3fc3c1882639f9c7d55f7991ce41f15d.tar.gz
Incorporate feedback
Diffstat (limited to 'app')
-rw-r--r--app/models/deployment.rb4
-rw-r--r--app/models/environment.rb6
-rw-r--r--app/models/merge_request.rb2
3 files changed, 6 insertions, 6 deletions
diff --git a/app/models/deployment.rb b/app/models/deployment.rb
index 67a4f3998ec..19b08f49d96 100644
--- a/app/models/deployment.rb
+++ b/app/models/deployment.rb
@@ -37,10 +37,10 @@ class Deployment < ActiveRecord::Base
deployable.try(:other_actions)
end
- def deployed_to(ref)
+ def deployed_to?(ref)
commit = project.commit(ref)
return false unless commit
- project.repository.merge_base(commit.id, sha) == commit.id
+ project.repository.is_ancestor?(commit.id, sha)
end
end
diff --git a/app/models/environment.rb b/app/models/environment.rb
index f6fdb8d1ecf..7247125f8a0 100644
--- a/app/models/environment.rb
+++ b/app/models/environment.rb
@@ -26,9 +26,9 @@ class Environment < ActiveRecord::Base
self.external_url = nil if self.external_url.blank?
end
- def deployed_to?(ref)
- return unless last_deployment
+ def deployed_from?(ref)
+ return false unless last_deployment
- last_deployment.deployed_to(ref)
+ last_deployment.deployed_to?(ref)
end
end
diff --git a/app/models/merge_request.rb b/app/models/merge_request.rb
index 85e4d1f6b51..945b0d76505 100644
--- a/app/models/merge_request.rb
+++ b/app/models/merge_request.rb
@@ -592,7 +592,7 @@ class MergeRequest < ActiveRecord::Base
def environments
target_project.environments.select do |environment|
- environment.deployed_to?(ref_path)
+ environment.deployed_from?(ref_path)
end
end