$user->isLoggedin() internal server error

[benbyf]

HELLO!

I'm using $user->isLoggedin() three times on one template (in the head.inc, template-page.php and foot.inc). I'm using a custom login in my template file:

// If they aren't logged in, then show the login form
if(!$user->isLoggedin()){

	// check for login before outputting markup
	if($input->post->user && $input->post->pass) {

		$user = $sanitizer->username($input->post->user);
		$pass = $input->post->pass;

		if($session->login($user, $pass)) {
			// login successful
			$session->redirect($homepage->url);
		} else {
			// login error
			$session->login_error = 'Login Failed. Please try again, or use the forgot password link below.';
		}
	}
	if($input->post->user && $input->post->pass) {
		$login = "<p class='error'>" . $session->login_error . "</p>";
	}
	$login .= "<form action='./' method='post'><div class='login'>
			<input type='text' id='user' name='user' placeholder='Enter your SP Company Code' />
			<input class='hidden' type='password' name='pass' value='' />
			<p><input type='submit' class='btn' name='submit' value='Login' /></p>
		</div></form>";

} // end !logged in

Using the correct logins works and everything is cool, but putting the wrong passwords in creates a Internal server error on both the head.inc and foot.inc $user->isLoggedin() functions.

I'm probably doing something absurdly stupid here, any ideas?

thanks

[clsource]

if($session->login($user, $pass)) {
            // login successful
            $session->redirect($homepage->url);
        } else {
            // login error
            $session->login_error = 'Login Failed. Please try again, or use the forgot password link below.';
        }

may be is causing an error

put them in a try catch block

try {
if($session->login($user, $pass)) {
			// login successful
			$session->redirect($homepage->url);
		} else {
			// login error
			
$session->login_error = 'Login Failed. Please try again, or use the forgot password link below.';
		}

} catch (Exception $e) {
   wire('log')->save('debug', $e->getMessage());
}

[benbyf]

error persists - it errors with the first call to $user->isLoggedin() (in the head.inc used to hide the nav if not logged in)

[DaveP]

This kind of error is usually because your code is overwriting the PW $user object. Try making your own user and password variable something like $u and $p, then there is no chance of that happening.

[benbyf]

That worked, I had $user overwritten, so i changed it for $u and $p