From 79335e46fd784f21ef5e9c8ae861e6820e422e29 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Tue, 30 May 2017 21:16:28 -0400 Subject: [PATCH] Refactor api/v1/apps controller (#3471) --- app/controllers/api/v1/apps_controller.rb | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/app/controllers/api/v1/apps_controller.rb b/app/controllers/api/v1/apps_controller.rb index 2ec7280af0..54f8d40b20 100644 --- a/app/controllers/api/v1/apps_controller.rb +++ b/app/controllers/api/v1/apps_controller.rb @@ -4,11 +4,24 @@ class Api::V1::AppsController < ApiController respond_to :json def create - @app = Doorkeeper::Application.create!(name: app_params[:client_name], redirect_uri: app_params[:redirect_uris], scopes: (app_params[:scopes] || Doorkeeper.configuration.default_scopes), website: app_params[:website]) + @app = Doorkeeper::Application.create!(application_options) end private + def application_options + { + name: app_params[:client_name], + redirect_uri: app_params[:redirect_uris], + scopes: app_scopes_or_default, + website: app_params[:website], + } + end + + def app_scopes_or_default + app_params[:scopes] || Doorkeeper.configuration.default_scopes + end + def app_params params.permit(:client_name, :redirect_uris, :scopes, :website) end