Port the MultiPing module to Drupal 6. Seeing how ping module is being removed from core, we should have a viable alternative to it in contrib. This means we'll have to update MultiPing to Drupal 6, and then port it to Drupal 7 when Drupal 7 starts being released. This way, people who use the Ping module will be able to move to MultiPing when they make the move to Drupal 7.

Comments

Skyr’s picture

Assigned: Unassigned » Skyr

These are my first efforts with Drupal 6... but I try to do a rough port of Multiping to Drupal 6. As soon as I get something done, I'll put it in the module's cvs repository.

Skyr’s picture

Assigned: Skyr » Unassigned

Ok, I think I've got something worth committing :)
The module doesn't cause any obvious errors when running in Drupal 6, but it has to be thoroughly tested. Perhaps someone with more D6 experience should have a glance at the code.

mcurry’s picture

subscribe

voxpelli’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev

There is still more work to do on this. Me and jonathan1055 are trying to find out what needs to be done, what has been done outside of Drupal.org and how to finally get this module officially stable for Drupal 6.

There's a follow up patch to this port in #517706: Code improvement patch, as well as some discussion by me and Jonathan on the future of the module, and there's a discussion regarding the future maintainer ship of Multiping in #537962: The future maintainership of Multiping

Skyr’s picture

At least on my test site, the dev version is working pretty nicely; I hesitated to make a release tag for two reasons:

  • There are pending patches which have to be included
  • I've heard of no success stories; it would be great to have some "works for me" confirmations before officially going green
jonathan1055’s picture

"works for me" ! Thanks.

Will post more later. I have lots of mods, but I think the first thing is to tidy up the code. I am going to review the patch(es) we have so far.

Jonathan

voxpelli’s picture

First beta released: http://drupal.org/node/866448

Hopefully we can have something good in time for Drupal 7.

jonathan1055’s picture

Excellent. I like the change to have separate multiping_ variables, instead of the fuss of converting all vars into one global variable stored as an array. Also using the proper system_settings_form. Well done.

mlncn’s picture

ping... per http://drupal.org/node/231437 this is the replacement for Ping module which was removed from core. It would be nice to have something to point people to, what are folks thoughts?

RobLoach’s picture

Title: Update MultiPing to Drupal 6 » Update MultiPing to Drupal 7
Component: Code » Miscellaneous

If anyone puts up a patch, for sure it'll be there.

RobLoach’s picture

Status: Active » Closed (duplicate)