optimize textures of thos stupid developers, later mckayshiour will change it

* register armor using `armor:register_armor` (fixes #4)
* improve code readability
* make `technic_worldgen` optional
* add screenshot
* update readme and mod.conf
* remove integration test (unnecessary for a simple mod)
* remove other unnecessary files

.github/workflows/luacheck.yml

@@ -0,0 +1,13 @@
+name: luacheck
+on: [push, pull_request]
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@main
+    - name: apt
+      run: sudo apt-get install -y luarocks
+    - name: luacheck install
+      run: luarocks install --local luacheck
+    - name: luacheck run
+      run: $HOME/.luarocks/bin/luacheck ./

.gitignore

@@ -1,10 +0,0 @@
-## Generic ignorable patterns and files
-*~
-.*.swp
-*bak*
-tags
-*.vim
-
-## Eclipse project files & directories
-.project
-.settings

.luacheckrc

@@ -0,0 +1,4 @@
+globals = {
+	"armor",
+	"minetest",
+}

LICENSE.txt

@@ -1,7 +1,4 @@
-[mod] Technic Armor [technic_armor]
-===================================
-
 License Source Code: Copyright (C) 2013-2018 Stuart Jones - LGPL v2.1
 
-License Textures: poet.nohit and numberZero - 2015-2018 WTFPL
+License Textures: poet-nohit and numberZero - 2015-2018 WTFPL
 

README.md

@@ -0,0 +1,9 @@
+# Technic Armor [technic_armor]
+
+[![luacheck](https://github.com/mt-mods/technic_armor/workflows/luacheck/badge.svg)](https://github.com/mt-mods/technic_armor/actions)
+
+Adds armor made from lead, brass, cast iron, carbon steel, stainless steel, tin and silver.
+
+![](screenshot.png?raw=true)
+
+Textures by @poet-nohit and @numberZero

README.txt

@@ -1,9 +0,0 @@
-[mod] Technic Armor [technic_armor]
-===================================
-
-Adds tin, silver and technic materials to 3d_armor.
-Requires technic (technic_worldgen at least) mod.
-
-Depends: 3d_armor, technic_worldgen
-
-Textures by poet.nohit and numberZero

depends.txt

@@ -1,3 +0,0 @@
-3d_armor
-technic_worldgen?
-moreores?

description.txt

@@ -1 +0,0 @@
-Adds tin, silver and technic materials to 3d_armor.

init.lua

@@ -1,66 +1,149 @@
--- support for i18n
-local S = armor_i18n.gettext
-local F = armor_i18n.fgettext
 
-if not minetest.get_modpath("technic_worldgen") then
+-- Use 3d_armor translator to take advantage of existing translations for armor parts
-	minetest.log("warning", S("technic_armor: Mod loaded but unused."))
+local S = minetest.get_translator("3d_armor")
-	return
+
+local materials = {}
+
+if minetest.get_modpath("technic_worldgen") then
+	materials.lead = {
+		name = S("Lead"),
+		craft_item = "technic:lead_ingot",
+		armor = 1.6,
+		heal = 0,
+		use = 500,
+		radiation = 88
+	}
+	materials.brass = {
+		name = S("Brass"),
+		craft_item = "technic:brass_ingot",
+		armor = 1.8,
+		heal = 0,
+		use = 650,
+		radiation = 43
+	}
+	materials.cast = {
+		name = S("Cast Iron"),
+		craft_item = "technic:cast_iron_ingot",
+		armor = 2.5,
+		heal = 8,
+		use = 200,
+		radiation = 40
+	}
+	materials.carbon = {
+		name = S("Carbon Steel"),
+		craft_item = "technic:carbon_steel_ingot",
+		armor = 2.7,
+		heal = 10,
+		use = 100,
+		radiation = 40
+	}
+	materials.stainless = {
+		name = S("Stainless Steel"),
+		craft_item = "technic:stainless_steel_ingot",
+		armor = 2.7,
+		heal = 10,
+		use = 75,
+		radiation = 40
+	}
+end
+
+local tin_ingot
+
+if minetest.registered_items["default:tin_ingot"] then
+	tin_ingot = "default:tin_ingot"
 end
 
-local stats = {
-	lead = { name=S("Lead"), material="technic:lead_ingot", armor=1.6, heal=0, use=500, radiation=80*1.1 },
-	brass = { name=S("Brass"), material="technic:brass_ingot", armor=1.8, heal=0, use=650, radiation=43 },
-	cast = { name=S("Cast Iron"), material="technic:cast_iron_ingot", armor=2.5, heal=8, use=200, radiation=40 },
-	carbon = { name=S("Carbon Steel"), material="technic:carbon_steel_ingot", armor=2.7, heal=10, use=100, radiation=40 },
-	stainless = { name=S("Stainless Steel"), material="technic:stainless_steel_ingot", armor=2.7, heal=10, use=75, radiation=40 },
-}
 if minetest.get_modpath("moreores") then
-	stats.tin = { name=S("Tin"), material="moreores:tin_ingot", armor=1.6, heal=0, use=750, radiation=37 }
+	tin_ingot = "moreores:tin_ingot"
-	stats.silver = { name=S("Silver"), material="moreores:silver_ingot", armor=1.8, heal=6, use=650, radiation=53 }
+	materials.silver = {
+		name = S("Silver"),
+		craft_item = "moreores:silver_ingot",
+		armor = 1.8,
+		heal = 6,
+		use = 650,
+		radiation = 53
+	}
+end
+
+if tin_ingot then
+	materials.tin = {
+		name = S("Tin"),
+		craft_item = tin_ingot,
+		armor = 1.6,
+		heal = 0,
+		use = 750,
+		radiation = 37
+	}
 end
 
 local parts = {
-	helmet = { place="head", name=S("Helmet"), level=5, radlevel = 0.10, craft={{1,1,1},{1,0,1}} },
+	helmet = {
-	chestplate = { place="torso", name=S("Chestplate"), level=8, radlevel = 0.35, craft={{1,0,1},{1,1,1},{1,1,1}} },
+		name = S("Helmet"),
-	leggings = { place="legs", name=S("Leggings"), level=7, radlevel = 0.15, craft={{1,1,1},{1,0,1},{1,0,1}} },
+		place = "head",
-	boots = { place="feet", name=S("Boots"), level=4, radlevel = 0.10, craft={{1,0,1},{1,0,1}} },
+		level = 5,
+		radlevel = 0.10,
+		craft = {{1, 1, 1}, {1, 0, 1}}
+	},
+	chestplate = {
+		name = S("Chestplate"),
+		place = "torso",
+		level = 8,
+		radlevel = 0.35,
+		craft = {{1, 0, 1}, {1, 1, 1}, {1, 1, 1}}
+	},
+	leggings = {
+		name = S("Leggings"),
+		place = "legs",
+		level = 7,
+		radlevel = 0.15,
+		craft = {{1, 1, 1}, {1, 0, 1}, {1, 0, 1}}
+	},
+	boots = {
+		name = S("Boots"),
+		place = "feet",
+		level = 4,
+		radlevel = 0.10,
+		craft = {{1, 0, 1}, {1, 0, 1}}
 	}
+}
+
 if minetest.get_modpath("shields") then
-	parts.shield = { place="shield", name=S("Shield"), level=5, radlevel=0.00, craft={{1,1,1},{1,1,1},{0,1,0}} }
+	parts.shield = {
+		name = S("Shield"),
+		place = "shield",
+		level = 5,
+		radlevel = 0.00,
+		craft = {{1, 1, 1}, {1, 1, 1}, {0, 1, 0}}
+	}
 end
 
--- Makes a craft recipe based on a template
+local function make_recipe(template, material)
--- template is a recipe-like table but indices are used instead of actual item names:
--- 0 means nothing, everything else is treated as an index in the materials table
-local function make_recipe(template, materials)
 	local recipe = {}
-	for j, trow in ipairs(template) do
+	for i, row in ipairs(template) do
-		local rrow = {}
+		recipe[i] = {}
-		for i, tcell in ipairs(trow) do
+		for j, item in ipairs(row) do
-			if tcell == 0 then
+			recipe[i][j] = item == 0 and "" or material
-				rrow[i] = ""
-			else
-				rrow[i] = materials[tcell]
 		end
 	end
-		recipe[j] = rrow
-	end
 	return recipe
 end
 
-for key, armor in pairs(stats) do
+for material, m in pairs(materials) do
-	for partkey, part in pairs(parts) do
+	for part, p in pairs(parts) do
-		local partname = "technic_armor:"..partkey.."_"..key
+		local name = "technic_armor:"..part.."_"..material
-		minetest.register_tool(partname, {
+		armor:register_armor(name, {
-			-- Translators: @1 stands for material and @2 for part of the armor, so that you could use a conjunction if in your language part name comes first then material (e.g. in french 'Silver Boots' is translated in 'Bottes en argent' by using '@2 en @1' as translated string)
+			description = S("@1 @2", m.name, p.name),
-			description = S("@1 @2", armor.name, part.name),
+			inventory_image = "technic_armor_inv_"..part.."_"..material..".png",
-			inventory_image = "technic_armor_inv_"..partkey.."_"..key..".png",
+			groups = {
-			groups = {["armor_"..part.place]=math.floor(part.level*armor.armor), armor_heal=armor.heal, armor_use=armor.use, armor_radiation=math.floor(part.radlevel*armor.radiation)},
+				["armor_"..p.place] = math.floor(p.level * m.armor),
-			wear = 0,
+				armor_heal = m.heal,
+				armor_use = m.use,
+				armor_radiation = math.floor(p.radlevel * m.radiation)
+			}
 		})
 		minetest.register_craft({
-			output = partname,
+			output = name,
-			recipe = make_recipe(part.craft, {armor.material}),
+			recipe = make_recipe(p.craft, m.craft_item),
 		})
 	end
 end

mod.conf

@@ -0,0 +1,4 @@
+name = technic_armor
+description = Adds armor made from lead, brass, cast iron, carbon steel, stainless steel, tin and silver
+depends = 3d_armor
+optional_depends = default, moreores, technic_worldgen

textures/preview_index.txt

@@ -1,41 +0,0 @@
-technic_armor/textures/technic_armor_helmet_brass.png:head
-technic_armor/textures/technic_armor_chestplate_brass.png:torso
-technic_armor/textures/technic_armor_leggings_brass.png:legs
-technic_armor/textures/technic_armor_boots_brass.png:feet
-technic_armor/textures/technic_armor_shield_brass.png:shield
-
-technic_armor/textures/technic_armor_helmet_cast.png:head
-technic_armor/textures/technic_armor_chestplate_cast.png:torso
-technic_armor/textures/technic_armor_leggings_cast.png:legs
-technic_armor/textures/technic_armor_boots_cast.png:feet
-technic_armor/textures/technic_armor_shield_cast.png:shield
-
-technic_armor/textures/technic_armor_helmet_stainless.png:head
-technic_armor/textures/technic_armor_chestplate_stainless.png:torso
-technic_armor/textures/technic_armor_leggings_stainless.png:legs
-technic_armor/textures/technic_armor_boots_stainless.png:feet
-technic_armor/textures/technic_armor_shield_stainless.png:shield
-
-technic_armor/textures/technic_armor_helmet_tin.png:head
-technic_armor/textures/technic_armor_chestplate_tin.png:torso
-technic_armor/textures/technic_armor_leggings_tin.png:legs
-technic_armor/textures/technic_armor_boots_tin.png:feet
-technic_armor/textures/technic_armor_shield_tin.png:shield
-
-technic_armor/textures/technic_armor_helmet_lead.png:head
-technic_armor/textures/technic_armor_chestplate_lead.png:torso
-technic_armor/textures/technic_armor_leggings_lead.png:legs
-technic_armor/textures/technic_armor_boots_lead.png:feet
-technic_armor/textures/technic_armor_shield_lead.png:shield
-
-technic_armor/textures/technic_armor_helmet_carbon.png:head
-technic_armor/textures/technic_armor_chestplate_carbon.png:torso
-technic_armor/textures/technic_armor_leggings_carbon.png:legs
-technic_armor/textures/technic_armor_boots_carbon.png:feet
-technic_armor/textures/technic_armor_shield_carbon.png:shield
-
-technic_armor/textures/technic_armor_helmet_silver.png:head
-technic_armor/textures/technic_armor_chestplate_silver.png:torso
-technic_armor/textures/technic_armor_leggings_silver.png:legs
-technic_armor/textures/technic_armor_boots_silver.png:feet
-technic_armor/textures/technic_armor_shield_silver.png:shield