refresh() method does not honors should_cache_fn function

Issue #107 new
Anonymous created an issue

CacheRegion.cache_on_arguments() has a parameter should_cache_fn which is used to control if content should be cached or not. the .refresh() method on the decorated function ignores it and goes directly to the backend.set()

Comments (2)

  1. Michael Bayer repo owner

    refresh probably should consult should_cache_fn (whereas set() should not?) , however I'm not comfortable w/ backwards incompatible behavior here. I'd rather clarify documentation and add .conditional_refresh().

  2. Log in to comment