[Bug 26393] New: Add cache on Koha::CirculationRules::get_effective_rule
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Bug ID: 26393 Summary: Add cache on Koha::CirculationRules::get_effective_rule Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Architecture, internals, and plumbing Assignee: koha-bugs@lists.koha-community.org Reporter: fridolin.somers@biblibre.com QA Contact: testopia@bugs.koha-community.org For performance of several circulation pages we should cache the result of method Koha::CirculationRules::get_effective_rule. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|koha-bugs@lists.koha-commun |fridolin.somers@biblibre.co |ity.org |m -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 --- Comment #1 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Created attachment 109670 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109670&action=edit Bug 26393: Add cache on Koha::CirculationRules::get_effective_rule For performance of several circulation pages we should cache the result of method Koha::CirculationRules::get_effective_rule. Adds a constant CACHE_NO_CIRCULATION_RULE_VALUE to store a special value when method must return undef. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 --- Comment #2 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- You may look at patch, test plan is comming -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |arthur.suzuki@biblibre.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |julian.maurice@biblibre.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #109670|0 |1 is obsolete| | --- Comment #3 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Created attachment 109827 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109827&action=edit Bug 26393: Add cache on Koha::CirculationRules::get_effective_rule For performance of several circulation pages we should cache the result of method Koha::CirculationRules::get_effective_rule. Adds a constant CACHE_NO_CIRCULATION_RULE_VALUE to store a special value when method must return undef. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #109827|0 |1 is obsolete| | --- Comment #4 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Created attachment 109839 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109839&action=edit Bug 26393: Add cache on Koha::CirculationRules::get_effective_rule For performance of several circulation pages we should cache the result of method Koha::CirculationRules::get_effective_rule. Adds a constant CACHE_NO_CIRCULATION_RULE_VALUE to store a special value when method must return undef. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #109839|0 |1 is obsolete| | --- Comment #5 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Created attachment 109840 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109840&action=edit Bug 26393: Add cache on Koha::CirculationRules::get_effective_rule For performance of several circulation pages we should cache the result of method Koha::CirculationRules::get_effective_rule. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #109840|0 |1 is obsolete| | --- Comment #6 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Created attachment 109846 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=109846&action=edit Bug 26393: Add cache on Koha::CirculationRules::get_effective_rule For performance of several circulation pages we should cache the result of method Koha::CirculationRules::get_effective_rule. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |In Discussion --- Comment #7 from Fridolin SOMERS <fridolin.somers@biblibre.com> --- Arf my tests dont prove a performance difference. Foud Bug 26424 is a real performance improvement. I set to in discussion in case someone is interested. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin SOMERS <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=26424 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26393 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|In Discussion |RESOLVED --- Comment #8 from Fridolin Somers <fridolin.somers@biblibre.com> --- *** This bug has been marked as a duplicate of bug 29623 *** -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org