Port the Signup Scheduler module to Drupal 6 and ensure compatibility with Signup for Drupal 6. This will include Views 2 integration.

CommentFileSizeAuthor
#2 signup_scheduler-336515.patch977 bytesmlsamuelson

Comments

slosa’s picture

subscribing

mlsamuelson’s picture

Status: Active » Needs review
StatusFileSize
new977 bytes

The following patch fixes an issue with the module's hook_nodeapi functions for the 'update' and 'insert' ops. They were not firing due to a check for $node->signup, which isn't on the $node object at that time. Changing the check to $node->signup_enabled resolves the issue.

I'm not sure how feature-complete (views 2 integration) this puts the module for D6, but it does put the basic functionality in working order.

--David--’s picture

Component: Code » Documentation
Assigned: jrbeeman » Unassigned

Thank you!

mlsamuelson’s picture

@--Me-- :) If the patch works for you, please post back here. It'll help move this towards getting commited.

devkinetic’s picture

Worked for me. Previously setting the close date would not enable signups, and when visiting the edit screen again after saving no settings were applied.

With the patch applied, settings save and signup gets enabled! Great job.

Balbo’s picture

Why is Drupal 6 release not "Recommended"?

devkinetic’s picture

the release is still buggy and in active development. Until a stable release is created, the 6.x branch will stay in dev