coa.7: guard Skills tab summary against unscanned skill ranks

GetColor() did floor(rank/..) with no nil-guard and riding did 'field >= 300';
skillRank1/2/cooking/firstaid/fishing/riding are nil for chars DataStore_Characters
hasn't scanned. GetColor now defaults rank to 0 and each field falls back to 0.

Altoholic/Altoholic.lua

@@ -249,10 +249,12 @@ end
 
 
 -- *** Event Handlers ***
+local hasScannedThisSession
 local function OnPlayerAlive()
-	-- print("Altoholic.lua") -- DEBUG 2025 07 21
-	if not UnitIsGhost("player") then return end -- only scan if player released spirit and went to graveyard
-	
+	-- CoA: scan once at login (see DataStore_Inventory / commit fdcb25a). FRIENDLIST_UPDATE also
+	-- keeps the friends list fresh; this just guarantees an initial scan without rescanning on res.
+	if hasScannedThisSession then return end
+	hasScannedThisSession = true
 	ScanFriends()
 end
 
@@ -585,6 +587,22 @@ function addon:GetDelayInDays(delay)
 	return floor((time() - delay) / 86400)
 end
 
+-- CoA: shared, nil-safe character display helpers.
+-- DataStore char-based getters return *no value* for any module that hasn't
+-- scanned a given character (DataStore.lua: "if not arg1.lastUpdate then return end").
+-- Fresh alts have partial per-module data, so every field is guarded here once
+-- instead of being copy-pasted (and missed) across the frames.
+function Altoholic:AddCharacterTooltipHeader(character)
+	AltoTooltip:AddDoubleLine(DS:GetColoredCharacterName(character) or "?", DS:GetColoredCharacterFaction(character) or "")
+	AltoTooltip:AddLine(format("%s %s |r%s %s", L["Level"],
+		GREEN..(DS:GetCharacterLevel(character) or 0), DS:GetCharacterRace(character) or "", DS:GetCharacterClass(character) or ""), 1, 1, 1)
+end
+
+function Altoholic:SetCharacterRowNameLevel(entry, i, icon, character)
+	_G[entry..i.."NameNormalText"]:SetText(icon .. format("%s (%s)", DS:GetColoredCharacterName(character) or "?", DS:GetCharacterClass(character) or ""))
+	_G[entry..i.."Level"]:SetText(GREEN .. (DS:GetCharacterLevel(character) or 0))
+end
+
 function Altoholic:FormatDelay(timeStamp)
 	-- timeStamp = value when time() was last called for a given variable (ex: last time the mailbox was checked)
 	if not timeStamp then
@@ -679,21 +697,18 @@ function Altoholic:ShowClassIcons()
 	local realm, account = Altoholic:GetCurrentRealm()
 	
 	
-	-- ####################
+	-- Sort characters by level first, then average item level. The getters yield no value
+	-- for alts whose Characters/Inventory module hasn't scanned them, so default [3]/[4] to 0.
 	local CharNameList = DS:GetCharacters(realm, account)
 	local CharNameList_sort = {}
-	
+
 	for k,v in pairs(CharNameList) do
-	   table.insert(CharNameList_sort,{k,v, DS:GetAverageItemLevel(v), DS:GetCharacterLevel(v)})
+	   table.insert(CharNameList_sort, {k, v, DS:GetAverageItemLevel(v) or 0, DS:GetCharacterLevel(v) or 0})
 	end
-	
-	-- sort for level first, avg iLevel secondly
+
 	table.sort(CharNameList_sort, function(a,b) return b[3]+b[4]*10000 < a[3]+a[4]*10000 end)
-	-- DEBUG_CHARLIST = CharNameList
-	-- print("-- altoholic DEBUG READY")
-	-- ####################
-	
-	
+
+
 	-- for characterName, character in pairs(DS:GetCharacters(realm, account)) do
 	for _,charTbl in ipairs(CharNameList_sort) do
 		local characterName, character = charTbl[1], charTbl[2]
@@ -763,10 +778,7 @@ function Altoholic:DrawCharacterTooltip(self, charName)
 	
 	AltoTooltip:SetOwner(self, "ANCHOR_LEFT");
 	AltoTooltip:ClearLines();
