Update translations from Crowdin

changelog.txt

@@ -1,4 +1,13 @@
 ---------------------------------------------------------------------------------------------------
+Version: 2.0.7
+Date: 2024-11-10
+  Optimizations:
+    - Add cleanup of empty global tables
+    - Move global table creation from on_surface_created to on_built_entity to save on event
+      listeners
+    - Review force rename/removal handling
+    - Code cleanup and comments
+---------------------------------------------------------------------------------------------------
 Version: 2.0.6
 Date: 2024-11-10
   Features:

control.lua

@@ -67,6 +67,13 @@ local function add_entity_to_list(event)
 	--Whenever an ammo-turret, vehicle or artillery type entity is built, add it to the global table.
 	local entity = event.created_entity or event.entity
 	local index = entity.surface.name.."_"..entity.force.name
+
+	-- Add table if it doesn't exist and a player of this force is connected
+	if entity.force.connected_players[1] and not storage.ammo_entities[index] then
+		storage.ammo_entities[index] = {}
+	end
+
+	-- Add entity to table
 	if storage.ammo_entities[index] then
 		table.insert(storage.ammo_entities[index], entity)
 	end
@@ -125,23 +132,11 @@ local function remove_force_from_list(event)
 		force = event.source
 	end
 
-	if force and not force.connected_players then
-		for surface_name,_ in pairs(game.surfaces) do
-			storage.ammo_entities[surface_name.."_"..force.name] = nil
-		end
-	end
-end
-
-local function add_surface_to_list(event)
-	--Whenever a surface is created, add an empty list to the global table for each force with a connected player.
-	local surface
-	if event.surface_index then
-		surface = game.surfaces[event.surface_index]
-	end
-
-	for _,force in pairs(game.forces) do
-		if surface and surface.valid and not storage.ammo_entities[surface.name.."_"..force.name] and force.connected_players[1] then
-			storage.ammo_entities[surface.name.."_"..force.name] = {}
+	if force then
+		if not force.valid or (force.valid and table_is_empty(force.connected_players)) then
+			for surface_name,_ in pairs(game.surfaces) do
+				storage.ammo_entities[surface_name.."_"..force.name] = nil
+			end
 		end
 	end
 end
@@ -193,52 +188,59 @@ local function generate_alerts()
 				ammo_entities = storage.ammo_entities[surface.name.."_"..player.force.name]
 			end
 
-			if ammo_entities and not table_is_empty(ammo_entities) then
-				for index,entity in pairs(ammo_entities) do
-					if entity.valid then
-						if entity.force == player.force then
+			if ammo_entities then
+				if not table_is_empty(ammo_entities) then
+					for index,entity in pairs(ammo_entities) do
+						if entity.valid then
+							if entity.force == player.force then
 
-							--Get ammo inventory based on entity type, skip vehicles without guns
-							local inventory
-							if turret_enabled and entity.type == "ammo-turret" then
-								inventory = entity.get_inventory(defines.inventory.turret_ammo)
-							elseif car_enabled and (entity.type == "car" or entity.type == "spider-vehicle") and not table_is_empty(entity.prototype.guns) then
-								inventory = entity.get_inventory(defines.inventory.car_ammo)
-							elseif artillery_enabled then
-								if entity.type == "artillery-turret" then
-									inventory = entity.get_inventory(defines.inventory.artillery_turret_ammo)
-								elseif entity.type == "artillery-wagon" then
-									inventory = entity.get_inventory(defines.inventory.artillery_wagon_ammo)
-								end
-							end
-
-							--Check for states of no or low ammo based on mode
-							local ammo_flag
-							if inventory and get_ammo_flag[mode] then
-								if entity.type == "ammo-turret" or entity.type == "artillery-turret" or entity.type == "artillery-wagon" then
-									ammo_flag = get_ammo_flag[mode](inventory, player_threshold)
-									if auto_full and entity.prototype.automated_ammo_count then
-										if entity.prototype.automated_ammo_count < player_threshold then
-											ammo_flag = get_ammo_flag[mode](inventory, entity.prototype.automated_ammo_count)
-										end
+								--Get ammo inventory based on entity type, skip vehicles without guns
+								local inventory
+								if turret_enabled and entity.type == "ammo-turret" then
+									inventory = entity.get_inventory(defines.inventory.turret_ammo)
+								elseif car_enabled and (entity.type == "car" or entity.type == "spider-vehicle") and not table_is_empty(entity.prototype.guns) then
+									inventory = entity.get_inventory(defines.inventory.car_ammo)
+								elseif artillery_enabled then
+									if entity.type == "artillery-turret" then
+										inventory = entity.get_inventory(defines.inventory.artillery_turret_ammo)
+									elseif entity.type == "artillery-wagon" then
+										inventory = entity.get_inventory(defines.inventory.artillery_wagon_ammo)
 									end
