From d000b4b20f3e48ce5c4a06856a4b8a435cd049ba Mon Sep 17 00:00:00 2001
From: OgelGames <olliverdc28@gmail.com>
Date: Sat, 20 Jun 2020 22:41:30 +1000
Subject: [PATCH] fix tool description for old and unused tools (#14)

---
 README.md |  2 +-
 init.lua  | 25 ++++++++++++++-----------
 2 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/README.md b/README.md
index 07edc66..4f30c9f 100644
--- a/README.md
+++ b/README.md
@@ -13,7 +13,7 @@ If so, to support this mod, add this code to your mod, after your tool's code:
 if minetest.get_modpath("toolranks") then
     minetest.override_item("mymod:mytool", {
         original_description = "My Tool",
-        description = toolranks.create_description("My Tool", 0, 1),
+        description = toolranks.create_description("My Tool"),
         after_use = toolranks.new_afteruse
     })
     end
diff --git a/init.lua b/init.lua
index 56fdad0..67dfd46 100644
--- a/init.lua
+++ b/init.lua
@@ -32,12 +32,19 @@ function toolranks.get_tool_type(description)
     elseif string.find(d, "sword") then
       return "sword"
     else
-    return "tool"
+      return "tool"
     end
   end
 end
 
-function toolranks.create_description(name, uses, level)
+function toolranks.get_level(uses)
+  if type(uses) == "number" and uses > 0 then
+    return math.min(max_level, math.floor(uses / level_digs))
+  end
+  return 0
+end
+
+function toolranks.create_description(name, uses)
   local description = name
   local tooltype = toolranks.get_tool_type(description)
   local newdesc = S(
@@ -45,18 +52,14 @@ function toolranks.create_description(name, uses, level)
     toolranks.colors.green,
     description,
     toolranks.colors.gold,
-    (level or 1),
+    toolranks.get_level(uses),
     S(tooltype),
     toolranks.colors.grey,
-    (uses or 0)
+    (type(uses) == "number" and uses or 0)
   )
   return newdesc
 end
 
-function toolranks.get_level(uses)
-  return math.min(max_level, math.floor(uses / level_digs))
-end
-
 function toolranks.new_afteruse(itemstack, user, node, digparams)
   local itemmeta = itemstack:get_meta()
   local itemdef = itemstack:get_definition()
@@ -107,7 +110,6 @@ function toolranks.new_afteruse(itemstack, user, node, digparams)
       to_player = user:get_player_name(),
       gain = 2.0,
     })
-    itemmeta:set_string("lastlevel", level)
 
     local speed_multiplier = 1 + (level * level_multiplier * (max_speed - 1))
     local use_multiplier = 1 + (level * level_multiplier * (max_use - 1))
@@ -125,7 +127,8 @@ function toolranks.new_afteruse(itemstack, user, node, digparams)
     itemmeta:set_tool_capabilities(caps)
   end
 
-  itemmeta:set_string("description", toolranks.create_description(itemdesc, dugnodes, level))
+  itemmeta:set_string("lastlevel", level)
+  itemmeta:set_string("description", toolranks.create_description(itemdesc, dugnodes))
   itemstack:add_wear(digparams.wear)
   return itemstack
 end
@@ -135,7 +138,7 @@ function toolranks.add_tool(name)
   local desc = ItemStack(name):get_definition().description
   minetest.override_item(name, {
     original_description = desc,
-    description = toolranks.create_description(desc, 0, 1),
+    description = toolranks.create_description(desc),
     after_use = toolranks.new_afteruse
   })
 end