Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

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
  • Loading branch information...
commit 6cb4ff7d89817a512a49bffd5fe3aa465f590ff4 1 parent de17640
@edavis10 edavis10 authored
Showing with 5 additions and 5 deletions.
  1. +5 −5 app/models/user.rb
View
10 app/models/user.rb
@@ -221,14 +221,14 @@ def notified_project_ids=(ids)
@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)
Please sign in to comment.
Something went wrong with that request. Please try again.