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

Paul Poulain <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|In Discussion               |Patch doesn't apply

--- Comment #27 from Paul Poulain <[email protected]> 2012-02-21 
12:44:08 UTC ---
Following discussion & decision on koha-devel, the current patch won't be
applied : we have decided to set caching at apache configuration level, bug
6193 has been applied and provide this feature/level of caching.

So the "usecache" syspref is now irrelevent (we use cache BEFORE reading
syspref)
Using Koha/Cache is probably a good idea though, but the patch must be
rewritten completly imo.

For now, all caching is made with the following code:

eval {
    if (C4::Context->ismemcached) {
        require Memoize::Memcached;
        import Memoize::Memcached qw(memoize_memcached);
        memoize_memcached('getTranslatedLanguages', memcached =>
C4::Context->memcached);
}

->ismemcached answer 1 if memcache is ON
C4::Context->memcached returns the handle of the memcache server

Some ideas behing this patch are great and should be reintroduced:
* use a central/unique Caching place = will let us change the caching system if
we want, it's something we can't do with current system
* the cache flushing is something we must add, definetly.

So you're more than welcomed to rewrite your patch !

-- 
Configure bugmail: 
http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- 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/

Reply via email to