Attachment 171800 Details for Bug 36083 – Bug 36083: Fix HTML additional contents for the logged in branchcode

[patch] Bug 36083: Fix HTML additional contents for the logged in branchcode

Bug-36083-Fix-HTML-additional-contents-for-the-log.patch (text/plain), 8.68 KB, created by Laura Escamilla on 2024-09-19 19:29:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Laura Escamilla

Created: 2024-09-19 19:29:24 UTC

Size: 8.68 KB

patch

obsolete

>From 3918fb6b08c1a3d77b67c1294f7e1dbcafa03b4e Mon Sep 17 00:00:00 2001
>From: Brendan Lawlor <blawlor@clamsnet.org>
>Date: Thu, 22 Aug 2024 16:39:55 +0000
>Subject: [PATCH] Bug 36083: Fix HTML additional contents for the logged in
> branchcode
>
>This patch updates the calls to AdditionalContents.get() to use
>Branches.GetLoggedInBranchcode instead of the logged in users
>homebranch for the library parameter.
>
>To test:
>1. In Tools > HTML Customizations
>2. Choose to create an entry in StaffAcquisitionsHome
>3. Assign it to a specific library
>   It's helpful to test with libraries other than Centerville
>4. Add text and publication date (important or it will never display)
>5. Save
>6. Go to the Acquisitions Home page - logged into that library, no text displays.
>7. Apply patch and refresh browser
>8. Notice the library specifc content displays
>9. Notice content created for all libraries still displays too
>10. Test HTML content for specific libraries works on all 8 pages
>    IntranetmainUserBlock
>    Staff AcquisitionsHome
>    StaffAuthoritiesHome
>    StaffCataloguing Home
>    StaffListsHome
>    StaffPatronsHome
>    StaffPOSHome
>    StaffSerialsHome
>
>Signed-off-by: Sam Lau <samalau@gmail.com>
>Signed-off-by: Laura_Escamilla <laura.escamilla@bywatersolutions.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt     | 2 +-
> .../prog/en/modules/authorities/authorities-home.tt             | 2 +-
> .../prog/en/modules/cataloguing/cataloging-home.tt              | 2 +-
> koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt        | 2 +-
> koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt       | 2 +-
> koha-tmpl/intranet-tmpl/prog/en/modules/pos/pay.tt              | 2 +-
> koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-home.tt | 2 +-
> .../intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt     | 2 +-
> 8 files changed, 8 insertions(+), 8 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt
>index 66a04d29e1..533cc7285e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/acqui-home.tt
>@@ -201,7 +201,7 @@
> </div>
> [% END %]
> 
>-    [%- SET StaffAcquisitionsHome = AdditionalContents.get( location => "StaffAcquisitionsHome", lang => lang, library => logged_in_user.branchcode ) -%]
>+    [%- SET StaffAcquisitionsHome = AdditionalContents.get( location => "StaffAcquisitionsHome", lang => lang, library => Branches.GetLoggedInBranchcode ) -%]
>     [%- FOREACH block IN StaffAcquisitionsHome.content -%]
>     <div class="page-section">
>         [%- block.content | $raw -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities-home.tt
>index b2bcb07099..de869cdbac 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities-home.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/authorities/authorities-home.tt
>@@ -34,7 +34,7 @@
> 
>             <div id="merge_in_progress" class="page-section bg-info" style="display:none;"></div>
> 
>-            [%- SET StaffAuthoritiesHome = AdditionalContents.get( location => "StaffAuthoritiesHome", lang => lang, library => logged_in_user.branchcode ) -%]
>+            [%- SET StaffAuthoritiesHome = AdditionalContents.get( location => "StaffAuthoritiesHome", lang => lang, library => Branches.GetLoggedInBranchcode ) -%]
>             [%- FOREACH block IN StaffAuthoritiesHome.content -%]
>             <div class="page-section">
>                 [%- block.content | $raw -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/cataloging-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/cataloging-home.tt
>index 8cde1a41a5..0bfd275638 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/cataloging-home.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/cataloging-home.tt
>@@ -219,7 +219,7 @@
>                     </div>
>                 </div>
> 
>-                [%- SET StaffCataloguingHome = AdditionalContents.get( location => "StaffCataloguingHome", lang => lang, library => logged_in_user.branchcode ) -%]
>+                [%- SET StaffCataloguingHome = AdditionalContents.get( location => "StaffCataloguingHome", lang => lang, library => Branches.GetLoggedInBranchcode ) -%]
>                 [%- FOREACH block IN StaffCataloguingHome.content -%]
>                 <div class="page-section">
>                     [%- block.content | $raw -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt
>index f624843057..65d793f319 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/intranet-main.tt
>@@ -276,7 +276,7 @@
>                     </div> 
>                 </div> 
> 
>-                [% SET IntranetmainUserblock = AdditionalContents.get( location => "IntranetmainUserblock", lang => lang, library => logged_in_user.branchcode || default_branch ) %]
>+                [% SET IntranetmainUserblock = AdditionalContents.get( location => "IntranetmainUserblock", lang => lang, library => Branches.GetLoggedInBranchcode || default_branch ) %]
> 
>                 [% IF ( IntranetmainUserblock.content && IntranetmainUserblock.content.count > 0 ) %]
>                     <div class="row">
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
>index 8684ac860e..49a0b98275 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
>@@ -120,7 +120,7 @@
>             </form>
>         [% END %]
> 
>-                [%- SET StaffPatronsHome = AdditionalContents.get( location => "StaffPatronsHome", lang => lang, library => logged_in_user.branchcode ) -%]
>+                [%- SET StaffPatronsHome = AdditionalContents.get( location => "StaffPatronsHome", lang => lang, library => Branches.GetLoggedInBranchcode ) -%]
>                 [%- FOREACH block IN StaffPatronsHome.content -%]
>                 <div class="page-section">
>                     [%- block.content | $raw -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/pos/pay.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/pos/pay.tt
>index e624d24f84..f30834d02a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/pos/pay.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/pos/pay.tt
>@@ -168,7 +168,7 @@
>         </form>
>         [% END %]
> 
>-        [%- SET StaffPOSHome = AdditionalContents.get( location => "StaffPOSHome", lang => lang, library => logged_in_user.branchcode ) -%]
>+        [%- SET StaffPOSHome = AdditionalContents.get( location => "StaffPOSHome", lang => lang, library => Branches.GetLoggedInBranchcode ) -%]
>         [%- FOREACH block IN StaffPOSHome.content -%]
>             <div class="page-section">
>                 [%- block.content | $raw -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-home.tt
>index 3153db6fa0..40727e470e 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-home.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-home.tt
>@@ -37,7 +37,7 @@
> 
>                 [% INCLUDE 'serials-advsearch.inc' %]
> 
>-                [%- SET StaffSerialsHome = AdditionalContents.get( location => "StaffSerialsHome", lang => lang, library => logged_in_user.branchcode ) -%]
>+                [%- SET StaffSerialsHome = AdditionalContents.get( location => "StaffSerialsHome", lang => lang, library => Branches.GetLoggedInBranchcode ) -%]
>                 [%- FOREACH block IN StaffSerialsHome.content -%]
>                 <div class="page-section">
>                     [%- block.content | $raw -%]
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
>index 25929c88e8..8a036da3e6 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/virtualshelves/shelves.tt
>@@ -547,7 +547,7 @@
>                     [% END # /WRAPPER tab_panels %]
>                 [% END # /WRAPPER tabs %]
> 
>-                [%- SET StaffListsHome = AdditionalContents.get( location => "StaffListsHome", lang => lang, library => logged_in_user.branchcode ) -%]
>+                [%- SET StaffListsHome = AdditionalContents.get( location => "StaffListsHome", lang => lang, library => Branches.GetLoggedInBranchcode ) -%]
>                 [%- FOREACH block IN StaffListsHome.content -%]
>                 <div class="page-section">
>                     [%- block.content | $raw -%]
>-- 
>2.39.2

Actions: View | Diff | Splinter Review

Attachments on bug 36083: 170612 | 170615 | 170616 | 170617 | 170618 | 171798 | 171799 | 171800 | 171801