Document new update manager hooks

Shouldn't this be included in the original issue report, rather than reporting as a separate issue?

I guess that other issue was too big so they closed it down and split it up.

Anyway, this is now part of the hooks meta-issue, and it's a bug report if this doc is missing (which it is).
#675046: Make sure all hooks in D7 have documentation

OK here's a start. The biggest problem I have is in hook_verify_update_archive(). So in #124 of the original issue, this is added with mention of it being used in contrib for things like code signing. However it is not anywhere in core (that I could find), so there's no easy place to grab a sample implementation from. Additionally, the calling code update_manager_archive_verify() indicates this should return $failures, but doesn't indicate what kind of data that is. It is checking !empty(), which seems weird, like it expects an array of some kind. If dww can provide some clarity there, that would be really helpful.

Otherwise the other two hooks seem pretty straightforward.

A few small style/punctuation changes:

+ * Provider information on updaters (classes that can update Drupal.)

--> Provide information on updaters (classes that can update Drupal).

+ *   This array can contain the following keys.

keys. -> keys:

+ * @return $failures

We generally don't put a $variable on the line with @return.

Anyway, it looks like we need dww or someone else who understands hook_verify_update_archive() to let us know what should be in that function doc and its example function body. The rest looks good to me so far.

Here's a new patch. I figured out what was going on and fixed up the documentation.

Another patch just got committed - here's a reroll that should work...

that has been download.

download -> downloaded
(probably came from my patch in the first place?)

Otherwise, looks fine to me. I'll just make that change...

(better wait for test bot though in case I screwed it up)

+++ modules/system/system.api.php	20 Apr 2010 21:24:29 -0000
@@ -3219,6 +3222,66 @@ function hook_token_info_alter(&$data) {
+ * Alter the Updater information array. ¶

Trailing white space

98 critical left. Go review some!

Looks good to me...

Committed to HEAD! Thanks!