- Issue created by @pfrenssen
- 🇨🇦Canada endless_wander
This was discussed in https://www.drupal.org/project/recurring_events/issues/3090286 ✨ Configuration options for registrant email address Active with no resolution.
I propose since email address is required for registering and triggers errors if not present, it makes sense to make it required. More functionality can be added for custom solutions where email is not required.
- Merge request !143Issue #3478268: Fatal error when registering for event without entering email. → (Open) created by dhruv.mittal
I have created MR that is solving the issue.
Also I noticed that if once a registration form is generated a error then it's status will be in pending status and when I try to delete the event series related to registrant it produces same error.
And the given MR solves this too. Please review and let me know if any assistance requires.
Thank you.- Status changed to RTBC
about 1 month ago 5:01am 8 April 2025 - 🇮🇳India kulpratap2002
The MR resolved the error, so moving this to RTBC.
Attaching a screenshot for reference.
- 🇧🇬Bulgaria pfrenssen Sofia
Thanks for working on this!
Instead of adding additional code to the form validate handler, let's instead mark the field as required in
Registrant::baseFieldDefinitions()
as was proposed in the issue summary.By doing this we will inform Drupal that the field is required. Drupal will then take care of the form validation for us. Also it will do some other helpful things like marking the field as required, support entity validation etc.
We do not need to make any changes to the form, it will all be handled automatically :)
- 🇮🇳India sagartiwari
Hello, I have verified the issue MR https://git.drupalcode.org/project/recurring_events/-/merge_requests/143... given in comment #9 fixes the issue. It adds a required validation on the Email Address field which prevents the page to encounter the error.
Thanks.