Summary: | Newly created report group is not selected after saving an SQL report | ||
---|---|---|---|
Product: | Koha | Reporter: | Christopher Brannon <cbrannon> |
Component: | Reports | Assignee: | Nick Clemens (kidclamp) <nick> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | fridolin.somers, lucas, martin.renvoize, nick |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18495 | ||
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: | 19957 | ||
Bug Blocks: | |||
Attachments: |
Bug 22278: Clear cache when creating new report groups or subgroups
Bug 22278: Clear cache when creating new report groups or subgroups Bug 22278: Clear cache when creating new report groups or subgroups |
Description
Christopher Brannon
2019-02-05 18:53:21 UTC
Correction: the group is created, but not linked when you save the report. You have to refresh to even see the group, then select and save the report again. I think the issue might be with the new "save and continue" feature. 0) Start a new report 1) Pick create for the group 2) Enter code and value 3) Save 4) You remain on the page and value and code are now empty (ok) 5) BUT: The pull down for the group remains "None" instead of reflecting the new group 6) Save again - group was created, but lost on the report Created attachment 84813 [details] [review] Bug 22278: Clear cache when creating new report groups or subgroups When a user adds a new group we create it, then fetch the groups from the cache. However, we aren't clearing the cache on creation. This patch fixes that To test: 0) Start a new report 1) Pick create for the group 2) Enter code and value 3) Save 4) You remain on the page and value and code are now empty (ok) 5) BUT: The pull down for the group remains "None" instead of reflecting the new group 6) Open a new tab and browse to reports - note the group exists and the report is in it 7) On original tab save report again, the group was not selected so is lost 8) Apply patch 9) Retest with a new report 10) Group is created, saved, and selected upon save 11) Subsequent edits include the new group 12) Test with subgroups as well Created attachment 84855 [details] [review] Bug 22278: Clear cache when creating new report groups or subgroups When a user adds a new group we create it, then fetch the groups from the cache. However, we aren't clearing the cache on creation. This patch fixes that To test: 0) Start a new report 1) Pick create for the group 2) Enter code and value 3) Save 4) You remain on the page and value and code are now empty (ok) 5) BUT: The pull down for the group remains "None" instead of reflecting the new group 6) Open a new tab and browse to reports - note the group exists and the report is in it 7) On original tab save report again, the group was not selected so is lost 8) Apply patch 9) Retest with a new report 10) Group is created, saved, and selected upon save 11) Subsequent edits include the new group 12) Test with subgroups as well Signed-off-by: Liz Rea <wizzyrea@gmail.com> Created attachment 84858 [details] [review] Bug 22278: Clear cache when creating new report groups or subgroups When a user adds a new group we create it, then fetch the groups from the cache. However, we aren't clearing the cache on creation. This patch fixes that To test: 0) Start a new report 1) Pick create for the group 2) Enter code and value 3) Save 4) You remain on the page and value and code are now empty (ok) 5) BUT: The pull down for the group remains "None" instead of reflecting the new group 6) Open a new tab and browse to reports - note the group exists and the report is in it 7) On original tab save report again, the group was not selected so is lost 8) Apply patch 9) Retest with a new report 10) Group is created, saved, and selected upon save 11) Subsequent edits include the new group 12) Test with subgroups as well Signed-off-by: Liz Rea <wizzyrea@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Sorry, Nick! Awesome work all! Pushed to master for 19.05 Pushed to 18.11.x for 18.11.03 backported to 18.05.x for 18.05.09 Depends on Bug 19957 not in 17.11.x |