Skip to content

Commit ac53186

Browse files
Christian PrescottDan Minkevitch
authored andcommitted
Fix ConditionalRelease::Service rules_cache caching nil
Change-Id: Ib656a703867af79554e459a04bf4f0cba75f7fba Reviewed-on: https://gerrit.instructure.com/90027 Tested-by: Jenkins Reviewed-by: Dan Minkevitch <dan@instructure.com> Product-Review: Dan Minkevitch <dan@instructure.com> QA-Review: Dan Minkevitch <dan@instructure.com>
1 parent e31f284 commit ac53186

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

app/models/conditional_release/service.rb

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -243,10 +243,8 @@ def rules_data(context, student, content_tags = [], session = {})
243243
end
244244
end
245245

246-
def rules_cache(context, student, force: false)
247-
Rails.cache.fetch(rules_cache_key(context, student), force: force) do
248-
yield if block_given?
249-
end
246+
def rules_cache(context, student, force: false, &block)
247+
Rails.cache.fetch(rules_cache_key(context, student), force: force, &block)
250248
end
251249

252250
def newer_than_cache?(items, cache)

0 commit comments

Comments
 (0)