From 2467697d81e4ab9e9d770d55d8cb33cb5779db22 Mon Sep 17 00:00:00 2001 From: Jacob Vosmaer Date: Thu, 23 Jun 2016 11:12:56 +0200 Subject: Simplify the GitAccessStatus class Make the optional 'message' argument required. Remove unused 'to_json' method. --- lib/gitlab_access_status.rb | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'lib') diff --git a/lib/gitlab_access_status.rb b/lib/gitlab_access_status.rb index 14ec1ef..7a5f7d5 100644 --- a/lib/gitlab_access_status.rb +++ b/lib/gitlab_access_status.rb @@ -1,10 +1,9 @@ require 'json' class GitAccessStatus - attr_accessor :status, :message - alias_method :allowed?, :status + attr_reader :message - def initialize(status, message = '') + def initialize(status, message) @status = status @message = message end @@ -14,7 +13,7 @@ class GitAccessStatus self.new(values["status"], values["message"]) end - def to_json - { status: @status, message: @message }.to_json + def allowed? + @status end end -- cgit v1.2.1