Attachment 140278 Details for Bug 31404 – Bug 31404: Sync Circulation sidebar with Circulation home

[patch] Bug 31404: Sync Circulation sidebar with Circulation home

Bug-31404-Sync-Circulation-sidebar-with-Circulatio.patch (text/plain), 9.21 KB, created by Katrin Fischer on 2022-09-07 12:41:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2022-09-07 12:41:14 UTC

Size: 9.21 KB

patch

obsolete

>From 4dd2a23f1146c4400e79b07b249a6b52adfa1168 Mon Sep 17 00:00:00 2001
>From: Isobel Graham <isobel.graham09@gmail.com>
>Date: Wed, 31 Aug 2022 16:09:43 +0100
>Subject: [PATCH] Bug 31404: Sync Circulation sidebar with Circulation home
>
>This patch syncs the content of the circulation sidebar navigation with
>the content of the circulation homepage
>
>Test plan
>1. Enable the `CircSidebar` system preference
>2. Compare the sidebar navigation on circulation pages with the content
>   of the circulation homepage.
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> .../prog/en/includes/circ-nav.inc             | 139 ++++++++++++------
> 1 file changed, 95 insertions(+), 44 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc
>index 3e2aaa81ae..4b0d15f9ce 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc
>+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/circ-nav.inc
>@@ -3,68 +3,119 @@
> <div id="navmenu">
>     <div id="navmenulist">
> 
>-        <ul>
>-            <li><a href="/cgi-bin/koha/circ/circulation-home.pl">Circulation home</a></li>
>-        </ul>
>-
>         <h5>Circulation</h5>
>         <ul>
>-            <li><a href="/cgi-bin/koha/circ/circulation.pl">Check out</a></li>
>-            <li><a href="/cgi-bin/koha/circ/returns.pl">Check in</a></li>
>-            <li><a href="/cgi-bin/koha/circ/renew.pl">Renew</a></li>
>-            [% IF !Koha.Preference('IndependentBranchesTransfers') || CAN_user_superlibrarian %]
>-                <li><a href="/cgi-bin/koha/circ/branchtransfers.pl">Transfer</a></li>
>-            [% END %]
>-            [% IF !( Koha.Preference('IndependentBranches') && !CAN_user_superlibrarian && !CAN_user_editcatalogue_edit_catalogue ) %]
>+            <li>
>+                <a href="/cgi-bin/koha/circ/circulation.pl">Check out</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/circ/returns.pl">Check in</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/circ/renew.pl">Renew</a>
>+            </li>
>+            [% UNLESS IndependentBranches %]
>+            <li>
>                 [% IF Koha.Preference('UseCirculationDesks') %]
>-                <li><a href="/cgi-bin/koha/circ/set-library.pl">Set library and desk</a></li>
>+                <a href="/cgi-bin/koha/circ/set-library.pl"> Set library and desk</a>
>                 [% ELSE %]
>-                <li><a href="/cgi-bin/koha/circ/set-library.pl">Set library</a></li>
>+                <a href="/cgi-bin/koha/circ/set-library.pl"> Set library</a>
>                 [% END %]
>-            [% ELSIF Koha.Preference('UseCirculationDesks') %]
>-                <li><a href="/cgi-bin/koha/circ/set-library.pl">Set desk</a></li>
>+            </li>
>+            [% END %]
>+            [% IF ( fast_cataloging && CAN_user_editcatalogue_fast_cataloging ) %]
>+            <li>
>+                <a href="/cgi-bin/koha/cataloguing/addbiblio.pl?frameworkcode=FA"> Fast cataloging</a>
>+            </li>
>+            [% END %]
>+            [% IF ( Koha.Preference('AllowCheckoutNotes') && CAN_user_circulate_manage_checkout_notes ) %]
>+            <li>
>+                <a href="/cgi-bin/koha/circ/checkout-notes.pl"> Checkout notes</a>
>+            </li>
>+            [% END %]
>+            [% IF Koha.Preference('OnSiteCheckouts') %]
>+            <li>
>+                <a href="/cgi-bin/koha/circ/on-site_checkouts.pl"> Pending on-site checkouts</a>
>+            </li>
>             [% END %]
>-            [% IF ( fast_cataloging ) %][% IF ( CAN_user_editcatalogue_fast_cataloging ) %]
>-                <li><a href="/cgi-bin/koha/cataloguing/addbiblio.pl?frameworkcode=FA">Fast cataloging</a></li>
>-            [% END %][% END %]
>-            [% IF Koha.Preference('AllowCheckoutNotes') && CAN_user_circulate_manage_checkout_notes %]
>-                <li><a href="/cgi-bin/koha/circ/checkout-notes.pl">Checkout notes</a> [% IF pending_checkout_notes.count %]<span class="number_box"><a href="/cgi-bin/koha/circ/checkout-notes.pl">[% pending_checkout_notes.count | html %]</a></span>[% END %]</li>
>+        </ul>
>+
>+        <h5>Holds</h5>
>+        <ul>
>+            <li>
>+                <a  href="/cgi-bin/koha/circ/view_holdsqueue.pl"> Holds queue</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/circ/pendingreserves.pl"> Holds to pull</a>
>+            </li>
>+            <li>
>+                <a  href="/cgi-bin/koha/circ/waitingreserves.pl"> Holds awaiting pickup</a>
>+            </li>
>+            [% IF ( Koha.Preference('CurbsidePickup') && CAN_user_circulate_manage_curbside_pickups ) %]
>+            <li>
>+                <a  href="/cgi-bin/koha/circ/curbside_pickups.pl"> Curbside pickups</a>
>+            </li>
>             [% END %]
>+            <li>
>+                <a - href="/cgi-bin/koha/circ/reserveratios.pl"> Hold ratios</a>
>+            </li>
>         </ul>
> 
>+        [% IF Koha.Preference('UseRecalls') and CAN_user_recalls %]
>+        <h5>Recalls</h5>
>+        <ul>
>+            <li>
>+                <a href="/cgi-bin/koha/recalls/recalls_queue.pl" title="All active recalls"> Recalls queue</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/recalls/recalls_to_pull.pl" title="Recalls that could be filled but have not been set waiting"> Recalls to pull</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/recalls/recalls_overdue.pl" title="Recalled items that are overdue to be returned"> Overdue recalls</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/recalls/recalls_waiting.pl" title="Recalled items awaiting pickup"> Recalls awaiting pickup</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/recalls/recalls_old_queue.pl" title="Inactive recalls"> Old recalls</a>
>+            </li>
>+        </ul>
>+        [% END %]
>+
>         [% IF Koha.Preference('ArticleRequests') %]
>         <h5>Patron request</h5>
>         <ul>
>-            <li><a href="/cgi-bin/koha/circ/article-requests.pl">Article requests</a></li>
>+            <li>
>+                <a href="/cgi-bin/koha/circ/article-requests.pl" title="Article requests"> Article requests</a>
>+            </li>
>         </ul>
>         [% END %]
> 
>-        <h5>Circulation reports</h5>
>+        <h5>Transfers</h5>
>         <ul>
>-            <li><a href="/cgi-bin/koha/circ/view_holdsqueue.pl">Holds queue</a></li>
>-            <li><a href="/cgi-bin/koha/circ/pendingreserves.pl">Holds to pull</a></li>
>-            <li><a href="/cgi-bin/koha/circ/waitingreserves.pl">Holds awaiting pickup</a></li>
>-            [% IF ( Koha.Preference('CurbsidePickup') && CAN_user_circulate_manage_curbside_pickups ) %]
>-                <li><a href="/cgi-bin/koha/circ/curbside_pickups.pl">Curbside pickups</a></li>
>-            [% END %]
>-            <li><a href="/cgi-bin/koha/circ/reserveratios.pl">Hold ratios</a></li>
>-            <li><a href="/cgi-bin/koha/circ/transferstoreceive.pl">Transfers to receive</a></li>
>-            [% IF ( CAN_user_circulate_overdues_report ) %]
>-                <li><a href="/cgi-bin/koha/circ/overdue.pl">Overdues</a></li>
>-            [% END %]
>-            <li><a href="/cgi-bin/koha/circ/branchoverdues.pl">Overdues with fines</a></li>
>-            [% IF Koha.Preference('OnSiteCheckouts') %]
>-                <li><a href="/cgi-bin/koha/circ/on-site_checkouts.pl">Pending on-site checkouts</a></li>
>-            [% END %]
>+        [% IF !Koha.Preference('IndependentBranchesTransfers') || CAN_user_superlibrarian %]
>+            <li>
>+                <a href="/cgi-bin/koha/circ/branchtransfers.pl"> Transfer</a>
>+            </li>
>+        [% END %]
>+             <li>
>+                <a href="/cgi-bin/koha/circ/transfers_to_send.pl"> Transfers to send</a>
>+            </li>
>+            <li>
>+                <a href="/cgi-bin/koha/circ/transferstoreceive.pl"> Transfers to receive</a>
>+            </li>
>+        </ul>
> 
>-            [% IF Koha.Preference('UseRecalls') and CAN_user_recalls %]
>-                <li><a href="/cgi-bin/koha/recalls/recalls_queue.pl" title="All active recalls">Recalls queue</a></li>
>-                <li><a href="/cgi-bin/koha/recalls/recalls_to_pull.pl" title="Recalls that could be filled but have not been set waiting">Recalls to pull</a></li>
>-                <li><a href="/cgi-bin/koha/recalls/recalls_overdue.pl" title="Recalled items that are overdue to be returned">Overdue recalls</a></li>
>-                <li><a href="/cgi-bin/koha/recalls/recalls_waiting.pl" title="Recalled items awaiting pickup">Recalls awaiting pickup</a></li>
>-                <li><a href="/cgi-bin/koha/recalls/recalls_old_queue.pl" title="Inactive recalls">Old recalls</a></li>
>+        <h5>Overdues</h5>
>+        <ul>
>+            [% IF ( CAN_user_circulate_overdues_report ) %]
>+            <li>
>+                <a href="/cgi-bin/koha/circ/overdue.pl" title="Warning: This report is very resource intensive on systems with large numbers of overdue items."> Overdues</a>
>+            </li>
>             [% END %]
>+            <li>
>+                <a href="/cgi-bin/koha/circ/branchoverdues.pl" title="Limited to your library.  See report help for other details."> Overdues with fines</a>
>+             </li>
>         </ul>
>-
>     </div>
> </div>
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 31404: 140006 | 140167 | 140278 | 140279