Librarians should be warned that a patron is restricted when placing holds.
Librarians should be warned that a patron is restricted when placing holds, as they are warned that a patron's acount has expired.
Created attachment 42814 [details] [review] Bug 14883 - Librarians are not warned if patron is restricted when placing holds Librarians should be warned that a patron is restricted when placing holds, as they are warned that a patron's acount has expired. Test Plan: 1) Apply this patch 2) Add a restriction for a patron 3) Attempt to place a hold for that patron 4) Note the notice at the top notifying you that the patron is restricted
Created attachment 42833 [details] [review] [Signed-off] Bug 14883 - Librarians are not warned if patron is restricted when placing holds Librarians should be warned that a patron is restricted when placing holds, as they are warned that a patron's acount has expired. Test Plan: 1) Apply this patch 2) Add a restriction for a patron 3) Attempt to place a hold for that patron 4) Note the notice at the top notifying you that the patron is restricted Works as expected. Signed-off-by: Marc Véron <veron@veron.ch>
Created attachment 42839 [details] [review] Bug 14883 - Librarians are not warned if patron is restricted when placing holds Librarians should be warned that a patron is restricted when placing holds, as they are warned that a patron's acount has expired. Test Plan: 1) Apply this patch 2) Add a restriction for a patron 3) Attempt to place a hold for that patron 4) Note the notice at the top notifying you that the patron is restricted Works as expected. Signed-off-by: Marc Véron <veron@veron.ch> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
It would have made sense to use the IsDebarred method from the Borrowers TT plugin, but to keep consistency with how it's done at the OPAC, it's certainly preferable to stick this way.
Patch pushed to master. Thanks Kyle!