-								elseif entity.type == "car" or entity.type == "spider-vehicle" then
-									ammo_flag = get_ammo_flag[mode](inventory, player_threshold, entity.selected_gun_index)
+								end
+
+								--Check for states of no or low ammo based on mode
+								local ammo_flag
+								if inventory and get_ammo_flag[mode] then
+									if entity.type == "ammo-turret" or entity.type == "artillery-turret" or entity.type == "artillery-wagon" then
+										ammo_flag = get_ammo_flag[mode](inventory, player_threshold)
+										-- Automated amme count override (if activated)
+										if auto_full and entity.prototype.automated_ammo_count then
+											if entity.prototype.automated_ammo_count < player_threshold then
+												ammo_flag = get_ammo_flag[mode](inventory, entity.prototype.automated_ammo_count)
+											end
+										end
+									elseif entity.type == "car" or entity.type == "spider-vehicle" then
+										ammo_flag = get_ammo_flag[mode](inventory, player_threshold, entity.selected_gun_index)
+									end
+								end
+
+								--Create alert for present state
+								if ammo_flag then
+									-- no ammo alert
+									player.add_custom_alert(entity, {type = "virtual", name = "ammo-icon-red"}, {"gun-turret-alerts.message-empty", entity.localised_name}, true)
+								elseif ammo_flag == false then
+									-- low ammo alert
+									player.add_custom_alert(entity, {type = "virtual", name = "ammo-icon-yellow"}, {"gun-turret-alerts.message-low", entity.localised_name}, true)
 								end
 							end
-
-							--Create alert for present state
-							if ammo_flag then
-								-- no ammo alert
-								player.add_custom_alert(entity, {type = "virtual", name = "ammo-icon-red"}, {"gun-turret-alerts.message-empty", entity.localised_name}, true)
-							elseif ammo_flag == false then
-								-- low ammo alert
-								player.add_custom_alert(entity, {type = "virtual", name = "ammo-icon-yellow"}, {"gun-turret-alerts.message-low", entity.localised_name}, true)
-							end
+						else
+							-- Cleanup if entity is invalid
+							table.remove(ammo_entities, index)
 						end
-					else
-						table.remove(ammo_entities, index)
 					end
+				else
+					-- Cleanup if table is empty
+					storage.ammo_entities[surface.name.."_"..player.force.name] = nil
 				end
 			end
 		end
@@ -258,7 +260,6 @@ script.on_event(defines.events.on_player_changed_force, remove_force_from_list)
 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_created, add_surface_to_list)
 script.on_event(defines.events.on_surface_renamed, remove_surface_from_list)
 script.on_event(defines.events.on_pre_surface_deleted, remove_surface_from_list)
 

info.json

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

locale/de/config.cfg

