Godot: A proper mean to get the list of InputMap actions

[SuperUserNameMan]

Operating system or device - Godot version:
Official 2.0.3stable win64

Issue description (what happened, and what was expected):
I've found no clean way to iterate through InputMap actions, and i've found no way to get the list of actions names.

If I do something like this, this ends up with an error.

    var index = 1 # <------ index 0 seems unavailable
    var action = InputMap.get_action_from_id(index)
    while action != "":
        print(action)
        index+=1
        action = InputMap.get_action_from_id(index)

ERROR: InputMap::get_action_from_id: Condition ' !input_id_map.has(p_id) ' is true. returned: StringName()
At: core\input_map.cpp:74

Steps to reproduce:

Link to minimal example project (optional but very welcome):

[akien-mga]

How about... InputMap.get_action_list()? :D
http://docs.godotengine.org/en/latest/classes/class_inputmap.html

[SuperUserNameMan]

@akien-mga :
Array InputMap.get_action_list(action) requires the name of an action as parameter.
So, I guess the "list" it returns is the list of inputs associated to the action.

What I need is to get the list of the name of all actions stored into the InputMap of the project settings.

[SuperUserNameMan]

I've done the test to be sure, and I was right. .get_action_list(action) returns the list of inputs associated to an action.

It is not what I need.

[akien-mga]

Hm indeed. It should be implemented then.

[J08nY]

I think get_action_list(var action) is rather misleading and should be renamed to action_list_events(var action) to go with: action_add_event, action_has_event, action_erase_event and something like get_actions() or even get_action_list() should be used for this new function.
@akien-mga Do you think it's possible to rename it, or will it break too much stuff?

[SuperUserNameMan]

or instead of changing the name or behavior of get_action_list() , there could be a get_list_of_actions() ?

[J08nY]

Well, then you would have InputMap.get_list_of_actions() and InputMap.get_action_list(var action) which is kind of misleading I think. Since it's really just a different wording of the same thing(while behavior is different), and the parameter doesn't explain much.

[neikeq]

Try:

var properties = Globals.get_property_list()

for property in properties:
    if not property.name.begins_with("input/"):
        continue
    var name = property.name.substr(property.name.find("/") + 1, property.name.length())
    var id = InputMap.get_action_id(name)

    assert(name == InputMap.get_action_from_id(id))

It's based on InputMap::load_from_globals().

[SuperUserNameMan]

@neikeq : great workaround ! thanks !