https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38040

Nick Clemens (kidclamp) <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA
                 CC|                            |[email protected]

--- Comment #13 from Nick Clemens (kidclamp) <[email protected]> ---
This patch only deals with the template, I can easily edit the html, remove the
disabled class, and still cancel the hold. A check needs to be added into the
perl scripts or the modules.

It also affects all non-superlibrarians, regardless of independentbranches
being enabled/disabled, we need to check that preference first.

I think maybe we need an additional permission for reserves here - which says
whether you can affect all holds or not. Taking away the ability to
modify/cancel holds would be a big change in behavior that I am not sure all
libraries using independent branches would agree to.

The trend has been to move parts of Independent Branches to more granular
system preferences ( IndependentBranchesPatronModifications , 
IndependentBranchesTransfers ) so maybe we need 'IndependentBranchesReserves'?

This current code could be adapted to work in IntranerUserJS if it is
sufficient for your library, happy to help with that or any of the changes
above if I can

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to