[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