Added autologin feature (disabled by default).

To enable this feature, go to administration settings and choose a duration for autologin.
When enabled, a checkbox on the login form lets users activate autologin.

git-svn-id: http://redmine.rubyforge.org/svn/trunk@514 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/controllers/account_controller.rb b/app/controllers/account_controller.rb
index 61c7155..9b54a90 100644 (file)
--- a/app/controllers/account_controller.rb
+++ b/app/controllers/account_controller.rb
@@ -42,6 +42,11 @@ class AccountController < ApplicationController
       user = User.try_to_login(params[:login], params[:password])
       if user
         self.logged_in_user = user
+        # generate a key and set cookie if autologin
+        if params[:autologin] && Setting.autologin?
+          token = Token.create(:user => user, :action => 'autologin')
+          cookies[:autologin] = { :value => token.value, :expires => 1.year.from_now }
+        end
         redirect_back_or_default :controller => 'my', :action => 'page'
       else
         flash.now[:notice] = l(:notice_account_invalid_creditentials)
@@ -51,6 +56,8 @@ class AccountController < ApplicationController
 
   # Log out current user and redirect to welcome page
   def logout
+    cookies.delete :autologin
+    Token.delete_all(["user_id = ? AND action = ?", logged_in_user.id, "autologin"]) if logged_in_user
     self.logged_in_user = nil
     redirect_to :controller => 'welcome'
   end

diff --git a/app/controllers/application.rb b/app/controllers/application.rb
index 2a8e151..54e4768 100644 (file)
--- a/app/controllers/application.rb
+++ b/app/controllers/application.rb
@@ -40,6 +40,13 @@ class ApplicationController < ActionController::Base
   
   # check if login is globally required to access the application
   def check_if_login_required
+    # no check needed if user is already logged in
+    return true if logged_in_user
+    # auto-login feature
+    autologin_key = cookies[:autologin]
+    if autologin_key && Setting.autologin?
+      self.logged_in_user = User.find_by_autologin_key(autologin_key)
+    end
     require_login if Setting.login_required?
   end 
   

diff --git a/app/models/setting.rb b/app/models/setting.rb
index 56f7a52..ecca01c 100644 (file)
--- a/app/models/setting.rb
+++ b/app/models/setting.rb
@@ -49,7 +49,7 @@ class Setting < ActiveRecord::Base
     end
 
     def self.#{name}?
-      self[:#{name}].to_s == "1"
+      self[:#{name}].to_i > 0
     end
 
     def self.#{name}=(value)

diff --git a/app/models/user.rb b/app/models/user.rb
index 6ae1301..1c56138 100644 (file)
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -141,6 +141,11 @@ class User < ActiveRecord::Base
     token = Token.find_by_value(key)
     token && token.user.active? ? token.user : nil
   end
+  
+  def self.find_by_autologin_key(key)
+    token = Token.find_by_action_and_value('autologin', key)
+    token && (token.created_on > Setting.autologin.to_i.day.ago) && token.user.active? ? token.user : nil
+  end
 
   def <=>(user)
     lastname == user.lastname ? firstname <=> user.firstname : lastname <=> user.lastname

diff --git a/app/views/account/login.rhtml b/app/views/account/login.rhtml
index 7148bdd..08d462c 100644 (file)
--- a/app/views/account/login.rhtml
+++ b/app/views/account/login.rhtml
@@ -3,23 +3,26 @@
 <h2 class="icon22 icon22-authent"><%=l(:label_please_login)%></h2>
 
 <% form_tag({:action=> "login"}, :class => "tabular") do %>
+
 <p><label for="login"><%=l(:field_login)%>:</label>
 <%= text_field_tag 'login', nil, :size => 25 %></p>
 
 <p><label for="password"><%=l(:field_password)%>:</label>
 <%= password_field_tag 'password', nil, :size => 25 %></p>
 
-<p><center><input type="submit" name="login" value="<%=l(:button_login)%> &#187;" class="primary" /></center>
+<% if Setting.autologin? %>
+<p><label for="autologin"><%= check_box_tag 'autologin' %> <%= l(:label_stay_logged_in) %></label></p>
+<% end %>
+
+<p><input type="submit" name="login" value="<%=l(:button_login)%> &#187;" class="primary" /></p>
 <% end %>
 <%= javascript_tag "Form.Element.focus('login');" %>
 
-<br>
 <% links = []
    links << link_to(l(:label_register), :action => 'register') if Setting.self_registration?
    links << link_to(l(:label_password_lost), :action => 'lost_password') if Setting.lost_password?
 %>
 <%= links.join(" | ") %>
-</p>
 
 </div>
 </center>
\ No newline at end of file

diff --git a/app/views/settings/edit.rhtml b/app/views/settings/edit.rhtml
index 057a988..f88d3f1 100644 (file)
--- a/app/views/settings/edit.rhtml
+++ b/app/views/settings/edit.rhtml
@@ -15,15 +15,6 @@
 <p><label><%= l(:setting_default_language) %></label>
 <%= select_tag 'settings[default_language]', options_for_select( lang_options_for_select(false), Setting.default_language) %></p>
 
-<p><label><%= l(:setting_login_required) %></label>
-<%= check_box_tag 'settings[login_required]', 1, Setting.login_required? %><%= hidden_field_tag 'settings[login_required]', 0 %></p>
-
-<p><label><%= l(:setting_self_registration) %></label>
-<%= check_box_tag 'settings[self_registration]', 1, Setting.self_registration? %><%= hidden_field_tag 'settings[self_registration]', 0 %></p>
-
-<p><label><%= l(:label_password_lost) %></label>
-<%= check_box_tag 'settings[lost_password]', 1, Setting.lost_password? %><%= hidden_field_tag 'settings[lost_password]', 0 %></p>
-
 <p><label><%= l(:setting_attachment_max_size) %></label>
 <%= text_field_tag 'settings[attachment_max_size]', Setting.attachment_max_size, :size => 6 %> KB</p>
 
@@ -52,6 +43,20 @@
 <%= check_box_tag 'settings[sys_api_enabled]', 1, Setting.sys_api_enabled? %><%= hidden_field_tag 'settings[sys_api_enabled]', 0 %></p>
 </div>
 
+<fieldset class="box"><legend><%= l(:label_authentication) %></legend>
+<p><label><%= l(:setting_login_required) %></label>
+<%= check_box_tag 'settings[login_required]', 1, Setting.login_required? %><%= hidden_field_tag 'settings[login_required]', 0 %></p>
+
+<p><label><%= l(:setting_autologin) %></label>
+<%= select_tag 'settings[autologin]', options_for_select( [[l(:label_disabled), "0"]] + [1, 7, 30, 365].collect{|days| [lwr(:actionview_datehelper_time_in_words_day, days), days.to_s]}, Setting.autologin) %></p>
+
+<p><label><%= l(:setting_self_registration) %></label>
+<%= check_box_tag 'settings[self_registration]', 1, Setting.self_registration? %><%= hidden_field_tag 'settings[self_registration]', 0 %></p>
+
+<p><label><%= l(:label_password_lost) %></label>
+<%= check_box_tag 'settings[lost_password]', 1, Setting.lost_password? %><%= hidden_field_tag 'settings[lost_password]', 0 %></p>
+</fieldset>
+
 <fieldset class="box"><legend><%= l(:text_issues_ref_in_commit_messages) %></legend>
 <p><label><%= l(:setting_commit_ref_keywords) %></label>
 <%= text_field_tag 'settings[commit_ref_keywords]', Setting.commit_ref_keywords, :size => 30 %><br /><em><%= l(:text_coma_separated) %></em></p>

diff --git a/config/settings.yml b/config/settings.yml
index 4632478..39e2e85 100644 (file)
--- a/config/settings.yml
+++ b/config/settings.yml
@@ -61,4 +61,8 @@ commit_fix_keywords:
 commit_fix_status_id:\r
   format: int\r
   default: 0\r
-  
\ No newline at end of file
+# autologin duration in days\r
+# 0 means autologin is disabled \r
+autologin:\r
+  format: int\r
+  default: 0\r

diff --git a/lang/bg.yml b/lang/bg.yml
index 6041f71..1e348cb 100644 (file)
--- a/lang/bg.yml
+++ b/lang/bg.yml
@@ -171,6 +171,7 @@ setting_autofetch_changesets: Автоматично обработване на
 setting_sys_api_enabled: Разрешаване на WS за управление на SVN склада
 setting_commit_ref_keywords: Отбелязващи ключови думи
 setting_commit_fix_keywords: Приключващи ключови думи
+setting_autologin: Autologin
 
 label_user: Потребител
 label_user_plural: Потребители
@@ -380,6 +381,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: Вход
 button_submit: Изпращане

diff --git a/lang/de.yml b/lang/de.yml
index 35e4328..1fc529b 100644 (file)
--- a/lang/de.yml
+++ b/lang/de.yml
@@ -171,6 +171,7 @@ setting_autofetch_changesets: Autofetch SVN commits
 setting_sys_api_enabled: Enable WS for repository management
 setting_commit_ref_keywords: Referencing keywords
 setting_commit_fix_keywords: Fixing keywords
+setting_autologin: Autologin
 
 label_user: Benutzer
 label_user_plural: Benutzer
@@ -380,6 +381,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: Einloggen
 button_submit: OK

diff --git a/lang/en.yml b/lang/en.yml
index fadcaad..1800b5d 100644 (file)
--- a/lang/en.yml
+++ b/lang/en.yml
@@ -171,6 +171,7 @@ setting_autofetch_changesets: Autofetch SVN commits
 setting_sys_api_enabled: Enable WS for repository management
 setting_commit_ref_keywords: Referencing keywords
 setting_commit_fix_keywords: Fixing keywords
+setting_autologin: Autologin
 
 label_user: User
 label_user_plural: Users
@@ -380,6 +381,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: Login
 button_submit: Submit

diff --git a/lang/es.yml b/lang/es.yml
index e6c9002..3a66e0b 100644 (file)
--- a/lang/es.yml
+++ b/lang/es.yml
@@ -171,6 +171,7 @@ setting_autofetch_changesets: Autofetch SVN commits
 setting_sys_api_enabled: Enable WS for repository management
 setting_commit_ref_keywords: Referencing keywords
 setting_commit_fix_keywords: Fixing keywords
+setting_autologin: Autologin
 
 label_user: Usuario
 label_user_plural: Usuarios
@@ -380,6 +381,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: Conexión
 button_submit: Someter

diff --git a/lang/fr.yml b/lang/fr.yml
index 394417a..bd1973b 100644 (file)
--- a/lang/fr.yml
+++ b/lang/fr.yml
@@ -171,6 +171,7 @@ setting_autofetch_changesets: Récupération auto. des commits SVN
 setting_sys_api_enabled: Activer les WS pour la gestion des dépôts
 setting_commit_ref_keywords: Mot-clés de référencement
 setting_commit_fix_keywords: Mot-clés de résolution
+setting_autologin: Autologin
 
 label_user: Utilisateur
 label_user_plural: Utilisateurs
@@ -380,6 +381,8 @@ label_end_to_start: début à fin
 label_end_to_end: fin à fin
 label_start_to_start: début à début
 label_start_to_end: début à fin
+label_stay_logged_in: Rester connecté
+label_disabled: désactivé
 
 button_login: Connexion
 button_submit: Soumettre

diff --git a/lang/it.yml b/lang/it.yml
index 2deb091..c1e9645 100644 (file)
--- a/lang/it.yml
+++ b/lang/it.yml
@@ -171,6 +171,7 @@ setting_autofetch_changesets: Acquisisci automaticamente le commit SVN
 setting_sys_api_enabled: Abilita WS per la gestione del repository
 setting_commit_ref_keywords: Referencing keywords
 setting_commit_fix_keywords: Fixing keywords
+setting_autologin: Autologin
 
 label_user: Utente
 label_user_plural: Utenti
@@ -380,6 +381,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: Login
 button_submit: Invia

diff --git a/lang/ja.yml b/lang/ja.yml
index f746faf..9557994 100644 (file)
--- a/lang/ja.yml
+++ b/lang/ja.yml
@@ -172,6 +172,7 @@ setting_autofetch_changesets: SVNコミットを自動取得する
 setting_sys_api_enabled: リポジトリ管理用のWeb Serviceを有効化する
 setting_commit_ref_keywords: Referencing keywords
 setting_commit_fix_keywords: Fixing keywords
+setting_autologin: Autologin
 
 label_user: ユーザ
 label_user_plural: ユーザ
@@ -381,6 +382,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: ログイン
 button_submit: 変更

diff --git a/lang/pt.yml b/lang/pt.yml
index 0d8bfc6..3c6debf 100644 (file)
--- a/lang/pt.yml
+++ b/lang/pt.yml
@@ -171,6 +171,7 @@ setting_autofetch_changesets: Autofetch SVN commits
 setting_sys_api_enabled: Ativa WS para gerenciamento do repositorio
 setting_commit_ref_keywords: Referencing keywords
 setting_commit_fix_keywords: Fixing keywords
+setting_autologin: Autologin
 
 label_user: Usuario
 label_user_plural: Usuarios
@@ -380,6 +381,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: Login
 button_submit: Enviar

diff --git a/lang/zh.yml b/lang/zh.yml
index 344d815..f9f2a75 100644 (file)
--- a/lang/zh.yml
+++ b/lang/zh.yml
@@ -174,6 +174,7 @@ setting_autofetch_changesets: Autofetch SVN commits
 setting_sys_api_enabled: Enable WS for repository management
 setting_commit_ref_keywords: Referencing keywords
 setting_commit_fix_keywords: Fixing keywords
+setting_autologin: Autologin
 
 label_user: 用户
 label_user_plural: 用户列表
@@ -383,6 +384,8 @@ label_end_to_start: start to end
 label_end_to_end: end to end
 label_start_to_start: start to start
 label_start_to_end: start to end
+label_stay_logged_in: Stay logged in
+label_disabled: disabled
 
 button_login: 登录
 button_submit: 提交