Attachment 167115 Details for Bug 36945 – Bug 36945: Fix several missed instance of breadcrumb WRAPPER use

[patch] Bug 36945: Fix several missed instance of breadcrumb WRAPPER use

Bug-36945-Fix-several-missed-instance-of-breadcrum.patch (text/plain), 8.87 KB, created by Owen Leonard on 2024-05-23 14:36:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2024-05-23 14:36:53 UTC

Size: 8.87 KB

patch

obsolete

>From 394ac77bb678ee844eeabaf26cd616569cb9caa9 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Thu, 23 May 2024 12:32:01 +0000
>Subject: [PATCH] Bug 36945: Fix several missed instance of breadcrumb WRAPPER
> use
>
>This patch corrects some templates which never got the breadcrumb
>WRAPPER update. Some breadcrumbs and page titles have been updated to be
>more consistent with page headings.
>
>To test, apply the patch and enable the "CatalogConcerns" system
>preference if necessary. Check the following pages to confirm that the
>breadcrumbs menu looks correct and that breadcrumbs are consistent with
>page titles and page headings.
>
>- Circulation -> Upload offline circulation file (.koc)
>  - Upload file -> Add to offline circulation queue
>  - View pending offline circulation actions
>- Cataloging -> Cataloging concerns
>- Reports -> Cash register stats
>
>Sponsored-By: Athens County Public Libraries
>---
> .../prog/en/modules/cataloguing/concerns.tt   | 25 +++++--------
> .../en/modules/offline_circ/enqueue_koc.tt    | 21 ++++-------
> .../prog/en/modules/offline_circ/list.tt      | 27 +++++---------
> .../en/modules/offline_circ/process_koc.tt    | 37 +++++++------------
> .../en/modules/reports/cash_register_stats.tt | 35 +++++++-----------
> 5 files changed, 55 insertions(+), 90 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/concerns.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/concerns.tt
>index c5d09a2213..270782a34c 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/concerns.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/concerns.tt
>@@ -18,28 +18,23 @@
>     [% INCLUDE 'header.inc' %]
>     [% INCLUDE 'cataloging-search.inc' %]
> 
>-    <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
>-        <ol>
>-            <li>
>-                <a href="/cgi-bin/koha/mainpage.pl">Home</a>
>-            </li>
>-            <li>
>+    [% WRAPPER 'sub-header.inc' %]
>+        [% WRAPPER breadcrumbs %]
>+            [% WRAPPER breadcrumb_item %]
>                 <a href="/cgi-bin/koha/cataloguing/cataloging-home.pl">Cataloging</a>
>-            </li>
>-            <li>
>-                <a href="#" aria-current="page">
>-                    Catalog concerns
>-                </a>
>-            </li>
>-        </ol>
>-    </nav>
>+            [% END %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Catalog concerns</span>
>+            [% END %]
>+        [% END #/ WRAPPER breadcrumbs %]
>+    [% END #/ WRAPPER sub-header.inc %]
> 
>     <div class="main container-fluid">
>         <div class="row">
>             <div class="col-sm-10 col-sm-push-2">
>                 <main>
>                     [% INCLUDE 'messages.inc' %]
>-                    <h1>Concerns</h1>
>+                    <h1>Catalog concerns</h1>
> 
>                     <div class="page-section">
>                         <fieldset class="action filters" style="cursor:pointer;">
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/enqueue_koc.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/enqueue_koc.tt
>index eac6c230ff..15ab7af037 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/enqueue_koc.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/enqueue_koc.tt
>@@ -14,20 +14,15 @@
> [% END %]
> 
> [% WRAPPER 'sub-header.inc' %]
>-<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
>-    <ol>
>-        <li>
>-            <a href="/cgi-bin/koha/mainpage.pl">Home</a>
>-        </li>
>-        <li>
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item %]
>             <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a>
>-        </li>
>-        <li>
>-            <a href="#" aria-current="page">Add offline circulations to queue</a>
>-        </li>
>-    </ol>
>-</nav>
>-[% END %]
>+        [% END %]
>+        [% WRAPPER breadcrumb_item bc_active= 1 %]
>+            <span>Add offline circulations to queue</span>
>+        [% END %]
>+    [% END #/ WRAPPER breadcrumbs %]
>+[% END #/ WRAPPER sub-header.inc %]
> 
> <div class="main container-fluid">
>     <div class="row">
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt
>index f1df1f6170..2710d0429d 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/list.tt
>@@ -17,23 +17,16 @@
>     [% INCLUDE 'circ-search.inc' %]
> [% END %]
> 
>-    [% WRAPPER 'sub-header.inc' %]
>-    <nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
>-        <ol>
>-            <li>
>-                <a href="/cgi-bin/koha/mainpage.pl">Home</a>
>-            </li>
>-            <li>
>-                <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a>
>-            </li>
>-            <li>
>-                <a href="#" aria-current="page">
>-                    Offline circulation
>-                </a>
>-            </li>
>-        </ol>
>-    </nav>
>-    [% END %]
>+[% WRAPPER 'sub-header.inc' %]
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item %]
>+            <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a>
>+        [% END %]
>+        [% WRAPPER breadcrumb_item bc_active= 1 %]
>+            <span>Offline circulation</span>
>+        [% END %]
>+    [% END #/ WRAPPER breadcrumbs %]
>+[% END #/ WRAPPER sub-header.inc %]
> 
> <div class="main container-fluid">
>     <div class="row">
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt
>index 272c51081b..3c6cd47f3b 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/offline_circ/process_koc.tt
>@@ -17,34 +17,25 @@
> [% END %]
> 
> [% WRAPPER 'sub-header.inc' %]
>-<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
>-    <ol>
>-        <li>
>-            <a href="/cgi-bin/koha/mainpage.pl">Home</a>
>-        </li>
>-        <li>
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item %]
>             <a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation</a>
>-        </li>
>+        [% END %]
> 
>         [% IF ( transactions_loaded ) %]
>-            <li>
>+            [% WRAPPER breadcrumb_item %]
>                 <a href="/cgi-bin/koha/offline_circ/process_koc.pl">Offline circulation file upload</a>
>-            </li>
>-            <li>
>-                <a href="#" aria-current="page">
>-                    Results
>-                </a>
>-            </li>
>+            [% END %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Results</span>
>+            [% END %]
>         [% ELSE %]
>-            <li>
>-                <a href="#" aria-current="page">
>-                    Offline circulation file upload
>-                </a>
>-            </li>
>-        [% END %]
>-    </ol>
>-</nav>
>-[% END %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Offline circulation file upload</span>
>+            [% END %]
>+        [% END # /IF transactions_loaded %]
>+    [% END #/ WRAPPER breadcrumbs %]
>+[% END #/ WRAPPER sub-header.inc %]
> 
> <div class="main container-fluid">
>     <div class="row">
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt
>index b1faf8f09a..e2d6fb831e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/reports/cash_register_stats.tt
>@@ -27,34 +27,25 @@
> [% END %]
> 
> [% WRAPPER 'sub-header.inc' %]
>-<nav id="breadcrumbs" aria-label="Breadcrumb" class="breadcrumb">
>-    <ol>
>-        <li>
>-            <a href="/cgi-bin/koha/mainpage.pl">Home</a>
>-        </li>
>-        <li>
>+    [% WRAPPER breadcrumbs %]
>+        [% WRAPPER breadcrumb_item %]
>             <a href="/cgi-bin/koha/reports/reports-home.pl">Reports</a>
>-        </li>
>+        [% END %]
> 
>         [% IF ( do_it ) %]
>-            <li>
>+            [% WRAPPER breadcrumb_item %]
>                 <a href="/cgi-bin/koha/reports/cash_register_stats.pl">Cash register statistics</a>
>-            </li>
>-            <li>
>-                <a href="#" aria-current="page">
>-                    Results
>-                </a>
>-            </li>
>+            [% END %]
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Results</span>
>+            [% END %]
>         [% ELSE %]
>-            <li>
>-                <a href="#" aria-current="page">
>-                    Cash register statistics
>-                </a>
>-            </li>
>+            [% WRAPPER breadcrumb_item bc_active= 1 %]
>+                <span>Cash register statistics</span>
>+            [% END %]
>         [% END %]
>-    </ol>
>-</nav>
>-[% END %]
>+    [% END #/ WRAPPER breadcrumbs %]
>+[% END #/ WRAPPER sub-header.inc %]
> 
> <div class="main container-fluid">
>     <div class="row">
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 36945: 167115 | 167138 | 170657