[Bug 30823] New: Recalls should use 'FILL' in action logs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Bug ID: 30823 Summary: Recalls should use 'FILL' in action logs 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: tomascohen@gmail.com QA Contact: testopia@bugs.koha-community.org Recalls are too close to holds but the action_logs action for filling a 'recall' has been coded as FULFILL. The same thing is coded FILL for holds, they should be named the same. -- 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=30823 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |rel_22_05_candidate Depends on| |19532 --- Comment #1 from Tomás Cohen Arazi <tomascohen@gmail.com> --- I think this should be fixed before the release, if agreed. This way we won't need to deal with an atomicupdate for action logs. Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19532 [Bug 19532] Recalls for Koha -- 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=30823 --- Comment #2 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Do they log as their own module? If so, using the same would make sense, it would also clean up the display of the log viewer to have less options. -- 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=30823 --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- To be a little clearer: different module, same action :) -- 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=30823 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Blocks| |29734 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29734 [Bug 29734] [OMNIBUS] Recalls for Koha -- 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=30823 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- 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=30823 --- Comment #4 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 135741 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135741&action=edit Bug 30823: Filling recalls uses 'FILL' action in action logs This enhancement changes recall fulfillment actions to log with the FILL action. It will also update existing recalls FULFILL actions in the database to use the FILL action. To test: 1) Enable the UseRecalls system preference and set up your recalls-related circulation rules. Confirm RecallsLog is enabled. 2) Check out an item to Patron B. 3) Log into the OPAC as Patron A and search for the item. 4) Place a recall on that item. 5) Go back to the staff client and check the item in. Confirm the recall as waiting for Patron A. 6) Check out the item for Patron A to fill the recall. 7) Go to Tools -> Log Viewer. Confirm there is a FULFILL action. Choose the following search params to browse system logs: - modules: recalls - actions: fill 8) Submit the search and confirm the recall DOES NOT show. 9) Apply the patch, update database, restart services. 10) Refresh the log viewer and repeat step 7. Submit the search and confirm the recall DOES show. Confirm there is no longer a FULFILL action as both holds and recalls will use FILL. 11) Check in the item. 12) Repeat steps 2-6. We are ensuring that future recalls are logged using the FILL action. 13) Repeat step 7. Confirm all test recalls are now showing in search results. Sponsored-by: Catalyst IT -- 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=30823 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Change sponsored?|--- |Sponsored Text to go in the| |This enhancement changes release notes| |recall fulfillment actions | |to log with the FILL | |action, same as holds. It | |will also update existing | |recalls FULFILL actions in | |the database to use the | |FILL action. CC| |aleisha@catalyst.net.nz Assignee|koha-bugs@lists.koha-commun |aleisha@catalyst.net.nz |ity.org | -- 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=30823 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #5 from David Nind <david@davidnind.com> --- I tried to test, but I'm getting lots of errors with recalls BEFORE applying the patch. I've logged these as bug 30924 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #135741|0 |1 is obsolete| | --- Comment #6 from David Nind <david@davidnind.com> --- Created attachment 136212 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136212&action=edit Bug 30823: Filling recalls uses 'FILL' action in action logs This enhancement changes recall fulfillment actions to log with the FILL action. It will also update existing recalls FULFILL actions in the database to use the FILL action. To test: 1) Enable the UseRecalls system preference and set up your recalls-related circulation rules. Confirm RecallsLog is enabled. 2) Check out an item to Patron B. 3) Log into the OPAC as Patron A and search for the item. 4) Place a recall on that item. 5) Go back to the staff client and check the item in. Confirm the recall as waiting for Patron A. 6) Check out the item for Patron A to fill the recall. 7) Go to Tools -> Log Viewer. Confirm there is a FULFILL action. Choose the following search params to browse system logs: - modules: recalls - actions: fill 8) Submit the search and confirm the recall DOES NOT show. 9) Apply the patch, update database, restart services. 10) Refresh the log viewer and repeat step 7. Submit the search and confirm the recall DOES show. Confirm there is no longer a FULFILL action as both holds and recalls will use FILL. 11) Check in the item. 12) Repeat steps 2-6. We are ensuring that future recalls are logged using the FILL action. 13) Repeat step 7. Confirm all test recalls are now showing in search results. Sponsored-by: Catalyst IT Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Bug 30823 depends on bug 19532, which changed state. Bug 19532 Summary: Recalls for Koha https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19532 What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |Needs documenting Resolution|FIXED |--- -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #136212|0 |1 is obsolete| | --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 137037 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=137037&action=edit Bug 30823: Filling recalls uses 'FILL' action in action logs This enhancement changes recall fulfillment actions to log with the FILL action. It will also update existing recalls FULFILL actions in the database to use the FILL action. To test: 1) Enable the UseRecalls system preference and set up your recalls-related circulation rules. Confirm RecallsLog is enabled. 2) Check out an item to Patron B. 3) Log into the OPAC as Patron A and search for the item. 4) Place a recall on that item. 5) Go back to the staff client and check the item in. Confirm the recall as waiting for Patron A. 6) Check out the item for Patron A to fill the recall. 7) Go to Tools -> Log Viewer. Confirm there is a FULFILL action. Choose the following search params to browse system logs: - modules: recalls - actions: fill 8) Submit the search and confirm the recall DOES NOT show. 9) Apply the patch, update database, restart services. 10) Refresh the log viewer and repeat step 7. Submit the search and confirm the recall DOES show. Confirm there is no longer a FULFILL action as both holds and recalls will use FILL. 11) Check in the item. 12) Repeat steps 2-6. We are ensuring that future recalls are logged using the FILL action. 13) Repeat step 7. Confirm all test recalls are now showing in search results. Sponsored-by: Catalyst IT Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to master Version(s)| |22.11.00 released in| | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 --- Comment #8 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Pushed to master for 22.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Pushed to master |Pushed to stable Version(s)|22.11.00 |22.11.00, 22.05.04 released in| | CC| |lucas@bywatersolutions.com --- Comment #9 from Lucas Gass <lucas@bywatersolutions.com> --- Backported to 22.05.x for 22.05.04 -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|Pushed to stable |RESOLVED CC| |arthur.suzuki@biblibre.com --- Comment #10 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- Depends on recalls. won't apply to 21.11.x -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Lucas Gass <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords|rel_22_05_candidate | -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Caroline Cyr La Rose <caroline.cyr-la-rose@inlibro.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Documentation| |https://gitlab.com/koha-com submission| |munity/koha-manual/-/merge_ | |requests/616 Documentation| |Caroline Cyr La Rose contact| | CC| |caroline.cyr-la-rose@inlibr | |o.com -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30823 Bug 30823 depends on bug 19532, which changed state. Bug 19532 Summary: Recalls for Koha https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19532 What |Removed |Added ---------------------------------------------------------------------------- Status|Needs documenting |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org