This patch change function "signup_admin_node_form_submit" to "signup_node_admin_summary_form_submit". This makes signup_status work with version 5.2.6 of the signup module.

Version 5.2.6 of the signup module add a lot of new code but this is the only problem I have found so far.

CommentFileSizeAuthor
signup_status_signup-5-2-6_compatibility.patch715 bytesfrjo

Comments

j.slemmer’s picture

j.slemmer commented

This code worked for the problem I was having, see:
http://drupal.org/node/342437

--
Joy Group
www.joygroup.nl

socialnicheguru’s picture

socialnicheguru commented

subscribing

is this necessary for signup 5.2.7 also?

dww’s picture

dww
we/he/they
commented
Priority: Normal » Critical
Status: Needs review » Reviewed & tested by the community

Yup, that's necesary.

@jrbeeman: Sorry for changing function names like this. I had no idea how deeply intwined with the core signup module signup_status is. :( At least all this is going to be getting pulled apart and made sane in the D6 version. ;)

dww’s picture

dww
we/he/they
commented
Status: Reviewed & tested by the community » Fixed

Now that Jeff gave me CVS access, and since I broke this, I figured I should commit it to signup_status's DRUPAL-5 branch. ;) That said, signup_status D5 is dying -- we're making things majorly better over at #330943: Drupal 6.x port of signup_status so don't expect too much more support for the DRUPAL-5 branch at this point...

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.