Rename cache key to exclude_account_ids_for:id, adjust formatting
This commit is contained in:
		
							parent
							
								
									2fd0473aa1
								
							
						
					
					
						commit
						f5cd0b4956
					
				| @ -8,11 +8,13 @@ class Block < ApplicationRecord | ||||
| 
 | ||||
|   validates :account_id, uniqueness: { scope: :target_account_id } | ||||
| 
 | ||||
|   after_create :remove_blocking_cache | ||||
|   after_create  :remove_blocking_cache | ||||
|   after_destroy :remove_blocking_cache | ||||
| 
 | ||||
|   private | ||||
| 
 | ||||
|   def remove_blocking_cache | ||||
|     Rails.cache.delete("blocked_account_ids:#{account_id}") | ||||
|     Rails.cache.delete("blocked_account_ids:#{target_account_id}") | ||||
|     Rails.cache.delete("exclude_account_ids_for:#{account_id}") | ||||
|     Rails.cache.delete("exclude_account_ids_for:#{target_account_id}") | ||||
|   end | ||||
| end | ||||
|  | ||||
| @ -8,10 +8,12 @@ class Mute < ApplicationRecord | ||||
| 
 | ||||
|   validates :account_id, uniqueness: { scope: :target_account_id } | ||||
| 
 | ||||
|   after_create :remove_blocking_cache | ||||
|   after_create  :remove_blocking_cache | ||||
|   after_destroy :remove_blocking_cache | ||||
| 
 | ||||
|   private | ||||
| 
 | ||||
|   def remove_blocking_cache | ||||
|     Rails.cache.delete("blocked_account_ids:#{account_id}") | ||||
|     Rails.cache.delete("exclude_account_ids_for:#{account_id}") | ||||
|   end | ||||
| end | ||||
|  | ||||
| @ -183,9 +183,7 @@ class Status < ApplicationRecord | ||||
|     private | ||||
| 
 | ||||
|     def filter_timeline(query, account) | ||||
|       blocked = Rails.cache.fetch("blocked_account_ids:#{account.id}") do | ||||
|         Block.where(account: account).pluck(:target_account_id) + Block.where(target_account: account).pluck(:account_id) + Mute.where(account: account).pluck(:target_account_id) | ||||
|       end | ||||
|       blocked = Rails.cache.fetch("exclude_account_ids_for:#{account.id}") { Block.where(account: account).pluck(:target_account_id) + Block.where(target_account: account).pluck(:account_id) + Mute.where(account: account).pluck(:target_account_id) } | ||||
|       query   = query.where('statuses.account_id NOT IN (?)', blocked) unless blocked.empty?  # Only give us statuses from people we haven't blocked, or muted, or that have blocked us | ||||
|       query   = query.where('accounts.silenced = TRUE') if account.silenced?                  # and if we're hellbanned, only people who are also hellbanned | ||||
|       query | ||||
|  | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user