To continue the process started by Bug 32507, this patch will update some circulation templates so that they use the new template wrapper for breadcrumbs. - circ/article-requests.tt - circ/bookcount.tt - circ/branchoverdues.tt - circ/branchtransfers.tt - circ/checkout-notes.tt - circ/circulation_batch_checkouts.tt - circ/circulation-home.tt
Created attachment 147793 [details] [review] Bug 33149: Use template wrapper for breadcrumbs: Circulation part 1 This patch updates several circulation templates so that they use the new WRAPPER for displaying breadcrumbs. The patch also makes a minor change to bookcount.pl to allow for correct display of the record title using the biblio-title include. Navigation context for bookcount.tt has been changed to "catalog" from "circulation" because the page is linked to from a catalog-related page. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - Circulation home page - Checkout notes - Article requests - Transfer - Overdues with fines - Check out -> Check out to patron -> - Batch check out - Catalog -> Search -> Bibliographic record -> Items - View item's checkout history. The updated page should show this new breadcrumb sequence: Home -> Catalog -> [title] -> Items -> Circulation statistics
Created attachment 148901 [details] [review] Bug 33149: Use template wrapper for breadcrumbs: Circulation part 1 This patch updates several circulation templates so that they use the new WRAPPER for displaying breadcrumbs. The patch also makes a minor change to bookcount.pl to allow for correct display of the record title using the biblio-title include. Navigation context for bookcount.tt has been changed to "catalog" from "circulation" because the page is linked to from a catalog-related page. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - Circulation home page - Checkout notes - Article requests - Transfer - Overdues with fines - Check out -> Check out to patron -> - Batch check out - Catalog -> Search -> Bibliographic record -> Items - View item's checkout history. The updated page should show this new breadcrumb sequence: Home -> Catalog -> [title] -> Items -> Circulation statistics Signed-off-by: Andrew Auld <andrew.auld@ptfs-europe.com>
Created attachment 149071 [details] [review] Bug 33149: Use template wrapper for breadcrumbs: Circulation part 1 This patch updates several circulation templates so that they use the new WRAPPER for displaying breadcrumbs. The patch also makes a minor change to bookcount.pl to allow for correct display of the record title using the biblio-title include. Navigation context for bookcount.tt has been changed to "catalog" from "circulation" because the page is linked to from a catalog-related page. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - Circulation home page - Checkout notes - Article requests - Transfer - Overdues with fines - Check out -> Check out to patron -> - Batch check out - Catalog -> Search -> Bibliographic record -> Items - View item's checkout history. The updated page should show this new breadcrumb sequence: Home -> Catalog -> [title] -> Items -> Circulation statistics Signed-off-by: Andrew Auld <andrew.auld@ptfs-europe.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Probably we should move this page to the catalog folder, however, the permission here is circ based, so it would be a bigger job and requires discussion. IN the context of breadcrumbs, the change makes sense and is small so passing QA
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work, thanks everyone! Pushed to 22.11.x for the next release.