Public/pics
2
2
Fork 2
Code Issues 6 Pull Requests 2 Releases Wiki Activity

Authentication: replace checkExists with Member::fromId

Browse Source
This commit is contained in:
Aaron van Geffen 2024-11-05 16:46:53 +01:00
parent 8eaeb6c332
commit 084658820e
1 changed files with 12 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
models/Authentication.php
View File

@ -12,22 +12,6 @@
*/ */
class Authentication class Authentication
{ {
/**
* Checks whether a user still exists in the database.
*/
public static function checkExists($id_user)
{
$res = Registry::get('db')->queryValue('
SELECT id_user
FROM users
WHERE id_user = {int:id}',
[
'id' => $id_user,
]);
return $res !== null;
}
/** /**
* Checks a password for a given username against the database. * Checks a password for a given username against the database.
*/ */
@ -78,8 +62,18 @@ class Authentication
*/ */
public static function isLoggedIn() public static function isLoggedIn()
{ {
// A user is logged in if a user id exists in the session and this id is (still) in the database. if (!isset($_SESSION['user_id']))
return isset($_SESSION['user_id']) && self::checkExists($_SESSION['user_id']); return false;
try
{
$exists = Member::fromId($_SESSION['user_id']);
return true;
}
catch (NotFoundException $e)
{
return false;
}
} }
/** /**