UX-wise, people expect that saving the profile will re-check links even without changing fields content. Bug-wise, `@account` was undefined. Regression from #8703
		
			
				
	
	
		
			28 lines
		
	
	
		
			745 B
		
	
	
	
		
			Ruby
		
	
	
	
	
	
			
		
		
	
	
			28 lines
		
	
	
		
			745 B
		
	
	
	
		
			Ruby
		
	
	
	
	
	
# frozen_string_literal: true
 | 
						|
 | 
						|
class UpdateAccountService < BaseService
 | 
						|
  def call(account, params, raise_error: false)
 | 
						|
    was_locked    = account.locked
 | 
						|
    update_method = raise_error ? :update! : :update
 | 
						|
 | 
						|
    account.send(update_method, params).tap do |ret|
 | 
						|
      next unless ret
 | 
						|
 | 
						|
      authorize_all_follow_requests(account) if was_locked && !account.locked
 | 
						|
      check_links(account)
 | 
						|
    end
 | 
						|
  end
 | 
						|
 | 
						|
  private
 | 
						|
 | 
						|
  def authorize_all_follow_requests(account)
 | 
						|
    AuthorizeFollowWorker.push_bulk(FollowRequest.where(target_account: account).select(:account_id, :target_account_id)) do |req|
 | 
						|
      [req.account_id, req.target_account_id]
 | 
						|
    end
 | 
						|
  end
 | 
						|
 | 
						|
  def check_links(account)
 | 
						|
    VerifyAccountLinksWorker.perform_async(account.id)
 | 
						|
  end
 | 
						|
end
 |