Skip to content

Fix TypeError: _validateParameters is not a function#8836

Closed
Chessing234 wants to merge 1 commit into
processing:mainfrom
Chessing234:fix-validate-parameters-8818
Closed

Fix TypeError: _validateParameters is not a function#8836
Chessing234 wants to merge 1 commit into
processing:mainfrom
Chessing234:fix-validate-parameters-8818

Conversation

@Chessing234
Copy link
Copy Markdown

Bug: Parameter validation was throwing a hard error when triggered, because p5.prototype._validateParameters was incorrectly aliased to the undefined p5.validateParameters instead of p5._validateParameters.

Root cause: The alias in validate_params.js referenced an undefined property instead of the static method defined earlier in the file. This prevented the FES (Friendly Error System) from correctly logging parameter errors and instead crashed the program when it attempted to execute the wrapper.

Why fix is correct: Pointing the alias to the correct static method p5._validateParameters restores the intended FES behavior without crashing the program, fixing Issue #8818.

Copy link
Copy Markdown
Collaborator

@perminder-17 perminder-17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, I think the issue has already been resolved. #8829

Thanks for your efforts, you can explore other issues from here;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants