From 63b533abf71de77eec7e0f9f0781dd687fd1ae40 Mon Sep 17 00:00:00 2001
From: Star Rauchenberger <fefferburbia@gmail.com>
Date: Fri, 14 Apr 2023 21:06:25 -0400
Subject: Receiving items shows the right name now

(Sending checks doesn't)
---
 Archipelago/client.gd | 76 +++++++++++++++++++++++++++++++--------------------
 1 file changed, 46 insertions(+), 30 deletions(-)

diff --git a/Archipelago/client.gd b/Archipelago/client.gd
index be0ef2d..335871e 100644
--- a/Archipelago/client.gd
+++ b/Archipelago/client.gd
@@ -11,9 +11,10 @@ var _client = WebSocketClient.new()
 var _last_state = WebSocketPeer.STATE_CLOSED
 var _should_process = false
 
-var _datapackage_checksum = ""
-var _item_name_to_id = {}
-var _location_name_to_id = {}
+var _datapackages = {}
+var _item_id_to_name = {}  # All games
+var _item_name_to_id = {}  # LINGO only
+var _location_name_to_id = {}  # LINGO only
 
 const uuid_util = preload("user://maps/Archipelago/vendor/uuid.gd")
 
@@ -58,11 +59,9 @@ func _init():
 		if data.size() > 2:
 			ap_pass = data[2]
 		if data.size() > 3:
-			_datapackage_checksum = data[3]
-		if data.size() > 4:
-			_item_name_to_id = data[4]
-		if data.size() > 5:
-			_location_name_to_id = data[5]
+			_datapackages = data[3]
+
+		processDatapackages()
 
 
 func _ready():
@@ -96,22 +95,26 @@ func _on_data():
 
 		if cmd == "RoomInfo":
 			_seed = message["seed_name"]
-			if message["datapackage_checksums"].has("Lingo"):
-				if _datapackage_checksum != message["datapackage_checksums"]["Lingo"]:
-					requestDatapackage()
-				else:
-					connectToRoom()
-
-		elif cmd == "DataPackage":
-			if message["data"]["games"].has("Lingo"):
-				var lingo_pkg = message["data"]["games"]["Lingo"]
-				_datapackage_checksum = lingo_pkg["checksum"]
-				_item_name_to_id = lingo_pkg["item_name_to_id"]
-				_location_name_to_id = lingo_pkg["location_name_to_id"]
-				saveSettings()
 
+			var needed_games = []
+			for game in message["datapackage_checksums"].keys():
+				if (
+					!_datapackages.has(game)
+					or _datapackages[game]["checksum"] != message["datapackage_checksums"][game]
+				):
+					needed_games.append(game)
+
+			if !needed_games.empty():
+				requestDatapackages(needed_games)
+			else:
 				connectToRoom()
 
+		elif cmd == "DataPackage":
+			_datapackages = message["data"]["games"]
+			saveSettings()
+			processDatapackages()
+			connectToRoom()
+
 		elif cmd == "Connected":
 			_authenticated = true
 			_team = message["team"]
@@ -196,9 +199,7 @@ func saveSettings():
 	var file = File.new()
 	file.open("user://settings/archipelago", File.WRITE)
 
-	var data = [
-		ap_server, ap_user, ap_pass, _datapackage_checksum, _item_name_to_id, _location_name_to_id
-	]
+	var data = [ap_server, ap_user, ap_pass, _datapackages]
 	file.store_var(data, true)
 	file.close()
 
@@ -239,8 +240,19 @@ func sendMessage(msg):
 	_client.get_peer(1).put_packet(payload.to_utf8())
 
 
-func requestDatapackage():
-	sendMessage([{"cmd": "GetDataPackage", "games": ["Lingo"]}])
+func requestDatapackages(games):
+	sendMessage([{"cmd": "GetDataPackage", "games": games}])
+
+
+func processDatapackages():
+	_item_id_to_name = {}
+	for package in _datapackages.values():
+		for name in package["item_name_to_id"].keys():
+			_item_id_to_name[package["item_name_to_id"][name]] = name
+
+	if _datapackages.has("Lingo"):
+		_item_name_to_id = _datapackages["Lingo"]["item_name_to_id"]
+		_location_name_to_id = _datapackages["Lingo"]["location_name_to_id"]
 
 
 func connectToRoom():
@@ -309,19 +321,23 @@ func processItem(item, index, from):
 	if _item_name_to_id["Progressive Orange Tower"] == item and _tower_floors < orange_tower.size():
 		var subitem_name = "Orange Tower - %s Floor" % orange_tower[_tower_floors]
 		global._print(subitem_name)
-		processItem(_item_name_to_id[subitem_name])
+		processItem(_item_name_to_id[subitem_name], null)
 		_tower_floors += 1
 
 	# Show a message about the item if it's new.
-	if index > _last_new_item:
+	if index != null and index > _last_new_item:
 		_last_new_item = index
 		saveLocaldata()
 
+		var item_name = "Unknown"
+		if _item_id_to_name.has(item):
+			item_name = _item_id_to_name[item]
+
 		var messages_node = get_tree().get_root().get_node("Spatial/AP_Messages")
 		if from == _slot:
-			messages_node.showMessage("Found %d" % item)
+			messages_node.showMessage("Found %s" % item_name)
 		else:
-			messages_node.showMessage("Received %d from %d" % [item, from])
+			messages_node.showMessage("Received %s from %d" % [item_name, from])
 
 
 func doorIsVanilla(door):
-- 
cgit 1.4.1