Summary: | Cache undef in L1 only | ||
---|---|---|---|
Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
Component: | Architecture, internals, and plumbing | Assignee: | Marcel de Rooy <m.de.rooy> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | julian.maurice, jweaver |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | 16044 | ||
Bug Blocks: | |||
Attachments: |
Bug 16412: Cache undef in L1 only
Bug 16412: Make sure Koha::Cache->set_in_cache will not warn anymore Bug 16412: Cache undef in L1 only [PASSED QA] Bug 16412: Make sure Koha::Cache->set_in_cache will not warn anymore [PASSED QA] Bug 16412: Cache undef in L1 only |
Description
Marcel de Rooy
2016-05-02 10:30:05 UTC
Created attachment 51086 [details] [review] Bug 16412: Cache undef in L1 only No need to cache undef in L2. Prevent numerous warnings like: Use of uninitialized value in subroutine entry at /usr/share/koha/masterclone/Koha/Cache.pm line 284 Test plan: Run t/Cache.t Run t/db_dependent/sysprefs.t Created attachment 51089 [details] [review] Bug 16412: Make sure Koha::Cache->set_in_cache will not warn anymore Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 51090 [details] [review] Bug 16412: Cache undef in L1 only No need to cache undef in L2. Prevent numerous warnings like: Use of uninitialized value in subroutine entry at /usr/share/koha/masterclone/Koha/Cache.pm line 284 Test plan: Run t/Cache.t Run t/db_dependent/sysprefs.t Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Amended patch: remove trailing space char and replace 'you' with 'we' to match existing wordings. Thanks, Jonathan. Just to clarify for QA: If you use memcached, this patch will remove numerous warnings like: Use of uninitialized value in subroutine entry at /usr/share/koha/masterclone/Koha/Cache.pm line 284 Created attachment 51310 [details] [review] [PASSED QA] Bug 16412: Make sure Koha::Cache->set_in_cache will not warn anymore Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Created attachment 51311 [details] [review] [PASSED QA] Bug 16412: Cache undef in L1 only No need to cache undef in L2. Prevent numerous warnings like: Use of uninitialized value in subroutine entry at /usr/share/koha/masterclone/Koha/Cache.pm line 284 Test plan: Run t/Cache.t Run t/db_dependent/sysprefs.t Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Amended patch: remove trailing space char and replace 'you' with 'we' to match existing wordings. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed for inclusion in 16.05, to be released in May 2015. Thanks for your work! May 2016, rather... Not needed in stable versions, status changed to RESOLVED (In reply to Julian Maurice from comment #9) > Not needed in stable versions, status changed to RESOLVED Now needed because bug 16044 was pushed to 3.22.x Patches pushed to 3.22.x, will be in 3.22.8 |