Module:RaritySort: Difference between revisions

← Older edit

@@ Line 1: / Line 1: @@
 local p = {}
-function p.getSortValue(rarity)
+-- Clean and extract processing info
-     if rarity == nil or type(rarity) ~= "string" then
+local function clean(rarity)
-         return 99
+     if not rarity or type(rarity) ~= "string" then return nil, false end
+    rarity = mw.text.trim(rarity)
+    local hasTilde = rarity:sub(1, 1) == "~"
+    if hasTilde then
+         rarity = rarity:sub(2)
      end
+    -- Strip commas (e.g. "1/10,000" → "1/10000")
+    rarity = rarity:gsub(",", "")
+    return rarity, hasTilde
+end
+-- RaritySort for sorting
+function p.getSortValue(frame)
+    local rarityRaw = frame.args[1]
+    local rarity, _ = clean(rarityRaw)
+    if not rarity then return 999999999 end
+    local lowered = rarity:lower()
-     local raw = rarity
+     if lowered == "1" or lowered == "always" then
-     rarity = mw.text.trim(rarity)
+        return 1
-     local r = mw.ustring.lower(rarity)
+     elseif lowered == "rare" then
+        return 999999
+     elseif lowered == "never" then
+        return 0
+    end
-     -- Debug: remove these after testing
+     -- Match 1/X and round down (floor)
-     if mw.title.getCurrentTitle().text == "Sandbox" then
+    local numStr = lowered:match("^1/(%d+%.?%d*)$")
-        return "RAW: " .. tostring(raw) .. " | TRIMMED: " .. rarity
+     if numStr then
+        local num = tonumber(numStr)
+        if num then
+            return math.floor(num)
+        end
      end
-     if r == "always" then
+    return 999999999
-        return 1
+end
-     elseif r:match("^1/%d+$") then
-        return 2
+-- Normalize display text
-    elseif r == "rare" then
+function p.normalizeDisplay(frame)
-        return 3
+    local original = frame.args[1]
-     elseif r == "never" then
+     if not original or type(original) ~= "string" then return "Unknown" end
-         return 4
-    else
+     local trimmed = mw.text.trim(original)
-        return 99
+    local hasTilde = trimmed:sub(1, 1) == "~"
+    local raw = hasTilde and trimmed:sub(2) or trimmed
+     if raw == "1" then
+         return "Always"
      end
+    return hasTilde and ("~" .. raw) or raw
 end
 return p