jro_apps/redmine Commit - r1029:4a729036bf0a

Prevent 'has already been taken' error messages for user login and email if these fields are left empty....

Jean-Philippe Lang -

r1029:4a729036bf0a

parent child

Context file:

r1029:4a729036bf0a

Collapse all files

app/models/user.rb +2 -1

              # redMine - project management software
              # Copyright (C) 2006-2007  Jean-Philippe Lang
              #
              # This program is free software; you can redistribute it and/or
              # modify it under the terms of the GNU General Public License
              # as published by the Free Software Foundation; either version 2
              # of the License, or (at your option) any later version.
              #
              # This program is distributed in the hope that it will be useful,
              # but WITHOUT ANY WARRANTY; without even the implied warranty of
              # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
              # GNU General Public License for more details.
              #
              # You should have received a copy of the GNU General Public License
              # along with this program; if not, write to the Free Software
              # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
              require "digest/sha1"
              class User < ActiveRecord::Base
                # Account statuses
                STATUS_ANONYMOUS  = 0
                STATUS_ACTIVE     = 1
                STATUS_REGISTERED = 2
                STATUS_LOCKED     = 3
                has_many :memberships, :class_name => 'Member', :include => [ :project, :role ], :conditions => "#{Project.table_name}.status=#{Project::STATUS_ACTIVE}", :order => "#{Project.table_name}.name", :dependent => :delete_all
                has_many :projects, :through => :memberships
                has_many :custom_values, :dependent => :delete_all, :as => :customized
                has_many :issue_categories, :foreign_key => 'assigned_to_id', :dependent => :nullify
                has_one :preference, :dependent => :destroy, :class_name => 'UserPreference'
                has_one :rss_token, :dependent => :destroy, :class_name => 'Token', :conditions => "action='feeds'"
                belongs_to :auth_source
                attr_accessor :password, :password_confirmation
                attr_accessor :last_before_login_on
                # Prevents unauthorized assignments
                attr_protected :login, :admin, :password, :password_confirmation, :hashed_password
                validates_presence_of :login, :firstname, :lastname, :mail, :if => Proc.new { |user| !user.is_a?(AnonymousUser) }
