Use `resource` in Group Variables routing scheme

2 files changed, 14 insertions, 3 deletions

diff --git a/app/controllers/groups/variables_controller.rb b/app/controllers/groups/variables_controller.rb
index 2f7058e95ea..c0eff63da18 100644
--- a/app/controllers/groups/variables_controller.rb
+++ b/app/controllers/groups/variables_controller.rb
@@ -2,7 +2,18 @@ module Groups
   class VariablesController < Groups::ApplicationController
     before_action :authorize_admin_build!
 
-    def save_multiple
+    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: variables }
+        end
+      end
+    end
+
+    def update
       respond_to do |format|
         format.json do
           return head :ok if @group.update(variables_params)
diff --git a/app/presenters/ci/group_variable_presenter.rb b/app/presenters/ci/group_variable_presenter.rb
index fb6dfcc1e4d..98d68bc7a83 100644
--- a/app/presenters/ci/group_variable_presenter.rb
+++ b/app/presenters/ci/group_variable_presenter.rb
@@ -11,11 +11,11 @@ module Ci
     end
 
     def edit_path
-      group_variables_save_multiple_path(group)
+      group_variables_path(group)
     end
 
     def delete_path
-      group_variables_save_multiple_path(group)
+      group_variables_path(group)
     end
   end
 end