[Koha-bugs] [Bug 7387] Add Template::Toolkit plugin to allow caching of includes

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7387

--- Comment #8 from Chris Cormack <chris at bigballofwax.co.nz> ---
(In reply to comment #7)
> (In reply to comment #3)
> > QA comment: chris, there is no explanation of the positive effect of this.
> > What would be the next step if this patch is included ? could we do that
> > with all includes ? more explanation welcomed !
> 
> Chris, before pushing this patch, i'd like to understand what will be the
> next step. Should all templates be switched to:
> -[% INCLUDE 'doc-head-open.inc' %]
> +[% USE cache = Cache %]
> +[% cache.inc(
> +    'template' => 'doc-head-open.inc',
> +    'keys' => {'dochead' => 'dochead'},
> +    'ttl' => 360
> +    )
> +%]
> 
> ?
> (if that's the case, then I'll add this to my next RM newsletter & send a
> mail to koha-devel about that)
> Do you already have a list of other .inc we could cache too ?

Nope d, in fact caching that one won't win us much at all it's fast to render
anyway. But looking into caching things that require computation, would be
good. This patch is just to enable people to cache includes when they find
ones, or even better create includes and cache them.

-- 
You are receiving this mail because:
You are watching all bug changes.