Skip to content

Expose get_autoload_list to gdscript#89198

Open
vaartis wants to merge 1 commit intogodotengine:masterfrom
vaartis:expose-autoload-list
Open

Expose get_autoload_list to gdscript#89198
vaartis wants to merge 1 commit intogodotengine:masterfrom
vaartis:expose-autoload-list

Conversation

@vaartis
Copy link
Copy Markdown
Contributor

@vaartis vaartis commented Mar 5, 2024

Expose get_autoload_list as a typed array to gdscript, since gdscript does not have an ordered array type. It's useful in GDScript too (I'd just tried to find something like this for my own project and saw it wasn't available in GDScript), so it should probably be available.

@vaartis vaartis requested review from a team as code owners March 5, 2024 20:32
@AThousandShips AThousandShips added this to the 4.x milestone Mar 5, 2024
@Mickeon Mickeon self-requested a review March 5, 2024 20:39
@vaartis vaartis force-pushed the expose-autoload-list branch from 08ab768 to efe2e4d Compare March 6, 2024 08:57
Comment thread doc/classes/ProjectSettings.xml Outdated
@vaartis vaartis force-pushed the expose-autoload-list branch from efe2e4d to 5fefda4 Compare March 6, 2024 09:22
@vaartis vaartis requested a review from bruvzg March 6, 2024 20:03
@Mickeon
Copy link
Copy Markdown
Member

Mickeon commented Mar 8, 2024

Although I'm not vehemently against this, could you provide the use case you need this for, and preferably open a Proposal to gather user interest? It is kinda possible to do this already, by opening the raw project.godot as a ConfigFile, but I can understand it's not exactly accessible, nor obvious.

Comment thread core/config/project_settings.cpp Outdated
Comment thread doc/classes/ProjectSettings.xml Outdated
Comment thread doc/classes/ProjectSettings.xml Outdated
@vaartis
Copy link
Copy Markdown
Contributor Author

vaartis commented Mar 8, 2024

My specific use case for this is as follows: I have a function that loads saved data for some scene. Together with that I need to also load data into some auto-loaded nodes, of which there are several. I'd like it if I could have a list of those. Admittedly this could be worked around, but since this function already exists it makes sense to me it should be exposed.

I guess I can open a proposal but there is already one that includes that one function and some others, it turns out godotengine/godot-proposals#4854. And another one that is specifically about this functionality but under a different name godotengine/godot-proposals#3705

@vaartis vaartis force-pushed the expose-autoload-list branch from 5fefda4 to 0745c44 Compare March 8, 2024 21:47
@vaartis vaartis requested a review from Mickeon March 8, 2024 21:54
Comment thread core/config/project_settings.cpp Outdated
Comment thread core/config/project_settings.cpp Outdated
Comment thread core/config/project_settings.h Outdated
@vaartis vaartis force-pushed the expose-autoload-list branch from 0745c44 to 8026cf4 Compare March 10, 2024 21:06
@vaartis vaartis requested a review from AThousandShips March 11, 2024 07:55
Copy link
Copy Markdown
Member

@Mickeon Mickeon left a comment

Choose a reason for hiding this comment

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

Documentation about this feature is entirely fine, if that was a bother.

@vaartis
Copy link
Copy Markdown
Contributor Author

vaartis commented Dec 23, 2025

Rebased on master and fixed the conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants