* Fix being able to follow oneself by moving to an account that was following the old one * Add specs * Add spec to catch MoveWorker issue with local followers following both accounts * Fix move worker breaking when a local account follows both source and target accounts * Fix migration from remote to local account not sending Undo Follow * Fix show_reblogs not being preserved for moved account's followers
		
			
				
	
	
		
			38 lines
		
	
	
		
			1.1 KiB
		
	
	
	
		
			Ruby
		
	
	
	
	
	
			
		
		
	
	
			38 lines
		
	
	
		
			1.1 KiB
		
	
	
	
		
			Ruby
		
	
	
	
	
	
| # frozen_string_literal: true
 | |
| 
 | |
| class MoveWorker
 | |
|   include Sidekiq::Worker
 | |
| 
 | |
|   def perform(source_account_id, target_account_id)
 | |
|     @source_account = Account.find(source_account_id)
 | |
|     @target_account = Account.find(target_account_id)
 | |
| 
 | |
|     if @target_account.local? && @source_account.local?
 | |
|       rewrite_follows!
 | |
|     else
 | |
|       queue_follow_unfollows!
 | |
|     end
 | |
|   rescue ActiveRecord::RecordNotFound
 | |
|     true
 | |
|   end
 | |
| 
 | |
|   private
 | |
| 
 | |
|   def rewrite_follows!
 | |
|     @source_account.passive_relationships
 | |
|                    .where(account: Account.local)
 | |
|                    .where.not(account: @target_account.followers.local)
 | |
|                    .where.not(account_id: @target_account.id)
 | |
|                    .in_batches
 | |
|                    .update_all(target_account_id: @target_account.id)
 | |
|   end
 | |
| 
 | |
|   def queue_follow_unfollows!
 | |
|     bypass_locked = @target_account.local?
 | |
| 
 | |
|     @source_account.followers.local.select(:id).find_in_batches do |accounts|
 | |
|       UnfollowFollowWorker.push_bulk(accounts.map(&:id)) { |follower_id| [follower_id, @source_account.id, @target_account.id, bypass_locked] }
 | |
|     end
 | |
|   end
 | |
| end
 |