]> cat aescling's git repositories - mastodon.git/commit
Fix UserTrackingConcern firing on every request, optimize some queries (#5368)
authorEugen Rochko <eugen@zeonfederated.com>
Fri, 13 Oct 2017 14:44:29 +0000 (16:44 +0200)
committerGitHub <noreply@github.com>
Fri, 13 Oct 2017 14:44:29 +0000 (16:44 +0200)
commitb8db386e056fe236ed3f41e563a824e30733c4ce
tree1753b0b08768582120309e788c30590b51e998d9
parent48f7a587999740709f234e1ee004994de0059ab3
Fix UserTrackingConcern firing on every request, optimize some queries (#5368)

- For some reason, :if option on before_action did not work. It got
  executed every time, returned false, and the action run anyway,
  which led to the current_sign_in_at and sign_in_count being
  updated on every request
- Return "do not filter" early in FeedManager#filter_from_home? if
  the status is authored by receiver. Usually this method is not
  called for own statuses at all, but it is called when Feed#get
  uses the database
- Return early if #reload_stale_associations! has nothing to load
  to save a database query with WHERE 1=0
app/controllers/concerns/user_tracking_concern.rb
app/lib/feed_manager.rb
app/models/notification.rb
app/models/status.rb