http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7387
--- Comment #8 from Chris Cormack <[email protected]> --- (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. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
