diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb index 4a4d4d855..599521265 100644 --- a/app/controllers/user_controller.rb +++ b/app/controllers/user_controller.rb @@ -2,9 +2,9 @@ class UserController < ApplicationController layout 'site' before_filter :authorize, :only => :preferences - before_filter :authorize_web, :only => :rename - - + before_filter :authorize_web, :only => [:rename, :account] + before_filter :require_user, :only => [:account, :display_name_edit] + def save @user = User.new(params[:user]) @user.set_defaults @@ -17,20 +17,15 @@ class UserController < ApplicationController render :action => 'new' end end - + def rename - new_name = params['display_name'] - if @user - @user.display_name = new_name + if params[:user] and params[:user][:display_name] + @user.display_name = params[:user][:display_name] if @user.save flash[:notice] = "User display name updated OK." - else - flash[:notice] = "Rename failed: #{ @user.errors.full_messages.join('; ') }." + redirect_to :controller => 'user', :action => 'account' end - else - flash[:notice] = 'not logged in' end - redirect_to :back end def lost_password diff --git a/app/views/layouts/site.rhtml b/app/views/layouts/site.rhtml index 1154237cb..d32490729 100644 --- a/app/views/layouts/site.rhtml +++ b/app/views/layouts/site.rhtml @@ -15,7 +15,7 @@ <% end %>
id | <%= @user.id %> |
<%= @user.email %> | |
display name | <%= @user.display_name %> (<%= link_to 'change...', :controller => 'user', :action => 'rename' %>) |
account created | <%= @user.creation_time %> |