@@ -4,6 +4,7 @@ gun-turret-alerts-car-enabled=Alarme für Vehikelmunition aktiviert
 gun-turret-alerts-artillery-enabled=Alarme für Artillerie aktiviert
 gun-turret-alerts-mode=Mehrplatz-Modus
 gun-turret-alerts-threshold=Grenze für wenig Munition
+gun-turret-alerts-z-show-planets=Auf allen Planeten anzeigen
 
 [mod-setting-description]
 gun-turret-alerts-enabled=Aktiviert Alarme für Geschützturmmunition für deinen Spieler

locale/en/config.cfg

@@ -24,8 +24,8 @@ gun-turret-alerts-mode-individually=Individually
 gun-turret-alerts-mode-selected=Selected
 
 [gun-turret-alerts]
-message-empty=__1__ out of ammo
-message-low=__1__ low ammo
+message-empty=__1__ is out of ammo
+message-low=__1__ is low on ammo
 
 [virtual-signal-name]
 ammo-icon-red=Out of ammo signal

locale/uk/config.cfg

@@ -4,6 +4,9 @@ gun-turret-alerts-car-enabled=Ввімкнено оповіщення про б
 gun-turret-alerts-artillery-enabled=Ввімкнено оповіщення про артилерійські боєприпаси
 gun-turret-alerts-mode=Багатослотовий режим
 gun-turret-alerts-threshold=Низький поріг боєприпасів
+gun-turret-alerts-z-automated-full=Автоматична сума дорівнює повній
+gun-turret-alerts-z-show-planets=Показати на всіх планетах
+gun-turret-alerts-z-show-platforms=Показати на всіх космічних платформах
 
 [mod-setting-description]
 gun-turret-alerts-enabled=Дозволяє додати сповіщення про боєприпаси у вежі до вашого плеєра.
@@ -11,6 +14,9 @@ gun-turret-alerts-car-enabled=Дозволяє додавати сповіщен
 gun-turret-alerts-artillery-enabled=Дозволяє додати сповіщення про артилерійські боєприпаси до вашого плеєра.
 gun-turret-alerts-mode=Керує, якщо гнізда для боєприпасів підсумовуються, кожен гніздо рахується окремо, або рахується лише вибране гніздо.\n(Застосовується лише для більш ніж одного гнізда для боєприпасів, наприклад, у транспортах)
 gun-turret-alerts-threshold=Якщо у вежі, транспорті або артилерії менше боєприпасів, гравцеві буде оголошено попередження.
+gun-turret-alerts-z-automated-full=Не надсилати сповіщення, коли у вежі або артилерії кількість боєприпасів дорівнює або перевищує кількість, яку автоматично вставляють в них маніпулятори.
+gun-turret-alerts-z-show-planets=Показувати сповіщення для інших планет, навіть якщо вас там немає. (За замовчуванням показувати тільки поточну планету/космічну платформу)
+gun-turret-alerts-z-show-platforms=Показувати сповіщення для інших космічних платформ, навіть якщо ви там не перебуваєте. (За замовчуванням показувати лише поточну планету/космічну платформу)
 
 [string-mod-setting]
 gun-turret-alerts-mode-added=Підсумок
@@ -18,14 +24,16 @@ gun-turret-alerts-mode-individually=Індивідуально
 gun-turret-alerts-mode-selected=Вибрані
 
 [gun-turret-alerts]
-message-empty=__1__ закінчилися боєприпаси
-message-low=__1__ мало боєприпасів
+message-empty=У __1__ закінчилися набої
+message-low=У __1__ мало боєприпасів
 
 [virtual-signal-name]
 ammo-icon-red=Сигнал про закінчення набоїв
 ammo-icon-yellow=Сигнал про низький рівень набоїв
 
 [mod-name]
+Gun_Turret_Alerts=Попередження про боєприпаси
 
 [mod-description]
+Gun_Turret_Alerts=Додає сповіщення на мапі для гравців, коли у гарматній вежі, транспортному засобі або артилерії закінчуються боєприпаси або у них мало боєприпасів.