[wp-trac] [WordPress Trac] #40821: Add filter to `wpmu_delete_blog()` to overwrite ability to drop tables

WordPress Trac noreply at wordpress.org
Fri May 19 18:53:48 UTC 2017


#40821: Add filter to `wpmu_delete_blog()` to overwrite ability to drop tables
--------------------------------+------------------------------
 Reporter:  mermel              |       Owner:
     Type:  enhancement         |      Status:  new
 Priority:  normal              |   Milestone:  Awaiting Review
Component:  Networks and Sites  |     Version:  trunk
 Severity:  normal              |  Resolution:
 Keywords:                      |     Focuses:  multisite
--------------------------------+------------------------------

Comment (by flixos90):

 I'm curious about a use-case for this. When would you want to deny
 dropping tables even though it is set? I'm not opposed to the filter, but
 would like to get some background.

 Also, as a preliminary reminder, we should consider using the current
 naming conventions for sites, networks and functions when thinking about a
 name for the filter, especially as we're planning to work on new improved
 functions for the functionality (in #40364 and some not-yet-created
 tickets that were briefly discussed during the multisite chat recently). I
 don't think we should use the `wpmu_` prefix for example. Might be weird
 to not have it there at the moment, but long-term it should make sense: If
 we have a function like `wp_drop_site_tables()` and the filter had a
 `wpmu_` prefix, it would be worse in my opinion.

 Also sorry for diving into detail too early. :)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40821#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list