[wp-trac] [WordPress Trac] #38918: delete_plugins() needs a proper singular and plural form for its error message
WordPress Trac
noreply at wordpress.org
Thu Nov 24 01:29:43 UTC 2016
#38918: delete_plugins() needs a proper singular and plural form for its error
message
--------------------------------------+------------------------------
Reporter: johnbillion | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Plugins | Version: 2.6
Severity: normal | Resolution:
Keywords: good-first-bug has-patch | Focuses:
--------------------------------------+------------------------------
Changes (by eddhurst):
* keywords: good-first-bug has-patch 2nd-opinion => good-first-bug has-
patch
Comment:
Tested a couple of older versions to check against what they were doing
and in the eventuality that multiple plugins failed to delete, only the
last message would show anyway (deleted 3 plugins, but was shown the
following error):
`Plugin could not be deleted due to an error: Could not fully remove the
plugin: rest-api/plugin.php.`
Given that all these errors have been brought in-line anyway now, each
error will show next to the relevant plugin that's failed (if multiple),
so a singular phrase seems fine.
I've added an extra patch to force errors on successfully removing a
plugin, just to make testing the output a little easier - the initial
patch is the correct one to apply going forward.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/38918#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list