Summary: | ILL breadcrumbs are wrong | ||
---|---|---|---|
Product: | Koha | Reporter: | Magnus Enger <magnus> |
Component: | ILL | Assignee: | Isobel Graham <isobel.graham09> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | minor | ||
Priority: | P5 - low | CC: | arthur.suzuki, lucas, martin.renvoize, victor |
Version: | 21.11 | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31494 | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.11.00, 22.05.07, 21.11.14
|
|
Circulation function: | |||
Attachments: |
Bug 30890: Fixed ILL breadcrumb
Bug 30890: Fixed ILL breadcrumb Bug 30890: Fixed ILL breadcrumb |
Description
Magnus Enger
2022-06-02 13:37:31 UTC
*** Bug 31494 has been marked as a duplicate of this bug. *** Created attachment 140037 [details] [review] Bug 30890: Fixed ILL breadcrumb Added correct breadrumb depth when managing requests. 1. Viewing a request on a ILL request page 2. Making sure the breadcrumb has three depths and the manage requests has the correct request id - [% ELSIF query_type == 'status' %] + [% ELSIF query_type == 'illview' %] Is the "status" query_type not used for anything? (In reply to Owen Leonard from comment #3) > - [% ELSIF query_type == 'status' %] > + [% ELSIF query_type == 'illview' %] > > Is the "status" query_type not used for anything? It's a good question.. as far as I can tell from the controller query_type will never be returned with a value of 'status'. I tried digging through the git logs to see when it was introduced or changed, but it appears to have never been returned going as far back as the original introduction of ILL into core Koha. Created attachment 140277 [details] [review] Bug 30890: Fixed ILL breadcrumb Added correct breadrumb depth when managing requests. 1. Viewing a request on a ILL request page 2. Making sure the breadcrumb has three depths and the manage requests has the correct request id Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 140280 [details] [review] Bug 30890: Fixed ILL breadcrumb Added correct breadrumb depth when managing requests. 1. Viewing a request on a ILL request page 2. Making sure the breadcrumb has three depths and the manage requests has the correct request id Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 22.11. Nice work everyone, thanks! Backported to 22.05.x for upcoming 22.05.07 release applied to 21.11 for 21.11.14 Not backported to oldoldstable (21.05.x). Feel free to ask if it's needed. Nothing to document it seems, marking resolved. |