diff options
| author | Lester Caine <lester@lsces.co.uk> | 2026-05-10 16:20:42 +0100 |
|---|---|---|
| committer | Lester Caine <lester@lsces.co.uk> | 2026-05-10 16:20:42 +0100 |
| commit | ff94347235de0c063f19521bbb35a6725a687a8e (patch) | |
| tree | c23df5abfbcad5ad9506ed3ee5bcbd3e53c79b5e | |
| parent | b9f5c5ad7f6fc68b236af01f6cb886776714b800 (diff) | |
| download | protector-ff94347235de0c063f19521bbb35a6725a687a8e.tar.gz protector-ff94347235de0c063f19521bbb35a6725a687a8e.tar.bz2 protector-ff94347235de0c063f19521bbb35a6725a687a8e.zip | |
php8.5 tidies
| -rwxr-xr-x | includes/classes/LibertyProtector.php | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/includes/classes/LibertyProtector.php b/includes/classes/LibertyProtector.php index 28240c9..de19e00 100755 --- a/includes/classes/LibertyProtector.php +++ b/includes/classes/LibertyProtector.php @@ -48,7 +48,7 @@ class LibertyProtector extends LibertyBase { } } } - return count( $this->mErrors ) == 0; + return \count( $this->mErrors ) == 0; } /** @@ -88,7 +88,7 @@ function protector_content_list() { $ret = [ 'join_sql' => " LEFT JOIN `" . BIT_DB_PREFIX . "liberty_content_role_map` lcrm ON ( lc.`content_id`=lcrm.`content_id` ) LEFT OUTER JOIN `" . BIT_DB_PREFIX . "users_roles_map` purm ON ( purm.`user_id` = " . $userId . " ) AND ( purm.`role_id`=lcrm.`role_id` ) ", 'where_sql' => " AND (lcrm.`content_id` IS null OR lcrm.`role_id` IN(" . implode( ',', array_fill( 0, count( $roles ), '?' ) ) . " ) OR purm.`user_id` = ? ) ", - 'bind_vars' => array_merge( $roles, [ $userId ] ), + 'bind_vars' => [ ...$roles, $userId ], ]; return $ret; } @@ -100,15 +100,15 @@ function protector_content_list() { */ function protector_content_load( $pContent = null ) { global $gBitUser; - $userId = $gBitUser->mUserId ?? 0; - $roles = \array_keys($gBitUser->getRoles( $userId ?? 0, true )); + $userId = $gBitUser->mUserId ?? -1; + $roles = \array_keys($gBitUser->getRoles( $userId, true )); protector_content_verify_access( $pContent, $roles ); $ret = [ 'join_sql' => " LEFT JOIN `" . BIT_DB_PREFIX . "liberty_content_role_map` lcrm ON ( lc.`content_id`=lcrm.`content_id` ) LEFT OUTER JOIN `" . BIT_DB_PREFIX . "users_roles_map` purm ON ( purm.`role_id`=lcrm.`role_id` ) ", 'where_sql' => " AND (lcrm.`content_id` IS null OR lcrm.`role_id` IN(" . implode( ',', array_fill( 0, count( $roles ), '?' ) ) . " ) OR purm.`user_id`=?) ", 'bind_vars' => [ $userId ], ]; - $ret['bind_vars'] = array_merge( $roles, $ret['bind_vars'] ); + $ret['bind_vars'] = [ ...$roles, ...$ret['bind_vars'] ]; return $ret; } |