Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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).
Comment | File | Size | Author |
---|---|---|---|
#2 | cck_signup.1039294-2.patch | 1.04 KB | jhedstrom |
#1 | cck_signup-1039294-2.patch | 2.5 KB | _toch |
Comments
Comment #1
_toch CreditAttribution: _toch commentedAnd the patch.
Comment #2
jhedstromHere'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.
Comment #3
jhedstromCommitted.
Comment #4
_toch CreditAttribution: _toch commentedThis 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.
Comment #5
jhedstromAh. 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)?
Comment #6
_toch CreditAttribution: _toch commentedOk, I'll check it tomorrow.
Comment #7
_toch CreditAttribution: _toch commentedsorry for the delay, some urgent work to finish.
I've tried the last revision in CVS. And it perfectly works. thanks jhedstrom.
Comment #8
jhedstromGlad to here it _toch. Thanks for your help with patches and testing.
Comment #9
_toch CreditAttribution: _toch commentedjhedstrom, could you tell me when do you release a new version with this fix?
Comment #10
jhedstromJust rolled a new release.
Comment #11
_toch CreditAttribution: _toch commentedThanks a lot.
Comment #12
marcusx CreditAttribution: marcusx commentedAhh 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.
Comment #13
jhedstrommarcusx, 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?