Context Navigation

← Previous Ticket
Next Ticket →

#18164 closed defect (bug) (fixed)

Users with capes don't need promote_users on their new role

Reported by:	nacin	Owned by:	nacin
Priority:	normal	Milestone:	3.3
Component:	Network Admin	Version:
Severity:	normal	Keywords:	has-patch
Cc:		Focuses:	multisite

Description

When setting your own role:

		// The new role of the current user must also have promote_users caps
		if ( $id == $current_user->ID && !$wp_roles->role_objects[$_REQUEST['new_role']]->has_cap('promote_users') ) {
			$update = 'err_admin_role';
			continue;
		}

There's no reason to prevent a super admin from demoting themselves on a site.

Attachments (2)

18164.diff (793 bytes ) - added by fonglh 15 years ago.
18164.2.diff (792 bytes ) - added by fonglh 15 years ago.

Download all attachments as: .zip

Change History (10)

#1 @johnbillion
15 years ago

How does one get a cape?

#2 @zippykid
15 years ago

Step 1 - Visit http://www.superherosupplies.com/

Step 2 - Add cape to cart

Step 3 - Go through check out procedure

Step 4 - Wait for delivery truck

Step 5 - Receive and Open package - find out it's shoes your wife ordered, go back outside to wait for delivery truck

Step 6 - Receive package - open - find cape

Step 7 - Wear cape.

Step 8 - Commence core hacking.

Last edited 15 years ago by zippykid (previous) (diff)

#3 @fonglh
15 years ago

Keywords has-patch added

Checks for cape before deciding if user can demote himself.

#4 @nacin
15 years ago

The more generic is_super_admin() would probably be a better fit here.

@fonglh
15 years ago

Attachment 18164.diff added

@fonglh
15 years ago

Attachment 18164.2.diff added

#5 @fonglh
15 years ago

Thanks for the tip! I've updated the patch to use is_super_admin()

#6 @nacin
15 years ago

Owner set to nacin
Resolution → fixed
Status new → closed

In [19024]:

Allow a super admin to demote themselves on a site to a role that does not contain the promote_users cap. props fonglh, fixes #18164.

#7 @nacin
15 years ago

Thanks for the patch, fonglh!

#8 @fonglh
15 years ago

No problem! Glad to have been able to make my first contribution to core.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#18164 closed defect (bug) (fixed)

Users with capes don't need promote_users on their new role

Description

Attachments (2)

Change History (10)

#1 @johnbillion 15 years ago

#2 @zippykid 15 years ago

#3 @fonglh 15 years ago

#4 @nacin 15 years ago

@fonglh 15 years ago

@fonglh 15 years ago

#5 @fonglh 15 years ago

#6 @nacin 15 years ago

#7 @nacin 15 years ago

#8 @fonglh 15 years ago

Download in other formats:

#1 @johnbillion
15 years ago

#2 @zippykid
15 years ago

#3 @fonglh
15 years ago

#4 @nacin
15 years ago

@fonglh
15 years ago

@fonglh
15 years ago

#5 @fonglh
15 years ago

#6 @nacin
15 years ago

#7 @nacin
15 years ago

#8 @fonglh
15 years ago