Switching back to previously loaded scene fails using load_scene_from_packed

[MM4096]

Switching back to previously loaded scene fails using load_scene_from_packed

[huwpascoe]

If the editor allowed for the creation of these cyclic references, that's a bug (please provide the workflow).

However if these files were manually changed outside the editor to try and bypass that behavior, that can't be helped.

I do have a ready to merge PR that might be an eventual fix to the preloading problem #91815.

[inhalt120g]

@AThousandShips

Circular references are indeed not supported, and there's clear errors to indicate this, so this isn't a bug and the error messages should make it clear that this isn't supported IMO

And in my own experience adding circular references in the editor is prevented, so how was this achieved?

Here is how to add a circular reference using the attached project (it's all about one node that spawns the next node when you press a button and removes itself, nodes S01 to S04 all use the same base scene).
Open S04 (it's one of the nodes in the root).
In S04, there's only one node. In the Inspector, you'll see that it has a "target" field. Drag, for example, S01 into it.
Save the project.
Quit to project list.
Reopen the project.
…It won't work.
engage set as handled comparison g43.zip

[AThousandShips]

Circular references are indeed not supported, and there's clear errors to indicate this, so this isn't a bug and the error messages should make it clear that this isn't supported IMO

And in my own experience adding circular references in the editor is prevented, so how was this achieved?

Unless this was not handled correctly in the editor, or the messages aren't clear enough, this isn't a bug and it is simply not supported, you can still work with it by loading things with paths, or with UIDs as mentioned above

If there aren't any of those issues here this should be discussed in this proposal instead:

• godotengine/godot-proposals#7363

[inhalt120g]

Not sure my instructions make sense, so here it is in video format too (only without saving / reopening steps):

2024-06-17.19.18.59.mov

[AThousandShips]

Then we need to check if something is missing when assigning here