2a90da1837
Comparison was downcasing only one side, therefore if previously existing account had a non-lowercase spelling, it would be ignored when checking for duplicates. New rake task `mastodon:maintenance:find_duplicate_usernames` will help find constraint violations that might have occured from the presence of this bug. Bump version to 2.3.3
62 lines
1.3 KiB
Ruby
62 lines
1.3 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
module AccountFinderConcern
|
|
extend ActiveSupport::Concern
|
|
|
|
class_methods do
|
|
def find_local!(username)
|
|
find_local(username) || raise(ActiveRecord::RecordNotFound)
|
|
end
|
|
|
|
def find_remote!(username, domain)
|
|
find_remote(username, domain) || raise(ActiveRecord::RecordNotFound)
|
|
end
|
|
|
|
def find_local(username)
|
|
find_remote(username, nil)
|
|
end
|
|
|
|
def find_remote(username, domain)
|
|
AccountFinder.new(username, domain).account
|
|
end
|
|
end
|
|
|
|
class AccountFinder
|
|
attr_reader :username, :domain
|
|
|
|
def initialize(username, domain)
|
|
@username = username
|
|
@domain = domain
|
|
end
|
|
|
|
def account
|
|
scoped_accounts.order(id: :asc).take
|
|
end
|
|
|
|
private
|
|
|
|
def scoped_accounts
|
|
Account.unscoped.tap do |scope|
|
|
scope.merge! with_usernames
|
|
scope.merge! matching_username
|
|
scope.merge! matching_domain
|
|
end
|
|
end
|
|
|
|
def with_usernames
|
|
Account.where.not(username: '')
|
|
end
|
|
|
|
def matching_username
|
|
Account.where(Account.arel_table[:username].lower.eq username.to_s.downcase)
|
|
end
|
|
|
|
def matching_domain
|
|
if domain.nil?
|
|
Account.where(domain: nil)
|
|
else
|
|
Account.where(Account.arel_table[:domain].lower.eq domain.to_s.downcase)
|
|
end
|
|
end
|
|
end
|
|
end
|