[wp-trac] [WordPress Trac] #49151: Show a warning for plugins in WP admin that haven't received updates in a long time
WordPress Trac
noreply at wordpress.org
Tue Jul 7 00:05:10 UTC 2020
#49151: Show a warning for plugins in WP admin that haven't received updates in a
long time
-------------------------------------+-------------------------------------
Reporter: vincenthasselgard | Owner: audrasjb
Type: feature request | Status: accepted
Priority: normal | Milestone: 5.5
Component: Plugins | Version:
Severity: normal | Resolution:
Keywords: has-screenshots has- | Focuses: ui, administration,
patch | ui-copy
-------------------------------------+-------------------------------------
Comment (by stuffradio):
Replying to [comment:23 elrae]:
> A few notes on the patch:
>
> - This introduces a new variable on line 835 $plugin_slug but I don't
see that variable actually being used within the foreach context. We
should either remove that new line or use it for its intended purpose
>
> - do we need a new function to get the plugin info? I thought there was
already a way to get all that information for plugins. If there isn't, an
minimum we should probably store this health data in a transient (with a
filter to allow custom plugins to tap into it) the same way the plugin
update transient works. This way you aren't hit with 60+ curl calls on
every page health load if you're loading it multiple times. Chances are if
you're visiting the health area it's to fix problems so you'll constantly
fix a problem, then reload to see if it's fixed.
Latest update fixes the fact that the variable on line 835 wasn't being
used anywhere.
For your second question/point, I would like feedback on how I should do
this. I was trying to figure out ways to get the plugin info easily. The
code I'm using comes from the file `wp-admin/includes/plugin-install.php`
and the function is `plugins_api()`. So if there's a way I could easily
use this and get the same result in my patch without rewriting everything,
I welcome suggestions. I will try and see if I can do this myself too.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/49151#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list