Refactor and documentation for User#find_by_login.

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3808 e93f8b46-1217-0410-a6f0-8f06a7374b81

Refactor and documentation for User#find_by_login.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3808 e93f8b46-1217-0410-a6f0-8f06a7374b81
6cb4ff7d · Eric Davis · de176404 · 6cb4ff7d
Commit 6cb4ff7d authored Jun 20, 2010 by Eric Davis
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 5 deletions

user.rb app/models/user.rb +5 -5

No files found.
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -221,14 +221,14 @@ class User < Principal
    @notified_projects_ids = nil
    notified_projects_ids
  end
-  
-  # case-insensitive fall-over
+
+  # Find a user account by matching the exact login and then a case-insensitive
+  # version.  Exact matches will be given priority.
  def self.find_by_login(login)
    # First look for an exact match
-    user = find(:first, :conditions => ["login = ?", login])
+    user = first(:conditions => {:login => login})
    # Fail over to case-insensitive if none was found
-    user = find(:first, :conditions => ["LOWER(login) = ?", login.to_s.downcase]) if user.nil?
-    return user
+    user ||= first(:conditions => ["LOWER(login) = ?", login.to_s.downcase])
  end

  def self.find_by_rss_key(key)