diff options
author | Folker Bernitt <fbernitt@thoughtworks.com> | 2014-07-11 14:55:36 +0200 |
---|---|---|
committer | Folker Bernitt <fbernitt@thoughtworks.com> | 2014-07-14 09:56:23 +0200 |
commit | 821b1444dcc147b9c1205326a2d28651c369a52d (patch) | |
tree | 451d67ee217d3f54ef783059aad5bca4269df93a /app/controllers/v1/users_controller.rb | |
parent | 07b141f3d677e993f02380b455738b20b9f0fe42 (diff) |
Moved check for allow_registration into filter.
Diffstat (limited to 'app/controllers/v1/users_controller.rb')
-rw-r--r-- | app/controllers/v1/users_controller.rb | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/app/controllers/v1/users_controller.rb b/app/controllers/v1/users_controller.rb index 881708a..abaefd8 100644 --- a/app/controllers/v1/users_controller.rb +++ b/app/controllers/v1/users_controller.rb @@ -5,6 +5,7 @@ module V1 before_filter :fetch_user, :only => [:update] before_filter :require_admin, :only => [:index] before_filter :require_token, :only => [:update] + before_filter :require_registration_allowed, only: :create respond_to :json @@ -19,12 +20,8 @@ module V1 end def create - if APP_CONFIG[:allow_registration] - @user = Account.create(params[:user]) - respond_with @user # return ID instead? - else - head :forbidden - end + @user = Account.create(params[:user]) + respond_with @user # return ID instead? end def update @@ -32,5 +29,11 @@ module V1 respond_with @user end + def require_registration_allowed + unless APP_CONFIG[:allow_registration] + head :forbidden + end + end + end end |