If RealTimeHoldsQueue is on, touch_all_biblios triggers a update_holds_queue_for_biblios background job for each affected record. This will result in a as many background jobs being queued up as records! It makes far more sense for this script to not do that which gives the administrator the option for running the holds queue builder if the changes would affect holdability, or to not run it at all.
Created attachment 167450 [details] [review] Bug 37037: touch_all_biblios.pl triggers rebuilding holds for all affected records when RealTimeHoldsQueue is enabled If RealTimeHoldsQueue is on, touch_all_biblios triggers a update_holds_queue_for_biblios background job for each affected record. This will result in a as many background jobs being queued up as records! It makes far more sense for this script to not do that which gives the administrator the option for running the holds queue builder if the changes would affect holdability, or to not run it at all. Test Plan: 1) Run touch_all_biblios.pl 3) Note a update_holds_queue_for_biblios background job is queued for each record touched 4) Apply this patch 5) Merge touch_all_biblios.pl again 6) Note that no update_holds_queue_for_biblios jobs were queued
Created attachment 167455 [details] [review] Bug 37037: touch_all_biblios.pl triggers rebuilding holds for all affected records when RealTimeHoldsQueue is enabled If RealTimeHoldsQueue is on, touch_all_biblios triggers a update_holds_queue_for_biblios background job for each affected record. This will result in a as many background jobs being queued up as records! It makes far more sense for this script to not do that which gives the administrator the option for running the holds queue builder if the changes would affect holdability, or to not run it at all. Test Plan: 1) Run touch_all_biblios.pl 3) Note a update_holds_queue_for_biblios background job is queued for each record touched 4) Apply this patch 5) Merge touch_all_biblios.pl again 6) Note that no update_holds_queue_for_biblios jobs were queued Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com>
Created attachment 167660 [details] [review] Bug 37037: touch_all_biblios.pl triggers rebuilding holds for all affected records when RealTimeHoldsQueue is enabled If RealTimeHoldsQueue is on, touch_all_biblios triggers a update_holds_queue_for_biblios background job for each affected record. This will result in a as many background jobs being queued up as records! It makes far more sense for this script to not do that which gives the administrator the option for running the holds queue builder if the changes would affect holdability, or to not run it at all. Test Plan: 1) Run touch_all_biblios.pl 3) Note a update_holds_queue_for_biblios background job is queued for each record touched 4) Apply this patch 5) Merge touch_all_biblios.pl again 6) Note that no update_holds_queue_for_biblios jobs were queued Signed-off-by: Matt Blenkinsop <matt.blenkinsop@ptfs-europe.com> Signed-off-by: Emily Lamancusa <emily.lamancusa@montgomerycountymd.gov>
Small, logical change. QA tool is happy. Passing QA
As this is a script and not a module, I'm not worried about a unit test here... but a warning at the end of the script run might be nice to give whoever ran the script a heads up that they may want to now run a rebuild of the holds queue might be helpful?
Thanks for all the hard work! Pushed to main for the next 24.11.00 release as RM Assistant
Backported to 24.05.x for upcoming 24.05.02
Pushed to 23.11.x for 23.11.07
not backporting to 23.05.x unless requested