Make WordPress Core

Opened 13 years ago

Last modified 2 weeks ago

#24398 new defect (bug)

Bulk Deleting Sites from Network Admin, No Confirmation message

Reported by: professor44 Owned by:
Priority: normal Milestone:
Component: Networks and Sites Version: 3.1
Severity: normal Keywords: has-patch close
Cc: Focuses: multisite, administration

Description

When you delete a single site from /wp-admin/network/sites.php you're presented with the message "You are about to delete the site SITENAME" and you have to click "confirm" to complete the deletion of the site.

However, if you bulk delete sites (selecting 2 or more sites) from the same location you are not presented with a confirmation message. Instead, the sites immediately delete. This can be quite hazardous, especially if you're expecting to see a confirmation message.

Attachments (1)

24398.patch (2.3 KB ) - added by tubiz 12 years ago.

Download all attachments as: .zip

Change History (9)

#1 @noam@…
13 years ago

In the Multisite Network Admin (/wp-admin/network/sites.php), if you hover over a site and click, "Delete", the confirmation message successfully displays before the site is deleted.

However, if you use the Bulk Actions select menu to delete a site, there is no confirmation message. To be clear: there is no confirmation message when using Bulk Actions, regardless of if you are trying to delete a single site or multiple sites.

#2 @DrewAPicture
13 years ago

  • Cc xoodrew@… added
  • Keywords needs-patch added
  • Version 3.5.13.1

#3 @nacin
13 years ago

  • Component Network AdminNetworks and Sites
  • Focuses administration added

@tubiz
12 years ago

#4 @tubiz
12 years ago

Confirms all bulk actions (Delete, Mark as Spam and Not Spam)

Mark As Spam
https://i.cloudup.com/k1WV8f-NU9-3000x3000.png

Delete Sites
https://i.cloudup.com/DSyLCs28L3-3000x3000.png

Mark as Not Spam
https://i.cloudup.com/eQHb989Ei5-3000x3000.png

Last edited 12 years ago by tubiz (previous) (diff)

#5 @tubiz
12 years ago

  • Keywords has-patch added; needs-patch removed

This ticket was mentioned in IRC in #wordpress-dev by tubiz. View the logs.


12 years ago

#8 @realloc
2 weeks ago

  • Keywords close added

The delete part of this report was resolved in 4.9 via #41177. wp-admin/network/sites.php now renders a confirmation screen for the bulk "Delete" action, listing the affected sites and requiring a separate delete_sites request with an ms-delete-sites nonce before anything is removed.

The only remaining part of the attached patch concerns the "Mark as Spam" and "Not Spam" bulk actions, which still run without confirmation. Since both are reversible from the same screen, a confirmation step there seems hard to justify, so I would not keep this ticket open for it.

Note: See TracTickets for help on using tickets.