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 Report][3.5.2] Maximum recursive updates exceeded for large forms #19161

Closed
N1XUS opened this issue Feb 2, 2024 · 4 comments
Closed

[Bug Report][3.5.2] Maximum recursive updates exceeded for large forms #19161

N1XUS opened this issue Feb 2, 2024 · 4 comments
Assignees
Labels
E: validation T: bug Functionality that does not work as intended/expected upstream Problem with a third party library that we may have to work around
Milestone

Comments

@N1XUS
Copy link

N1XUS commented Feb 2, 2024

Environment

Vuetify Version: 3.5.2
Vue Version: 3.4.15
Browsers: Chrome 121.0.0.0
OS: Mac OS 10.15.7

Steps to reproduce

Create form with large amount of controls that have validation rules applied to it

Expected Behavior

No error

Actual Behavior

Error

Reproduction Link

https://play.vuetifyjs.com/#...

@N1XUS
Copy link
Author

N1XUS commented Feb 8, 2024

More information: if you create same form with lazy validation, yet. trigger form validation when component mounted, error is not present.

@johnleider johnleider added T: bug Functionality that does not work as intended/expected E: validation and removed S: triage labels Feb 12, 2024
@KaelWD KaelWD added the upstream Problem with a third party library that we may have to work around label Feb 13, 2024
@KaelWD
Copy link
Member

KaelWD commented Feb 13, 2024

There isn't actually an infinite loop here, a watcher is just being called for each component in the form and vue has an arbitrary limit of 100 calls in development.

@N1XUS
Copy link
Author

N1XUS commented Feb 19, 2024

But if you change the validation to be async, and still call the validation over full form after it's being rendered, then there's no issue. I'd say there something wrong with refs/computed triggers, as they trigger multiple times per form control

@KaelWD KaelWD added this to the v3.5.x milestone Feb 20, 2024
@KaelWD KaelWD closed this as completed in 07d91b2 Feb 20, 2024
VIXI0 pushed a commit to VIXI0/vuetify that referenced this issue Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E: validation T: bug Functionality that does not work as intended/expected upstream Problem with a third party library that we may have to work around
Projects
None yet
Development

No branches or pull requests

4 participants