Module "ping.module" extended, reorganised and partly recoded !

Well, since you emailed me:

1) We prefer patches to posting whole modules
2) You need to conform to the coding standards
3) Can't you do everything you intend to do just by implementing hook_ping in a module and sending the pings you want? Basically what you need is a module, say, 'mega_ping' and it implements 'mega_ping_ping()' which then does all of the pings you list here.

Additionally, it seems like it would be better to have an array:

  function mega_ping_services() {
    return array(
      "http://ping.blo.gs/",
      "http://blogsearch.google.com/ping/RPC2",
      ...
    );
  }

  function mega_ping_ping() {
    foreach (mega_ping_services() as $service) {
      ... do ping ...
    }
  }

There is already a patch which allows admins to specify the sites that should be notified: 11148. Does this version add anything else that warrants another patch?

@MobilePublisher: Yes, ping.module has a ping_cron(). The patch doesn't remove it.

Make sure you use the patch in comment #8, the other ones are for Drupal 4.6/4.7. It may be that it doesn't apply any more because of other changes to the module. Haven't tried it recently.

I mentioned the other patch only to avoid duplicate work. There is already a solution for letting admins add their own ping services. So my question is, did you change anything else in ping.module that should also be taken into account?

You should also know that neither your version or mine will result in a new ping.module in Drupal 5 or 6 as these versions are closed for new features. If you want to see a new version included in Drupal, you should aim for Drupal 7. Or you could release it as a contributed module with a different name than Ping.

Ping module was removed from core : #231437: Remove ping module from core