ping.module patch (Druapl for Bloggers)

Setting status to 'patch' so this issue shows up in the patch queue.

Some verbiage problems on this one... My changes in brackets: "There are three different ways to notify different search engine[[[s]]] or websphere trackers. If you don't know what these mean[[[s; remove]]], then just leave it as-is. Pings are sen[[[t]]] out once every few hours, depending [[[on]]] how frequent you set your cron.php (see Section 6 on CRON TASKS in INSTALL). Pings will not be sen[[[t]]] unless you configure [[[your; remove]]] the [[[']]]name[[[']]] and 'slogan' of your site."

I also think we should remove, specifically, the words "Section 6" from the instructions. Leaving them in will merely cause problems when, and if, the documentation is revised, and the cron stuff becomes Section 5, or whatever.

The documentation mentions that the pings will not be sent unless the 'name' is configured, but the code doesn't seem to check for the default name of 'drupal'. In fact, it assigns the 'name' to 'drupal' as a default, and then /merely/ checks for the existence of the slogan. Why default to 'drupal' then? The name should default to nothing, and there should be an existence check on it.

Have you seen http://drupal.org/node/view/7905?

New features won't be added to 4.4. Please send new patch or close request.

No updated patch, several release cycles later, I think we have ourselves ANOTHER closed issue.