Bug 37030 - Use template wrapper for breadcrumbs: Cash register stats
Summary: Use template wrapper for breadcrumbs: Cash register stats
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor
Assignee: Owen Leonard
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on: 33436
Blocks: 35402
  Show dependency treegraph
 
Reported: 2024-06-04 17:05 UTC by Owen Leonard
Modified: 2024-08-02 17:01 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.11.00
Circulation function:


Attachments
Bug 37030: Use template wrapper for breadcrumbs: Cash register stats (2.48 KB, patch)
2024-06-04 18:13 UTC, Owen Leonard
Details | Diff | Splinter Review
Bug 37030: Use template wrapper for breadcrumbs: Cash register stats (2.53 KB, patch)
2024-06-04 19:05 UTC, David Nind
Details | Diff | Splinter Review
Bug 37030: Use template wrapper for breadcrumbs: Cash register stats (2.62 KB, patch)
2024-08-02 09:31 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2024-06-04 17:05:49 UTC
This fix was supposed to be in Bug 33436 but I apparently missed it!
Comment 1 Owen Leonard 2024-06-04 18:13:15 UTC
Created attachment 167411 [details] [review]
Bug 37030: Use template wrapper for breadcrumbs: Cash register stats

This patch updates the cash register statistics template so that they
use the new WRAPPER for displaying breadcrumbs.

To test, apply the patch and test page and its variations.
Breadcrumbs should look correct, and each link should be correct.

- Reports ->
  - Cash register statistics
    - Results

Sponsored-By: Athens County Public Libraries
Comment 2 David Nind 2024-06-04 19:05:13 UTC
Created attachment 167418 [details] [review]
Bug 37030: Use template wrapper for breadcrumbs: Cash register stats

This patch updates the cash register statistics template so that they
use the new WRAPPER for displaying breadcrumbs.

To test, apply the patch and test page and its variations.
Breadcrumbs should look correct, and each link should be correct.

- Reports ->
  - Cash register statistics
    - Results

Sponsored-By: Athens County Public Libraries
Signed-off-by: David Nind <david@davidnind.com>
Comment 3 Marcel de Rooy 2024-08-02 09:31:35 UTC
Created attachment 169995 [details] [review]
Bug 37030: Use template wrapper for breadcrumbs: Cash register stats

This patch updates the cash register statistics template so that they
use the new WRAPPER for displaying breadcrumbs.

To test, apply the patch and test page and its variations.
Breadcrumbs should look correct, and each link should be correct.

- Reports ->
  - Cash register statistics
    - Results

Sponsored-By: Athens County Public Libraries
Signed-off-by: David Nind <david@davidnind.com>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Katrin Fischer 2024-08-02 17:01:26 UTC
Pushed for 24.11!

Well done everyone, thank you!