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

Bug in field_groups.js #48

Closed
onyxnz opened this issue Oct 8, 2024 · 1 comment · Fixed by #49
Closed

Bug in field_groups.js #48

onyxnz opened this issue Oct 8, 2024 · 1 comment · Fixed by #49
Labels

Comments

@onyxnz
Copy link

onyxnz commented Oct 8, 2024

The API changed for JQuery UI 1.9+, so line 104 should be

$firstErrorItem = $(this).parent().accordion('option', 'active' , i);

otherwise an error is encountered:
Uncaught Error: no such method 'activate' for accordion widget instance

cf https://stackoverflow.com/questions/1899307/jquery-ui-accordion-activate
Cheers!

@herbdool
Copy link
Collaborator

Steps to recreate:

  1. Add an accordion group, and two accordion items to a content type.
  2. In each accordion item but a field.
  3. Make either one of those fields required.
  4. Go to create a node.
  5. Leave the required field blank and attempt to save.
  6. Console shows error.

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 a pull request may close this issue.

2 participants