https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18015 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |mirko@abunchofthings.net |ity.org | Status|NEW |ASSIGNED -- 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=18015 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- 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=18015 Claire Gravely <claire_gravely@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #59646|0 |1 is obsolete| | --- Comment #2 from Claire Gravely <claire_gravely@hotmail.com> --- Created attachment 59649 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59649&action=edit Bug 18015 - On shelf holds allowed > "If all unavailable" ignores notforloan If in the circ rules matrix you set "On shelf holds allowed" to "If all unavailable", items with status "Not for loan" are considered available and break the functionality. Test plan: - Set "On shelf holds allowed" to "If all unavailable" for your patron and item category (or everyone and everything) - Have two items for a record. Check out one - Set 7 - Not for loan: "Not For Loan" for the second item - Try to place a hold. Does not work. - Apply the patch - Try to place a hold. Should work now. Signed-off-by: Claire Gravely <claire_gravely@hotmail.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18015 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #59649|0 |1 is obsolete| | --- Comment #3 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 59676 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59676&action=edit Bug 18015 - On shelf holds allowed > "If all unavailable" ignores notforloan If in the circ rules matrix you set "On shelf holds allowed" to "If all unavailable", items with status "Not for loan" are considered available and break the functionality. Test plan: - Set "On shelf holds allowed" to "If all unavailable" for your patron and item category (or everyone and everything) - Have two items for a record. Check out one - Set 7 - Not for loan: "Not For Loan" for the second item - Try to place a hold. Does not work. - Apply the patch - Try to place a hold. Should work now. Signed-off-by: Claire Gravely <claire_gravely@hotmail.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18015 Mirko Tietgen <mirko@abunchofthings.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18015 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master CC| |kyle@bywatersolutions.com --- Comment #5 from Kyle M Hall <kyle@bywatersolutions.com> --- Pushed to master for 17.05, thanks Mirko! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18015 --- Comment #7 from Mason James <mtj@kohaaloha.com> --- (In reply to Jonathan Druart from comment #4)

Must be backported to 16.11 and 16.05

Pushed to 16.05.x, for 16.05.10 release -- You are receiving this mail because: You are watching all bug changes.