refactor(items): ignore non-string item names in getItem

People repeatedly send number values here.
I've yet to figure out their thought patterns, though they may lack any sort of conscious thought.

modules/items/client.lua

@@ -16,17 +16,17 @@ exports('displayMetadata', displayMetadata)
 ---@param name string?
 ---@return table?
 local function getItem(_, name)
-	if name then
-		name = name:lower()
+    if not name then return Items end
 
-		if name:sub(0, 7) == 'weapon_' then
-			name = name:upper()
-		end
+	if type(name) ~= 'string' then return end
 
-		return Items[name]
-	end
+    name = name:lower()
+
+    if name:sub(0, 7) == 'weapon_' then
+        name = name:upper()
+    end
 
-	return Items
+    return Items[name]
 end
 
 setmetatable(Items --[[@as table]], {

modules/items/server.lua

@@ -25,17 +25,17 @@ local trash = {
 ---@param name string?
 ---@return table?
 local function getItem(_, name)
-	if name then
-		name = name:lower()
+    if not name then return ItemList end
 
-		if name:sub(0, 7) == 'weapon_' then
-			name = name:upper()
-		end
+	if type(name) ~= 'string' then return end
 
-		return ItemList[name]
-	end
+    name = name:lower()
+
+    if name:sub(0, 7) == 'weapon_' then
+        name = name:upper()
+    end
 
-	return ItemList
+    return ItemList[name]
 end
 
 setmetatable(Items --[[@as table]], {