From e2f4b2e01f50071f6b812bdfdcdd8b7a03b40070 Mon Sep 17 00:00:00 2001 From: Galen Charlton Date: Wed, 19 Feb 2014 16:40:06 +0000 Subject: [PATCH] Bug 11779: (follow-up) improve GetLoggedInBranchcode() and add test cases This patch adjusts the new GetLoggedInBranchcode() template function so that it returns the empty string rather than undef if there is no active user environment. That way, there won't be lots of undefined value warnings if/when this function gets used in the OPAC. This patch also adds test cases. To test: [1] Verify that there are no regressions in the main test plan for this bug. [2] Verify that prove -v t/db_dependent/Koha_template_plugin_Branches.t passes. Signed-off-by: Galen Charlton (cherry picked from commit 065a3a5efa9b90723460a607ac85591c17945b00) Signed-off-by: Fridolin Somers Signed-off-by: Kyle M Hall (cherry picked from commit 1b02c031a34e7e3e3caabbe0184fb5fe71447d95 ) --- Koha/Template/Plugin/Branches.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Koha/Template/Plugin/Branches.pm b/Koha/Template/Plugin/Branches.pm index 0031bfe89d..04ec4140c8 100644 --- a/Koha/Template/Plugin/Branches.pm +++ b/Koha/Template/Plugin/Branches.pm @@ -39,7 +39,9 @@ sub GetName { sub GetLoggedInBranchcode { my ($self) = @_; - return C4::Context->userenv->{'branch'} if C4::Context->userenv; + return C4::Context->userenv ? + C4::Context->userenv->{'branch'} : + ''; } 1; -- 2.39.5