From 095146c5243397b8c7de893b374f813b7352cb43 Mon Sep 17 00:00:00 2001
From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Date: Thu, 26 Mar 2020 13:49:57 +0000
Subject: [PATCH] Bug 24201: (follow-up) Use GetLoggedInDeskName

This patch updates the templates to use the new TT plugin instead of
adding additional variables to the already crouded get_template_and_user.

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>

Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
---
 C4/Auth.pm                                          |  3 ---
 Koha/Template/Plugin/Desks.pm                       | 16 ++++++++++++++++
 koha-tmpl/intranet-tmpl/prog/en/includes/header.inc | 10 +++++-----
 3 files changed, 21 insertions(+), 8 deletions(-)

diff --git a/C4/Auth.pm b/C4/Auth.pm
index 35b794c3a5..e8af5389af 100644
--- a/C4/Auth.pm
+++ b/C4/Auth.pm
@@ -462,7 +462,6 @@ sub get_template_and_user {
         OpenLibraryCovers                                                  => C4::Context->preference("OpenLibraryCovers"),
         KohaAdminEmailAddress                                              => "" . C4::Context->preference("KohaAdminEmailAddress"),
         LoginBranchcode => ( C4::Context->userenv ? C4::Context->userenv->{"branch"}    : undef ),
-        LoginDeskname                                                      => ( C4::Context->userenv ? C4::Context->userenv->{"desk_name"} : undef ),
         LoginFirstname  => ( C4::Context->userenv ? C4::Context->userenv->{"firstname"} : "Bel" ),
         LoginSurname    => C4::Context->userenv ? C4::Context->userenv->{"surname"}      : "Inconnu",
         emailaddress    => C4::Context->userenv ? C4::Context->userenv->{"emailaddress"} : undef,
@@ -491,7 +490,6 @@ sub get_template_and_user {
             IntranetmainUserblock                                                      => C4::Context->preference("IntranetmainUserblock"),
             LibraryName                                                                => C4::Context->preference("LibraryName"),
             LoginBranchname                                                            => ( C4::Context->userenv ? C4::Context->userenv->{"branchname"} : undef ),
-            LoginDeskname                                                              => ( C4::Context->userenv ? C4::Context->userenv->{"desk_name"} : undef ),
             advancedMARCEditor                                                         => C4::Context->preference("advancedMARCEditor"),
             canreservefromotherbranches                                                => C4::Context->preference('canreservefromotherbranches'),
             intranetcolorstylesheet                                                    => C4::Context->preference("intranetcolorstylesheet"),
@@ -560,7 +558,6 @@ sub get_template_and_user {
             LibraryName                           => "" . C4::Context->preference("LibraryName"),
             LibraryNameTitle                      => "" . $LibraryNameTitle,
             LoginBranchname                       => C4::Context->userenv ? C4::Context->userenv->{"branchname"} : "",
-            LoginDeskname                         => C4::Context->userenv ? C4::Context->userenv->{"desk_name"} : "",
             OPACAmazonCoverImages                 => C4::Context->preference("OPACAmazonCoverImages"),
             OPACFRBRizeEditions                   => C4::Context->preference("OPACFRBRizeEditions"),
             OpacHighlightedWords                  => C4::Context->preference("OpacHighlightedWords"),
diff --git a/Koha/Template/Plugin/Desks.pm b/Koha/Template/Plugin/Desks.pm
index 99fe2329e3..a3be8c365f 100644
--- a/Koha/Template/Plugin/Desks.pm
+++ b/Koha/Template/Plugin/Desks.pm
@@ -67,6 +67,22 @@ sub GetLoggedInDeskId {
         '';
 }
 
+=head3 GetLoggedInDeskName
+
+[% Desks.GetLoggedInDeskName %]
+
+Return the desk name that is attached to the session or empty string
+
+=cut
+
+sub GetLoggedInDeskName {
+    my ($self) = @_;
+
+    return C4::Context->userenv ?
+        C4::Context->userenv->{'desk_name'} :
+        '';
+}
+
 =head3 all
 
 [% Desks.all %]
diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc
index 2af835004c..37b16132c8 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/header.inc
@@ -89,10 +89,10 @@
                         [% IF (Desks.defined) %]
                             <span class="separator">|</span>
                             <strong>
-                                [% IF ( LoginDeskname == '') %]
+                                [% IF ( Desks.GetLoggedInDeskName == '' ) %]
                                    <span class="logged-in-desk-name">NO DESK SET</span>
                                 [% ELSE %]
-                                    <span class="logged-in-desk-name">[% LoginDeskname | html %]</span>
+                                    <span class="logged-in-desk-name">[% Desks.GetLoggedInDeskName | html %]</span>
                                     <span class="logged-in-desk-id content_hidden">[% Desks.GetLoggedInDeskId | html %]</span>
                                 [% END %]
                             </strong>
@@ -119,10 +119,10 @@
                     </li>
                     <li class="loggedin-menu-label">
                         Desk: <br />
-                        [% IF ( LoginDeskname == '' AND Desks.defined ) %]
+                        [% IF ( Desks.GetLoggedInDeskName == '' AND Desks.defined ) %]
                            <span class="logged-in-desk-name">NO DESK SET</span>
-                        [% ELSIF ( LoginDeskname != '' ) %]
-                            <span class="logged-in-desk-name">[% LoginDeskname | html %]</span>
+                        [% ELSIF ( Desks.GetLoggedInDeskName != '' ) %]
+                            <span class="logged-in-desk-name">[% Desks.GetLoggedInDeskName | html %]</span>
                             <span class="logged-in-desk-id content_hidden">[% Desks.GetLoggedInDeskId | html %]</span>
                         [% END %]
                     </li>
-- 
2.11.0