Wrap methods of ProcessFeedService::ProcessEntry in classes (#4151)

ProcessFeedService::ProcessEntry had many methods, so wrap them in classes
representing activities.
lolsob-rspec
Akihiko Odaki (@fn_aki@pawoo.net) 2017-07-11 20:37:05 +09:00 committed by Eugen Rochko
parent 2e3f13788f
commit 002ed7dc62
1 changed files with 225 additions and 208 deletions

View File

@ -22,268 +22,285 @@ class ProcessFeedService < BaseService
class ProcessEntry class ProcessEntry
def call(xml, account) def call(xml, account)
@account = account @account = account
@xml = xml @fetched = Activity.new(xml)
return if skip_unsupported_type? return unless [:activity, :note, :comment].include?(@fetched.type)
case verb klass = case @fetched.verb
when :post, :share when :post
return create_status PostActivity
when :delete when :share
return delete_status ShareActivity
end when :delete
DeletionActivity
else
return
end
@fetched = klass.new(xml, account)
@fetched.perform
rescue ActiveRecord::RecordInvalid => e rescue ActiveRecord::RecordInvalid => e
Rails.logger.debug "Nothing was saved for #{id} because: #{e}" Rails.logger.debug "Nothing was saved for #{id} because: #{e}"
nil nil
end end
private class Activity
def initialize(xml, account = nil)
def create_status @xml = xml
if redis.exists("delete_upon_arrival:#{@account.id}:#{id}") @account = account
Rails.logger.debug "Delete for status #{id} was queued, ignoring"
return
end end
status, just_created = nil def verb
raw = @xml.at_xpath('./activity:verb', activity: TagManager::AS_XMLNS).content
Rails.logger.debug "Creating remote status #{id}" TagManager::VERBS.key(raw)
rescue
if verb == :share :post
original_status = shared_status_from_xml(@xml.at_xpath('.//activity:object', activity: TagManager::AS_XMLNS))
return nil if original_status.nil?
end end
ApplicationRecord.transaction do def type
status, just_created = status_from_xml(@xml) raw = @xml.at_xpath('./activity:object-type', activity: TagManager::AS_XMLNS).content
TagManager::TYPES.key(raw)
rescue
:activity
end
return if status.nil? def id
return status unless just_created @xml.at_xpath('./xmlns:id', xmlns: TagManager::XMLNS).content
end
if verb == :share def url
status.reblog = original_status.reblog? ? original_status.reblog : original_status link = @xml.at_xpath('./xmlns:link[@rel="alternate"]', xmlns: TagManager::XMLNS)
link.nil? ? nil : link['href']
end
private
def find_status(uri)
if TagManager.instance.local_id?(uri)
local_id = TagManager.instance.unique_tag_to_local_id(uri, 'Status')
return Status.find_by(id: local_id)
end end
status.save! Status.find_by(uri: uri)
end end
if thread?(@xml) && status.thread.nil? def redis
Rails.logger.debug "Trying to attach #{status.id} (#{id(@xml)}) to #{thread(@xml).first}" Redis.current
ThreadResolveWorker.perform_async(status.id, thread(@xml).second)
end
notify_about_mentions!(status) unless status.reblog?
notify_about_reblog!(status) if status.reblog? && status.reblog.account.local?
Rails.logger.debug "Queuing remote status #{status.id} (#{id}) for distribution"
LinkCrawlWorker.perform_async(status.id) unless status.spoiler_text?
DistributionWorker.perform_async(status.id)
status
end
def notify_about_mentions!(status)
status.mentions.includes(:account).each do |mention|
mentioned_account = mention.account
next unless mentioned_account.local?
NotifyService.new.call(mentioned_account, mention)
end end
end end
def notify_about_reblog!(status) class CreationActivity < Activity
NotifyService.new.call(status.reblog.account, status) def perform
end if redis.exists("delete_upon_arrival:#{@account.id}:#{id}")
Rails.logger.debug "Delete for status #{id} was queued, ignoring"
return
end
def delete_status Rails.logger.debug "Creating remote status #{id}"
Rails.logger.debug "Deleting remote status #{id}" # Return early if status already exists in db
status = Status.find_by(uri: id, account: @account) status = find_status(id)
if status.nil? return [status, false] unless status.nil?
redis.setex("delete_upon_arrival:#{@account.id}:#{id}", 6 * 3_600, id)
else
RemoveStatusService.new.call(status)
end
end
def skip_unsupported_type? return [nil, false] if @account.suspended?
!([:post, :share, :delete].include?(verb) && [:activity, :note, :comment].include?(type))
end
def shared_status_from_xml(entry) status = Status.create!(
status = find_status(id(entry)) uri: id,
url: url,
account: @account,
reblog: reblog,
text: content,
spoiler_text: content_warning,
created_at: published,
reply: thread?,
language: content_language,
visibility: visibility_scope,
conversation: converstation_to_persistent&.first,
thread: thread? ? find_status(thread.first) : nil
)
return status unless status.nil? save_mentions(status)
save_hashtags(status)
save_media(status)
FetchRemoteStatusService.new.call(url(entry)) if thread? && status.thread.nil?
end Rails.logger.debug "Trying to attach #{status.id} (#{id}) to #{thread.first}"
ThreadResolveWorker.perform_async(status.id, thread.second)
end
def status_from_xml(entry) Rails.logger.debug "Queuing remote status #{status.id} (#{id}) for distribution"
# Return early if status already exists in db
status = find_status(id(entry))
return [status, false] unless status.nil? LinkCrawlWorker.perform_async(status.id) unless status.spoiler_text?
DistributionWorker.perform_async(status.id)
account = @account [status, true]
return [nil, false] if account.suspended?
status = Status.create!(
uri: id(entry),
url: url(entry),
account: account,
text: content(entry),
spoiler_text: content_warning(entry),
created_at: published(entry),
reply: thread?(entry),
language: content_language(entry),
visibility: visibility_scope(entry),
conversation: find_or_create_conversation(entry),
thread: thread?(entry) ? find_status(thread(entry).first) : nil
)
mentions_from_xml(status, entry)
hashtags_from_xml(status, entry)
media_from_xml(status, entry)
[status, true]
end
def find_or_create_conversation(xml)
uri = xml.at_xpath('./ostatus:conversation', ostatus: TagManager::OS_XMLNS)&.attribute('ref')&.content
return if uri.nil?
if TagManager.instance.local_id?(uri)
local_id = TagManager.instance.unique_tag_to_local_id(uri, 'Conversation')
return Conversation.find_by(id: local_id)
end end
Conversation.find_by(uri: uri) || Conversation.create!(uri: uri) def content
end @xml.at_xpath('./xmlns:content', xmlns: TagManager::XMLNS).content
def find_status(uri)
if TagManager.instance.local_id?(uri)
local_id = TagManager.instance.unique_tag_to_local_id(uri, 'Status')
return Status.find_by(id: local_id)
end end
Status.find_by(uri: uri) def content_language
end @xml.at_xpath('./xmlns:content', xmlns: TagManager::XMLNS)['xml:lang']&.presence || 'en'
def mentions_from_xml(parent, xml)
processed_account_ids = []
xml.xpath('./xmlns:link[@rel="mentioned"]', xmlns: TagManager::XMLNS).each do |link|
next if [TagManager::TYPES[:group], TagManager::TYPES[:collection]].include? link['ostatus:object-type']
mentioned_account = account_from_href(link['href'])
next if mentioned_account.nil? || processed_account_ids.include?(mentioned_account.id)
mentioned_account.mentions.where(status: parent).first_or_create(status: parent)
# So we can skip duplicate mentions
processed_account_ids << mentioned_account.id
end end
end
def account_from_href(href) def content_warning
url = Addressable::URI.parse(href).normalize @xml.at_xpath('./xmlns:summary', xmlns: TagManager::XMLNS)&.content || ''
if TagManager.instance.web_domain?(url.host)
Account.find_local(url.path.gsub('/users/', ''))
else
Account.where(uri: href).or(Account.where(url: href)).first || FetchRemoteAccountService.new.call(href)
end end
end
def hashtags_from_xml(parent, xml) def visibility_scope
tags = xml.xpath('./xmlns:category', xmlns: TagManager::XMLNS).map { |category| category['term'] }.select(&:present?) @xml.at_xpath('./mastodon:scope', mastodon: TagManager::MTDN_XMLNS)&.content&.to_sym || :public
ProcessHashtagsService.new.call(parent, tags) end
end
def media_from_xml(parent, xml) def published
do_not_download = DomainBlock.find_by(domain: parent.account.domain)&.reject_media? @xml.at_xpath('./xmlns:published', xmlns: TagManager::XMLNS).content
end
xml.xpath('./xmlns:link[@rel="enclosure"]', xmlns: TagManager::XMLNS).each do |link| def thread?
next unless link['href'] !@xml.at_xpath('./thr:in-reply-to', thr: TagManager::THR_XMLNS).nil?
end
media = MediaAttachment.where(status: parent, remote_url: link['href']).first_or_initialize(account: parent.account, status: parent, remote_url: link['href']) def thread
parsed_url = Addressable::URI.parse(link['href']).normalize thr = @xml.at_xpath('./thr:in-reply-to', thr: TagManager::THR_XMLNS)
[thr['ref'], thr['href']]
end
next if !%w(http https).include?(parsed_url.scheme) || parsed_url.host.empty? private
media.save def converstation_to_persistent
uri = @xml.at_xpath('./ostatus:conversation', ostatus: TagManager::OS_XMLNS)&.attribute('ref')&.content
return if uri.nil?
next if do_not_download if TagManager.instance.local_id?(uri)
local_id = TagManager.instance.unique_tag_to_local_id(uri, 'Conversation')
return Conversation.find_by(id: local_id)
end
begin found = Conversation.find_by(uri: uri)
media.file_remote_url = link['href'] found ? [found, false] : [Conversation.create!(uri: uri), true]
media.save! end
rescue ActiveRecord::RecordInvalid
next def save_mentions(parent)
processed_account_ids = []
@xml.xpath('./xmlns:link[@rel="mentioned"]', xmlns: TagManager::XMLNS).each do |link|
next if [TagManager::TYPES[:group], TagManager::TYPES[:collection]].include? link['ostatus:object-type']
mentioned_account = account_from_href(link['href'])
next if mentioned_account.nil? || processed_account_ids.include?(mentioned_account.id)
mentioned_account.mentions.where(status: parent).first_or_create(status: parent)
# So we can skip duplicate mentions
processed_account_ids << mentioned_account.id
end
end
def save_hashtags(parent)
tags = @xml.xpath('./xmlns:category', xmlns: TagManager::XMLNS).map { |category| category['term'] }.select(&:present?)
ProcessHashtagsService.new.call(parent, tags)
end
def save_media(parent)
do_not_download = DomainBlock.find_by(domain: parent.account.domain)&.reject_media?
@xml.xpath('./xmlns:link[@rel="enclosure"]', xmlns: TagManager::XMLNS).each do |link|
next unless link['href']
media = MediaAttachment.where(status: parent, remote_url: link['href']).first_or_initialize(account: parent.account, status: parent, remote_url: link['href'])
parsed_url = Addressable::URI.parse(link['href']).normalize
next if !%w(http https).include?(parsed_url.scheme) || parsed_url.host.empty?
media.save
next if do_not_download
begin
media.file_remote_url = link['href']
media.save!
rescue ActiveRecord::RecordInvalid
next
end
end
end
def account_from_href(href)
url = Addressable::URI.parse(href).normalize
if TagManager.instance.web_domain?(url.host)
Account.find_local(url.path.gsub('/users/', ''))
else
Account.where(uri: href).or(Account.where(url: href)).first || FetchRemoteAccountService.new.call(href)
end end
end end
end end
def id(xml = @xml) class ShareActivity < CreationActivity
xml.at_xpath('./xmlns:id', xmlns: TagManager::XMLNS).content def perform
status, just_created = super
NotifyService.new.call(reblog.account, status) if reblog&.account&.local? && just_created
status
end
def object
@xml.at_xpath('.//activity:object', activity: TagManager::AS_XMLNS)
end
private
def status
reblog && super
end
def reblog
return @reblog if defined? @reblog
original_status = RemoteActivity.new(object).perform
@reblog = original_status.reblog? ? original_status.reblog : original_status
end
end end
def verb(xml = @xml) class PostActivity < CreationActivity
raw = xml.at_xpath('./activity:verb', activity: TagManager::AS_XMLNS).content def perform
TagManager::VERBS.key(raw) status, just_created = super
rescue
:post if just_created
status.mentions.includes(:account).each do |mention|
mentioned_account = mention.account
next unless mentioned_account.local?
NotifyService.new.call(mentioned_account, mention)
end
end
status
end
private
def reblog
nil
end
end end
def type(xml = @xml) class DeletionActivity < Activity
raw = xml.at_xpath('./activity:object-type', activity: TagManager::AS_XMLNS).content def perform
TagManager::TYPES.key(raw) Rails.logger.debug "Deleting remote status #{id}"
rescue status = Status.find_by(uri: id, account: @account)
:activity
if status.nil?
redis.setex("delete_upon_arrival:#{@account.id}:#{id}", 6 * 3_600, id)
else
RemoveStatusService.new.call(status)
end
end
end end
def url(xml = @xml) class RemoteActivity < Activity
link = xml.at_xpath('./xmlns:link[@rel="alternate"]', xmlns: TagManager::XMLNS) def perform
link.nil? ? nil : link['href'] find_status(id) || FetchRemoteStatusService.new.call(url)
end end
def content(xml = @xml)
xml.at_xpath('./xmlns:content', xmlns: TagManager::XMLNS).content
end
def content_language(xml = @xml)
xml.at_xpath('./xmlns:content', xmlns: TagManager::XMLNS)['xml:lang']&.presence || 'en'
end
def content_warning(xml = @xml)
xml.at_xpath('./xmlns:summary', xmlns: TagManager::XMLNS)&.content || ''
end
def visibility_scope(xml = @xml)
xml.at_xpath('./mastodon:scope', mastodon: TagManager::MTDN_XMLNS)&.content&.to_sym || :public
end
def published(xml = @xml)
xml.at_xpath('./xmlns:published', xmlns: TagManager::XMLNS).content
end
def thread?(xml = @xml)
!xml.at_xpath('./thr:in-reply-to', thr: TagManager::THR_XMLNS).nil?
end
def thread(xml = @xml)
thr = xml.at_xpath('./thr:in-reply-to', thr: TagManager::THR_XMLNS)
[thr['ref'], thr['href']]
end
def account?(xml = @xml)
!xml.at_xpath('./xmlns:author', xmlns: TagManager::XMLNS).nil?
end
def redis
Redis.current
end end
end end
end end