In our library viewing the Hold Ratios is more a function of Acquisitions than Circulation. It would be helpful to add a link to the Hold Ratios on the Acquisitions home page so that staff doesn't have to go to the Circulation home page to access.
It's always confused me that Holds Ratio is in circ. I'd be in favor of moving it to acq entirely.
Created attachment 116249 [details] [review] Bug 27498: Add a link for the Hold Ratios to Acquisitions home page This patch adds a link to the hold ratios report in the Acquisitions sidebar menu under the "reports" heading. To test, apply the patch and go to Acquisitions. The link to the hold ratios report should be there and it should work correctly.
Created attachment 116407 [details] [review] Bug 27498: Add a link for the Hold Ratios to Acquisitions home page This patch adds a link to the hold ratios report in the Acquisitions sidebar menu under the "reports" heading. To test, apply the patch and go to Acquisitions. The link to the hold ratios report should be there and it should work correctly. Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov>
I signed off because the link was where it was supposed to be and it correctly went to the holds ratio page. I did want to point out that I placed a bunch of holds that should have triggered results in the holds ratio report but did not in case that's an issue.
Hi Owen, I believe the permission check is not quite right. The hold ratios page is asking for circulation_remaining_permissions. Can you please check?
Created attachment 116616 [details] [review] Bug 27498: Add a link for the Hold Ratios to Acquisitions home page This patch adds a link to the hold ratios report in the Acquisitions sidebar menu under the "reports" heading. To test, apply the patch and go to Acquisitions. The link to the hold ratios report should be there and it should work correctly.
I've tried testing this bug several times in a sandbox and I'm getting an error with the patch applied and even without it applied. The error starts out with the below statement and then lists 56 specific errors. I'm unable to test until something gets fixed. Plack::App::WrapCGI::call(): DBI Exception: DBD::mysql::st execute failed: 'koha_bug27498.reserves.reservedate' isn't in GROUP BY [for Statement "SELECT reservedate . . . . .
Can confirm that before the patch is applied on master using koha-testing-docker that there is a system error when clicking on Circulation > Holds ratios. Tested Koha 20.11.x and it works then, so something has broken it. The same error is recorded in Bug 21941 but that was created in 2018! I'm not smart enough to figure out what bug was introduced after 20.11.x that is causing it...
The GROUP BY type of error is usually related to using scrict MySQL settings. I'll link it to the omnibus.
Created attachment 118565 [details] [review] Bug 27498: Add a link for the Hold Ratios to Acquisitions home page This patch adds a link to the hold ratios report in the Acquisitions sidebar menu under the "reports" heading. To test, apply the patch and go to Acquisitions. The link to the hold ratios report should be there and it should work correctly. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 118566 [details] [review] Bug 27498: (follow-up) Update to display logic Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
@David and Barbara I think the problem you are seeing is another bug, not related to this patch. Could you check you see the same error when opening the report from the circulation start page please and make a report if you do? @Owen I've taken a look and have a suggestion to change up the logic a bit to avoid repeating the link. Could you have a look if you like it?
Created attachment 118567 [details] [review] Bug 27498: (follow-up) Update to display logic Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
@Katrin I’m still getting the same error I reported in comment #7 when clicking on the link from the circ start page and the new link from acq home page.
(In reply to koha-US bug tracker from comment #14) > @Katrin I’m still getting the same error I reported in comment #7 when > clicking on the link from the circ start page and the new link from acq home > page. Yes, it means the error is unrelaed - this only adds a link in another spot. We need to file a bug for what you reported in comment#7 separately. Btw - I don't see it, but I think that just means that my setup is a little different. I suspect a MySQL/MariaDB issue.
I realized there already is a report for it that David pointed out: bug 21941
Not able to apply the second patch Bug 27498: (follow-up) Update to display logic Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> -------------------------- Apply? [y]es/[n]o/[e]dit/[v]iew patch/[a]ccept all: a Applying: Bug 27498: (follow-up) Update to display logic fatal: sha1 information is lacking or useless (koha-tmpl/intranet-tmpl/prog/en/includes/acquisitions-menu.inc). error: could not build fake ancestor Patch failed at 0001 Bug 27498: (follow-up) Update to display logic The copy of the patch that failed is found in: .git/rebase-apply/patch When you have resolved this problem, run "git am --continue". If you prefer to skip this patch, run "git am --skip" instead. To restore the original branch and stop patching, run "git am --abort".
Hi Amit, can you make sure your branch is current master? I am not sure why the first woudl apply but the second not. Are you using git bz?
Created attachment 118709 [details] [review] Bug 27498: Add a link for the Hold Ratios to Acquisitions home page This patch adds a link to the hold ratios report in the Acquisitions sidebar menu under the "reports" heading. To test, apply the patch and go to Acquisitions. The link to the hold ratios report should be there and it should work correctly.
Created attachment 118710 [details] [review] Bug 27498: (follow-up) Update to display logic Hi Katrin, Sorry applied one patch only. Now applied both the patches and working fine. Now giving signoff. Thank you.
Created attachment 120707 [details] [review] Bug 27498: Add a link for the Hold Ratios to Acquisitions home page This patch adds a link to the hold ratios report in the Acquisitions sidebar menu under the "reports" heading. To test, apply the patch and go to Acquisitions. The link to the hold ratios report should be there and it should work correctly. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Amit Gupta <amitddng135@gmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 120708 [details] [review] Bug 27498: (follow-up) Update to display logic Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Amit Gupta <amitddng135@gmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Pushed to master for 21.11, thanks to everybody involved!
Pushed to 21.05.x for 21.05.02
I prefer not impact display on oldstable 20.11.x