diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2016-07-15 02:22:29 +0800 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2016-07-15 02:22:29 +0800 |
commit | 53a9dee6cb54b75fa2999b4a33a59928b3b73ec3 (patch) | |
tree | 7607cc1e0cc0cb3e8dce5f4d163802d4355554d7 /app/controllers/projects/artifacts_controller.rb | |
parent | 4bb3787eee282434263c37194a443edf6a93c1b7 (diff) | |
download | gitlab-ce-53a9dee6cb54b75fa2999b4a33a59928b3b73ec3.tar.gz |
Introduce Project#latest_success_builds_for:
So it's more accessible for views to access the names of jobs.
Only filter Build#name from where we really need to download it.
Diffstat (limited to 'app/controllers/projects/artifacts_controller.rb')
-rw-r--r-- | app/controllers/projects/artifacts_controller.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/controllers/projects/artifacts_controller.rb b/app/controllers/projects/artifacts_controller.rb index 3e487c24cbd..1b3c4ec9bd8 100644 --- a/app/controllers/projects/artifacts_controller.rb +++ b/app/controllers/projects/artifacts_controller.rb @@ -61,9 +61,9 @@ class Projects::ArtifactsController < Projects::ApplicationController def build_from_ref if params[:ref_name] - builds = project.builds_for(params[:job], params[:ref_name]) + builds = project.latest_success_builds_for(params[:ref_name]) - builds.success.latest.first + builds.where(name: params[:job]).first end end |