]> cat aescling's git repositories - mastodon.git/commitdiff
Eliminate unnecessary queries and query clauses with none and all
authorBrad Urani <bradurani@gmail.com>
Wed, 5 Apr 2017 06:02:58 +0000 (06:02 +0000)
committerBrad Urani <bradurani@gmail.com>
Wed, 5 Apr 2017 06:02:58 +0000 (06:02 +0000)
app/models/status.rb

index daf1285720fd17bc0ef52d73630506b25337d453..6948ad77c20f4b2b6b845db8c97bf1bd93ce52ce 100644 (file)
@@ -161,9 +161,9 @@ class Status < ApplicationRecord
       return where.not(visibility: [:private, :direct]) if account.nil?
 
       if target_account.blocking?(account) # get rid of blocked peeps
-        where('1 = 0')
+        none
       elsif account.id == target_account.id # author can see own stuff
-        where('1 = 1')
+        all
       elsif account.following?(target_account) # followers can see followers-only stuff, but also things they are mentioned in
         joins('LEFT OUTER JOIN mentions ON statuses.id = mentions.status_id AND mentions.account_id = ' + account.id.to_s)
           .where('statuses.visibility != ? OR mentions.id IS NOT NULL', Status.visibilities[:direct])