Use existing SeveredRelationship.about_local_account scope in more places (#29673)
				
					
				
			This commit is contained in:
		
							parent
							
								
									718ee72c80
								
							
						
					
					
						commit
						77897cd24c
					
				| @ -33,7 +33,7 @@ class SeveredRelationshipsController < ApplicationController | |||||||
| 
 | 
 | ||||||
|   def following_data |   def following_data | ||||||
|     CSV.generate(headers: ['Account address', 'Show boosts', 'Notify on new posts', 'Languages'], write_headers: true) do |csv| |     CSV.generate(headers: ['Account address', 'Show boosts', 'Notify on new posts', 'Languages'], write_headers: true) do |csv| | ||||||
|       @event.severed_relationships.active.where(local_account: current_account).includes(:remote_account).reorder(id: :desc).each do |follow| |       @event.severed_relationships.active.about_local_account(current_account).includes(:remote_account).reorder(id: :desc).each do |follow| | ||||||
|         csv << [acct(follow.target_account), follow.show_reblogs, follow.notify, follow.languages&.join(', ')] |         csv << [acct(follow.target_account), follow.show_reblogs, follow.notify, follow.languages&.join(', ')] | ||||||
|       end |       end | ||||||
|     end |     end | ||||||
| @ -41,7 +41,7 @@ class SeveredRelationshipsController < ApplicationController | |||||||
| 
 | 
 | ||||||
|   def followers_data |   def followers_data | ||||||
|     CSV.generate(headers: ['Account address'], write_headers: true) do |csv| |     CSV.generate(headers: ['Account address'], write_headers: true) do |csv| | ||||||
|       @event.severed_relationships.passive.where(local_account: current_account).includes(:remote_account).reorder(id: :desc).each do |follow| |       @event.severed_relationships.passive.about_local_account(current_account).includes(:remote_account).reorder(id: :desc).each do |follow| | ||||||
|         csv << [acct(follow.account)] |         csv << [acct(follow.account)] | ||||||
|       end |       end | ||||||
|     end |     end | ||||||
|  | |||||||
| @ -29,6 +29,6 @@ class AccountRelationshipSeveranceEvent < ApplicationRecord | |||||||
|   private |   private | ||||||
| 
 | 
 | ||||||
|   def set_relationships_count! |   def set_relationships_count! | ||||||
|     self.relationships_count = severed_relationships.where(local_account: account).count |     self.relationships_count = severed_relationships.about_local_account(account).count | ||||||
|   end |   end | ||||||
| end | end | ||||||
|  | |||||||
| @ -48,13 +48,13 @@ module Account::Merging | |||||||
|       record.update_attribute(:account_warning_id, id) |       record.update_attribute(:account_warning_id, id) | ||||||
|     end |     end | ||||||
| 
 | 
 | ||||||
|     SeveredRelationship.where(local_account_id: other_account.id).reorder(nil).find_each do |record| |     SeveredRelationship.about_local_account(other_account).reorder(nil).find_each do |record| | ||||||
|       record.update_attribute(:local_account_id, id) |       record.update_attribute(:local_account_id, id) | ||||||
|     rescue ActiveRecord::RecordNotUnique |     rescue ActiveRecord::RecordNotUnique | ||||||
|       next |       next | ||||||
|     end |     end | ||||||
| 
 | 
 | ||||||
|     SeveredRelationship.where(remote_account_id: other_account.id).reorder(nil).find_each do |record| |     SeveredRelationship.about_remote_account(other_account).reorder(nil).find_each do |record| | ||||||
|       record.update_attribute(:remote_account_id, id) |       record.update_attribute(:remote_account_id, id) | ||||||
|     rescue ActiveRecord::RecordNotUnique |     rescue ActiveRecord::RecordNotUnique | ||||||
|       next |       next | ||||||
|  | |||||||
| @ -26,6 +26,7 @@ class SeveredRelationship < ApplicationRecord | |||||||
|   } |   } | ||||||
| 
 | 
 | ||||||
|   scope :about_local_account, ->(account) { where(local_account: account) } |   scope :about_local_account, ->(account) { where(local_account: account) } | ||||||
|  |   scope :about_remote_account, ->(account) { where(remote_account: account) } | ||||||
| 
 | 
 | ||||||
|   scope :active, -> { where(direction: :active) } |   scope :active, -> { where(direction: :active) } | ||||||
|   scope :passive, -> { where(direction: :passive) } |   scope :passive, -> { where(direction: :passive) } | ||||||
|  | |||||||
| @ -21,13 +21,13 @@ | |||||||
|               %td{ rowspan: 2 }= t('severed_relationships.purged') |               %td{ rowspan: 2 }= t('severed_relationships.purged') | ||||||
|             - else |             - else | ||||||
|               %td |               %td | ||||||
|                 - count = event.severed_relationships.active.where(local_account: current_account).count |                 - count = event.severed_relationships.active.about_local_account(current_account).count | ||||||
|                 - if count.zero? |                 - if count.zero? | ||||||
|                   = t('generic.none') |                   = t('generic.none') | ||||||
|                 - else |                 - else | ||||||
|                   = table_link_to 'download', t('severed_relationships.download', count: count), following_severed_relationship_path(event, format: :csv) |                   = table_link_to 'download', t('severed_relationships.download', count: count), following_severed_relationship_path(event, format: :csv) | ||||||
|               %td |               %td | ||||||
|                 - count = event.severed_relationships.passive.where(local_account: current_account).count |                 - count = event.severed_relationships.passive.about_local_account(current_account).count | ||||||
|                 - if count.zero? |                 - if count.zero? | ||||||
|                   = t('generic.none') |                   = t('generic.none') | ||||||
|                 - else |                 - else | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user