goostengine / goost

A general-purpose, extensible and customizable C++ extension for Godot Engine.
https://goostengine.github.io/
MIT License
481 stars 18 forks source link

Loading existing `MixinScript` into itself causes editor crash #128

Closed Xrayez closed 3 years ago

Xrayez commented 3 years ago

Goost and Godot version: Goost 625a381, Godot 3.4.beta

OS/platform/device including version: Windows

Steps to reproduce: Create a new MixinScript (by attaching a script to a node, for instance), and then try to load the same script as a mixin one:

image

Minimal reproduction project: N/A

Xrayez commented 3 years ago

The fix likely requires checking if a script is the same as the MixinScript and simply prevent loading. I could fix this myself, but not high priority at the moment (not until Godot 3.4-stable is released), so this could be a good first issue to fix if someone would like to take this.