Hello,

There is anonymous signup on my site and there are some signup-enabled events at the same time.

Noticed such problem:
If some guest signups in 2 nodes and admin cancels his signup in one node, signup in other node is canceled too.

best regards
Jonas

CommentFileSizeAuthor
#4 321653-signup-patch-5.x-2.x-4.patch583 bytesshanefjordan

Comments

dww’s picture

dww
we/he/they
commented

That seems very surprising. Can anyone else reproduce this and confirm? ;) Thanks.

shanefjordan’s picture

shanefjordan commented

I just reproduced this. Signup 5.x-2.dev with the patch for backend date support. I signed an anonymous user up to three different nodes. When in as an administrator, and cancelled the signup for one node for that user and it removed the user from all of the signups that the anonymous user belonged to. This does not remove all anonymous users, just the one that has the e-mail address.

- Shane

shanefjordan’s picture

shanefjordan commented
Assigned: Unassigned » shanefjordan

I'll write a patch for this

shanefjordan’s picture

shanefjordan commented
Status: Active » Needs review
StatusFileSize
new 321653-signup-patch-5.x-2.x-4.patch583 bytes

Please try this patch and see if it works. It is now working on my system.

- Shane

naiptol’s picture

naiptol commented

Shane,

It works on my 2 systems.

Thank you.

Jonas

stborchert’s picture

stborchert
he/him
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Confirmed the bug and after applying the patch its gone.
I'm curious about it was so long in the module and noone found it :-).

dww’s picture

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

Wow, that's a pretty nasty bug. ;) CVS blame points back to the original commit when anonymous signup support was added, over at #68497: anonymous signup support. Can't believe I missed this while reviewing hunmonk's patch -- oh well. ;)

Thanks for the patch, shane_jordan, that's perfect. Reviewed, tested and committed to both DRUPAL-5 and HEAD.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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