Bug 18761

Summary: AutomaticItemModificationByAge.t tests are failing
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Galen Charlton <gmcharlt>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, mtj
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Bug 18761: Fix tests for AutomaticItemModificationByAge.t - clear cache
[SIGNED OFF] Bug 18761: AutomaticItemModification

Description Jonathan Druart 2017-06-08 17:17:50 UTC
The cache must be cleared!
Comment 1 Jonathan Druart 2017-06-08 17:20:12 UTC
Created attachment 64113 [details] [review]
Bug 18761: Fix tests for AutomaticItemModificationByAge.t - clear cache

The mapping structure is cached and may be wrong if already populated.
We need to clear the cache when a framework is modified
Comment 2 Jonathan Druart 2017-06-08 17:21:02 UTC
Test plan:
populate the cache before using Koha (edit a biblio), then launch the tests.
Should fail before the patch and pass after applying it.
Comment 3 Lee Jamison 2017-06-11 05:34:55 UTC
Created attachment 64176 [details] [review]
[SIGNED OFF] Bug 18761: AutomaticItemModification

The mapping structure is cached and may be wrong if already populated.
We need to clear the cache when a framework is modified

Signed-off-by: Lee Jamison <ldjamison@marywood.edu>
Works as described in test plan and passes QA tools.
Comment 4 Jonathan Druart 2017-06-12 13:44:13 UTC
Hi Lee,
Why did you modify the commit message? :)
Comment 5 Jonathan Druart 2017-06-14 17:05:14 UTC
Skipping QA.
Comment 6 Jonathan Druart 2017-06-14 20:39:13 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 7 Fridolin Somers 2017-06-15 12:36:05 UTC
Pushed to 17.05.x, will be in 17.05.01
Comment 8 Katrin Fischer 2017-06-15 20:23:04 UTC
This patch has been pushed to 16.11.x and will be in 16.11.09.
Comment 9 Mason James 2017-07-31 11:24:44 UTC
Pushed to 16.05.x, for 16.05.15 release