Misc tidying and clean ups (#3445)

* Remove trailing whitespace in i18n mailers

* Use query methods instead of #present? on AR attributes

* Delegate Status#account_domain method

* Delegate Mention #account_username and #account_acct methods
This commit is contained in:
Matt Jankowski 2017-05-31 14:38:17 -04:00 committed by Eugen Rochko
parent 5236a62861
commit b25e42a77f
8 changed files with 18 additions and 9 deletions

View File

@ -328,7 +328,7 @@ class AtomSerializer
end end
def conversation_uri(conversation) def conversation_uri(conversation)
return conversation.uri if conversation.uri.present? return conversation.uri if conversation.uri?
TagManager.instance.unique_tag(conversation.created_at, conversation.id, 'Conversation') TagManager.instance.unique_tag(conversation.created_at, conversation.id, 'Conversation')
end end

View File

@ -17,4 +17,11 @@ class Mention < ApplicationRecord
has_one :notification, as: :activity, dependent: :destroy has_one :notification, as: :activity, dependent: :destroy
validates :account, uniqueness: { scope: :status } validates :account, uniqueness: { scope: :status }
delegate(
:username,
:acct,
to: :account,
prefix: true
)
end end

View File

@ -73,6 +73,8 @@ class Status < ApplicationRecord
cache_associated :account, :application, :media_attachments, :tags, :stream_entry, mentions: :account, reblog: [:account, :application, :stream_entry, :tags, :media_attachments, mentions: :account], thread: :account cache_associated :account, :application, :media_attachments, :tags, :stream_entry, mentions: :account, reblog: [:account, :application, :stream_entry, :tags, :media_attachments, mentions: :account], thread: :account
delegate :domain, to: :account, prefix: true
def reply? def reply?
!in_reply_to_id.nil? || attributes['reply'] !in_reply_to_id.nil? || attributes['reply']
end end
@ -288,7 +290,7 @@ class Status < ApplicationRecord
def filter_from_context?(status, account) def filter_from_context?(status, account)
should_filter = account&.blocking?(status.account_id) should_filter = account&.blocking?(status.account_id)
should_filter ||= account&.domain_blocking?(status.account.domain) should_filter ||= account&.domain_blocking?(status.account_domain)
should_filter ||= account&.muting?(status.account_id) should_filter ||= account&.muting?(status.account_id)
should_filter ||= (status.account.silenced? && !account&.following?(status.account_id)) should_filter ||= (status.account.silenced? && !account&.following?(status.account_id))
should_filter ||= !StatusPolicy.new(account, status).show? should_filter ||= !StatusPolicy.new(account, status).show?

View File

@ -34,7 +34,7 @@ class PostStatusService < BaseService
process_mentions_service.call(status) process_mentions_service.call(status)
process_hashtags_service.call(status) process_hashtags_service.call(status)
LinkCrawlWorker.perform_async(status.id) unless status.spoiler_text.present? LinkCrawlWorker.perform_async(status.id) unless status.spoiler_text?
DistributionWorker.perform_async(status.id) DistributionWorker.perform_async(status.id)
Pubsubhubbub::DistributionWorker.perform_async(status.stream_entry.id) Pubsubhubbub::DistributionWorker.perform_async(status.stream_entry.id)

View File

@ -77,7 +77,7 @@ class ProcessFeedService < BaseService
Rails.logger.debug "Queuing remote status #{status.id} (#{id}) for distribution" Rails.logger.debug "Queuing remote status #{status.id} (#{id}) for distribution"
LinkCrawlWorker.perform_async(status.id) unless status.spoiler_text.present? LinkCrawlWorker.perform_async(status.id) unless status.spoiler_text?
DistributionWorker.perform_async(status.id) DistributionWorker.perform_async(status.id)
status status

View File

@ -1,4 +1,4 @@
node(:url) { |mention| TagManager.instance.url_for(mention.account) } node(:url) { |mention| TagManager.instance.url_for(mention.account) }
node(:acct) { |mention| mention.account.acct } node(:acct) { |mention| mention.account_acct }
node(:id) { |mention| mention.account_id } node(:id) { |mention| mention.account_id }
node(:username) { |mention| mention.account.username } node(:username) { |mention| mention.account_username }