https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |19532 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.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 --- Comment #2 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 139441 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=139441&action=edit Issue 1: Scheduling cron to convert recalls to reserves (cherry picked from commit 5af81d5a5adcce07adb93915bf94e76b3066d88c) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on|19532 |31391 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=19532 [Bug 19532] Recalls for Koha https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31391 [Bug 31391] Staff-side recalls -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Small patch Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the| |This patch adds a script release notes| |which can be used to | |automatically convert the | |oldest reserve on a record | |to a recall, if there are a | |specified minimum number of | |reserves on the record. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #139737|0 |1 is obsolete| | --- Comment #6 from David Nind <david@davidnind.com> --- Created attachment 145796 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=145796&action=edit Bug 31415: Script to automate converting reserves to recalls This patch adds a script which can be used to automatically convert the oldest reserve on a record to a recall, if there are a specified minimum number of reserves on the record. To test: 1) Enable UseRecalls and configure circulation rules relevant to recalls. Keep the 'on shelf recalls allowed' setting to 'any' for the sake of testing ('all' would require all items to be checked out for the reserve to be converted to a recall) 2) Check out items from multiple records (at least 2 different records) to Patron A. 3) Place multiple reserves on each record (at least 2 reserves). 4) Place multiple reserves on another record which has no items checked out. 5) We now have multiple records where the oldest reserve is eligible for recall, plus a record where the oldest reserve is NOT eligible for recall (because no items are checked out). 6) In your terminal, get into the shell and run the script. -v means verbose and will print the reserves that are converted into recalls. --min 1 means there must be more than 1 reserve on the record to be considered. sudo koha-shell INSTANCE perl misc/cronjobs/recalls/convert_reserves_to_recalls.pl -v --min 1 7) Confirm that, for the records where items are checked out and there are multiple reserves, the oldest reserve was converted into a recall 8) Confirm that, for the record where no items are checked out, the reserves all remain and not converted into a recall. 9) Confirm documentation for the script looks good perldoc misc/cronjobs/recalls/convert_reserves_to_recalls.pl Sponsored-by: Auckland University of Technology 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=31415 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |kyle@bywatersolutions.com --- Comment #7 from Kyle M Hall <kyle@bywatersolutions.com> --- Easy to fix issue, can you please change usage of the word "reserve" to "hold" for consistency with Koha standard terminology: https://wiki.koha-community.org/wiki/Terminology#R -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #145796|0 |1 is obsolete| | --- Comment #8 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 148112 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=148112&action=edit Bug 31415: Script to automate converting holds to recalls This patch adds a script which can be used to automatically convert the oldest hold on a record to a recall, if there are a specified minimum number of holds on the record. To test: 1) Enable UseRecalls and configure circulation rules relevant to recalls. Keep the 'on shelf recalls allowed' setting to 'any' for the sake of testing ('all' would require all items to be checked out for the hold to be converted to a recall) 2) Check out items from multiple records (at least 2 different records) to Patron A. 3) Place multiple holds on each record (at least 2 holds). 4) Place multiple holds on another record which has no items checked out. 5) We now have multiple records where the oldest hold is eligible for recall, plus a record where the oldest hold is NOT eligible for recall (because no items are checked out). 6) In your terminal, get into the shell and run the script. -v means verbose and will print the holds that are converted into recalls. --min 1 means there must be more than 1 hold on the record to be considered. sudo koha-shell INSTANCE perl misc/cronjobs/recalls/convert_holds_to_recalls.pl -v --min 1 7) Confirm that, for the records where items are checked out and there are multiple holds, the oldest hold was converted into a recall 8) Confirm that, for the record where no items are checked out, the holds all remain and not converted into a recall. 9) Confirm documentation for the script looks good perldoc misc/cronjobs/recalls/convert_holds_to_recalls.pl Sponsored-by: Auckland University of Technology 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=31415 Michal Denar <black23@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |black23@gmail.com -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com Status|Needs Signoff |Failed QA --- Comment #9 from David Nind <david@davidnind.com> --- I'm getting this error when running the script: perl misc/cronjobs/recalls/convert_holds_to_recalls.pl -v --min 1 DBIx::Class::Storage::DBI::_dbh_execute(): DBI Exception: DBD::mysql::st execute failed: Unknown column 'holddate' in 'order clause' at /kohadevbox/koha/Koha/Objects.pm line 394 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #148112|0 |1 is obsolete| | --- Comment #10 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 157002 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=157002&action=edit Bug 31415: Script to automate converting holds to recalls This patch adds a script which can be used to automatically convert the oldest hold on a record to a recall, if there are a specified minimum number of holds on the record. To test: 1) Enable UseRecalls and configure circulation rules relevant to recalls. Keep the 'on shelf recalls allowed' setting to 'any' for the sake of testing ('all' would require all items to be checked out for the hold to be converted to a recall) 2) Check out items from multiple records (at least 2 different records) to Patron A. 3) Place multiple holds on each record (at least 2 holds). 4) Place multiple holds on another record which has no items checked out. 5) We now have multiple records where the oldest hold is eligible for recall, plus a record where the oldest hold is NOT eligible for recall (because no items are checked out). 6) In your terminal, get into the shell and run the script. -v means verbose and will print the holds that are converted into recalls. --min 1 means there must be more than 1 hold on the record to be considered. sudo koha-shell INSTANCE perl misc/cronjobs/recalls/convert_holds_to_recalls.pl -v --min 1 7) Confirm that, for the records where items are checked out and there are multiple holds, the oldest hold was converted into a recall 8) Confirm that, for the record where no items are checked out, the holds all remain and not converted into a recall. 9) Confirm documentation for the script looks good perldoc misc/cronjobs/recalls/convert_holds_to_recalls.pl Sponsored-by: Auckland University of Technology 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=31415 --- Comment #12 from Aleisha Amohia <aleisha@catalyst.net.nz> --- Created attachment 159010 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=159010&action=edit Bug 31415: Exclude found holds and fix hold ID call -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #157002|0 |1 is obsolete| | --- Comment #13 from David Nind <david@davidnind.com> --- Created attachment 159016 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=159016&action=edit Bug 31415: Script to automate converting holds to recalls This patch adds a script which can be used to automatically convert the oldest hold on a record to a recall, if there are a specified minimum number of holds on the record. To test: 1) Enable UseRecalls and configure circulation rules relevant to recalls. Keep the 'on shelf recalls allowed' setting to 'any' for the sake of testing ('all' would require all items to be checked out for the hold to be converted to a recall) 2) Check out items from multiple records (at least 2 different records) to Patron A. 3) Place multiple holds on each record (at least 2 holds). 4) Place multiple holds on another record which has no items checked out. 5) We now have multiple records where the oldest hold is eligible for recall, plus a record where the oldest hold is NOT eligible for recall (because no items are checked out). 6) In your terminal, get into the shell and run the script. -v means verbose and will print the holds that are converted into recalls. --min 1 means there must be more than 1 hold on the record to be considered. sudo koha-shell INSTANCE perl misc/cronjobs/recalls/convert_holds_to_recalls.pl -v --min 1 7) Confirm that, for the records where items are checked out and there are multiple holds, the oldest hold was converted into a recall 8) Confirm that, for the record where no items are checked out, the holds all remain and not converted into a recall. 9) Confirm documentation for the script looks good perldoc misc/cronjobs/recalls/convert_holds_to_recalls.pl Sponsored-by: Auckland University of Technology 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=31415 --- Comment #15 from David Nind <david@davidnind.com> --- All looks good and now working as expected. The documentation looks fine to me. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Circulation| |recalls function| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|BLOCKED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31415 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Comma delimited| |Auckland University of list of Sponsors| |Technology 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=31415 --- Comment #18 from David Nind <david@davidnind.com> --- Created attachment 202579 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202579&action=edit Bug 31415: Script to automate converting holds to recalls This patch adds a script which can be used to automatically convert the oldest hold on a record to a recall, if there are a specified minimum number of holds on the record. To test: 1) Enable UseRecalls and configure circulation rules relevant to recalls. Keep the 'on shelf recalls allowed' setting to 'any' for the sake of testing ('all' would require all items to be checked out for the hold to be converted to a recall) 2) Check out items from multiple records (at least 2 different records) to Patron A. 3) Place multiple holds on each record (at least 2 holds). 4) Place multiple holds on another record which has no items checked out. 5) We now have multiple records where the oldest hold is eligible for recall, plus a record where the oldest hold is NOT eligible for recall (because no items are checked out). 6) In your terminal, get into the shell and run the script. -v means verbose and will print the holds that are converted into recalls. --min 1 means there must be more than 1 hold on the record to be considered. sudo koha-shell INSTANCE perl misc/cronjobs/recalls/convert_holds_to_recalls.pl -v --min 1 7) Confirm that, for the records where items are checked out and there are multiple holds, the oldest hold was converted into a recall 8) Confirm that, for the record where no items are checked out, the holds all remain and not converted into a recall. 9) Confirm documentation for the script looks good perldoc misc/cronjobs/recalls/convert_holds_to_recalls.pl Sponsored-by: Auckland University of Technology 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=31415 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Text to go in the|This patch adds a script |This enhancement adds a release notes|which can be used to |script |automatically convert the |(misc/cronjobs/recalls/conv |oldest hold on a record to |ert_holds_to_recalls.pl) |a recall, if there are a |which can be used to |specified minimum number of |automatically convert the |holds on the record. |oldest hold on a record to | |a recall, if there are a | |specified minimum number of | |holds on the record. | | | |Example: | |- perl | |misc/cronjobs/recalls/conve | |rt_holds_to_recalls.pl -v | |--min 1 | |- Explanation: | | . converts the first hold | |for a record to a recall | |(where it has an item that | |is checked out), where | |there is more than 1 hold | |and a recall can be placed | | . if a record has a hold, | |but no items checked out, | |then no holds are converted | |to recalls -- You are receiving this mail because: You are watching all bug changes.