search

McManning / BlueGraph

Visual Scripting Framework for Unity
MIT License
240 stars 33 forks source link

Prevent copying nodes onto graphs with different module rules #3

Closed McManning closed 4 years ago

McManning commented 4 years ago

There are currently no checks in CopyPasteGraph to prevent someone from taking nodes from a graph of one set of [IncludeModules] and copying them over to another graph with a different [IncludeModules].

Simple example:

I have a subgraph defined as:

[IncludeModules("Subgraph", "Flow Control", ...)]
public class Subgraph : Graph
{
    ...
}

I can copy+paste any nodes from that over to another graph without checking IncludeModules first:

[IncludeModules("UAI")]
public class UtilityAI : Graph
{
    ...
}
McManning commented 4 years ago

Calling this a priority for initial release due to the risk of potentially generating broken graph assets by doing this.

github-actions[bot] commented 3 years ago

:tada: This issue has been resolved in version 1.0.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: