Redmine

JB, Eric: the commit for this is here: http://github.com/thegcat/redmine/commit/7ab0baa96a93f0c50d0bc99cefcb52e6fdcba1f4

The change is small: it forces the select list to return 0 instead of '' for "autologin disabled". The problem with having an empty string returned for autologin is that autologin is defined as an int in the settings and thus checked for numericality, which fails for the empty string.

Jan, if you feel up to applying a patch to your installation, here's the diff:

diff --git a/app/views/settings/_authentication.rhtml b/app/views/settings/_authentication.rhtml
index da29bb7..1122053 100644
--- a/app/views/settings/_authentication.rhtml
+++ b/app/views/settings/_authentication.rhtml
@@ -3,7 +3,7 @@
 <div class="box tabular settings">
 <p><%= setting_check_box :login_required %></p>

-<p><%= setting_select :autologin, [1, 7, 30, 365].collect{|days| [l('datetime.distance_in_words.x_days', :count => days), days.to_s]}, :blank => :label_disabled %></p>
+<p><%= setting_select :autologin, [[l(:label_disabled), 0]] + [1, 7, 30, 365].collect{|days| [l('datetime.distance_in_words.x_days', :count => days), days.to_s]} %></p>

 <p><%= setting_select :self_registration, [[l(:label_disabled), "0"],
                                                                                     [l(:label_registration_activation_by_email), "1"],

Applied in r4306, thanks.

Merged into 1.0-stable for release in 1.0.3