Fix surface handling fuckup for real

changelog.txt

@@ -1,4 +1,19 @@
 ---------------------------------------------------------------------------------------------------
+Version: 2.0.5
+Date: 2024-11-09
+  Bugfixes:
+    - Really fix the crash on surface deletion
+---------------------------------------------------------------------------------------------------
+Version: 2.0.4
+Date: 2024-11-09
+  Bugfixes:
+    - Fix another crash on surface deletion
+---------------------------------------------------------------------------------------------------
+Version: 2.0.3
+Date: 2024-11-08
+  Bugfixes:
+    - Fixed crash on invalid entity
+---------------------------------------------------------------------------------------------------
 Version: 2.0.2
 Date: 2024-11-08
   Changes:

control.lua

@@ -90,15 +90,17 @@ local function add_force_to_list(event)
 	local player, force
 	if event.player_index then
 		player = game.get_player(event.player_index)
+		if player and player.valid then
 			force = player.force
+		end
 	elseif event.force then
 		force = event.force
-		if force.connected_players then
+		if force.valid and force.connected_players then
 			player = force.connected_players[1]
 		end
 	end
 
-	if player and force and not storage.ammo_entities[player.surface.name.."_"..force.name] then
+	if player and player.valid and force and force.valid and not storage.ammo_entities[player.surface.name.."_"..force.name] then
 		for _,surface in pairs(game.surfaces) do
 			storage.ammo_entities[surface.name.."_"..force.name] = surface.find_entities_filtered{type = {"ammo-turret","car","artillery-turret","artillery-wagon","spider-vehicle"}, force = force, to_be_deconstructed = false}
 		end
@@ -132,13 +134,13 @@ end
 
 local function remove_surface_from_list(event)
 	--Whenever a surface is renamed or deleted, move/remove all entities in/from the global table.
-	for force,_ in pairs(game.forces) do
+	for _,force in pairs(game.forces) do
 		local index
 		if event.new_name then
-			storage.ammo_entities[new_name.."_"..force.name] = table.deepcopy(storage.ammo_entities[old_name.."_"..force.name])
+			storage.ammo_entities[event.new_name.."_"..force.name] = table.deepcopy(storage.ammo_entities[event.old_name.."_"..force.name])
-			index = old_name.."_"..force.name
+			index = event.old_name.."_"..force.name
 		elseif event.surface_index then
-			index = game.surfaces[surface_index].name.."_"..force.name
+			index = game.surfaces[event.surface_index].name.."_"..force.name
 		end
 
 		if index then
@@ -170,7 +172,7 @@ local function generate_alerts()
 		local ammo_entities = storage.ammo_entities[player.surface.name.."_"..player.force.name]
 
 		if ammo_entities then
-			for _,entity in pairs(ammo_entities) do
+			for index,entity in pairs(ammo_entities) do
 				if entity.valid then
 					if entity.force == player.force then
 
@@ -213,9 +215,7 @@ local function generate_alerts()
 						end
 					end
 				else
-					local param = {}
+					ammo_entities[index] = nil
-					param.entity = entity
-					remove_entity_from_list(param)
 				end
 			end
 		end
@@ -236,7 +236,7 @@ script.on_event(defines.events.on_force_created, add_force_to_list)
 script.on_event(defines.events.on_forces_merged, remove_force_from_list)
 
 script.on_event(defines.events.on_surface_renamed, remove_surface_from_list)
-script.on_event(defines.events.on_surface_deleted, remove_surface_from_list)
+script.on_event(defines.events.on_pre_surface_deleted, remove_surface_from_list)
 
 script.on_event(defines.events.on_built_entity, add_entity_to_list, {{filter="type", type = "ammo-turret"},{filter="type", type = "car"},{filter="type", type = "spider-vehicle"},{filter="type", type = "artillery-turret"},{filter="type", type = "artillery-wagon"}})
 script.on_event(defines.events.on_robot_built_entity, add_entity_to_list, {{filter="type", type = "ammo-turret"},{filter="type", type = "car"},{filter="type", type = "spider-vehicle"},{filter="type", type = "artillery-turret"},{filter="type", type = "artillery-wagon"}})

info.json

@@ -1,6 +1,6 @@
 {
     "name": "Gun_Turret_Alerts",
-    "version": "2.0.2",
+    "version": "2.0.5",
     "title": "Ammo Alerts",
     "author": "Wyrrrd",
     "factorio_version": "2.0",

locale/de/config.cfg

@@ -24,3 +24,8 @@ message-low=__1__ hat wenig Munition
 [virtual-signal-name]
 ammo-icon-red=Hat keine Munition
 ammo-icon-yellow=Hat wenig Munition
+
+[mod-name]
+
+[mod-description]
+

locale/fr/config.cfg

@@ -24,3 +24,8 @@ message-low=__1__ peu de munitions
 [virtual-signal-name]
 ammo-icon-red=Signal d'absence de munition
 ammo-icon-yellow=Signal de manque de munitions
+
+[mod-name]
+
+[mod-description]
+

locale/uk/config.cfg

@@ -24,3 +24,8 @@ message-low=__1__ мало боєприпасів
 [virtual-signal-name]
 ammo-icon-red=Сигнал про закінчення набоїв
 ammo-icon-yellow=Сигнал про низький рівень набоїв
+
+[mod-name]
+
+[mod-description]
+