Plugin update notification should generate bubble like comment notification bubble

[mdawaffe]

Attached fires the plugin update notification script more frequently and adds a little bubble on the plugins menu item when there are plugins ready to be updated.

Tested in FF3 and Safari only.

[mdawaffe]

on every page load

[matt]

(In [8305]) Check for plugin updates more frequently, notify with double bubble trouble. Hat tip: mdawaffe. Fixes #7265.

[DD32]

The action 'init' is fired before any administration includes are included, As such, The plugin update check is not run on any pages within the admin area.

One work around is to hook the check onto both init and admin_init, as admin_init runs after the admin includes have been included.

The other is to include admin includes before init is run.

See the 2 patches for the possible ways

[DD32]

Er, Actually..

Theres no point hooking the plugin check to init at all, The code is not loaded for the WordPress front end, Switching it to hook admin_init only is the better solution. (7265.admin_init.diff)

[DD32]

Or theres the 4th option of actually moving the plugin check into the front-end loaded code.. Which my patch does, My reasoning is the same as for the Core update checker, So that plugins may notify the blog owner of an update.

[DD32]

Also included a grammer correction to the PHPDoc..

strpos($_SERVER['PHP_SELF'], 'install.php') !== false

Removed to allow admin pages with 'install.php' in them to be able to trigger update checks as well (eg. 'plugin-install.php' which is similar to #6990 - WP_INSTALLING is defined anyway.

[DD32]

Eugh, Its late, Didnt notice function dependancies on get_plugins().

Honestly, I dont see the need for either of the update checks to be be run on non-admin hooks..

Final patch.. Someone else can decide on which route to take :)

[mdawaffe]

I think both core and plugin checks should run blog-side; I agree with DD32's original rationale: a plugin (or future core) could send emails or something when there are updates.

So I'd go +1 on .3.

And a +1000 to DD32 for catching the bug :)

[mdawaffe]

Messed up the attachment name, but 7265.5.2.diff:

Same as DD32's .3, but checks to see if is_admin and calls the appropriate hook. If blog-side, includes wp-admin/include/plugin.php (only if it hasn't checked in the last 12 hours) and goes from there.

Changing back to admin_init hook is definitely easier, but I think it cripples the feature a bit.

[ryan]

(In [8317]) Run plugin checks both admin and blog side. Props DD32 and mdawaffe. fixes #7265

[DD32]

Thanks for fixing up the code mdawaffe :)