ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

add request cache around feature_flag_account_ids 

gets called multiple times per request

Change-Id: Id630760d1b34724d3d089e7dd48ad1c8268078b8
Reviewed-on: https://gerrit.instructure.com/82632
Tested-by: Jenkins
Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
Product-Review: James Williams  <jamesw@instructure.com>
QA-Review: James Williams  <jamesw@instructure.com>
This commit is contained in:
James Williams 2016-06-15 12:50:03 -06:00
parent 4e729f8fe9
commit 304667fa85
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
lib/feature_flags.rb
View File

@ -77,14 +77,16 @@ module FeatureFlags
# each account that needs to be searched for a feature flag, in priority order,
# starting with site admin
def feature_flag_account_ids
Rails.cache.fetch(['feature_flag_account_ids', self].cache_key) do
if is_a?(User)
chain = [Account.site_admin]
else
chain = account_chain(include_site_admin: true)
chain.shift if is_a?(Account)
RequestCache.cache('feature_flag_account_ids', self) do
Rails.cache.fetch(['feature_flag_account_ids', self].cache_key) do
if is_a?(User)
chain = [Account.site_admin]
else
chain = account_chain(include_site_admin: true)
chain.shift if is_a?(Account)
end
chain.reverse.map(&:global_id)
end
chain.reverse.map(&:global_id)
end
end