Add flags parameter to Node.duplicate() #7855
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
to decide whether signals, groups and/or scripts should be set in the copied nodes or not; it's default value makes the method work as usual, that is, including everything. So non-breaking change.
(This is the version for 2.1 of #7856.)
Motivation: my use case is to copy a hierarchy of nodes in a behaviorless fashion because I just want a visual copy of the original node so I can apply certain effects on it. With these flags I can control what kind of duplicate I want.
Furthermore, it avoids some error prints (which lag the game) about signals with targets outside the subtree being copied.
I believe more people will find this useful.