From 5f8299cf4af98c2075d6a18b5fec0c74414e91b5 Mon Sep 17 00:00:00 2001
From: Bernd Bestel <bernd@berrnd.de>
Date: Thu, 6 Jan 2022 13:46:20 +0100
Subject: [PATCH] Also show the logout button when using externally managed
 auth (fixes #1709)

---
 changelog/66_UNRELEASED_xxxx-xx-xx.md | 1 +
 views/layout/default.blade.php        | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/changelog/66_UNRELEASED_xxxx-xx-xx.md b/changelog/66_UNRELEASED_xxxx-xx-xx.md
index 9577e561..c4d9e610 100644
--- a/changelog/66_UNRELEASED_xxxx-xx-xx.md
+++ b/changelog/66_UNRELEASED_xxxx-xx-xx.md
@@ -5,3 +5,4 @@
 - Fixed that formatted (HTML) text for the (hidden by default) product description column on the stock overview page was not correctly displayed
 - Fixed that numeric and date-time sorting of table columns on the stock entries page did not work correctly (thanks @MasterofJOKers)
 - Fixed that the barcode lookup for the "Stock by-barcode" API endpoints was case sensitive
+- Fixed that the logout button/menu was missing when using external authentication (e.g. LDAP)
diff --git a/views/layout/default.blade.php b/views/layout/default.blade.php
index 4eb3fb8d..ff42a5ce 100644
--- a/views/layout/default.blade.php
+++ b/views/layout/default.blade.php
@@ -483,10 +483,10 @@
 					</a>
 
 					<div class="dropdown-menu dropdown-menu-right">
-						@if(!defined('GROCY_EXTERNALLY_MANAGED_AUTHENTICATION'))
 						<a class="dropdown-item logout-button discrete-link"
 							href="{{ $U('/logout') }}"><i class="fas fa-sign-out-alt"></i>&nbsp;{{ $__t('Logout') }}</a>
 						<div class="dropdown-divider"></div>
+						@if(!defined('GROCY_EXTERNALLY_MANAGED_AUTHENTICATION'))
 						<a class="dropdown-item logout-button discrete-link"
 							href="{{ $U('/user/' . GROCY_USER_ID . '?changepw=true') }}"><i class="fas fa-key"></i>&nbsp;{{ $__t('Change password') }}</a>
 						@else