From 46fd485c58c8c72156a9ebe4ec1775a3e57934de Mon Sep 17 00:00:00 2001 From: Kamil Trzcinski Date: Fri, 19 Feb 2016 18:07:05 +0100 Subject: [PATCH] Fix API implementation --- lib/api/builds.rb | 4 +--- lib/api/entities.rb | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/api/builds.rb b/lib/api/builds.rb index f5ae8e57e23..2b104f90aa7 100644 --- a/lib/api/builds.rb +++ b/lib/api/builds.rb @@ -79,9 +79,7 @@ module API return redirect_to build.artifacts_file.url end - unless artifacts_file.exists? - not_found! - end + return not_found! unless artifacts_file.exists? present_file!(artifacts_file.path, artifacts_file.filename) end diff --git a/lib/api/entities.rb b/lib/api/entities.rb index eaf42bee113..a3b5f1eb8d3 100644 --- a/lib/api/entities.rb +++ b/lib/api/entities.rb @@ -406,7 +406,7 @@ module API repo_obj.artifacts_download_url end end - expose :artifacts_file, using: BuildArtifactFile, if: lambda { |build, opts| build.artifacts? } + expose :artifacts_file, using: BuildArtifactFile, if: -> (build, opts) { build.artifacts? } expose :commit, with: RepoCommit do |repo_obj, _options| if repo_obj.respond_to?(:commit) repo_obj.commit.commit_data