From 47ed6a5135dd1c6bc01169f430ebb243e29f2694 Mon Sep 17 00:00:00 2001
From: Andreas Fischer <bantu@owncloud.com>
Date: Wed, 2 Oct 2013 15:31:46 +0200
Subject: [PATCH] Move backend finding into its own method.

---
 lib/private/user.php | 42 ++++++++++++++++++++++++++----------------
 1 file changed, 26 insertions(+), 16 deletions(-)

diff --git a/lib/private/user.php b/lib/private/user.php
index 0284e8044f..04cd06b08b 100644
--- a/lib/private/user.php
+++ b/lib/private/user.php
@@ -247,18 +247,15 @@ class OC_User {
 	 *          null: not handled / no backend available
 	 */
 	public static function handleApacheAuth() {
-		foreach (self::$_usedBackends as $backend) {
-			if ($backend instanceof OCP\Authentication\IApacheBackend) {
-				if ($backend->isSessionActive()) {
-					OC_App::loadApps();
+		$backend = self::findFirstActiveUsedBackend();
+		if ($backend) {
+			OC_App::loadApps();
 
-					//setup extra user backends
-					self::setupBackends();
-					self::unsetMagicInCookie();
+			//setup extra user backends
+			self::setupBackends();
+			self::unsetMagicInCookie();
 
-					return self::loginWithApache($backend);
-				}
-			}
+			return self::loginWithApache($backend);
 		}
 
 		return null;
@@ -319,12 +316,9 @@ class OC_User {
 	 * @return string with one or more HTML attributes.
 	 */
 	public static function getLogoutAttribute() {
-		foreach (self::$_usedBackends as $backend) {
-			if ($backend instanceof OCP\Authentication\IApacheBackend) {
-				if ($backend->isSessionActive()) {
-					return $backend->getLogoutAttribute();
-				}
-			}
+		$backend = self::findFirstActiveUsedBackend();
+		if ($backend) {
+			return $backend->getLogoutAttribute();
 		}
 
 		return "href=" . link_to('', 'index.php') . "?logout=true";
@@ -568,4 +562,20 @@ class OC_User {
 	public static function unsetMagicInCookie() {
 		self::getUserSession()->unsetMagicInCookie();
 	}
+
+	/**
+	 * @brief Returns the first active backend from self::$_usedBackends.
+	 * @return null if no backend active, otherwise OCP\Authentication\IApacheBackend
+	 */
+	private static function findFirstActiveUsedBackend() {
+		foreach (self::$_usedBackends as $backend) {
+			if ($backend instanceof OCP\Authentication\IApacheBackend) {
+				if ($backend->isSessionActive()) {
+					return $backend;
+				}
+			}
+		}
+
+		return null;
+	}
 }