- Koha-bugs - lists.koha-community.org

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 --- Comment #76 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 79699 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79699&action=ed… Bug 15486: Modify request.tt to handle the new error string This patch makes reserve/request.pl display a convenient error description when a hold cannot be placed due to the new daily limit configuration. To test: - Apply this patchset - Upgrade - Configure your circulation rules so there's a daily limit for holds - Place holds so the patron reaches the maximum - Place one more hold - Notice the hold cannot be placed and a convenient error message is displayed. - Sign off :-D Signed-off-by: Séverine QUEUNE <severine.queune(a)bulac.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 --- Comment #75 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 79698 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79698&action=ed… Bug 15486: Extend CanItemBeReserved so it handles daily holds limits This patch implements the required changes in C4::Reserves::CanItemBeReserved so it implements a daily limit on holds. It returns the 'tooManyReservesToday' string if the policy doesn't allow placing the hold. It returns 'OK' (current behaviour) otherwise. To test: - Run: $ sudo koha-shell kohadev k$ cd kohaclone k$ prove t/db_dependent/Holds.t => FAIL: Tests fail because the error condition is not making CanItemBeReserved return the desired error code. - Apply this patch - Run: k$ prove t/db_dependent/Holds.t => SUCCESS: Tests pass! - Sign off :-D Signed-off-by: Séverine QUEUNE <severine.queune(a)bulac.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 --- Comment #74 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 79697 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79697&action=ed… Bug 15486: Unit tests This patch introduces unit tests for the new circulation rules option that allows setting a max holds per day limit. To test: - Apply the patch - Run: $ sudo koha-shell kohadev k$ cd kohaclone k$ prove t/db_dependent/Holds.t => FAIL: CanItemBeReserved doesn't check the amount of holds per day and the introduced error code is not returned. OK is returned instead. Signed-off-by: Séverine QUEUNE <severine.queune(a)bulac.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 --- Comment #73 from sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> --- Created attachment 79696 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=79696&action=ed… Bug 15486: DB structure change This patch adds a new column __max_holds_per_day__ to the issuingrules table. It's going to be used to set a daily limit for holds. Signed-off-by: Tomas Cohen Arazi <tomascohen(a)theke.io> Signed-off-by: Séverine QUEUNE <severine.queune(a)bulac.fr> -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79324|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79323|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79322|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79321|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79320|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0

[Bug 15486] Restrict number of holds placed by day
by bugzilla-daemon＠bugs.koha-community.org 01 Oct '18

01 Oct '18

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15486 sandboxes(a)biblibre.com <sandboxes(a)biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #79319|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

1 0