From 0a4db90d36870211df2dc1e606b01602666706c8 Mon Sep 17 00:00:00 2001 From: Jacob Vosmaer Date: Tue, 2 Feb 2016 14:35:09 +0100 Subject: Use begin/end instead of defined? --- app/models/ci/commit.rb | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'app/models/ci/commit.rb') diff --git a/app/models/ci/commit.rb b/app/models/ci/commit.rb index 96786ac4573..ecbd2078b1d 100644 --- a/app/models/ci/commit.rb +++ b/app/models/ci/commit.rb @@ -205,11 +205,11 @@ module Ci end def ci_yaml_file - return @ci_yaml_file if defined?(@ci_yaml_file) - - blob = project.repository.blob_at(sha, '.gitlab-ci.yml') - blob.load_all_data!(project.repository) - @ci_yaml_file = blob.data + @ci_yaml_file ||= begin + blob = project.repository.blob_at(sha, '.gitlab-ci.yml') + blob.load_all_data!(project.repository) + blob.data + end rescue nil end -- cgit v1.2.1