Add in overdues.pl filters on item home branch and holding branch.
Created attachment 15291 [details] [review] Proposed patch See commit message
Hi, I applied this patch and did the following tests: 1. Patron A had one item overdue that had both home and holding branch of Centerville. 2. Patron B had one item overdue that had both home and holding branch of Centerville, and one item overdue that had home branch of Rutland and holding branch of Centerville. 3. Patron C had one item overdue that had home branch of Rutland and holding branch of Fairfield. I then used the filters on overdues and found that they gave correct results for item home branch and item holding branch. So I am signing off! However, I do have two questions which could possibly be addressed with a follow up patch: - When looking at overdues, in addition to the item home branch and item holding branch filters that you've added, I also see one for just "Home branch" (with no dropdown menu) as well as one for "Library of the patron." "Library of the patron" is clear and appears to work fine, but I wasn't sure what "Home branch" is supposed to do now that we have a choice of item home branch. Should "Home branch" be removed now? I got no results using Home branch each time that I tried. - When looking at a record (here, for example: http://staff-melia.test.bywatersolutions.com/cgi-bin/koha/catalogue/detail.pl?biblionumber=502), we have columns for Current Location (the same as holding branch) and Home Library. Should we rename your holding branch filter as Current Location for consistency? Thanks, Melia
Created attachment 15349 [details] [review] [Signed off] Bug 9604: filter overdues with homebranch and holdingbranch This patch adds in overdues.pl filters on item home branch and holding branch. Test plan : Play with item home branch and holding branch filters and check it filters well results. Signed-off-by: Melia Meggs <melia@bywatersolutions.com>
(In reply to comment #2) > I also see one for just "Home branch" I do not see this in master version. > Should we rename your holding branch filter as Current Location for consistency? Goof idea for me.
Created attachment 16804 [details] [review] [PASSED QA] Bug 9604: filter overdues with homebranch and holdingbranch This patch adds in overdues.pl filters on item home branch and holding branch. Test plan : Play with item home branch and holding branch filters and check it filters well results. Signed-off-by: Melia Meggs <melia@bywatersolutions.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Filters are added to the list of overdues in the circulation module. All tests and QA script pass.
I think in order to be really consistent we should use library as we agreed on this some time ago (last hackfest even?).
Created attachment 16806 [details] [review] [PASSED QA] Bug 9604: filter overdues with homebranch and holdingbranch This patch adds in overdues.pl filters on item home branch and holding branch. Test plan : Play with item home branch and holding branch filters and check it filters well results. Signed-off-by: Melia Meggs <melia@bywatersolutions.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Filters are added to the list of overdues in the circulation module. All tests and QA script pass.
Created attachment 16807 [details] [review] Bug 9604 - Follow-up: Changing labels from branch to library Relabelling the new filters for consistency.
Created attachment 16808 [details] [review] [SIGNED-OFF] Bug 9604 - Follow-up: Changing labels from branch to library Relabelling the new filters for consistency. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
This patch has been pushed to master.