Don't store deprecated auto_translate
property
#90685
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.
With the introduction of
Node.auto_translate_mode
,Control.auto_translate
becomes a proxy for it.But
auto_translate
is currently still stored intscn
files. This results in unexpected behavior: If a parent node is set to have disabled translate mode, all child nodes will set the mode to "disabled" when that scene is instantiated (instead of keeping "inherit") .This PR makes
Control.auto_translate
property not stored. It is still OK to open old scenes usingauto_translate
. But the property will not exist anymore once the scene is saved.CC @YeldhamDev
To reproduce the problem, save the following script and run
godot --headless -s /path/to/mrp.gd
.mrp.gd
Output:
The "Instantiated" section should be the same as the "Disable Parent" section.