https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16770
Bug ID: 16770
Summary: Remove wrong uses of Memoize::Memcached
Change sponsored?: ---
Product: Koha
Version: master
Hardware: All
OS: All
Status: ASSIGNED
Severity: enhancement
Priority: P5 - low
Component: Architecture, internals, and plumbing
Assignee: [email protected]
Reporter: [email protected]
QA Contact: [email protected]
>From C4/languages.pm:
32 eval {
33 if (C4::Context->ismemcached) {
34 require Memoize::Memcached;
35 import Memoize::Memcached qw(memoize_memcached);
36
37 memoize_memcached('getTranslatedLanguages', memcached =>
C4::Context->memcached);
38 memoize_memcached('getFrameworkLanguages' , memcached =>
C4::Context->memcached);
39 memoize_memcached('getAllLanguages', memcached =>
C4::Context->memcached);
40 }
41 };
Looking at this code, you might think these subroutines are cached, but
actually they are not.
The eval surrounding the code hides a bug, if you remove it, you will get:
Invalid memcached argument (expected a hash)
--
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/
