Skip to content

Fix scene properties in the variable action and condition#8504

Merged
4ian merged 3 commits intomasterfrom
fix-scene-property-as-variable
Apr 13, 2026
Merged

Fix scene properties in the variable action and condition#8504
4ian merged 3 commits intomasterfrom
fix-scene-property-as-variable

Conversation

@D8H
Copy link
Copy Markdown
Collaborator

@D8H D8H commented Apr 12, 2026

No description provided.

@D8H D8H requested a review from 4ian as a code owner April 12, 2026 14:23
Comment on lines 74 to 78
gd::EventsFunctionTools::PropertiesToVariablesContainer(
eventsBasedBehavior.GetSharedPropertyDescriptors(), propertyVariablesContainer);
variablesContainersList.Push(propertyVariablesContainer);

gd::EventsFunctionTools::PropertiesToVariablesContainer(
eventsBasedBehavior.GetPropertyDescriptors(), propertyVariablesContainer);
eventsBasedBehavior.GetPropertyDescriptors(),
eventsBasedBehavior.GetSharedPropertyDescriptors(),
propertyVariablesContainer);
variablesContainersList.Push(propertyVariablesContainer);
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Shared properties were cleared since the same variable container is used.

@4ian 4ian merged commit 5f26035 into master Apr 13, 2026
5 checks passed
@4ian 4ian deleted the fix-scene-property-as-variable branch April 13, 2026 07:12
NBForgeLab added a commit to NBForgeLab/GDevelop that referenced this pull request Apr 18, 2026
* Make the home page navigation sidebar collapsible

* Fix potential crashes in the scene editor

* Improve logs in case of UseAfterFreeError

Only show in developer changelog

* Fix potential crashes in the scene editor

Don't show in changelog

* Display extension details side by side with the extension list on large screens (4ian#8482)

* Fix hot-reloading of global object instances (4ian#8489)

* Fix "Add or edit variables" button always creating variables for object variables (4ian#8493)

* Fix "Add or edit variables" button opening scene variables instead of local variables (4ian#8496)

* Add save/restore support for linked object relationships (4ian#8499)

Fix 4ian#8497

* Fix potential crash during resource reloading (4ian#8500)

* Make clear in Storage actions if a number or text is being written (4ian#8498)

* Add "Reload Project" menu action (4ian#8501)

Co-authored-by: Gleb Volkov <glebusheg@playtika.com>

* Remove deprecated, unused GitHub workflow

Don't show in changelog

* Fix scene properties not usable via the variable action and condition (4ian#8504)

* [Auto PR] Update translations (4ian#8474)

* Bump newIDE version

* Add experimental option in tabs context menu to open some editors (Event Sheets, Extensions...) into separate windows (4ian#8492)

* Fix wavesurfer version (4ian#8508)

* Allow AI to create objects via exact asset ID without specifying type (4ian#8509)

* Fix crashes when using invalid object names in object folders

* Support partial asset ID matching in asset search (4ian#8512)

Don't show in changelog

* Fix missing globally used resources when unloading a scene that uses these resources (4ian#8513)

- It only happens when "Unload at scene exit" is selected in project settings and a global object share some resources with a scene object.

* Fix tentatively crash in popped-out window when a theme is missing a color (4ian#8518)

Fix 4ian#8517

---------

Co-authored-by: Florian Rival <Florian.rival@gmail.com>
Co-authored-by: D8H <Davy.Helard@gmail.com>
Co-authored-by: LuniMoon <103995399+LuniMoon@users.noreply.github.com>
Co-authored-by: Gleb Volkov <glebusheg@playtika.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
NBForgeLab added a commit to NBForgeLab/GDevelop that referenced this pull request Apr 18, 2026
* Make the home page navigation sidebar collapsible

* Fix potential crashes in the scene editor

* Improve logs in case of UseAfterFreeError

Only show in developer changelog

* Fix potential crashes in the scene editor

Don't show in changelog

* Display extension details side by side with the extension list on large screens (4ian#8482)

* Fix hot-reloading of global object instances (4ian#8489)

* Fix "Add or edit variables" button always creating variables for object variables (4ian#8493)

* Fix "Add or edit variables" button opening scene variables instead of local variables (4ian#8496)

* Add save/restore support for linked object relationships (4ian#8499)

Fix 4ian#8497

* Fix potential crash during resource reloading (4ian#8500)

* Make clear in Storage actions if a number or text is being written (4ian#8498)

* Add "Reload Project" menu action (4ian#8501)



* Remove deprecated, unused GitHub workflow

Don't show in changelog

* Fix scene properties not usable via the variable action and condition (4ian#8504)

* [Auto PR] Update translations (4ian#8474)

* Bump newIDE version

* Add experimental option in tabs context menu to open some editors (Event Sheets, Extensions...) into separate windows (4ian#8492)

* Fix wavesurfer version (4ian#8508)

* Allow AI to create objects via exact asset ID without specifying type (4ian#8509)

* Fix crashes when using invalid object names in object folders

* Support partial asset ID matching in asset search (4ian#8512)

Don't show in changelog

* Fix missing globally used resources when unloading a scene that uses these resources (4ian#8513)

- It only happens when "Unload at scene exit" is selected in project settings and a global object share some resources with a scene object.

* Fix tentatively crash in popped-out window when a theme is missing a color (4ian#8518)

Fix 4ian#8517

---------

Co-authored-by: Florian Rival <Florian.rival@gmail.com>
Co-authored-by: D8H <Davy.Helard@gmail.com>
Co-authored-by: LuniMoon <103995399+LuniMoon@users.noreply.github.com>
Co-authored-by: Gleb Volkov <glebusheg@playtika.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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