Fixed migration breaking multiplayer

Removed migration, fixed missing car index through on_config_changed

README.md

@@ -1,10 +1,9 @@
-# Gun Turret Alerts
+# Ammo Alerts
 <img src=https://raw.githubusercontent.com/Wyrrrd/Gun_Turret_Alerts/master/thumbnail.png width="128" height="128">
 
-(Please note: This is a quick and dirty port to 0.18. As soon as the original is ported to 0.18, this will be removed.)
+### Description
+Adds alerts when a car or gun turret is out of ammo or has low ammo.
 
-Description
-- Adds alerts when a gun turret is out of ammo or has low ammo. 
-
-Credit:
-- Thanks to [unhott](https://mods.factorio.com/user/unhott) for the [original mod](https://mods.factorio.com/mod/GunTurretAlerts).
+### 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.

changelog.txt

@@ -1,8 +1,46 @@
 ---------------------------------------------------------------------------------------------------
+Version: 1.1.4
+Date: 2021-04-02
+  Bugfixes:
+    - Fix multiplayer incompatibility by clearing index on game version/mod version/config change
+  Locale:
+    - Added alerts to locale
+    - Customized alerts with localised name of entities
+---------------------------------------------------------------------------------------------------
+Version: 1.1.3
+Date: 2021-03-29
+  Changes:
+    - Icon and message overhaul (by snouz)
+---------------------------------------------------------------------------------------------------
+Version: 1.1.2
+Date: 2021-03-28
+  Bugfixes:
+    - Added migration to add car index to existing saves
+---------------------------------------------------------------------------------------------------
+Version: 1.1.1
+Date: 2021-03-28
+  Changes:
+    - Added car ammo alerts, if car has guns (this should apply to modded cars and tanks)
+    - Added config options to display/hide car and/or turret ammo alerts
+    - Added german locale
+    - Mod display name is now less focused on turrets
+---------------------------------------------------------------------------------------------------
+Version: 1.1.0
+Date: 2020-12-02
+  Changes:
+    - Version bump for base game 1.1
+---------------------------------------------------------------------------------------------------
+Version: 0.18.3
+Date: 2020-03-21
+  Changes:
+    - Migrated on_event to on_nth_tick for performance
+  Bugfixes:
+    - Added missing initialization of turret index
+---------------------------------------------------------------------------------------------------
 Version: 0.18.2
 Date: 2020-03-20
   Changes:
-    - Redesigned surface handling again to be more perfomant
+    - Redesigned surface handling again to be more performant
 ---------------------------------------------------------------------------------------------------
 Version: 0.18.1
 Date: 2020-03-19

control.lua

@@ -1,33 +1,62 @@
 --control.lua
---This mod scans the map for gun-turrets and places alerts when turrets are low. 
+--This mod scans the map for cars and gun-turrets and places alerts when they are low. 
 
-script.on_event({defines.events.on_tick}, function (event)
-	--Every minute the surface is rescanned for ammo-turret type entities. This is stored in the global table. 
-	if event.tick%3600 == 0 or global.turret_entities == nil then
-		local planet = game.surfaces
-		for index,_ in pairs(planet) do
-			global.turret_entities[index] = planet[index].find_entities_filtered{type = "ammo-turret"}
-		end
+script.on_init(function (event)
+	-- index init
+	global.turret_entities = {}
+	global.car_entities = {}
+end)
+
+script.on_configuration_changed(function (event)
+	-- index init fix
+	global.turret_entities = {}
+	global.car_entities = {}
+end)
+
+script.on_nth_tick(3600, function (event)
+	--Every minute the surface is rescanned for car and ammo-turret type entities. This is stored in two global tables. 
+	for index,surface in pairs(game.surfaces) do
+		global.turret_entities[index] = surface.find_entities_filtered{type = "ammo-turret"}
+		global.car_entities[index] = surface.find_entities_filtered{type = "car"}
 	end
-	if event.tick%600 == 0 then
-	--Every 10 seconds recheck and give alerts to players for ammo-turret entities on the same force as them. 
-		for _,player in pairs(game.connected_players) do
-			
-			GTA_enabled = player.mod_settings["gun-turret-alerts-enabled"].value
-			player_threshold = player.mod_settings["gun-turret-alerts-threshold"].value
-			turret_entities = global.turret_entities[player.surface.name]
+end)
 
-			if GTA_enabled and turret_entities then
-				for _,turret_entity in pairs(turret_entities) do
-					if turret_entity.valid and turret_entity.force == player.force then
-						inv_var = turret_entity.get_inventory(defines.inventory.turret_ammo)
-						if inv_var.is_empty() then
-							-- no ammo alert
-							player.add_custom_alert(turret_entity, {type = "item", name = "piercing-rounds-magazine"}, "Out of ammo", true)
-						elseif inv_var[1].count < player_threshold then
-							-- low ammo alert
-							player.add_custom_alert(turret_entity, {type = "item", name = "firearm-magazine"}, "Ammo low", true)
-						end
+script.on_nth_tick(600, function (event)
+	--Every 10 seconds recheck and give alerts to players for car and ammo-turret entities on the same force as them. 
+	for _,player in pairs(game.connected_players) do
+		
+		GTA_turret_enabled = player.mod_settings["gun-turret-alerts-enabled"].value
+		GTA_car_enabled = player.mod_settings["gun-turret-alerts-car-enabled"].value
+		player_threshold = player.mod_settings["gun-turret-alerts-threshold"].value
+		turret_entities = global.turret_entities[player.surface.name]
+		car_entities = global.car_entities[player.surface.name]
+
+		if GTA_turret_enabled and turret_entities then
+			for _,turret_entity in pairs(turret_entities) do
+				if turret_entity.valid and turret_entity.force == player.force then
+					inv_var = turret_entity.get_inventory(defines.inventory.turret_ammo)
+					if inv_var.is_empty() then
+						-- no ammo alert
+						player.add_custom_alert(turret_entity, {type = "virtual", name = "ammo-icon-red"}, {"gun-turret-alerts.message-empty", turret_entity.localised_name}, true)
+					elseif inv_var[1].count < player_threshold then
+						-- low ammo alert
+						player.add_custom_alert(turret_entity, {type = "virtual", name = "ammo-icon-yellow"}, {"gun-turret-alerts.message-low", turret_entity.localised_name}, true)
+					end
+				end
+			end
+		end
+		
+		if GTA_car_enabled and car_entities then
+			for _,car_entity in pairs(car_entities) do
+				-- extra check if car has gun
+				if car_entity.valid and car_entity.force == player.force and car_entity.selected_gun_index then
+					inv_var = car_entity.get_inventory(defines.inventory.car_ammo)
+					if inv_var.is_empty() then
+						-- no ammo alert
+						player.add_custom_alert(car_entity, {type = "virtual", name = "ammo-icon-red"}, {"gun-turret-alerts.message-empty", car_entity.localised_name}, true)
+					elseif inv_var[1].count < player_threshold then
+						-- low ammo alert
+						player.add_custom_alert(car_entity, {type = "virtual", name = "ammo-icon-yellow"}, {"gun-turret-alerts.message-low", car_entity.localised_name}, true)
 					end
 				end
 			end

data.lua

@@ -1 +1,2 @@
 --data.lua
+require("prototypes.signals")

info.json

@@ -1,9 +1,9 @@
 {
     "name": "Gun_Turret_Alerts",
-    "version": "0.18.2",
-    "title": "Gun_Turret_Alerts",
+    "version": "1.1.4",
+    "title": "Ammo Alerts",
     "author": "Wyrrrd",
-    "factorio_version": "0.18",
-    "dependencies": ["base >= 0.18"],
-    "description": "Adds map alerts for players when a turret is either out of ammo or has low ammo"
+    "factorio_version": "1.1",
+    "dependencies": ["base >= 1.1.0"],
+    "description": "Adds map alerts for players when a turret or car is either out of ammo or has low ammo"
 }

locale/de/config.cfg

@@ -0,0 +1,13 @@
+[mod-setting-name]
+gun-turret-alerts-enabled=Alarme für Geschützturmmunition aktiviert
+gun-turret-alerts-car-enabled=Alarme für Fahrzeugmunition aktiviert
+gun-turret-alerts-threshold=Grenze für wenig Munition
+
+[mod-setting-description]
+gun-turret-alerts-enabled=Aktiviert Alarme für Geschützturmmunition für deinen Spieler
+gun-turret-alerts-car-enabled=Aktiviert Alarme für Fahrzeugmunition für deinen Spieler
+gun-turret-alerts-threshold=Wenn ein Fahrzeug oder Geschützturm weniger als diese Anzahl Munition hat, wird ein Alarm zu deinem Spieler hinzugefügt
+
+[gun-turret-alerts]
+message-empty=__1__ hat keine Munition
+message-low=__1__ hat wenig Munition

locale/en/config.cfg

@@ -1,7 +1,13 @@
 [mod-setting-name]
-gun-turret-alerts-enabled=Gun turret alerts enabled
+gun-turret-alerts-enabled=Turret ammo alerts enabled
+gun-turret-alerts-car-enabled=Car ammo alerts enabled
 gun-turret-alerts-threshold=Low ammo threshold
 
 [mod-setting-description]
-gun-turret-alerts-enabled=Enables alerts to be added to your player
-gun-turret-alerts-threshold=If a turret has less than this much ammo an alert will be added to your player
+gun-turret-alerts-enabled=Enables turret ammo alerts to be added to your player
+gun-turret-alerts-car-enabled=Enables car ammo alerts to be added to your player
+gun-turret-alerts-threshold=If a car or turret has less than this much ammo an alert will be added to your player
+
+[gun-turret-alerts]
+message-empty=__1__ out of ammo
+message-low=__1__ ammo low

prototypes/signals.lua

@@ -0,0 +1,14 @@
+data:extend({
+	{
+		type = "virtual-signal",
+  		name = "ammo-icon-red",
+  		icon = "__Gun_Turret_Alerts__/graphics/ammo-icon-red.png",
+  		icon_size = 64,
+	},
+	{
+		type = "virtual-signal",
+  		name = "ammo-icon-yellow",
+  		icon = "__Gun_Turret_Alerts__/graphics/ammo-icon-yellow.png",
+  		icon_size = 64,
+	}
+})

settings.lua

@@ -7,6 +7,12 @@ data:extend({
         name = "gun-turret-alerts-enabled",
         setting_type = "runtime-per-user",
         default_value = true
+    },
+    {
+        type = "bool-setting",
+        name = "gun-turret-alerts-car-enabled",
+        setting_type = "runtime-per-user",
+        default_value = true
     },
 	{
         type = "int-setting",