X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmysql-manager.php;h=91a16286fef6f3f40c91ba36c4fdb724377f4bc1;hp=f6bc8baaeca843f8454a8a61ddd702ffc50030a0;hb=2763c536e15b838a5df284bca71b7430870edc47;hpb=c744215dba419b52b84291622179c5e3a6ab17e2

diff --git a/inc/mysql-manager.php b/inc/mysql-manager.php
index f6bc8baaec..91a16286fe 100644
--- a/inc/mysql-manager.php
+++ b/inc/mysql-manager.php
@@ -368,58 +368,53 @@ function isMember () {
 	// @TODO Try to rewrite this to one or more functions
 	if ((!isset($GLOBALS['last_online'])) || (!is_array($GLOBALS['last_online']))) $GLOBALS['last_online'] = array();
 
-	// is the cache entry there?
+	// Is the cache entry there?
 	if (isset($GLOBALS[__FUNCTION__])) {
 		// Then return it
 		//* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'CACHED! (' . intval($GLOBALS[__FUNCTION__]) . ')');
 		return $GLOBALS[__FUNCTION__];
 	} elseif ((!isSessionVariableSet('userid')) || (!isSessionVariableSet('u_hash'))) {
-		// No member
+		// Destroy any existing user session data
+		destroyMemberSession();
 		//* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'No member set in cookie/session.');
+
+		// Abort further processing
 		return false;
-	} else {
-		// Get it secured from session
-		setMemberId(getSession('userid'));
-		setCurrentUserId(getMemberId());
-		//* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'userid=' . getSession('userid') . ' used from cookie/session.');
 	}
 
+	// Get userid secured from session
+	setMemberId(getSession('userid'));
+
+	// ... and set it as currently handled user id
+	setCurrentUserId(getMemberId());
+
 	// Init user data array
 	initUserData();
 
 	// Fix "deleted" cookies first
 	fixDeletedCookies(array('userid', 'u_hash'));
 
-	// Are cookies set?
-	if ((isMemberIdSet()) && (isSessionVariableSet('u_hash'))) {
-		// Cookies are set with values, but are they valid?
-		if (fetchUserData(getMemberId()) === true) {
-			// Validate password by created the difference of it and the secret key
-			$valPass = encodeHashForCookie(getUserData('password'));
+	// Are cookies set and can the member data be loaded?
+	if ((isMemberIdSet()) && (isSessionVariableSet('u_hash')) && (fetchUserData(getMemberId()) === true)) {
+		// Validate password by created the difference of it and the secret key
+		$valPass = encodeHashForCookie(getUserData('password'));
 
+		// So did we now have valid data and an unlocked user?
+		if ((getUserData('status') == 'CONFIRMED') && ($valPass == getSession('u_hash'))) {
 			// Transfer last module and online time
 			$GLOBALS['last_online']['module'] = getUserData('last_module');
 			$GLOBALS['last_online']['online'] = getUserData('last_online');
 
-			// So did we now have valid data and an unlocked user?
-			if ((getUserData('status') == 'CONFIRMED') && ($valPass == getSession('u_hash'))) {
-				// Account is confirmed and all cookie data is valid so he is definely logged in! :-)
-				$ret = true;
-			} else {
-				// Maybe got locked etc.
-				//* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'status=' . getUserData('status') . ',' . $valPass . '(' . strlen($valPass) . ')/' . getSession('u_hash') . '(' . strlen(getSession('u_hash')) . ')/' . getUserData('password') . '(' . strlen(getUserData('password')) . ')');
-				destroyMemberSession();
-			}
-		} else {
-			// Cookie data is invalid!
-			//* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Cookie data invalid or user not found.');
-			destroyMemberSession();
-		}
-	} else {
-		// Cookie data is invalid!
-		//* DEBUG: */ logDebugMessage(__FUNCTION__, __LINE__, 'Cookie data not complete.');
+			// Account is confirmed and all cookie data is valid so he is definely logged in! :-)
+			$ret = true;
+		} // END - if
+	} // END - if
+
+	// Is $ret still false?
+	if ($ret === false) {
+		// Yes, so destroy the session
 		destroyMemberSession();
-	}
+	} // END - if
 
 	// Cache status
 	$GLOBALS[__FUNCTION__] = $ret;