From 47be60407bcfb35aa91c344fb6a48b3b1393797c Mon Sep 17 00:00:00 2001 From: digimint Date: Sun, 24 Mar 2024 20:22:16 -0500 Subject: [PATCH] Fix user lookup by ID When calling `get_users` with an array of ids, the function attempts to pop the first id from the `names` array rather than the `ids` array. This commit fixes that issue. --- addons/gift/api_connection.gd | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/gift/api_connection.gd b/addons/gift/api_connection.gd index 5fe4784..ca58dba 100644 --- a/addons/gift/api_connection.gd +++ b/addons/gift/api_connection.gd @@ -89,7 +89,7 @@ func get_users(names : Array[String], ids : Array[String]) -> Dictionary: if (params.length() > 1): params += "&" if (ids.size() > 0): - params += "id=%s" % names.pop_back() + params += "id=%s" % ids.pop_back() while(ids.size() > 0): params += "&id=%s" % ids.pop_back() var response = await(request(HTTPClient.METHOD_GET,"/helix/users/%s" % params, headers)) -- 2.47.2