summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorKamil Trzcinski <ayufan@ayufan.eu>2016-02-19 18:07:05 +0100
committerKamil Trzcinski <ayufan@ayufan.eu>2016-02-19 18:30:43 +0100
commit46fd485c58c8c72156a9ebe4ec1775a3e57934de (patch)
tree7e728cde6b610c70f817f551cd6444f927840612 /lib
parent55701d9c6c5b86274990c8b75b768330978a1f07 (diff)
downloadgitlab-ce-46fd485c58c8c72156a9ebe4ec1775a3e57934de.tar.gz
Fix API implementation
Diffstat (limited to 'lib')
-rw-r--r--lib/api/builds.rb4
-rw-r--r--lib/api/entities.rb2
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