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 CustomType error spams in CreateDialog #25675

Conversation

willnationsdev
Copy link
Contributor

This should close #24041. It adds a bunch of logic to account for CustomTypes in the various checks it performs.

It should be noted that the CreateDialog's inheritance and type checking is becoming increasingly complex, so this solution is more of a bandaid. If/when something like #22181 is integrated, allowing scenes to be named entities as well, then there'll be all the more need for something like the ClassType module it includes to greatly simplify all of the logic.

@willnationsdev willnationsdev requested a review from reduz as a code owner February 6, 2019 23:37
@willnationsdev willnationsdev force-pushed the fix-customtype-errorspam branch from 27e7a50 to ca771a7 Compare February 6, 2019 23:59
core/class_db.cpp Outdated Show resolved Hide resolved
@akien-mga
Copy link
Member

Is this still necessary after #26818? (cc @vnen).

@akien-mga
Copy link
Member

As #24041 was fixed in the master branch, I guess this PR is no longer necessary.

Feel free to rebase and reopen in case it still handles different use cases than what #24041 described.

@akien-mga akien-mga closed this Oct 30, 2019
@willnationsdev willnationsdev deleted the fix-customtype-errorspam branch May 16, 2020 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Node in the editor is giving an error when filtering the name.
4 participants