Correct typo

README.md

@@ -5,11 +5,8 @@
 Adds alerts when a gun turret, vehicle or artillery is out of ammo or has low ammo. The amount of ammo considered low can be configured. Alerts for turrets, vehicles and artillery can be disabled separately. Behaviour on vehicles with multiple ammo slots can be configured (for all vehicles at once). Alerts on other planets or space platforms can be enabled.
 
 ### Locale
-If you want to contribute by translating this mod, you can view the existing translations on [Crowdin](https://crowdin.com/project/factorio-mods-localization). I'd be happy to add your language and credits to the next release.
+If you want to contribute by translating this mod, you can view the existing translations on [Crowdin](https://crowdin.com/project/factorio-mods-localization). I'd be happy to add your language to the next release.
 
 ### Credit
 Thanks to [unhott](https://mods.factorio.com/user/unhott) for the [original mod](https://mods.factorio.com/mod/GunTurretAlerts).  
 Thanks to [snouz](https://mods.factorio.com/user/snouz) for the new icons and thumbnail.  
-Thanks to [Friendch](https://mods.factorio.com/user/Friendch) for the french locale.  
-Thanks to [eugenikus8](https://github.com/eugenikus8) for the russian locale.  
-Thanks to [Met_en_Bouldry](https://crowdin.com/profile/Met_en_Bouldry) for the ukrainian locale.  

changelog.txt

@@ -1,4 +1,28 @@
 ---------------------------------------------------------------------------------------------------
+Version: 2.0.10
+Date: 2025-09-02
+  Bugfixes:
+    - Typo, messed up syntax
+---------------------------------------------------------------------------------------------------
+Version: 2.0.9
+Date: 2025-09-02
+  Bugfixes:
+    - Fix missing requirements
+---------------------------------------------------------------------------------------------------
+Version: 2.0.8
+Date: 2024-12-19
+  Locale:
+    - Update locales from Crowdin
+---------------------------------------------------------------------------------------------------
+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

@@ -1,6 +1,10 @@
 --control.lua
 --This mod scans the map for gun-turrets, vehicles and artillery and places alerts when they are low on ammo.
 
+--requires
+
+require "util"
+
 --util functions
 
 function table_is_empty(table)
@@ -67,6 +71,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 +136,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 +192,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 +264,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.10",
     "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=Додає сповіщення на мапі для гравців, коли у гарматній вежі, транспортному засобі або артилерії закінчуються боєприпаси або у них мало боєприпасів.