Context Navigation

← Previous Ticket
Next Ticket →

#28529 closed defect (bug) (fixed)

Remove bulk action checkbox for Super Admin

Reported by:	UmeshSingla	Owned by:	SergeyBiryukov
Priority:	normal	Milestone:	4.4
Component:	Users	Version:	3.9
Severity:	normal	Keywords:	has-patch commit
Cc:		Focuses:	administration, multisite

Description

The Bulk action checkbox serves no purpose for Super Admins in a multisite or admin in a normal site.

The only action allowed for Superadmin in a multisite is to mark it as not spam, which is doubtful in its place, Why should a spam user be allowed to get Superadmin privileges in first place?

I suggest to add a filter with default value as false to hide the bulk action checkbox for Superadmins or just not show it permanently without any filter.

If it is suppose to be considered, I can submit a patch

Attachments (2)

28529.png (34.4 KB ) - added by jeremyfelt 11 years ago.
28529.1.diff (773 bytes ) - added by bordoni 11 years ago.: Version 01

Download all attachments as: .zip

Change History (9)

#1 @Clorith
12 years ago

Focuses multisite added
Version trunk → 3.9

Which screen are you referencing here that only has a mark as not spam option?

#2 @Clorith
12 years ago

Keywords reporter-feedback added

@jeremyfelt
11 years ago

Attachment 28529.png added

#3 @jeremyfelt
11 years ago

Keywords needs-patch added; reporter-feedback removed
Milestone Awaiting Review → Future Release

This makes sense. We do the same thing with a network's main site right now in the sites list table. If we aren't going to allow the action here, we can make it easier not to accidentally select it.

#4 @jeremyfelt
11 years ago

Milestone Future Release → 4.4
Type enhancement → defect (bug)

#5 @bordoni
11 years ago

Keywords has-patch dev-feedback added; needs-patch removed

I create a solution that removes the Checkbox, attached the .diff

PS: Not 100% sure if I should add dev-feedback, If I did it wrong, please let me know for the next time.

Last edited 11 years ago by bordoni (previous) (diff)

@bordoni
11 years ago

Attachment 28529.1.diff added

Version 01

#6 @SergeyBiryukov
11 years ago

Keywords commit added; dev-feedback removed

Looks good.

#7 @SergeyBiryukov
11 years ago

Owner set to SergeyBiryukov
Resolution → fixed
Status new → closed

In 33777:

Network Admin: Hide the bulk actions checkbox for super admins.

See [25125] for hiding the checkbox for the main site in Sites table.

props bordoni.
fixes #28529.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#28529 closed defect (bug) (fixed)

Remove bulk action checkbox for Super Admin

Description

Attachments (2)

Change History (9)

#1 @Clorith 12 years ago

#2 @Clorith 12 years ago

@jeremyfelt 11 years ago

#3 @jeremyfelt 11 years ago

#4 @jeremyfelt 11 years ago

#5 @bordoni 11 years ago

@bordoni 11 years ago

#6 @SergeyBiryukov 11 years ago

#7 @SergeyBiryukov 11 years ago