Wrong variable name in hook_menu_delete documentation

Good catch!

While we're fixing this, first line should be changed from "Inform modules that..." to say "Respond to...".
http://drupal.org/node/1354#hooks

This got fixed for d8 in #1814916: Convert menus into entities

D7 patch for #1.

There is still a problem in D8 that needs to be addressed (which is addressed in this d7 patch, but we need to fix d8 before fixing d7): the first lines of all the hook docs in menu.api.php are incorrect. So can we get a patch for d8 that fixes them as in the existing d7 patch? Thanks!

Oh actually, the d7 patch says things like:

Respond to a custom menu was created.

That isn't very good English. How about "Respond to custom menu creation"? And similar for the other hooks.

oops,seems i missed #1..sorry for the noise.restoring tag as well

Ok. I will follow up on this.

This D8 patch corrects the subject line.

Thanks! Please change "Responds" to "Respond" -- see
http://drupal.org/node/1354#hooks

Hi, the correction is included in this version.

Looks good! I'll get this committed to 8.x shortly, and then we can return to the 7.x patch.

Thanks again -- Committed the patch in #10 to Drupal 8.

Now we can return to the patch in #3 for Drupal 7, which needs the first lines of the hook documentation to look like the patch in #10.

How about this? See patch...

Hi, -13 patch needs another fix in hook_menu_delete. You can look at -1.patch. This @link parameter is fixed in D8. This is why it is not included in the D8 patch.

Added that change. How about now?

Looks good -- thanks! I'll get this committed shortly.

Thanks again -- this is committed to 7.x.