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
		
			
				
	
	
		
			27 lines
		
	
	
	
		
			745 B
		
	
	
	
		
			Ruby
		
	
	
	
	
	
			
		
		
	
	
			27 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
 |