Opened 3 years ago

Closed 3 years ago

#11808 closed defect (bug) (fixed)

is_super_admin() should try to get missing user_id

Reported by: filosofo Owned by: ryan
Priority: normal Milestone: 3.0
Component: Role/Capability Version: 3.0
Severity: normal Keywords: has-patch commit multisite
Cc:

Description

If its user_id argument is missing, is_super_admin() should assume we want the current user's ID and try to get that before returning false.

Attachments (1)

is_super_admin-get_userid.diff (714 bytes) - added by filosofo 3 years ago.

Download all attachments as: .zip

Change History (5)

To clarify, instead of checking the global $current_user directly is_super_admin() should rely on the current user API as the functions used are pluggable and thus they might not be setting $current_user in predictable ways.

  • Keywords commit added; is_super_admin removed

Gets my vote.

Changed patch to use ! empty instead of isset since $current_user can get set to false instead of being an object.

comment:4   ryan3 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.