WordPress.org

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#8781 closed feature request (fixed)

Plugins Dashboard Widget: Dont show installed plugins (or offer to install them)

Reported by: DD32 Owned by: Viper007Bond
Milestone: 2.8 Priority: normal
Severity: normal Version: 2.7
Component: Widgets Keywords: has-patch needs-testing
Focuses: Cc:

Description

As Chris Moody suggested: http://comox.textdrive.com/pipermail/wp-testers/2009-January/011011.html

Personally i think skipping the plugin if its already installed would be best, Not sure how much overhead it'd introduce matching the plugins up (Slug/Name in Widget vs. the actual plugins WP has installed vs. reading plugins from directory to get names, etc.)

Could possibly be a good candidate to introduce a cached option inside WP for a WP-Slug => Local Plugin file/Plugin Name lookup, Which would aid in displaying which plugins couldnt be automatically checked for updates too..

Attachments (9)

8781.patch (2.5 KB) - added by Viper007Bond 13 years ago.
8781.2.patch (4.9 KB) - added by Viper007Bond 13 years ago.
Cache slugs, avoid using get_plugins() on every page load
8781.3.patch (5.1 KB) - added by Viper007Bond 13 years ago.
Make sure we never infinite loop and don't autoload the option
8781.4.patch (5.1 KB) - added by Viper007Bond 13 years ago.
Code improvements care of rmccue and DD32
8781.5.patch (5.0 KB) - added by Viper007Bond 13 years ago.
More code streamlining care of rmccue
8781.6.patch (4.9 KB) - added by Viper007Bond 13 years ago.
More redundancy removal thanks to rmccue (why don't I think of these things when writing it?)
8781.7.patch (4.9 KB) - added by Viper007Bond 13 years ago.
Don't consolidate too much, we still need $plugin_slugs
8781.8.patch (4.9 KB) - added by Viper007Bond 13 years ago.
Refresh for 2.8 / SimplePie
8781.9.patch (3.6 KB) - added by Viper007Bond 13 years ago.
Use the new transient API instead of an option to cache the plugin slugs

Download all attachments as: .zip

Change History (16)

#1 @Viper007Bond
13 years ago

  • Owner set to Viper007Bond

@Viper007Bond
13 years ago

#2 @Viper007Bond
13 years ago

  • Keywords has-patch needs-testing added; needs-patch removed

Needs testing and the while() could be troublesome if it somehow gets stuck in an infinite loop, but I tested it on my localhost and it seems to work great. Akismet no longer shows up under "Most Popular" for example.

#3 @Viper007Bond
13 years ago

  • Component changed from Plugins to Administration

@Viper007Bond
13 years ago

Cache slugs, avoid using get_plugins() on every page load

@Viper007Bond
13 years ago

Make sure we never infinite loop and don't autoload the option

@Viper007Bond
13 years ago

Code improvements care of rmccue and DD32

@Viper007Bond
13 years ago

More code streamlining care of rmccue

@Viper007Bond
13 years ago

More redundancy removal thanks to rmccue (why don't I think of these things when writing it?)

@Viper007Bond
13 years ago

Don't consolidate too much, we still need $plugin_slugs

#4 @ryan
13 years ago

  • Component changed from Administration to Widgets

Needs update now that SimplePie has landed.

@Viper007Bond
13 years ago

Refresh for 2.8 / SimplePie

@Viper007Bond
13 years ago

Use the new transient API instead of an option to cache the plugin slugs

#5 @Viper007Bond
13 years ago

Should be good to go now. Installed a couple of the current "Most Popular" plugins and they stopped showing up in the list.

The transient API is really slick BTW. Gotta remember to make more use of that in the future. I set the caching time to 24 hours though, dunno if that's an acceptable cache time for caching the list of installed plugin slugs (it's updated on a visit to the plugins page anyway).

#6 @ryan
13 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [10738]) Don't show already installed plugins in the Plugins dashboard widget. Props Viper007Bond. fixes #8781

#7 @ryan
13 years ago

  • Milestone changed from 2.9 to 2.8
Note: See TracTickets for help on using tickets.