diff options
| author | Lester Caine <lester@lsces.co.uk> | 2026-05-16 20:26:15 +0100 |
|---|---|---|
| committer | Lester Caine <lester@lsces.co.uk> | 2026-05-16 20:26:15 +0100 |
| commit | d9b30bccb967388a548528fe2bd940250f112599 (patch) | |
| tree | 6adb3510cd1c11c41e3c266b8b364a5122042d67 /includes | |
| parent | aed053fc746883bea02745c2c299c05e47a8a99f (diff) | |
| download | users-d9b30bccb967388a548528fe2bd940250f112599.tar.gz users-d9b30bccb967388a548528fe2bd940250f112599.tar.bz2 users-d9b30bccb967388a548528fe2bd940250f112599.zip | |
Correct logic around getRoles to correctly return just the roles a user is assigned to.
Diffstat (limited to 'includes')
| -rwxr-xr-x | includes/classes/RoleUser.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/includes/classes/RoleUser.php b/includes/classes/RoleUser.php index 98537ed..e1f2708 100755 --- a/includes/classes/RoleUser.php +++ b/includes/classes/RoleUser.php @@ -2679,8 +2679,8 @@ class RoleUser extends \Bitweaver\Liberty\LibertyMime { * @access public * @return int|array|bool false on failure - mErrors will contain reason for failure */ - public function getRoles( int $pUserId = -1, bool $pForceRefresh = false ) { - $userId = $pUserId ?? ( $this->mUserId ?? -1 ); + public function getRoles( ?int $pUserId = null, bool $pForceRefresh = false ) { + $userId = $pUserId ?? $this->mUserId ?? -1; if( !isset( $this->cUserRoles[$userId] ) || $pForceRefresh ) { $query = " SELECT ur.`role_id`, ur.`role_name`, ur.`user_id` as role_owner_user_id |