-	AltoTooltip:AddDoubleLine(DS:GetColoredCharacterName(character), DS:GetColoredCharacterFaction(character))
-
-	AltoTooltip:AddLine(format("%s %s |r%s %s", L["Level"],
-		GREEN..(DS:GetCharacterLevel(character) or 0), DS:GetCharacterRace(character) or "",	DS:GetCharacterClass(character) or ""),1,1,1)
+	Altoholic:AddCharacterTooltipHeader(character)
 
 	local zone, subZone = DS:GetLocation(character)
 	AltoTooltip:AddLine(format("%s: %s |r(%s|r)", L["Zone"], GOLD..(zone or "?"), GOLD..(subZone or "")),1,1,1)

Altoholic/Altoholic.toc

@@ -13,7 +13,7 @@
 
 ## Author: Thaoky, Telkar-RG
 ## X-Edited-By: Exiles (Sub-Net) — florian.berthold@sub-net.at
-## Version: 3.3.002b-coa.4
+## Version: 3.3.002b-coa.7
 ## X-Category: Inventory, Tradeskill, Mail
 ## X-Localizations: enUS, frFR, zhCN, zhTW, deDE, koKR, esES, esMX, ruRU
 ## X-Website: http://wow.curse.com/downloads/wow-addons/details/altoholic.aspx

Altoholic/Frames/AccountSharing.lua

