fix: Return a logout URL even when SAML SLO is not configured

Signed-off-by: Carl Schwan <carlschwan@kde.org>

Author: CarlSchwan

core/AppInfo/Application.php

@@ -99,5 +99,4 @@ public function register(IRegistrationContext $context): void {
 	public function boot(IBootContext $context): void {
 		// ...
 	}
-
 }

lib/private/NavigationManager.php

@@ -282,17 +282,15 @@ private function init(bool $resolveClosures = true): void {
 			}
 
 			$logoutUrl = \OC_User::getLogoutUrl($this->urlGenerator);
-			if ($logoutUrl !== '') {
-				// Logout
-				$this->add([
-					'type' => 'settings',
-					'id' => 'logout',
-					'order' => 99999,
-					'href' => $logoutUrl,
-					'name' => $l->t('Log out'),
-					'icon' => $this->urlGenerator->imagePath('core', 'actions/logout.svg'),
-				]);
-			}
+			// Logout
+			$this->add([
+				'type' => 'settings',
+				'id' => 'logout',
+				'order' => 99999,
+				'href' => $logoutUrl,
+				'name' => $l->t('Log out'),
+				'icon' => $this->urlGenerator->imagePath('core', 'actions/logout.svg'),
+			]);
 
 			if ($this->isSubadmin()) {
 				// User management

lib/private/legacy/OC_User.php

@@ -277,6 +277,7 @@ public static function isIncognitoMode(): bool {
 
 	/**
 	 * Returns the current logout URL valid for the currently logged-in user
+	 * @return non-empty-string
 	 */
 	public static function getLogoutUrl(IURLGenerator $urlGenerator): string {
 		$backend = self::findFirstActiveUsedBackend();
@@ -288,7 +289,10 @@ public static function getLogoutUrl(IURLGenerator $urlGenerator): string {
 		if ($user instanceof IUser) {
 			$backend = $user->getBackend();
 			if ($backend instanceof ICustomLogout) {
-				return $backend->getLogoutUrl();
+				$logoutUrl = $backend->getLogoutUrl();
+				if ($logoutUrl !== '') {
+					return $logoutUrl;
+				}
 			}
 		}