[wp-trac] [WordPress Trac] #38112: Deleting an Active Plugin via Bulk Actions Fails Silently

[WordPress Trac]

#38112: Deleting an Active Plugin via Bulk Actions Fails Silently
-------------------------------------------------+-------------------------
 Reporter:  mrwweb                               |       Owner:  swissspidy
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  Future
Component:  Plugins                              |  Release
 Severity:  normal                               |     Version:
 Keywords:  needs-testing shiny-updates needs-   |  Resolution:
  patch                                          |     Focuses:
-------------------------------------------------+-------------------------
Changes (by swissspidy):

 * keywords:  has-patch needs-testing shiny-updates => needs-testing shiny-
     updates needs-patch
 * milestone:  4.7 => Future Release


Comment:

 I couldn't reproduce the `undefined` bug and I don't know why deleting a
 network enabled theme is even possible.

 [attachment:38112.2.diff] changes the selector in
 `wp.updates.deletePluginError` to show the bulk admin notice ''and'' the
 inline notice. This makes the bug at least more obvious for both screens
 during testing.
 Unfortunately, I haven't found a good solution yet to hide inline admin
 notices during a bulk action.  The bulk handler would need to pass custom
 success & error callbacks instead of hooking into the triggers inside
 `wp.updates.deletePluginError()` et al. Ideally, the inline notices should
 not be added in `wp.updates.deletePluginError()` directly, but also via
 the hook.

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