X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/34189cf761a871f366775843da580121b39b4e25..3aa69ffbdda5f111aed60d77e1a1eec8cdcad3ba:/app/models/user.rb diff --git a/app/models/user.rb b/app/models/user.rb index 9b5bfd595..e6d7a4e46 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -1,107 +1,258 @@ class User < ActiveRecord::Base - require 'xml/libxml' - - has_many :traces - has_many :diary_entries, :order => 'created_at DESC' - has_many :messages, :foreign_key => :to_user_id, :order => 'sent_on DESC' - has_many :new_messages, :class_name => "Message", :foreign_key => :to_user_id, :conditions => "message_read = 0", :order => 'sent_on DESC' - has_many :sent_messages, :class_name => "Message", :foreign_key => :from_user_id, :order => 'sent_on DESC' - has_many :friends, :include => :befriendee, :conditions => "users.visible = 1" + require "xml/libxml" + + has_many :traces, -> { where(:visible => true) } + has_many :diary_entries, -> { order(:created_at => :desc) } + has_many :diary_comments, -> { order(:created_at => :desc) } + has_many :diary_entry_subscriptions, :class_name => "DiaryEntrySubscription" + has_many :diary_subscriptions, :through => :diary_entry_subscriptions, :source => :diary_entry + has_many :messages, -> { where(:to_user_visible => true).order(:sent_on => :desc).preload(:sender, :recipient) }, :foreign_key => :to_user_id + has_many :new_messages, -> { where(:to_user_visible => true, :message_read => false).order(:sent_on => :desc) }, :class_name => "Message", :foreign_key => :to_user_id + has_many :sent_messages, -> { where(:from_user_visible => true).order(:sent_on => :desc).preload(:sender, :recipient) }, :class_name => "Message", :foreign_key => :from_user_id + has_many :friends, -> { joins(:befriendee).where(:users => { :status => %w(active confirmed) }) } + has_many :friend_users, :through => :friends, :source => :befriendee has_many :tokens, :class_name => "UserToken" has_many :preferences, :class_name => "UserPreference" + has_many :changesets, -> { order(:created_at => :desc) } + has_many :changeset_comments, :foreign_key => :author_id + has_and_belongs_to_many :changeset_subscriptions, :class_name => "Changeset", :join_table => "changesets_subscribers", :foreign_key => "subscriber_id" + has_many :note_comments, :foreign_key => :author_id + has_many :notes, :through => :note_comments - validates_presence_of :email, :display_name - validates_confirmation_of :email, :message => 'Email addresses must match' - validates_confirmation_of :pass_crypt, :message => 'Password must match the confirmation password' - validates_uniqueness_of :display_name, :allow_nil => true - validates_uniqueness_of :email - validates_length_of :pass_crypt, :minimum => 8 - validates_length_of :display_name, :minimum => 3, :allow_nil => true - validates_format_of :email, :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i - validates_format_of :display_name, :with => /^[^\/;.,?]*$/ - validates_numericality_of :home_lat, :allow_nil => true - validates_numericality_of :home_lon, :allow_nil => true - validates_numericality_of :home_zoom, :only_integer => true, :allow_nil => true + has_many :client_applications + has_many :oauth_tokens, -> { order(:authorized_at => :desc).preload(:client_application) }, :class_name => "OauthToken" - before_save :encrypt_password + has_many :blocks, :class_name => "UserBlock" + has_many :blocks_created, :class_name => "UserBlock", :foreign_key => :creator_id + has_many :blocks_revoked, :class_name => "UserBlock", :foreign_key => :revoker_id - file_column :image, :magick => { :geometry => "100x100>" } + has_many :roles, :class_name => "UserRole" - def after_initialize - self.creation_time = Time.now if self.creation_time.nil? - end + scope :visible, -> { where(:status => %w(pending active confirmed)) } + scope :active, -> { where(:status => %w(active confirmed)) } + scope :identifiable, -> { where(:data_public => true) } - def encrypt_password - if pass_crypt_confirmation - self.pass_salt = OSM::make_token(8) - self.pass_crypt = OSM::encrypt_password(pass_crypt, pass_salt) - end - end + has_attached_file :image, + :default_url => "/assets/:class/:attachment/:style.png", + :styles => { :large => "100x100>", :small => "50x50>" } + + validates :display_name, :presence => true, :allow_nil => true, :length => 3..255, + :exclusion => %w(new terms save confirm confirm-email go_public reset-password forgot-password suspended) + validates :display_name, :if => proc { |u| u.display_name_changed? }, + :uniqueness => { :case_sensitive => false } + validates :display_name, :if => proc { |u| u.display_name_changed? }, + :format => { :with => %r{\A[^\x00-\x1f\x7f\ufffe\uffff/;.,?%#]*\z} } + validates :display_name, :if => proc { |u| u.display_name_changed? }, + :format => { :with => /\A\S/, :message => "has leading whitespace" } + validates :display_name, :if => proc { |u| u.display_name_changed? }, + :format => { :with => /\S\z/, :message => "has trailing whitespace" } + validates :email, :presence => true, :confirmation => true + validates :email, :if => proc { |u| u.email_changed? }, + :uniqueness => { :case_sensitive => false } + validates :pass_crypt, :confirmation => true, :length => 8..255 + validates :home_lat, :home_lon, :allow_nil => true, :numericality => true + validates :home_zoom, :allow_nil => true, :numericality => { :only_integer => true } + validates :preferred_editor, :inclusion => Editors::ALL_EDITORS, :allow_nil => true + validates :image, :attachment_content_type => { :content_type => %r{\Aimage/.*\Z} } + validates :auth_uid, :unless => proc { |u| u.auth_provider.nil? }, + :uniqueness => { :scope => :auth_provider } + + validates_email_format_of :email, :if => proc { |u| u.email_changed? } + validates_email_format_of :new_email, :allow_blank => true, :if => proc { |u| u.new_email_changed? } + + after_initialize :set_defaults + before_save :encrypt_password + after_save :spam_check def self.authenticate(options) - if options[:username] and options[:password] - user = find(:first, :conditions => ["email = ? OR display_name = ?", options[:username], options[:username]]) - user = nil if user and user.pass_crypt != OSM::encrypt_password(options[:password], user.pass_salt) + if options[:username] && options[:password] + user = find_by("email = ? OR display_name = ?", options[:username], options[:username]) + + if user.nil? + users = where("LOWER(email) = LOWER(?) OR LOWER(display_name) = LOWER(?)", options[:username], options[:username]) + + user = users.first if users.count == 1 + end + + if user && PasswordHash.check(user.pass_crypt, user.pass_salt, options[:password]) + if PasswordHash.upgrade?(user.pass_crypt, user.pass_salt) + user.pass_crypt, user.pass_salt = PasswordHash.create(options[:password]) + user.save + end + else + user = nil + end elsif options[:token] - token = UserToken.find(:first, :include => :user, :conditions => ["user_tokens.token = ?", options[:token]]) + token = UserToken.find_by(:token => options[:token]) user = token.user if token end - if user - user = nil unless user.visible? and (user.active? or options[:inactive]) + if user && + (user.status == "deleted" || + (user.status == "pending" && !options[:pending]) || + (user.status == "suspended" && !options[:suspended])) + user = nil end - token.update_attribute(:expiry, 1.week.from_now) if token and user + token.update_column(:expiry, 1.week.from_now) if token && user - return user - end + user + end def to_xml doc = OSM::API.new.get_xml_doc - doc.root << to_xml_node() - return doc + doc.root << to_xml_node + doc end def to_xml_node - el1 = XML::Node.new 'user' - el1['display_name'] = self.display_name.to_s - el1['account_created'] = self.creation_time.xmlschema - if self.home_lat and self.home_lon - home = XML::Node.new 'home' - home['lat'] = self.home_lat.to_s - home['lon'] = self.home_lon.to_s - home['zoom'] = self.home_zoom.to_s + el1 = XML::Node.new "user" + el1["display_name"] = display_name.to_s + el1["account_created"] = creation_time.xmlschema + if home_lat && home_lon + home = XML::Node.new "home" + home["lat"] = home_lat.to_s + home["lon"] = home_lon.to_s + home["zoom"] = home_zoom.to_s el1 << home end - return el1 + el1 + end + + def description + RichText.new(self[:description_format], self[:description]) + end + + def languages + attribute_present?(:languages) ? self[:languages].split(/ *[, ] */) : [] + end + + def languages=(languages) + self[:languages] = languages.join(",") + end + + def preferred_language + languages.find { |l| Language.exists?(:code => l) } end - def nearby(radius = 50, num = 10) - if self.home_lon and self.home_lat - gc = OSM::GreatCircle.new(self.home_lat, self.home_lon) - bounds = gc.bounds(radius) - nearby = User.find(:all, :conditions => "visible = 1 and home_lat between #{bounds[:minlat]} and #{bounds[:maxlat]} and home_lon between #{bounds[:minlon]} and #{bounds[:maxlon]} and data_public = 1 and id != #{self.id}") - nearby.delete_if { |u| gc.distance(u.home_lat, u.home_lon) > radius } - nearby.sort! { |u1,u2| gc.distance(u1.home_lat, u1.home_lon) <=> gc.distance(u2.home_lat, u2.home_lon) } + def preferred_languages + @locales ||= Locale.list(languages) + end + + def nearby(radius = NEARBY_RADIUS, num = NEARBY_USERS) + if home_lon && home_lat + gc = OSM::GreatCircle.new(home_lat, home_lon) + sql_for_distance = gc.sql_for_distance("home_lat", "home_lon") + nearby = User.where("id != ? AND status IN (\'active\', \'confirmed\') AND data_public = ? AND #{sql_for_distance} <= ?", id, true, radius).order(sql_for_distance).limit(num) else nearby = [] end - return nearby.first(num) + nearby end def distance(nearby_user) - return OSM::GreatCircle.new(self.home_lat, self.home_lon).distance(nearby_user.home_lat, nearby_user.home_lon) + OSM::GreatCircle.new(home_lat, home_lon).distance(nearby_user.home_lat, nearby_user.home_lon) end def is_friends_with?(new_friend) - res = false - @new_friend = new_friend - self.friends.each do |friend| - if friend.friend_user_id == @new_friend.id - return true - end + friends.where(:friend_user_id => new_friend.id).exists? + end + + ## + # returns true if a user is visible + def visible? + %w(pending active confirmed).include? status + end + + ## + # returns true if a user is active + def active? + %w(active confirmed).include? status + end + + ## + # returns true if the user has the moderator role, false otherwise + def moderator? + has_role? "moderator" + end + + ## + # returns true if the user has the administrator role, false otherwise + def administrator? + has_role? "administrator" + end + + ## + # returns true if the user has the requested role + def has_role?(role) + roles.any? { |r| r.role == role } + end + + ## + # returns the first active block which would require users to view + # a message, or nil if there are none. + def blocked_on_view + blocks.active.detect(&:needs_view?) + end + + ## + # delete a user - leave the account but purge most personal data + def delete + self.display_name = "user_#{id}" + self.description = "" + self.home_lat = nil + self.home_lon = nil + self.image = nil + self.email_valid = false + self.new_email = nil + self.auth_provider = nil + self.auth_uid = nil + self.status = "deleted" + save + end + + ## + # return a spam score for a user + def spam_score + changeset_score = changesets.size * 50 + trace_score = traces.size * 50 + diary_entry_score = diary_entries.inject(0) { |acc, elem| acc + elem.body.spam_score } + diary_comment_score = diary_comments.inject(0) { |acc, elem| acc + elem.body.spam_score } + + score = description.spam_score / 4.0 + score += diary_entries.where("created_at > ?", 1.day.ago).count * 10 + score += diary_entry_score / diary_entries.length unless diary_entries.empty? + score += diary_comment_score / diary_comments.length unless diary_comments.empty? + score -= changeset_score + score -= trace_score + + score.to_i + end + + ## + # perform a spam check on a user + def spam_check + if status == "active" && spam_score > SPAM_THRESHOLD + update_column(:status, "suspended") end - return false end + ## + # return an oauth access token for a specified application + def access_token(application_key) + ClientApplication.find_by(:key => application_key).access_token_for_user(self) + end + + private + + def set_defaults + self.creation_time = Time.now.getutc unless attribute_present?(:creation_time) + end + + def encrypt_password + if pass_crypt_confirmation + self.pass_crypt, self.pass_salt = PasswordHash.create(pass_crypt) + self.pass_crypt_confirmation = nil + end + end end