elisseck / com.elisseck.civihoneypot

Simple honeypot fields for CiviCRM Contribute forms
Other
3 stars 6 forks source link

CiviHoneypot breaks event registration in Civi 5.40.0/5.40.1 #16

Closed MegaphoneJon closed 3 years ago

MegaphoneJon commented 3 years ago

This is a bug in core, but I haven't seen it affect anything (so far) except when CiviHoneypot is enabled, at which point you can't submit an event registration (the "Register" button simply doesn't work).

I have a fix here for folks who need it: https://github.com/civicrm/civicrm-core/pull/21124

I don't 100% understand why crmValidate()'s absence + CiviHoneypot breaks event registration. Well, I do - you can't submit a form with a field that is both hidden and required. I guess I don't understand how crmValidate fixes it, but I'm reasonably certain my patch is correct.

MegaphoneJon commented 3 years ago

This is fixed in 5.40.2.