If the user doesn't have the superserials permission, or isn't superlibrarian, you can still edit a subscription accessing the subscription detail page through the serial collection page (for records with multiple subscriptions) or accessing the detail page directly.
Perhaps we should move JD's patches from bug 12080 here? I don't like the thought that his patches are hidden by being obsoleted on another bug.
(In reply to M. Tompsett from comment #1) > Perhaps we should move JD's patches from bug 12080 here? I don't like the > thought that his patches are hidden by being obsoleted on another bug. Jonathan have them on his repo, and he will re-post them here. That's my understanding. If you prefer, or would like to test them now, I can attach his patches here as they are. Just tell.
Jonathan said he would be gone for a week - so reattaching the patches here would be nice.
Created attachment 27242 [details] [review] Refactor can_*_subscription in C4::Serials
Created attachment 27243 [details] [review] Fix C4::Serials::can_edit_subscription
Created attachment 27280 [details] [review] Bug 12080: Fix C4::Serials::can_edit_subscription This patch adds some unit tests for the can_edit_subscription and add a new can_show_subscription subroutines. http://bugs.koha-community.org/show_bug.cgi?id=12098 Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 27281 [details] [review] Bug 12080: Refactor can_*_subscription in C4::Serials http://bugs.koha-community.org/show_bug.cgi?id=12098 Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
After applying all the patches for 12048 - 12080 - 12098 t/db_dependent/Bookseller.t fails for me. I can imagine the test might be wrong now or needs to be rewritten a bit. I will continue testing for now.
Created attachment 27290 [details] [review] [PASSED QA] Bug 12098: Fix C4::Serials::can_edit_subscription This patch adds some unit tests for the can_edit_subscription and add a new can_show_subscription subroutines. http://bugs.koha-community.org/show_bug.cgi?id=12098 Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Notes on second patch.
Created attachment 27291 [details] [review] [PASSED QA] Bug 12098: Refactor can_*_subscription in C4::Serials http://bugs.koha-community.org/show_bug.cgi?id=12098 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.
Pushed to master, along with an important follow-up that puts the C4::Serials::can_show_subscription routine into use (as otherwise, it would be a routine that had zero actual affect). BUT NOTE: Note that there is user-visible change: if a subscription has a blank library, all users with serials permissions will be able to view and/or edit it. It remains to be determined whether we *want* such subscriptions to exist, or if they should only be tied to specific libraries. If this change in behavior isn't desirable, please open a new bug. Thanks, Jonathan!
Pushed to 3.14.x, will be in 3.14.10