Skip to content

Add graph property to GaeaNodeResource and remove the old reference passed around#574

Merged
BenjaTK merged 1 commit intogaea-godot:2.0from
Zehir:add_graph_property
Feb 15, 2026
Merged

Add graph property to GaeaNodeResource and remove the old reference passed around#574
BenjaTK merged 1 commit intogaea-godot:2.0from
Zehir:add_graph_property

Conversation

@Zehir
Copy link
Collaborator

@Zehir Zehir commented Feb 9, 2026

This PR is the first step for #573

The idea here is to have a reference of the "owning" GareGraph on the GaeaNodeResource instead of passing around the reference. Because a lot of funtions inside GaeaNodeResource does use the reference and a GaeaNodeResource can have only one GareGraph.

Also simplify the reference to the graph inside the output node and parameter nodes. The methods was using the node->graph_edit->graph reference that is not defined on an exported game. Not sure how that was working before during gameplay

Note: Any custom nodes will need to remove the graph param so it's better to do that now than later

@Zehir Zehir force-pushed the add_graph_property branch from 397578d to 72ba562 Compare February 9, 2026 22:35
@Zehir Zehir force-pushed the add_graph_property branch from 72ba562 to a9bcb2c Compare February 9, 2026 22:38
@Zehir Zehir requested review from BenjaTK and cullumi February 9, 2026 22:38
@Zehir Zehir added ‼️breaking A change that could break already existing projects. organization Movement of code around, refactoring, removal of code. topic: graph-nodes New nodes or changes related to GaeaNodeResource. labels Feb 10, 2026
Copy link
Collaborator

@BenjaTK BenjaTK left a comment

Choose a reason for hiding this comment

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

LGTM

@BenjaTK BenjaTK merged commit 1127f6d into gaea-godot:2.0 Feb 15, 2026
5 checks passed
@BenjaTK BenjaTK changed the title Add a graph property to GaeaNodeResource and remove the old reference passed around in the methods. Add graph property to GaeaNodeResource and remove the old reference passed around Feb 15, 2026
@Zehir Zehir deleted the add_graph_property branch February 15, 2026 21:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

‼️breaking A change that could break already existing projects. organization Movement of code around, refactoring, removal of code. topic: graph-nodes New nodes or changes related to GaeaNodeResource.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants