]> git.koha-community.org Git - koha.git/commit
Bug 12098: Refactor can_*_subscription in C4::Serials
authorJonathan Druart <jonathan.druart@biblibre.com>
Wed, 16 Apr 2014 14:58:36 +0000 (16:58 +0200)
committerFridolin Somers <fridolin.somers@biblibre.com>
Thu, 7 Aug 2014 08:26:50 +0000 (10:26 +0200)
commit9de01117b268b8ba878cf5e50aaabfecb8bed6fe
treebaa8b077d8a3bb89504de764319ed39a4e3bc373
parent38189555a346751287315453bce46880d13c701e
Bug 12098: Refactor can_*_subscription in C4::Serials

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested on top of patches for 12048 and 12080.

Subscription search
- superlibrarian, IndyBranches on/off - always sees all subscriptions
- superserials, IndyBranches on/off - always sees all subscriptions
- no superserials, IndyBranches on - only sees own subscriptions
Note: Subscriptions without branches will only show, when all subscriptions
      are visible. In a future enh it might be good to enforce setting a
      branch, when IndyBranches is used.
- no superserials, IndyBranches off - always sees all subscriptions

Subscription editing
- superlibrarian, IndyBranches on/off - can edit all subscriptions
- superserials, IndyBranches on/off - can edit all subscriptions
- no superserials, IndyBranches on - can only edit own subscriptons and
  subscriptions without branch
  NOTE: it would make sense to also allow Edit > Edit as new (duplicate)
  here, so one can copy the subscription from another branch to modify
  it for the own branch.

Passes tests in t, xt and QA script, also newly provided unit tests.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit 4d78b9588a9f5fc68142b9d1556686072e7b5dba)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
C4/Serials.pm