18 tooltip showing wrong itemrankings (#19)

* moved table sorting from tooltip trigger to db import function

* increased version

LootplanIt.toc

@@ -2,7 +2,7 @@
 ## Title: LootPlanIt
 ## Author: zahdr
 ## Notes: Make Loot Council easy!
-## Version: v0.2.3
+## Version: v0.2.4
 ## DefaultState: enabled
 ## SavedVariables: LootPlanItDB
 ## X-Curse-Project-ID: 876058

classes/db.lua

@@ -3,6 +3,23 @@ local _, LPI = ...;
 LPI.db = {}
 db = LPI.db
 
+
+local function sortByItemRating(a, b)
+    local ratingA = tonumber(string.sub(a["itemRating"], 1, -2))
+    local ratingB = tonumber(string.sub(b["itemRating"], 1, -2))
+
+    return ratingA > ratingB
+end
+
+local function sortSubTablesByItemRating(tbl)
+    for key, subTable in pairs(tbl) do
+        if #subTable > 1 then
+            table.sort(subTable, sortByItemRating)
+        end
+    end
+end
+
+
 function db:init()
     if not LootPlanItDB then
         LootPlanItDB = {}
@@ -31,6 +48,8 @@ function db:resetLootCouncilData()
 end
 
 function LPI:importLootCouncilData(data)
+    local tmpTbl = {}
+
     if type(data) == "string" then
         data = LPI:convertStringToTable(data)
     end
@@ -40,9 +59,9 @@ function LPI:importLootCouncilData(data)
         local itemRating = entry[2]
         local itemPrio = entry[3]
         local playerName = entry[4]  
-        
-        if not LootPlanItDB.lootCouncil.itemIds[itemId] then
-            LootPlanItDB.lootCouncil.itemIds[itemId] = {}
+
+        if not tmpTbl[itemId] then
+            tmpTbl[itemId] = {}
         end
 
         local record = {
@@ -51,8 +70,11 @@ function LPI:importLootCouncilData(data)
             playerName = playerName
         }
 
-        table.insert(LootPlanItDB.lootCouncil.itemIds[itemId], record)
-    end       
+        table.insert(tmpTbl[itemId], record)
+    end
+
+    sortSubTablesByItemRating(tmpTbl)
+    LootPlanItDB.lootCouncil.itemIds = tmpTbl     
     LPI:infoMessage("Loot Council Items imported successfully!")
 end
 

classes/helper.lua

@@ -27,5 +27,6 @@ function LPI:convertStringToTable(string)
         end
         table.insert(dataTable, tmpTable)
     end
+
     return dataTable
-end
+end

classes/tooltip.lua

@@ -1,19 +1,11 @@
 local _, LPI = ...;
 
-local function compareItemRating(a, b)
-    local ratingA = tonumber(a.itemRating:sub(1, -2)) / 100
-    local ratingB = tonumber(b.itemRating:sub(1, -2)) / 100
-
-    return ratingA > ratingB
-end
-
 function tooltipAddPrios(tooltip)
 	local _, itemLink = tooltip:GetItem()
 	local itemId = tostring(GetItemInfoFromHyperlink(itemLink))
 	local loopStop = 0
 
 	if LootPlanItDB.lootCouncil.itemIds[itemId] then
-		table.sort(LootPlanItDB.lootCouncil.itemIds[itemId], compareItemRating)
 		for _, record in pairs(LootPlanItDB.lootCouncil.itemIds[itemId]) do
 			if loopStop == 0 then
 				loopStop = 1