[Koha-bugs] [Bug 6261] Consolidate patron toolbar include files

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Dec 26 10:48:37 CET 2012


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6261

Jonathan Druart <jonathan.druart at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Passed QA
                 CC|                            |jonathan.druart at biblibre.co
                   |                            |m

--- Comment #8 from Jonathan Druart <jonathan.druart at biblibre.com> ---
QA Comments:

Looks good to me, no regression found.
The patch merges 2 include template files into 1 (members-toolbar.inc), changes
the permission for printing (from 'borrowers => 1' to 'circulate =>
circulate_remaining_permissions') and fixes a typo in the setstatus.pl script
(bornum with borrowernumber).

A future improvement could be done by managing the different files which use
this include file specifying the destination variable (in order to be
redirected to the page where the user was).

Marked to Passed QA

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.


More information about the Koha-bugs mailing list