@@ -279,7 +279,7 @@ local ContentScrollFrame_Desc = {
 		[CHARACTER_HEADER_LINE] = {
 			GetText = function(self, line)
 					local _, realm, name = strsplit(".", line.key)
-					return format("%s|r / %s", WHITE..realm, DataStore:GetColoredCharacterName(line.key))
+					return format("%s|r / %s", WHITE..realm, DataStore:GetColoredCharacterName(line.key) or "?")
 				end,
 			GetOffset = function(self, line)
 					return 20

Altoholic/Frames/AccountSummary.lua

@@ -249,10 +249,7 @@ function ns:Update()
 					_G[entry..i.."Name"]:SetWidth(170)
 					_G[entry..i.."Name"]:SetPoint("TOPLEFT", 10, 0)
 					_G[entry..i.."NameNormalText"]:SetWidth(170)
-					-- CoA: DataStore char-based getters return *no value* for any module that hasn't scanned this char
-					-- (DataStore.lua: "if not arg1.lastUpdate then return end"). Fresh alts have partial module data, so guard every result.
-					_G[entry..i.."NameNormalText"]:SetText(icon .. format("%s (%s)", DS:GetColoredCharacterName(character) or "?", DS:GetCharacterClass(character) or ""))
-					_G[entry..i.."Level"]:SetText(GREEN .. (DS:GetCharacterLevel(character) or 0))
+					addon:SetCharacterRowNameLevel(entry, i, icon, character)
 
 					_G[entry..i.."Money"]:SetText(addon:GetMoneyString(DS:GetMoney(character) or 0))
 					_G[entry..i.."Played"]:SetText(addon:GetTimeString(DS:GetPlayTime(character) or 0))
@@ -344,9 +341,7 @@ function ns:Level_OnEnter(frame)
 	AltoTooltip:ClearLines();
 	AltoTooltip:SetOwner(frame, "ANCHOR_RIGHT");
 	
-	AltoTooltip:AddDoubleLine(DS:GetColoredCharacterName(character), DS:GetColoredCharacterFaction(character))
-	AltoTooltip:AddLine(format("%s %s |r%s %s", L["Level"],
-		GREEN..(DS:GetCharacterLevel(character) or 0), DS:GetCharacterRace(character) or "",	DS:GetCharacterClass(character) or ""),1,1,1)
+	addon:AddCharacterTooltipHeader(character)
 
 	local zone, subZone = DS:GetLocation(character)
 	AltoTooltip:AddLine(format("%s: %s |r(%s|r)", L["Zone"], GOLD..(zone or "?"), GOLD..(subZone or "")),1,1,1)

Altoholic/Frames/Activity.lua

@@ -100,8 +100,7 @@ function ns:Update()
 					_G[entry..i.."Name"]:SetWidth(170)
 					_G[entry..i.."Name"]:SetPoint("TOPLEFT", 10, 0)
 					_G[entry..i.."NameNormalText"]:SetWidth(170)
-					_G[entry..i.."NameNormalText"]:SetText(icon .. format("%s (%s)", DS:GetColoredCharacterName(character) or "?", DS:GetCharacterClass(character) or ""))
-					_G[entry..i.."Level"]:SetText(GREEN .. (DS:GetCharacterLevel(character) or 0))
+					addon:SetCharacterRowNameLevel(entry, i, icon, character)
 				
 					local color
 					local num = DS:GetNumMails(character) or 0
@@ -181,9 +180,7 @@ function ns:OnEnter(self)
 	AltoTooltip:ClearLines();
 	AltoTooltip:SetOwner(self, "ANCHOR_RIGHT");
 	
-	AltoTooltip:AddDoubleLine(DS:GetColoredCharacterName(character), DS:GetColoredCharacterFaction(character))
-	AltoTooltip:AddLine(format("%s %s |r%s %s", L["Level"],
-		GREEN..(DS:GetCharacterLevel(character) or 0), DS:GetCharacterRace(character) or "",	DS:GetCharacterClass(character) or ""),1,1,1)
+	addon:AddCharacterTooltipHeader(character)
 
 	local zone, subZone = DS:GetLocation(character)
 	AltoTooltip:AddLine(format("%s: %s |r(%s|r)", L["Zone"], GOLD..(zone or "?"), GOLD..(subZone or "")),1,1,1)

Altoholic/Frames/BagUsage.lua

@@ -96,8 +96,7 @@ function ns:Update()
 					_G[entry..i.."Name"]:SetWidth(170)
 					_G[entry..i.."Name"]:SetPoint("TOPLEFT", 10, 0)
 					_G[entry..i.."NameNormalText"]:SetWidth(170)
-					_G[entry..i.."NameNormalText"]:SetText(icon .. format("%s (%s)", DS:GetColoredCharacterName(character) or "?", DS:GetCharacterClass(character) or ""))
-					_G[entry..i.."Level"]:SetText(GREEN .. (DS:GetCharacterLevel(character) or 0))
+					addon:SetCharacterRowNameLevel(entry, i, icon, character)
 
 					_G[entry..i.."FreeBags"]:SetText(GREEN .. (DS:GetNumFreeBagSlots(character) or 0))
 					_G[entry..i.."FreeBank"]:SetText(GREEN .. (DS:GetNumFreeBankSlots(character) or 0))
@@ -182,9 +181,7 @@ function ns:OnEnter(self)
 	
 	AltoTooltip:ClearLines();
 	AltoTooltip:SetOwner(self, "ANCHOR_RIGHT");
-	AltoTooltip:AddDoubleLine(DS:GetColoredCharacterName(character), DS:GetColoredCharacterFaction(character))
-	AltoTooltip:AddLine(format("%s %s |r%s %s", L["Level"],
-		GREEN..(DS:GetCharacterLevel(character) or 0), DS:GetCharacterRace(character) or "",	DS:GetCharacterClass(character) or ""),1,1,1)
+	addon:AddCharacterTooltipHeader(character)
 	AltoTooltip:AddLine(" ",1,1,1);
 
 	local id = self:GetID()

Altoholic/Frames/Keys.lua

@@ -789,7 +789,7 @@ function ns:Item_OnEnter(frame)
 		
 		AltoTooltip:SetOwner(frame, "ANCHOR_LEFT")
 		AltoTooltip:ClearLines()
-		AltoTooltip:AddLine( format("%s|r: %s", DS:GetColoredCharacterName(character), nameKey) )
+		AltoTooltip:AddLine( format("%s|r: %s", DS:GetColoredCharacterName(character) or "?", nameKey) )
 		AltoTooltip:AddLine(" ")
 		
 		local questDone, cr,cg,cb
@@ -848,7 +848,7 @@ function ns:Item_OnEnter(frame)
 		else
 			AltoTooltip:SetOwner(frame, "ANCHOR_LEFT")
 			AltoTooltip:ClearLines()
-			AltoTooltip:AddLine( format("%s|r: %s", DS:GetColoredCharacterName(character), nameKey) )
+			AltoTooltip:AddLine( format("%s|r: %s", DS:GetColoredCharacterName(character) or "?", nameKey) )
 			AltoTooltip:AddLine(" ")
 			
 			local questDone, cr,cg,cb
@@ -912,7 +912,7 @@ function ns:Item_OnEnter(frame)
 		else
 			AltoTooltip:SetOwner(frame, "ANCHOR_LEFT")
 			AltoTooltip:ClearLines()
-			AltoTooltip:AddLine( format("%s|r: %s", DS:GetColoredCharacterName(character), nameKey) )
+			AltoTooltip:AddLine( format("%s|r: %s", DS:GetColoredCharacterName(character) or "?", nameKey) )
 			AltoTooltip:AddLine(" ")
 			AltoTooltip:AddLine(L["Required reputation"] .. ":",1,1,1)
 			

Altoholic/Frames/Skills.lua

@@ -109,8 +109,7 @@ function ns:Update()
 					_G[entry..i.."Name"]:SetWidth(170)
 					_G[entry..i.."Name"]:SetPoint("TOPLEFT", 10, 0)
 					_G[entry..i.."NameNormalText"]:SetWidth(170)
-					_G[entry..i.."NameNormalText"]:SetText(icon .. format("%s (%s)", DS:GetColoredCharacterName(character), DS:GetCharacterClass(character)))
-					_G[entry..i.."Level"]:SetText(GREEN .. DS:GetCharacterLevel(character))
+					addon:SetCharacterRowNameLevel(entry, i, icon, character)
 					
 					-- profession 1
 					local field = Characters:GetField(line, "spellID1")
@@ -120,7 +119,7 @@ function ns:Update()
 					else
 						icon = ""
 					end
-					field = Characters:GetField(line, "skillRank1")
+					field = Characters:GetField(line, "skillRank1") or 0
 					_G[entry..i.."Skill1NormalText"]:SetText(icon .. ns:GetColor(field) .. field)
 					
 					-- profession 2
@@ -131,29 +130,29 @@ function ns:Update()
 					else
 						icon = ""
 					end
-					field = Characters:GetField(line, "skillRank2")
+					field = Characters:GetField(line, "skillRank2") or 0
 					_G[entry..i.."Skill2NormalText"]:SetText(icon .. ns:GetColor(field) .. field)
 					
 					-- cooking
 					-- icon = addon:TextureToFontstring(addon:GetSpellIcon(2550), size, size) .. " "
 					icon = addon:TextureToFontstring2(addon:GetSpellIcon(2550), size, size, inset, inset, inset, inset) .. " "
-					field = Characters:GetField(line, "cooking")
+					field = Characters:GetField(line, "cooking") or 0
 					_G[entry..i.."CookingNormalText"]:SetText(icon .. ns:GetColor(field) .. field)
 					
 					-- first aid
 					-- icon = addon:TextureToFontstring(addon:GetSpellIcon(3273), size, size) .. " "
 					icon = addon:TextureToFontstring2(addon:GetSpellIcon(3273), size, size, inset, inset, inset, inset) .. " "
-					field = Characters:GetField(line, "firstaid")
+					field = Characters:GetField(line, "firstaid") or 0
 					_G[entry..i.."FirstAidNormalText"]:SetText(icon .. ns:GetColor(field) .. field)
 					
 					-- fishing
 					-- icon = addon:TextureToFontstring(addon:GetSpellIcon(7733), size, size) .. " "
 					icon = addon:TextureToFontstring2(addon:GetSpellIcon(7733), size, size, inset, inset, inset, inset) .. " "
-					field = Characters:GetField(line, "fishing")
+					field = Characters:GetField(line, "fishing") or 0
 					_G[entry..i.."FishingNormalText"]:SetText(icon .. ns:GetColor(field) .. field)
 					
 					-- riding
-					field = Characters:GetField(line, "riding")
+					field = Characters:GetField(line, "riding") or 0
 					if field >= 300 then
 						-- icon = addon:TextureToFontstring("Interface\\Icons\\Ability_Mount_Gryphon_01", size, size) .. " "
 						icon = addon:TextureToFontstring2("Interface\\Icons\\ability_mount_drake_bronze", size, size, inset, inset, inset, inset)
@@ -251,6 +250,7 @@ function ns:OnEnter(frame)
 	local DS = DataStore
 	local character = DS:GetCharacter(Characters:GetInfo(line))
 	local curRank, maxRank = DS:GetSkillInfo(character, skillName)
+	curRank, maxRank = curRank or 0, maxRank or 0		-- CoA: getter returns no value for skills DataStore_Skills hasn't scanned
 	local profession = DS:GetProfession(character, skillName)
 	
 	if (id >= 1) and (id <= 6) then
@@ -268,6 +268,7 @@ function ns:OnEnter(frame)
 		skillName = L["Rogue Proficiencies"]
 		
 		local curLock, maxLock = DS:GetSkillInfo(character, L["Lockpicking"])
+		curLock, maxLock = curLock or 0, maxLock or 0		-- CoA: guard unscanned lockpicking
 		rank = TEAL .. L["Lockpicking"] .. " " .. curLock .. "/" .. maxLock
 		suggestion = addon:GetSuggestion(L["Lockpicking"], curLock)
 	end
@@ -399,6 +400,7 @@ end
 local skillColors = { RECIPE_GREY, RED, ORANGE, YELLOW, GREEN }
 
 function ns:GetColor(rank, skillCap)
+	rank = rank or 0		-- CoA: skill fields are nil for chars DataStore_Characters hasn't scanned
 	skillCap = skillCap or 450
 	return skillColors[ floor(rank / (skillCap/4)) + 1 ]
 end

DataStore_Inventory/DataStore_Inventory.lua

@@ -167,10 +167,13 @@ function ScanInventory()
 end
 
 -- *** Event Handlers ***
+local hasScannedThisSession
 local function OnPlayerAlive()
-	-- print("DataStore_Inventory.lua") -- DEBUG 2025 07 21
-	if not UnitIsGhost("player") then return end -- only scan if player released spirit and went to graveyard
-	
+	-- CoA: scan once at login. PLAYER_ALIVE also fires on resurrect / Feign-Death cancel where gear
+	-- is unchanged, so skip those. (The previous "only when ghost" gate also skipped login, so iLvl
+	-- never populated and UNIT_INVENTORY_CHANGED was the only scan path - see commit fdcb25a.)
+	if hasScannedThisSession then return end
+	hasScannedThisSession = true
 	ScanInventory()
 end
 

DataStore_Inventory/DataStore_Inventory.toc

@@ -3,7 +3,7 @@
 ## Notes: Stores information about character inventory
 ## Author: Thaoky (EU-Marécages de Zangar)
 ## X-Edited-By: Exiles (Sub-Net)
-## Version: 3.3.002-coa.2
+## Version: 3.3.002-coa.5
 ## Dependencies: DataStore
 ## OptionalDeps: Ace3
 ## SavedVariables: DataStore_InventoryDB

README.md

@@ -5,6 +5,12 @@ Altoholic: modified development for WotLK
 
 Ported for the Ascension CoA (Vol'jin) 3.3.5a client by the Exiles guild. Released as `*-coa.N` tags via Gitea Actions; see `Exiles/coa-altoholic`.
 
+- **3.3.002b-coa.7** — Skills tab: `GetColor()` now nil-safe and the per-skill rank fields (`skillRank1/2`, `cooking`, `firstaid`, `fishing`, `riding`) default to `0` — they're nil for chars `DataStore_Characters` hasn't scanned, which crashed the Skills summary (`floor(rank/…)` arithmetic and the `>= 300` riding check).
+- **3.3.002b-coa.6** — Final straggler: guarded `AccountSharing.lua` realm/name line (name getter was the last `format` arg, so a no-value collapsed it to a format error). Concludes the frame sweep.
+- **3.3.002b-coa.5** — Refactor + completeness pass:
+  - Extracted the duplicated character header/row blocks into `Altoholic:AddCharacterTooltipHeader()` and `Altoholic:SetCharacterRowNameLevel()` — the nil-guards now live in one place instead of being copy-pasted across frames.
+  - Fixed crash sites the per-frame sweep had missed: `Skills.lua` (row + skill-rank tooltip), `Keys.lua` (×3 `format` with possibly-nil name), and the latent `ShowClassIcons` sort (`Altoholic.lua` — getters bypass their own `or 0` via the DataStore wrapper).
+  - Restored login scanning: `OnPlayerAlive` in `Altoholic.lua` + `DataStore_Inventory` was gated to ghost-only (commit fdcb25a), so inventory/iLvl never populated on login. Now scans once per session (still skips resurrect/Feign-Death rescans). Removed dated DEBUG leftovers.
 - **3.3.002b-coa.4** — Rebranded to the Exiles fork (title `Altoholic (Exiles)`; Thaoky/Telkar-RG still credited as Author). Hardened **all** Altoholic frames against partial alt records: DataStore char-based getters return *no value* for any module that hasn't scanned a char, and the frames assumed full data everywhere. Guarded every `format`/concat/arithmetic/`pairs` site across AccountSummary, Activity, BagUsage, Quests, Reputations, TabCharacters, `DrawCharacterTooltip`, and the recipe tooltip. No DataStore contract change.
 - **3.3.002b-coa.3** — More partial-record guards in `DataStore_Characters` (own alts seen via guild comm but never fully scanned):
   - `GetXPRate` — guard nil/zero `XPMax` (crashed AccountSummary; also fixes div-by-zero at max level).