Bug 8781

Summary: Access to Transfer page while "independent branches" is on
Product: Koha Reporter: Adrien SAURAT <adrien.saurat>
Component: CirculationAssignee: Adrien SAURAT <adrien.saurat>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: minor    
Priority: P5 - low CC: chris, gmcharlt, kyle.m.hall, melia
Version: 3.8   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: proposed patch
[Signed off] Bug 8781: hiding of transfer page when it's useless
[SIGNED-OFF] Bug 8781: hiding of transfer page when it's useless

Description Adrien SAURAT 2012-09-18 08:01:07 UTC
With the system preference  "IndependantBranches" set to On, only a superlibrarian can make transfers.
The page is still displayed for others librarians with circulation rights, but they can only chose their own library as a destination, making the page useless for them.

The link to this page could be hidden in such cases.
Comment 1 Adrien SAURAT 2012-09-18 08:32:46 UTC Comment hidden (obsolete)
Comment 2 Melia Meggs 2012-11-08 20:13:33 UTC
Thanks for the test plan!  I followed steps #1-4, and everything behaves as you described.  I am signing off.
Comment 3 Melia Meggs 2012-11-08 20:14:31 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2012-12-02 13:16:42 UTC
Created attachment 13820 [details] [review]
[SIGNED-OFF] Bug 8781: hiding of transfer page when it's useless

The transfer page is only useful when the system
preference "IndependantBranches" is Off of when the user
is a superlibrarian. Otherwise it can be hidden.

Signed-off-by: Melia Meggs <melia@test.bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
All tests and perlcritic pass.

TEST PLAN :
It's about testing the display of the "Transfer" link on "circulation-home.pl".
The link needs to appear if IndependantBranches is off or if the user is a superlibrarian.

1) IndependantBranches = Off, user is superlibrarian : link appears
2) IndependantBranches = Off, user is not superlibrarian : link appears
3) IndependantBranches = On, user is superlibrarian : link appears
4) IndependantBranches = On, user is not superlibrarian : link does not appear
Comment 5 Katrin Fischer 2012-12-02 13:18:02 UTC
Passed QA.

Note: The page is still accessible (and not useful) when indybranches is ON and user is not superlibrarian. But this is an improvement.
Comment 6 Jared Camins-Esakov 2012-12-02 14:28:20 UTC
This patch has been pushed to master.
Comment 7 Chris Cormack 2012-12-03 06:56:25 UTC
Bug fix really, pushed to 3.10.x and 3.8.x, will be in 3.10.1 and 3.8.8