https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16229

            Bug ID: 16229
           Summary: Koha::Cache should be on the safer side
 Change sponsored?: ---
           Product: Koha
           Version: master
          Hardware: All
                OS: All
            Status: ASSIGNED
          Severity: major
          Priority: P5 - low
         Component: Architecture, internals, and plumbing
          Assignee: jonathan.dru...@bugs.koha-community.org
          Reporter: jonathan.dru...@bugs.koha-community.org
        QA Contact: testo...@bugs.koha-community.org
        Depends on: 16044
            Blocks: 16221

Theoretically our Koha::Cache is not as safe as possible, the set_in_cache and
get_from_cache subroutines should always clone the structure to avoid
uncontrollable behaviors.

There are, at least, 2 issues to cover:
- get_from_cache does not deep copy on first call
- set_in_cache does not deep copy

Note that bug 16221 plans to replace Clone::clone with Storable::dclone, it
should be built on top of this patch.


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16044
[Bug 16044] Define a L1 cache for all objects set in cache
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16221
[Bug 16221] Use Storable::dclone() instead of Clone::clone() for L1 cache
deep-copying mode
-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
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