about summary refs log tree commit diff
path: root/app/controllers/api/v1
diff options
context:
space:
mode:
authorMatt Jankowski <mjankowski@thoughtbot.com>2017-05-30 21:16:28 -0400
committerGitHub <noreply@github.com>2017-05-30 21:16:28 -0400
commit79335e46fd784f21ef5e9c8ae861e6820e422e29 (patch)
treeb59af8217c8c2d29b8ba5b7134463861436ef393 /app/controllers/api/v1
parent7c6e02aaf3e3771448ada4197eda66322dff714f (diff)
Refactor api/v1/apps controller (#3471)
Diffstat (limited to 'app/controllers/api/v1')
-rw-r--r--app/controllers/api/v1/apps_controller.rb15
1 files changed, 14 insertions, 1 deletions
diff --git a/app/controllers/api/v1/apps_controller.rb b/app/controllers/api/v1/apps_controller.rb
index 2ec7280af..54f8d40b2 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