summaryrefslogtreecommitdiff
path: root/app/controllers/profiles_controller.rb
diff options
context:
space:
mode:
authorDJ Mountney <david@twkie.net>2018-04-04 12:49:08 -0700
committerDJ Mountney <david@twkie.net>2018-04-04 12:49:08 -0700
commit39bb3720381af64039673cf21705fcf95cf78314 (patch)
treec32aa05ee2eaa2e0b93ec811a6ad8f3aa71a207e /app/controllers/profiles_controller.rb
parentf7f92836400ec86c74d1140223c009d3d328aea9 (diff)
parent166b4575a6353668c894fea0ba234d0b371dee03 (diff)
downloadgitlab-ce-39bb3720381af64039673cf21705fcf95cf78314.tar.gz
Merge remote-tracking branch 'origin/master' into dev-master
Diffstat (limited to 'app/controllers/profiles_controller.rb')
-rw-r--r--app/controllers/profiles_controller.rb6
1 files changed, 5 insertions, 1 deletions
diff --git a/app/controllers/profiles_controller.rb b/app/controllers/profiles_controller.rb
index dbf61a17724..3d27ae18b17 100644
--- a/app/controllers/profiles_controller.rb
+++ b/app/controllers/profiles_controller.rb
@@ -51,7 +51,7 @@ class ProfilesController < Profiles::ApplicationController
end
def update_username
- result = Users::UpdateService.new(current_user, user: @user, username: user_params[:username]).execute
+ result = Users::UpdateService.new(current_user, user: @user, username: username_param).execute
options = if result[:status] == :success
{ notice: "Username successfully changed" }
@@ -72,6 +72,10 @@ class ProfilesController < Profiles::ApplicationController
return render_404 unless @user.can_change_username?
end
+ def username_param
+ @username_param ||= user_params.require(:username)
+ end
+
def user_params
@user_params ||= params.require(:user).permit(
:avatar,