I need to disallow multiple signup nodes per event. This has to be checked during the validation of the new signup (this is a todo in the code).

CommentFileSizeAuthor
#2 cck_signup.1039294-2.patch1.04 KBjhedstrom
#1 cck_signup-1039294-2.patch2.5 KB_toch

Comments

_toch’s picture

_toch commented
StatusFileSize
new cck_signup-1039294-2.patch2.5 KB

And the patch.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Active » Needs review
StatusFileSize
new cck_signup.1039294-2.patch1.04 KB

Here's a slightly different patch, again, moving logic from Volunteer Rally back here. This implements hook_cck_signup_validate, and in combination with #1039274: remove link if already signed up should work for most use-cases. Please test this with your setup, and if it looks good, I'll commit it.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs review » Fixed

Committed.

_toch’s picture

_toch commented

This one doesn't work for me. The user can create two signups for the same event.

The cck_signup_cck_signup_validate function is never called.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Fixed » Needs work

Ah. It looks like this hook wasn't being called for event types that weren't using a signup status field. Can you try the latest version in CVS (I just committed a fix that should make it work)?

_toch’s picture

_toch commented
Component: Code » CCK Signup core

Ok, I'll check it tomorrow.

_toch’s picture

_toch commented
Version: 6.x-1.0-alpha4 » 6.x-1.0-beta1

sorry for the delay, some urgent work to finish.

I've tried the last revision in CVS. And it perfectly works. thanks jhedstrom.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs work » Fixed

Glad to here it _toch. Thanks for your help with patches and testing.

_toch’s picture

_toch commented

jhedstrom, could you tell me when do you release a new version with this fix?

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented

Just rolled a new release.

_toch’s picture

_toch commented

Thanks a lot.

marcusx’s picture

marcusx commented

Ahh Sorry - is there a switch somewhere I can switch this off, again? I need the old behaviour were it is allowed to create more signups for the same event. My events are holiday activity for kids. An the parents need to sigup for several siblings not only for one.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented

marcusx, have you looked at the CCK Signup Group sub-module? This allows a user to signup more than one person. If that doesn't work for your use case, could you open a new feature request to make this check configurable?

Status: Fixed » Closed (fixed)

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