From 9d56cb6a0d562ba7e2c9429af1d2ab0677871a37 Mon Sep 17 00:00:00 2001 From: Shinya Maeda <gitlab.shinyamaeda@gmail.com> Date: Fri, 17 Mar 2017 17:05:27 +0900 Subject: [PATCH] Using fetch instead of direct refference for hash --- app/serializers/build_serializer.rb | 2 +- app/serializers/pipeline_serializer.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/serializers/build_serializer.rb b/app/serializers/build_serializer.rb index 2c9cf99811e..c5dbb391ce0 100644 --- a/app/serializers/build_serializer.rb +++ b/app/serializers/build_serializer.rb @@ -7,6 +7,6 @@ class BuildSerializer < BaseSerializer def represent_status(resource) data = represent(resource, { only: [:status] }) - data[:status] + data.fetch(:status, {}) end end diff --git a/app/serializers/pipeline_serializer.rb b/app/serializers/pipeline_serializer.rb index 62f31d4c5ad..384e881da06 100644 --- a/app/serializers/pipeline_serializer.rb +++ b/app/serializers/pipeline_serializer.rb @@ -25,6 +25,6 @@ class PipelineSerializer < BaseSerializer def represent_status(resource) data = represent(resource, { only: [{ details: [:status] }] }) - data[:details][:status] + data.fetch(:details, {}).fetch(:status, {}) end end -- GitLab