-               validates_uniqueness_of :login, :mail
+               validates_uniqueness_of :login, :if => Proc.new { |user| !user.login.blank? }
+               validates_uniqueness_of :mail, :if => Proc.new { |user| !user.mail.blank? }
                # Login must contain lettres, numbers, underscores only
                validates_format_of :login, :with => /^[a-z0-9_\-@\.]*$/i
                validates_length_of :login, :maximum => 30
                validates_format_of :firstname, :lastname, :with => /^[\w\s\'\-]*$/i
                validates_length_of :firstname, :lastname, :maximum => 30
                validates_format_of :mail, :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i, :allow_nil => true
                validates_length_of :mail, :maximum => 60, :allow_nil => true
                validates_length_of :password, :minimum => 4, :allow_nil => true
                validates_confirmation_of :password, :allow_nil => true
                validates_associated :custom_values, :on => :update
                def before_create
                  self.mail_notification = false
                  true
                end
                def before_save
                  # update hashed_password if password was set
                  self.hashed_password = User.hash_password(self.password) if self.password
                end
                def self.active
                  with_scope :find => { :conditions => [ "status = ?", STATUS_ACTIVE ] } do
                    yield
                  end
                end
                def self.find_active(*args)
                  active do
                    find(*args)
                  end
                end
                # Returns the user that matches provided login and password, or nil
                def self.try_to_login(login, password)
                  user = find(:first, :conditions => ["login=?", login])
                  if user
                    # user is already in local database
                    return nil if !user.active?
                    if user.auth_source
                      # user has an external authentication method
                      return nil unless user.auth_source.authenticate(login, password)
                    else
                      # authentication with local password
                      return nil unless User.hash_password(password) == user.hashed_password
                    end
                  else
                    # user is not yet registered, try to authenticate with available sources
                    attrs = AuthSource.authenticate(login, password)
                    if attrs
                      onthefly = new(*attrs)
                      onthefly.login = login
                      onthefly.language = Setting.default_language
                      if onthefly.save
                        user = find(:first, :conditions => ["login=?", login])
                        logger.info("User '#{user.login}' created on the fly.") if logger
                      end
                    end
                  end
                  user.update_attribute(:last_login_on, Time.now) if user
                  user
                  rescue => text
                    raise text
                end
                # Return user's full name for display
                def name
                  "#{firstname} #{lastname}"
                end
                def active?
                  self.status == STATUS_ACTIVE
                end
                def registered?
                  self.status == STATUS_REGISTERED
                end
                def locked?
                  self.status == STATUS_LOCKED
                end
                def check_password?(clear_password)
                  User.hash_password(clear_password) == self.hashed_password
                end
                def pref
                  self.preference ||= UserPreference.new(:user => self)
                end
                def time_zone
                  self.pref.time_zone.nil? ? nil : TimeZone[self.pref.time_zone]
                end
                # Return user's RSS key (a 40 chars long string), used to access feeds
                def rss_key
                  token = self.rss_token || Token.create(:user => self, :action => 'feeds')
                  token.value
                end
                # Return an array of project ids for which the user has explicitly turned mail notifications on
                def notified_projects_ids
                  @notified_projects_ids ||= memberships.select {|m| m.mail_notification?}.collect(&:project_id)
                end
                def notified_project_ids=(ids)
                  Member.update_all("mail_notification = #{connection.quoted_false}", ['user_id = ?', id])
                  Member.update_all("mail_notification = #{connection.quoted_true}", ['user_id = ? AND project_id IN (?)', id, ids]) if ids && !ids.empty?
                  @notified_projects_ids = nil
                  notified_projects_ids
                end
                def self.find_by_rss_key(key)
                  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)
                  user.nil? ? -1 : (lastname == user.lastname ? firstname <=> user.firstname : lastname <=> user.lastname)
                end
                def to_s
                  name
                end
                def logged?
                  true
                end
                # Return user's role for project
                def role_for_project(project)
                  # No role on archived projects
                  return nil unless project && project.active?
                  if logged?
                    # Find project membership
                    membership = memberships.detect {|m| m.project_id == project.id}
                    if membership
                      membership.role
                    else
                      @role_non_member ||= Role.non_member
                    end
                  else
                    @role_anonymous ||= Role.anonymous
                  end
                end
                # Return true if the user is a member of project
                def member_of?(project)
                  role_for_project(project).member?
                end
                # Return true if the user is allowed to do the specified action on project
                # action can be:
                # * a parameter-like Hash (eg. :controller => 'projects', :action => 'edit')
                # * a permission Symbol (eg. :edit_project)
                def allowed_to?(action, project)
                  # No action allowed on archived projects
                  return false unless project.active?
                  # No action allowed on disabled modules
                  return false unless project.allows_to?(action)
                  # Admin users are authorized for anything else
                  return true if admin?
                  role = role_for_project(project)
                  return false unless role
                  role.allowed_to?(action) && (project.is_public? || role.member?)
                end
                def self.current=(user)
                  @current_user = user
                end
                def self.current
                  @current_user ||= User.anonymous
                end
                def self.anonymous
                  return @anonymous_user if @anonymous_user
                  anonymous_user = AnonymousUser.find(:first)
                  if anonymous_user.nil?
                    anonymous_user = AnonymousUser.create(:lastname => 'Anonymous', :firstname => '', :mail => '', :login => '', :status => 0)
                    raise 'Unable to create the anonymous user.' if anonymous_user.new_record?
                  end
                  @anonymous_user = anonymous_user
                end
              private
                # Return password digest
                def self.hash_password(clear_password)
                  Digest::SHA1.hexdigest(clear_password || "")
                end
              end
              class AnonymousUser < User
                def validate_on_create
                  # There should be only one AnonymousUser in the database
                  errors.add_to_base 'An anonymous user already exists.' if AnonymousUser.find(:first)
                end
                # Overrides a few properties
                def logged?; false end
                def admin; false end
                def name; 'Anonymous' end
                def mail; nil end
                def time_zone; nil end
                def rss_key; nil end
              end

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages