Make the tooltips for the tabs a bit more consistent and make the edit

[rails.git] / app / controllers / user_controller.rb

diff --git a/app/controllers/user_controller.rb b/app/controllers/user_controller.rb
index 3a2bc7f0d333afc45626413517575cae7b4cc5dd..430e56cb05c1ae0719d5540874a6e681e721ab94 100644 (file)
--- a/app/controllers/user_controller.rb
+++ b/app/controllers/user_controller.rb
@@ -67,7 +67,12 @@ class UserController < ApplicationController
           flash.now[:notice] = t 'user.account.flash update success'
         else
           flash.now[:notice] = t 'user.account.flash update success confirm needed'
           flash.now[:notice] = t 'user.account.flash update success'
         else
           flash.now[:notice] = t 'user.account.flash update success confirm needed'

-          Notifier.deliver_email_confirm(@user, @user.tokens.create)
+
+          begin
+            Notifier.deliver_email_confirm(@user, @user.tokens.create)
+          rescue
+            # Ignore errors sending email
+          end

         end
       end
     else
         end
       end
     else


@@ -107,7 +112,8 @@ class UserController < ApplicationController
       if user
         token = user.tokens.create
         Notifier.deliver_lost_password(user, token)
       if user
         token = user.tokens.create
         Notifier.deliver_lost_password(user, token)

-        flash.now[:notice] = t 'user.lost_password.notice email on way'
+        flash[:notice] = t 'user.lost_password.notice email on way'
+        redirect_to :action => 'login'

       else
         flash.now[:error] = t 'user.lost_password.notice email cannot find'
       end
       else
         flash.now[:error] = t 'user.lost_password.notice email cannot find'
       end


@@ -157,6 +163,7 @@ class UserController < ApplicationController
       user = User.authenticate(:username => email_or_display_name, :password => pass)
       if user
         session[:user] = user.id
       user = User.authenticate(:username => email_or_display_name, :password => pass)
       if user
         session[:user] = user.id

+        session_expires_after 1.month if params[:remember_me]

       elsif User.authenticate(:username => email_or_display_name, :password => pass, :inactive => true)
         flash.now[:error] = t 'user.login.account not active'
       else
       elsif User.authenticate(:username => email_or_display_name, :password => pass, :inactive => true)
         flash.now[:error] = t 'user.login.account not active'
       else


@@ -192,6 +199,7 @@ class UserController < ApplicationController
       session[:token] = nil
     end
     session[:user] = nil
       session[:token] = nil
     end
     session[:user] = nil

+    session_expires_automatically

     if params[:referer]
       redirect_to params[:referer]
     else
     if params[:referer]
       redirect_to params[:referer]
     else