From e2c8a2231bcc00e5993411e4ad6c0c0db7e1a297 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Matija=20=C4=8Cupi=C4=87?= Date: Tue, 30 Jan 2018 02:14:39 +0100 Subject: [PATCH] Remove usage of VariablePresenter in controller --- app/controllers/groups/variables_controller.rb | 10 ++-------- app/controllers/projects/variables_controller.rb | 10 ++-------- 2 files changed, 4 insertions(+), 16 deletions(-) diff --git a/app/controllers/groups/variables_controller.rb b/app/controllers/groups/variables_controller.rb index 32c024d78fb..3e78711dabf 100644 --- a/app/controllers/groups/variables_controller.rb +++ b/app/controllers/groups/variables_controller.rb @@ -5,10 +5,7 @@ module Groups def show respond_to do |format| format.json do - variables = @group.variables - .map { |variable| variable.present(current_user: current_user) } - - render status: :ok, json: { variables: GroupVariableSerializer.new.represent(variables) } + render status: :ok, json: { variables: GroupVariableSerializer.new.represent(@group.variables) } end end end @@ -17,10 +14,7 @@ module Groups respond_to do |format| format.json do if @group.update(variables_params) - variables = @group.variables - .map { |variable| variable.present(current_user: current_user) } - - return render status: :ok, json: { variables: GroupVariableSerializer.new.represent(variables) } + return render status: :ok, json: { variables: GroupVariableSerializer.new.represent(@group.variables) } end render status: :bad_request, json: @group.errors.full_messages diff --git a/app/controllers/projects/variables_controller.rb b/app/controllers/projects/variables_controller.rb index b5635ca1b3b..18225218a1b 100644 --- a/app/controllers/projects/variables_controller.rb +++ b/app/controllers/projects/variables_controller.rb @@ -4,10 +4,7 @@ class Projects::VariablesController < Projects::ApplicationController def show respond_to do |format| format.json do - variables = @project.variables - .map { |variable| variable.present(current_user: current_user) } - - render status: :ok, json: { variables: VariableSerializer.new.represent(variables) } + render status: :ok, json: { variables: VariableSerializer.new.represent(@project.variables) } end end end @@ -16,10 +13,7 @@ class Projects::VariablesController < Projects::ApplicationController respond_to do |format| format.json do if @project.update(variables_params) - variables = @project.variables - .map { |variable| variable.present(current_user: current_user) } - - return render status: :ok, json: { variables: VariableSerializer.new.represent(variables) } + return render status: :ok, json: { variables: VariableSerializer.new.represent(@project.variables) } end render status: :bad_request, json: @project.errors.full_messages