Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix crmValidate() not loading on event registration pages #21124

Merged
merged 1 commit into from
Aug 13, 2021

Conversation

MegaphoneJon
Copy link
Contributor

Overview

There is a logic error in #20543 that prevents the CiviCRM-specific client-side validation loading on event registration pages (and probably elsewhere).

Before

crmValidate() not loading.

After

crmValidate() loading.

Comments

My understanding if #20543 is:

  • It's a refactor in that it's not intended to have any noticeable effect;
  • The goal is to ensure that we're not attempting to access uninitialized variables, array elements, etc.

I think this fix satisfies both.

@civibot
Copy link

civibot bot commented Aug 13, 2021

(Standard links)

@civibot civibot bot added the master label Aug 13, 2021
@MegaphoneJon
Copy link
Contributor Author

Oh:

Steps for replication

  • Visit an event registration page.
  • Open the browser's Dev Toolbar.
  • On the "inspector" tab, search for crmValidate.

Also flagging that the regression is new to 5.40.0.

@eileenmcnaughton
Copy link
Contributor

I agree with your analysis

@eileenmcnaughton eileenmcnaughton merged commit fa62d72 into civicrm:5.41 Aug 13, 2021
@MegaphoneJon MegaphoneJon deleted the fix-event-validation branch August 13, 2021 20:19
@eileenmcnaughton
Copy link
Contributor

5.40 port #21128

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants