Scheduler module: rewritten

Submitted by ted on 22 July 2006 - 7:36pm

On the long plane ride home from London today (I was out there building the new MTV.co.uk site that is going live early August 2006, see some pictures), I decided to completely rewrite the scheduler module. I based the rewrite upon an earlier article I wrote about the TWiT.tv site where I was using a simple publishing nodes technique.

I talked with Gerhard Killesreiter who maintains the Drupal CVS repository and he said the scheduler module was mostly abandoned and needed some love. So I stepped up to the plate, took a big swing, and got the module back in working shape. Not only that, but I cleaned up the issue queue too. Not too shabby for 2 hours of sleep and switching back 5 hours from one timezone to another in the same day :-)

Hi Ted, great work, thanks

Submitted by Gerald Schmidhuber (not verified) on 23 July 2006 - 6:51am.

Hi Ted,

great work, thanks for your effort!

Bye, Gerald

This is great news and one

Submitted by bertboerland (not verified) on 23 July 2006 - 12:07pm.

This is great news and one of the modules that needed a lot of love and a maintainer as well.

So thank you!

Thanks a lot! We use this

Submitted by Gabor Hojtsy (not verified) on 24 July 2006 - 3:08pm.

Thanks a lot! We use this heavily on Weblabor.hu (Drupal 4.6), and we intend to upgrade, so an upgraded scheduler is very good news for us.

Hi Ted!! Your great work and

Submitted by Ben Bicais (not verified) on 10 August 2006 - 1:07am.

Hi Ted!!

Your great work and effort kicks major ass!!! Bookmarked!!!

Thanx for it!! Ben

I got scheduler from the

Submitted by Jim Miller (not verified) on 10 August 2006 - 1:11pm.

I got scheduler from the main drupal site (http://ftp.osuosl.org/pub/drupal/files/projects/scheduler-4.7.0 — I’m assuming this is your version?) and had some problems with nodes not publishing/unpublishing at the right times. (For instance, a node scheduled to publish in ten minutes would be published as soon as cron ran.) Here’s what I had to do to scheduler_cron to get it to behave

(a) I added the timezone of the scheduled event into the queries looking for nodes that are ready to be published/unpublished (lines 168 and 190):

  $nodes = db_query('SELECT * FROM {scheduler} s LEFT JOIN {node} n ON s.nid = n.nid WHERE n.status = 0 AND s.publish_on > 0 AND s.publish_on < %d + s.timezone', time());

(b) line 174 has a typo: it should read:

    if ($node->unpublish_on == 0) {

Does this make sense? It seems to be behaving properly on my installation with these changes.

Just checking / hoping to help, Jim

Thanks Jim! I've actually

Submitted by ted on 13 August 2006 - 12:39pm.

Thanks Jim! I’ve actually just committed both of this fixes, should be good now I hope. I’ve learned that dealing with timezones can really make your headspin, eek!

ted serbinski