Bump ClassicAPI minimum version to 10905

Replace GetSpellRecField (nampower) with the C_Spell field readers.

libdebuff reads the dispel type with C_Spell.GetSpellDispelType.

swingtimer detects on-next-swing spells with C_Spell.IsNextMeleeSpell,
and the swing-reset rule with C_Spell.ResetsMeleeSwing. The on-next-swing
check now covers both attribute bits, and the freeze check folds in the
keep-swings attribute, both matching the server rule.

Note: needs the ClassicAPI release that adds these functions. Bump
PFUI_CLASSIC_API_MIN when that version is out.

api/api.lua

@@ -204,12 +204,6 @@ function pfUI.api.UnitHasBuff(unit, name)
   return C_UnitAuras.GetAuraDataBySpellName(unit, name, "HELPFUL") ~= nil or nil
 end
 
--- [ IsPlayerGuid ]
--- Returns whether a GUID or unit token refers to the local player.
--- guid         [string]        A unit GUID (or unitID) to test.
--- return:      [bool]          true if it is the player otherwise "false"
-pfUI.api.IsPlayerGuid = _G.IsPlayerGuid
-
 -- [ GetUnbuffedRoster ]
 -- Returns a comma-joined, colored list of group members missing the named aura.
 -- name         [string]        the localized aura name to check for
@@ -403,33 +397,14 @@ end
 -- Returns an itemLink for the given itemname
 -- 'name'       [string]         name of the item
 -- returns:     [string]         entire itemLink for the given item
-local itemLinkCache = {}
-local itemLinkMisses = {}
-local ITEMLINK_MAX_SCANS = 2
 function pfUI.api.GetItemLinkByName(name)
-  -- GetInboxItem() hands us a nil name for attachment-less mail
-  if not name then return end
-  -- cache successful resolutions so repeated lookups (e.g. per inbox click) are free
-  if itemLinkCache[name] then return itemLinkCache[name] end
-
-  -- Failures have to be counted, not just retried. An unresolvable name walks
-  -- the entire id range and finds nothing -- a visible hitch on every tooltip
-  -- hover. Allow a couple of attempts (rendering the tooltip caches the item,
-  -- so the next hover usually resolves), then stop scanning for that name.
-  local misses = itemLinkMisses[name] or 0
-  if misses >= ITEMLINK_MAX_SCANS then return end
-
-  -- Octo/Turtle custom items live well past the 25818 vanilla ceiling
-  for itemID = 1, 61000 do
+  for itemID = 1, 25818 do
     local itemName = C_Item.GetItemNameByID(itemID)
     if itemName and itemName == name then
       local _, itemLink = C_Item.GetItemInfo(itemID)
-      itemLinkCache[name] = itemLink
       return itemLink
     end
   end
-
-  itemLinkMisses[name] = misses + 1
 end
 
 -- [ FindItem ]
@@ -671,10 +646,6 @@ end
 function pfUI.api.CreateGoldString(money)
   if type(money) ~= "number" then return "-" end
 
-  if _G.GetCoinTextureString then
-    return "|cffffffff" .. _G.GetCoinTextureString(money) .. "|r"
-  end
-
   local gold = floor(money/ 100 / 100)
   local silver = floor(mod((money/100),100))
   local copper = floor(mod(money,100))

api/config.lua

@@ -554,6 +554,7 @@ function pfUI:LoadConfig()
 
     pfUI:UpdateConfig("unitframes", unit,      "glowaggro",        "1")
     pfUI:UpdateConfig("unitframes", unit,      "glowcombat",       "1")
+    pfUI:UpdateConfig("unitframes", unit,      "showtooltip",      "1")
     pfUI:UpdateConfig("unitframes", unit,      "healthcolor",      "1")
     pfUI:UpdateConfig("unitframes", unit,      "powercolor",       "1")
     pfUI:UpdateConfig("unitframes", unit,      "levelcolor",       "1")
@@ -785,9 +786,6 @@ function pfUI:LoadConfig()
   pfUI:UpdateConfig("tooltip",    nil,           "itemid",           "0")
   pfUI:UpdateConfig("tooltip",    nil,           "movespeed",        "0")
   pfUI:UpdateConfig("tooltip",    nil,           "aurasource",       "0")
-  pfUI:UpdateConfig("tooltip",    nil,           "spellid",          "0")
-  pfUI:UpdateConfig("tooltip",    nil,           "unitid",           "0")
-  pfUI:UpdateConfig("tooltip",    nil,           "showbuffs",        "0")
   pfUI:UpdateConfig("tooltip",    nil,           "alpha",            "0.8")
   pfUI:UpdateConfig("tooltip",    nil,           "alwaysperc",       "0")
   pfUI:UpdateConfig("tooltip",    "compare",     "basestats",        "1")
@@ -966,7 +964,6 @@ function pfUI:LoadConfig()
   pfUI:UpdateConfig("thirdparty", "dpsmate",     "dock",             "0")
   pfUI:UpdateConfig("thirdparty", "shagudps",    "skin",             "0")
   pfUI:UpdateConfig("thirdparty", "shagudps",    "dock",             "0")
-  pfUI:UpdateConfig("thirdparty", "greedmeter",  "dock",             "0")
   pfUI:UpdateConfig("thirdparty", "swstats",     "skin",             "0")
   pfUI:UpdateConfig("thirdparty", "swstats",     "dock",             "0")
   pfUI:UpdateConfig("thirdparty", "ktm",         "skin",             "0")

api/ui-widgets.lua

@@ -587,7 +587,7 @@ end
 function pfUI.api.SetHighlight(frame, cr, cg, cb)
   if not frame then return end
   if not cr or not cg or not cb then
-    cr, cg, cb = PFUI_CLASS_COLORS[UnitClassBase('player')]:GetRGB()
+    cr, cg, cb = GetClassColor(UnitClassBase('player'))
   end
 
   frame.cr, frame.cg, frame.cb = cr, cg, cb, ca
@@ -632,7 +632,7 @@ function pfUI.api.SkinButton(button, cr, cg, cb, icon, disableHighlight)
   if not b then b = button end
   if not b then return end
   if not cr or not cg or not cb then
-    cr, cg, cb = PFUI_CLASS_COLORS[UnitClassBase('player')]:GetRGB()
+    cr, cg, cb = GetClassColor(UnitClassBase('player'))
   end
   pfUI.api.CreateBackdrop(b, nil, true)
   b:SetNormalTexture("")
@@ -709,7 +709,7 @@ end
 function pfUI.api.SkinRotateButton(button)
   pfUI.api.CreateBackdrop(button)
 
-  local cr, cg, cb = PFUI_CLASS_COLORS[UnitClassBase('player')]:GetRGB()
+  local cr, cg, cb = GetClassColor(UnitClassBase('player'))
 
   local btnW, btnH = button:GetSize()
   button:SetSize(btnW - 18, btnH - 18)
@@ -907,7 +907,7 @@ function pfUI.api.SkinDropDown(frame, cr, cg, cb, useSmall)
   end
 
   if not cr or not cg or not cb then
-    cr, cg, cb = PFUI_CLASS_COLORS[UnitClassBase('player')]:GetRGB()
+    cr, cg, cb = GetClassColor(UnitClassBase('player'))
   end
 
   SetHighlight(button, cr, cg, cb)

api/unitframes.lua

@@ -140,43 +140,32 @@ function pfUI.api.GetUnitStats(unitstr)
 end
 
 local aggrodata = { }
-local aggroScan = nil -- static { u, t, tt } triples, built once so the hot scan allocates nothing
 function pfUI.api.UnitHasAggro(unit)
-  local now = GetTime()
-  local data = aggrodata[unit]
-  -- Cache positive results 1s (so aggro clears fast) AND negative results 0.3s
-  -- (so we don't rescan the whole unit table on every call while nothing has aggro).
-  if data and now < data.check + (data.state > 0 and 1 or 0.3) then
-    return data.state
+  -- Only cache positive results to allow instant detection when aggro changes
+  if aggrodata[unit] and aggrodata[unit].state > 0 and GetTime() < aggrodata[unit].check + 1 then
+    return aggrodata[unit].state
   end
 
-  if not data then data = { }; aggrodata[unit] = data end
-  data.check = now
-  data.state = 0
+  aggrodata[unit] = aggrodata[unit] or { }
+  aggrodata[unit].check = GetTime()
+  aggrodata[unit].state = 0
 
   if UnitExists(unit) and UnitIsFriend(unit, "player") then
-    -- pfValidUnits never changes after load, so precompute the "<u>target" /
-    -- "<u>targettarget" token strings once instead of concatenating per call.
-    if not aggroScan then
-      aggroScan = {}
-      for u in pairs(pfValidUnits) do
-        local t = u .. "target"
-        table.insert(aggroScan, { u = u, t = t, tt = t .. "target" })
-      end
-    end
+    for u in pairs(pfValidUnits) do
+      local t = u .. "target"
+      local tt = t .. "target"
 
-    for i = 1, table.getn(aggroScan) do
-      local s = aggroScan[i]
-      if UnitExists(s.t) and UnitIsUnit(s.t, unit) and UnitCanAttack(s.u, unit) then
-        data.state = data.state + 1
+      if UnitExists(t) and UnitIsUnit(t, unit) and UnitCanAttack(u, unit) then
+        aggrodata[unit].state = aggrodata[unit].state + 1
       end
-      if UnitExists(s.tt) and UnitIsUnit(s.tt, unit) and UnitCanAttack(s.t, unit) then
-        data.state = data.state + 1
+
+      if UnitExists(tt) and UnitIsUnit(tt, unit) and UnitCanAttack(t, unit) then
+        aggrodata[unit].state = aggrodata[unit].state + 1
       end
     end
   end
 
-  return data.state
+  return aggrodata[unit].state
 end
 
 pfUI.uf.glow = CreateFrame("Frame", nil, UIParent)
@@ -205,13 +194,8 @@ function pfUI.uf:UpdateVisibility()
 
   -- cache result of strsub to avoid repeating calls
   if not self.cache_raid then
-    local name = self:GetName()
-    if strsub(name,0,9) == "pfRaidPet" then
-      -- pet grid slot (pfRaidPet1..40); used to mirror party pets below
-      self.cache_raid = 0
-      self.cache_raidpet = tonumber(strsub(name,10)) or 0
-    elseif strsub(name,0,6) == "pfRaid" then
-      self.cache_raid = tonumber(strsub(name,7,8)) or 0
+    if strsub(self:GetName(),0,6) == "pfRaid" then
+      self.cache_raid = tonumber(strsub(self:GetName(),7,8)) or 0
     else
       self.cache_raid = 0
     end
@@ -243,26 +227,6 @@ function pfUI.uf:UpdateVisibility()
     end
   end
 
-  -- show raidpet frames as party pets when a party is shown as a raid grid
-  if self.cache_raidpet then
-    if not IsInRaid() and IsInGroup() and C.unitframes.raidforgroup == "1" then
-      local id = self.cache_raidpet
-      if id == 1 then
-        -- grid slot 1 mirrors the player, so its pet is the player's pet
-        self.id = ""
-        self.label = "pet"
-      elseif id <= 5 then
-        self.id = id - 1
-        self.label = "partypet"
-      end
-
-    -- reset to regular raidpet unitstrings after leaving party mode
-    elseif self.label == "pet" or self.label == "partypet" then
-      self.id = self.cache_raidpet
-      self.label = "raidpet"
-    end
-  end
-
   -- display every unit as player while pfUI.uf.showall is set
   if pfUI.uf.showall then
     self._label = self._label or self.label
@@ -291,10 +255,9 @@ function pfUI.uf:UpdateVisibility()
     -- frame shall not be visible
     visibility = "hide"
     self.visible = nil
-  elseif hide_group and self.cache_raid == 0 and not self.cache_raidpet and self.label and strsub(self.label,0,5) == "party" then
-    -- hide group while shown as a raid grid and option is set (raid player
-    -- frames carry label "party" and raid pet frames carry label "partypet"
-    -- under raidforgroup, so exclude them by cache_raid / cache_raidpet)
+  elseif hide_group and self.cache_raid == 0 and self.label and strsub(self.label,0,5) == "party" then
+    -- hide group while shown as a raid grid and option is set (raid frames
+    -- carry label "party" under raidforgroup, so exclude them by cache_raid)
     visibility = "hide"
     self.visible = nil
   elseif ( self.fname == "Group0" or self.fname == "PartyPet0" or self.fname == "Party0Target" )
@@ -1446,7 +1409,7 @@ function pfUI.uf:CreateUnitFrame(unit, id, config, tick)
     f.group:SetFont(pfUI.font_unit, 8, "OUTLINE")
     f.group:SetTextColor(1,1,1,.8)
     f.group:SetHeight(16)
-    f.group:SetText(T["Group"] .. " " .. group)
+    f.group:SetText("Group " .. group)
     f.group:Hide()
   end
 
@@ -2178,9 +2141,6 @@ function pfUI.uf:EnableClickCast()
           self:SetAttribute(prefix .. "type" .. bid, "target")
         elseif low == "focus" then
           self:SetAttribute(prefix .. "type" .. bid, "focus")
-        elseif string.find(low, "^macro:") then
-          self:SetAttribute(prefix .. "type" .. bid, "macro")
-          self:SetAttribute(prefix .. "macro" .. bid, string.gsub(string.sub(action, 7), "^%s+", ""))
         elseif string.find(action, "^/") then
           self:SetAttribute(prefix .. "type" .. bid, "macro")
           self:SetAttribute(prefix .. "macrotext" .. bid, action)

env/translations_deDE.lua

@@ -51,12 +51,10 @@ pfUI_translation["deDE"] = {
   ["Auto Sell Grey Items"] = nil,
   ["Average Per Hour"] = nil,
   ["Background Color"] = nil,
-  ["Bags"] = "Taschen",
   ["Bags & Bank"] = nil,
   ["Bags Border Size"] = nil,
   ["Bagslots Per Row"] = nil,
   ["Bagspace"] = nil,
-  ["Bank"] = "Bank",
   ["Banker"] = nil,
   ["Bankslots Per Row"] = nil,
   ["Bar Background"] = nil,
@@ -292,6 +290,7 @@ pfUI_translation["deDE"] = {
   ["Enable Mana Ticks"] = nil,
   ["Enable Micro Bar"] = nil,
   ["Enable Mouselook With Right Click"] = nil,
+  ["Enable Mouseover Tooltip"] = nil,
   ["Enable Movable Bags"] = nil,
   ["Enable Offscreen Frame Positions"] = nil,
   ["Enable Overlap"] = nil,
@@ -312,7 +311,6 @@ pfUI_translation["deDE"] = {
   ["Encode"] = nil,
   ["Ended"] = nil,
   ["Energy Color"] = nil,
-  ["Equipped"] = "Angelegt",
   ["Equipped Item Color"] = nil,
   ["Estimate Debuffs"] = nil,
   ["Estimate Enemy Health Points"] = nil,
@@ -821,7 +819,6 @@ pfUI_translation["deDE"] = {
   ["Top Actionbar"] = nil,
   ["Top Left"] = nil,
   ["Top Right"] = nil,
-  ["Total"] = "Gesamt",
   ["Total Gold"] = nil,
   ["Totem Direction"] = nil,
   ["Totem Icons"] = nil,

env/translations_enUS.lua

@@ -52,19 +52,15 @@ pfUI_translation["enUS"] = {
   ["Auto Sell Grey Items"] = nil,
   ["Average Per Hour"] = nil,
   ["Background Color"] = nil,
-  ["Bags"] = nil,
   ["Bags & Bank"] = nil,
   ["Bags Border Size"] = nil,
   ["Bagslots Per Row"] = nil,
   ["Bagspace"] = nil,
-  ["Bank"] = nil,
   ["Banker"] = nil,
   ["Bankslots Per Row"] = nil,
   ["Bar Background"] = nil,
   ["Bar Texture"] = nil,
-  ["Battleground Frames"] = nil,
   ["Battleground Statistics"] = nil,
-  ["BEHIND"] = nil,
   ["Blacklist"] = nil,
   ["Blinding Powder"] = nil,
   ["Blue Border On Friendly Players"] = nil,
@@ -297,6 +293,7 @@ pfUI_translation["enUS"] = {
   ["Enable Mana Ticks"] = nil,
   ["Enable Micro Bar"] = nil,
   ["Enable Mouselook With Right Click"] = nil,
+  ["Enable Mouseover Tooltip"] = nil,
   ["Enable Movable Bags"] = nil,
   ["Enable Offscreen Frame Positions"] = nil,
   ["Enable Overlap"] = nil,
@@ -317,7 +314,6 @@ pfUI_translation["enUS"] = {
   ["Encode"] = nil,
   ["Ended"] = nil,
   ["Energy Color"] = nil,
-  ["Equipped"] = nil,
   ["Equipped Item Color"] = nil,
   ["Estimate Debuffs"] = nil,
   ["Estimate Enemy Health Points"] = nil,
@@ -330,7 +326,6 @@ pfUI_translation["enUS"] = {
   ["Export"] = nil,
   ["Face"] = nil,
   ["Fade To Custom Color"] = nil,
-  ["FARM MODE"] = nil,
   ["Fast"] = nil,
   ["Filter Mode"] = nil,
   ["Finish"] = nil,
@@ -541,7 +536,6 @@ pfUI_translation["enUS"] = {
   ["Next Memory Cleanup"] = nil,
   ["No"] = nil,
   ["No Anchor"] = nil,
-  ["NO LOS"] = nil,
   ["None"] = nil,
   ["Not a valid button!"] = nil,
   ["No tracking spell active"] = nil,
@@ -777,8 +771,6 @@ pfUI_translation["enUS"] = {
   ["Small"] = nil,
   ["Some settings need to reload the UI to take effect.\nDo you want to reload now?"] = nil,
   ["Some settings need to reload the UI to take effect.\nDo you want to reloadUI now?"] = nil,
-  ["Sort Bags"] = nil,
-  ["Sort Bank"] = nil,
   ["Sort Order"] = nil,
   ["Soul Shard"] = nil,
   ["Soulshard Counter"] = nil,
@@ -843,7 +835,6 @@ pfUI_translation["enUS"] = {
   ["Top Actionbar"] = nil,
   ["Top Left"] = nil,
   ["Top Right"] = nil,
-  ["Total"] = nil,
   ["Total Gold"] = nil,
   ["Totem Direction"] = nil,
   ["Totem Icons"] = nil,

env/translations_esES.lua

@@ -51,12 +51,10 @@ pfUI_translation["esES"] = {
   ["Auto Sell Grey Items"] = "Vender objetos grises automáticamente",
   ["Average Per Hour"] = "Promedio por hora",
   ["Background Color"] = "Color del fondo",
-  ["Bags"] = "Bolsas",
   ["Bags & Bank"] = "Bolsas y banco",
   ["Bags Border Size"] = "Tamaño del borde de las bolsas",
   ["Bagslots Per Row"] = "Casillas de la bolsa por fila",
   ["Bagspace"] = "Huecos en las bolsas",
-  ["Bank"] = "Banco",
   ["Banker"] = "Banquero",
   ["Bankslots Per Row"] = "Casillas del banco por fila",
   ["Bar Background"] = "Fondo de la barra",
@@ -292,6 +290,7 @@ pfUI_translation["esES"] = {
   ["Enable Mana Ticks"] = "Activar el indicador de pulsos del maná",
   ["Enable Micro Bar"] = "Activar microbarra",
   ["Enable Mouselook With Right Click"] = "Mirar usando el puntero con clic derecho",
+  ["Enable Mouseover Tooltip"] = "Activar la descripción emergente al pasar el puntero",
   ["Enable Movable Bags"] = "Permitir mover las bolsas",
   ["Enable Offscreen Frame Positions"] = "Permitir mover los marcos fuera de la pantalla",
   ["Enable Overlap"] = "Permitir superposición",
@@ -312,7 +311,6 @@ pfUI_translation["esES"] = {
   ["Encode"] = "Codificar",
   ["Ended"] = "Terminado",
   ["Energy Color"] = "Color de energía",
-  ["Equipped"] = "Equipado",
   ["Equipped Item Color"] = "Color del objeto equipado",
   ["Estimate Debuffs"] = "Estimar los perjuicios",
   ["Estimate Enemy Health Points"] = "Estimar los puntos de salud del enemigo",
@@ -821,7 +819,6 @@ pfUI_translation["esES"] = {
   ["Top Actionbar"] = "Barra de acción superior",
   ["Top Left"] = "Superior derecha",
   ["Top Right"] = "Superior izquierda",
-  ["Total"] = "Total",
   ["Total Gold"] = "Oro Total",
   ["Totem Direction"] = "Dirección del tótem",
   ["Totem Icons"] = "Iconos de los tótems",

env/translations_frFR.lua

@@ -51,12 +51,10 @@ pfUI_translation["frFR"] = {
   ["Auto Sell Grey Items"] = "Vente automatique des objets gris",
   ["Average Per Hour"] = "Moyenne par heure",
   ["Background Color"] = "Couleur de l'arrière-plan",
-  ["Bags"] = "Sacs",
   ["Bags & Bank"] = "Sacs et Banque",
   ["Bags Border Size"] = "Taille de la bordure des Sacs",
   ["Bagslots Per Row"] = "Emplacements des sacs par rangée",
   ["Bagspace"] = "Place libre des sacs",
-  ["Bank"] = "Banque",
   ["Banker"] = "Banquier",
   ["Bankslots Per Row"] = "Emplacements de la banque par rangée",
   ["Bar Background"] = "Arrière plan de la barre",
@@ -292,6 +290,7 @@ pfUI_translation["frFR"] = {
   ["Enable Mana Ticks"] = "Activer les Ticks de mana",
   ["Enable Micro Bar"] = "Activer la barre des menus miniature",
   ["Enable Mouselook With Right Click"] = "Activer le déplacement de la caméra avec le clic droit",
+  ["Enable Mouseover Tooltip"] = "Activer l'infobulle au passage de la souris",
   ["Enable Movable Bags"] = "Activer les sacs mobiles",
   ["Enable Offscreen Frame Positions"] = "Activer le positionnement des cadres en dehors de l'écran",
   ["Enable Overlap"] = "Activer la superposition",
@@ -312,7 +311,6 @@ pfUI_translation["frFR"] = {
   ["Encode"] = "Encoder",
   ["Ended"] = "Terminé",
   ["Energy Color"] = "Couleur de l'energie",
-  ["Equipped"] = "Équipé",
   ["Equipped Item Color"] = "Couleur de l'objet équipé",
   ["Estimate Debuffs"] = "Estimer les Affaiblissements",
   ["Estimate Enemy Health Points"] = "Estimer les points de vie ennemis",
@@ -821,7 +819,6 @@ pfUI_translation["frFR"] = {
   ["Top Actionbar"] = "Barre d'action supérieure",
   ["Top Left"] = "Haut Gauche",
   ["Top Right"] = "Haut Droit",
-  ["Total"] = "Total",
   ["Total Gold"] = "Or Total",
   ["Totem Direction"] = "Direction des totems",
   ["Totem Icons"] = "Icones des totems",

env/translations_koKR.lua

@@ -51,12 +51,10 @@ pfUI_translation["koKR"] = {
   ["Auto Sell Grey Items"] = "회색아이템 자동 판매",
   ["Average Per Hour"] = nil,
   ["Background Color"] = "백그라운드 색상",
-  ["Bags"] = "가방",
   ["Bags & Bank"] = "가방&은행",
   ["Bags Border Size"] = "가방 테두리 크기",
   ["Bagslots Per Row"] = nil,
   ["Bagspace"] = "가방 공간",
-  ["Bank"] = "은행",
   ["Banker"] = nil,
   ["Bankslots Per Row"] = nil,
   ["Bar Background"] = nil,
@@ -292,6 +290,7 @@ pfUI_translation["koKR"] = {
   ["Enable Mana Ticks"] = nil,
   ["Enable Micro Bar"] = "미니바 보기(블리자드 마이크로바)",
   ["Enable Mouselook With Right Click"] = "오른쪽 클릭 Mouselook 사용",
+  ["Enable Mouseover Tooltip"] = nil,
   ["Enable Movable Bags"] = nil,
   ["Enable Offscreen Frame Positions"] = "오프스크린 프레임 위치 활성화",
   ["Enable Overlap"] = "겹쳐서 표시",
@@ -312,7 +311,6 @@ pfUI_translation["koKR"] = {
   ["Encode"] = nil,
   ["Ended"] = nil,
   ["Energy Color"] = nil,
-  ["Equipped"] = "착용 중",
   ["Equipped Item Color"] = nil,
   ["Estimate Debuffs"] = nil,
   ["Estimate Enemy Health Points"] = nil,
@@ -821,7 +819,6 @@ pfUI_translation["koKR"] = {
   ["Top Actionbar"] = nil,
   ["Top Left"] = nil,
   ["Top Right"] = nil,
-  ["Total"] = "총",
   ["Total Gold"] = "총 금",
   ["Totem Direction"] = nil,
   ["Totem Icons"] = nil,

env/translations_ruRU.lua

@@ -51,12 +51,10 @@ pfUI_translation["ruRU"] = {
   ["Auto Sell Grey Items"] = "Автоматическая продажа серых предметов",
   ["Average Per Hour"] = "Среднее за час",
   ["Background Color"] = "Цвет фона",
-  ["Bags"] = "Сумки",
   ["Bags & Bank"] = "Сумки & Банк",
   ["Bags Border Size"] = "Размер границы сумок",
   ["Bagslots Per Row"] = "Количество ячеек в сумке на строку",
   ["Bagspace"] = "Место в сумках",
-  ["Bank"] = "Банк",
   ["Banker"] = "Банкир",
   ["Bankslots Per Row"] = "Количество ячеек в банке на строку",
   ["Bar Background"] = "Фон панели",
@@ -292,6 +290,7 @@ pfUI_translation["ruRU"] = {
   ["Enable Mana Ticks"] = "Включить восполнение маны",
   ["Enable Micro Bar"] = "Включить панель микро меню",
   ["Enable Mouselook With Right Click"] = "Включить нажатие правой кнопки мыши на индикаторе здоровья",
+  ["Enable Mouseover Tooltip"] = "Включить подсказку при наведении мыши",
   ["Enable Movable Bags"] = "Включить подвижные сумки",
   ["Enable Offscreen Frame Positions"] = "Включить расположение окон за пределами экрана",
   ["Enable Overlap"] = "Включить перекрытие",
@@ -312,7 +311,6 @@ pfUI_translation["ruRU"] = {
   ["Encode"] = "Кодировать",
   ["Ended"] = "Окончание",
   ["Energy Color"] = "Цвет энергии",
-  ["Equipped"] = "Экипировано",
   ["Equipped Item Color"] = "Цвет экипированных предметов",
   ["Estimate Debuffs"] = "Оценка дебафов",
   ["Estimate Enemy Health Points"] = "Оценка очков здоровья врага",
@@ -821,7 +819,6 @@ pfUI_translation["ruRU"] = {
   ["Top Actionbar"] = "Верхняя",
   ["Top Left"] = "Сверху слева",
   ["Top Right"] = "Сверху справа",
-  ["Total"] = "Всего",
   ["Total Gold"] = "Всего золота",
   ["Totem Direction"] = "Направление тотемов",
   ["Totem Icons"] = "Иконки тотемов",

env/translations_zhCN.lua

@@ -51,12 +51,10 @@ pfUI_translation["zhCN"] = {
   ["Auto Sell Grey Items"] = "自动贩卖灰色物品",
   ["Average Per Hour"] = "平均每小时",
   ["Background Color"] = "背景颜色",
-  ["Bags"] = "背包",
   ["Bags & Bank"] = "背包和银行",
   ["Bags Border Size"] = "背包边框大小",
   ["Bagslots Per Row"] = "每行包裹槽数量",
   ["Bagspace"] = "背包空间",
-  ["Bank"] = "银行",
   ["Banker"] = "工会银行角色",
   ["Bankslots Per Row"] = "每行银行槽数量",
   ["Bar Background"] = "动作条背景",
@@ -292,6 +290,7 @@ pfUI_translation["zhCN"] = {
   ["Enable Mana Ticks"] = "显示法力值刻度",
   ["Enable Micro Bar"] = "显示菜单栏",
   ["Enable Mouselook With Right Click"] = "启动右键移动镜头",
+  ["Enable Mouseover Tooltip"] = "启用鼠标悬停工具提示",
   ["Enable Movable Bags"] = "启用可移动包裹",
   ["Enable Offscreen Frame Positions"] = "允许本插件所有框体移出屏幕边缘",
   ["Enable Overlap"] = "叠加显示",
@@ -312,7 +311,6 @@ pfUI_translation["zhCN"] = {
   ["Encode"] = "编码",
   ["Ended"] = "结束",
   ["Energy Color"] = "能量值颜色",
-  ["Equipped"] = "已装备",
   ["Equipped Item Color"] = "已装备物品颜色",
   ["Estimate Debuffs"] = "预估Debuffs",
   ["Estimate Enemy Health Points"] = "预估敌人的生命值",
@@ -822,7 +820,6 @@ pfUI_translation["zhCN"] = {
   ["Top Actionbar"] = "上方动作条",
   ["Top Left"] = "左上方",
   ["Top Right"] = "右上方",
-  ["Total"] = "总计",
   ["Total Gold"] = "总黄金",
   ["Totem Direction"] = "图腾方向",
   ["Totem Icons"] = "图腾图标",

env/translations_zhTW.lua

@@ -51,12 +51,10 @@ pfUI_translation["zhTW"] = {
   ["Auto Sell Grey Items"] = "自動販賣灰色物品",
   ["Average Per Hour"] = "平均每小時",
   ["Background Color"] = "背景顏色",
-  ["Bags"] = "背包",
   ["Bags & Bank"] = "背包和銀行",
   ["Bags Border Size"] = "背包邊框大小",
   ["Bagslots Per Row"] = "每行包裹槽數量",
   ["Bagspace"] = "背包空間",
-  ["Bank"] = "銀行",
   ["Banker"] = "工會銀行角色",
   ["Bankslots Per Row"] = "每行銀行槽數量",
   ["Bar Background"] = nil,
@@ -292,6 +290,7 @@ pfUI_translation["zhTW"] = {
   ["Enable Mana Ticks"] = nil,
   ["Enable Micro Bar"] = "顯示功能表列",
   ["Enable Mouselook With Right Click"] = "啟動右鍵移動鏡頭",
+  ["Enable Mouseover Tooltip"] = "啟用滑鼠懸停工具提示",
   ["Enable Movable Bags"] = "啟用可移動包裹",
   ["Enable Offscreen Frame Positions"] = "啟用螢幕框架固定",
   ["Enable Overlap"] = "疊加顯示",
@@ -312,7 +311,6 @@ pfUI_translation["zhTW"] = {
   ["Encode"] = "編碼",
   ["Ended"] = nil,
   ["Energy Color"] = nil,
-  ["Equipped"] = "已裝備",
   ["Equipped Item Color"] = nil,
   ["Estimate Debuffs"] = nil,
   ["Estimate Enemy Health Points"] = nil,
@@ -821,7 +819,6 @@ pfUI_translation["zhTW"] = {
   ["Top Actionbar"] = nil,
   ["Top Left"] = "左上方",
   ["Top Right"] = "右上方",
-  ["Total"] = "全部",
   ["Total Gold"] = "總金量",
   ["Totem Direction"] = nil,
   ["Totem Icons"] = nil,

init/modules.xml

@@ -81,5 +81,4 @@
   <Include file="..\modules\equipmentmanager.lua"/>
   <Include file="..\modules\loothistory.lua"/>
   <Include file="..\modules\newitem.lua"/>
-  <Include file="..\modules\friendnotes.lua"/>
 </Ui>

init/skins.xml

@@ -41,6 +41,8 @@
   <!-- Turtle WoW -->
   <Include file="..\skins\blizzard\barbershop.lua"/>
   <Include file="..\skins\blizzard\transmog.lua"/>
+  <Include file="..\skins\blizzard\custom_merchant.lua"/>
+  <Include file="..\skins\blizzard\arena_score.lua"/>
   <Include file="..\skins\blizzard\ebc.lua"/>
 
 </Ui>

libs/libdebuff.lua

@@ -591,13 +591,37 @@ end
 -- DURATION FUNCTIONS
 -- ============================================================================
 
--- Report the live, talent-accurate duration of a matching aura on the player.
--- ClassicAPI's C_UnitAuras carries the real (mod-folded) duration straight from
--- the engine, so there's no static duration table to maintain. Returns 0 when
--- the player has no such aura (e.g. a debuff that only ever lands on enemies).
 function libdebuff:GetDuration(effect, rank)
-  local aura = C_UnitAuras.GetAuraDataBySpellName("player", effect)
-  return aura and aura.duration or 0
+  if L["debuffs"][effect] then
+    local rank = rank and tonumber((string.gsub(rank, RANK, ""))) or 0
+    local rank = L["debuffs"][effect][rank] and rank or libdebuff:GetMaxRank(effect)
+    local duration = L["debuffs"][effect][rank]
+
+    if effect == L["dyndebuffs"]["Demoralizing Shout"] then
+      local _,_,_,_,count = GetTalentInfo(2,1)
+      if count and count > 0 then duration = duration + ( duration / 100 * (count*10)) end
+    elseif effect == L["dyndebuffs"]["Shadow Word: Pain"] then
+      local _,_,_,_,count = GetTalentInfo(3,4)
+      if count and count > 0 then duration = duration + count * 3 end
+    elseif effect == L["dyndebuffs"]["Frostbolt"] then
+      local _,_,_,_,count = GetTalentInfo(3,7)
+      if count and count > 0 then duration = duration + count end
+    elseif effect == L["dyndebuffs"]["Gouge"] then
+      local _,_,_,_,count = GetTalentInfo(3,3)
+      if count and count > 0 then duration = duration + (count*.5) end
+    end
+    return duration
+  else
+    return 0
+  end
+end
+
+function libdebuff:GetMaxRank(effect)
+  local max = 0
+  for id in pairs(L["debuffs"][effect]) do
+    if id > max then max = id end
+  end
+  return max
 end
 
 function libdebuff:UpdateDuration(unit, unitlevel, effect, duration)
@@ -623,6 +647,7 @@ libdebuff.objects = {}
 
 function libdebuff:AddPending(unit, unitlevel, effect, duration, caster, rank)
   if not unit or duration <= 0 then return end
+  if not L["debuffs"][effect] then return end
   if libdebuff.pending[3] then return end
 
   libdebuff.pending[1] = unit
@@ -701,25 +726,39 @@ end
 -- API: GetBestAuraCast (for libpredict HoT tracking)
 -- ============================================================================
 
--- Report the active aura for spellName on `guid`, read straight from
--- C_UnitAuras. expirationTime is the true, caster-modified remaining time, so
--- no cast-event bookkeeping is needed -- and a unit only ever holds one instance
--- of a given spell, so this is inherently the "best" one.
--- Returns (startTime, duration, timeleft, rank, casterGuid) or nil.
 function libdebuff:GetBestAuraCast(guid, spellName)
   if not guid or not spellName then return nil end
-
-  local aura = C_UnitAuras.GetAuraDataBySpellName(guid, spellName)
-  if not aura or not aura.expirationTime or aura.expirationTime == 0 then return nil end
-
-  local timeleft = aura.expirationTime - GetTime()
-  if timeleft <= 0 then return nil end
-
-  local duration = (aura.duration and aura.duration > 0) and aura.duration or timeleft
-  local startTime = aura.expirationTime - duration
-  local rank = aura.spellId and tonumber((string.gsub(C_Spell.GetSpellSubtext(aura.spellId) or "", RANK, ""))) or nil
-
-  return startTime, duration, timeleft, rank, aura.sourceGUID
+  
+  -- Check ownDebuffs first (for our casts)
+  if ownDebuffs[guid] and ownDebuffs[guid][spellName] then
+    local data = ownDebuffs[guid][spellName]
+    local timeleft = (data.startTime + data.duration) - GetTime()
+    if timeleft > 0 then
+      return data.startTime, data.duration, timeleft, data.rank, GetPlayerGuid()
+    end
+  end
+  
+  -- Check allAuraCasts (for any caster)
+  if allAuraCasts[guid] and allAuraCasts[guid][spellName] then
+    local bestData = nil
+    local bestCaster = nil
+    local bestTimeleft = 0
+    
+    for casterGuid, data in pairs(allAuraCasts[guid][spellName]) do
+      local timeleft = (data.startTime + data.duration) - GetTime()
+      if timeleft > bestTimeleft then
+        bestTimeleft = timeleft
+        bestData = data
+        bestCaster = casterGuid
+      end
+    end
+    
+    if bestData and bestTimeleft > 0 then
+      return bestData.startTime, bestData.duration, bestTimeleft, bestData.rank, bestCaster
+    end
+  end
+  
+  return nil
 end
 
 -- ============================================================================

libs/libpredict.lua

@@ -1222,7 +1222,7 @@ libpredict.sender:SetScript("OnEvent", function()
 end)
 
 function libpredict:GetHotDuration(unit, spell)
-  if unit == UNKNOWNOBJECT or unit == UKNOWNBEING then return end
+  if unit == UNKNOWNOBJECT or unit == UNKOWNBEING then return end
   
   -- NEW: Try libdebuff first (Nampower AURA_CAST events)
   if pfUI.api.libdebuff and pfUI.api.libdebuff.GetBestAuraCast then

libs/libunitscan.lua

@@ -96,12 +96,13 @@ libunitscan:SetScript("OnEvent", function()
     RememberByUnit("player", name, class)
 
   elseif event == "FRIENDLIST_UPDATE" then
+    local name, class, level
     for i = 1, GetNumFriends() do
-      local info = C_FriendList.GetFriendInfoByIndex(i)
-      if info then
-        local level = info.level > 0 and info.level or nil
-        AddData("players", info.name, info.classFilename, level)
-      end
+      name, level, class = GetFriendInfo(i)
+      class = L["class"][class] or nil
+      -- friendlist updates due to friend going off-line return level 0, let's not overwrite good older values
+      level = level > 0 and level or nil
+      AddData("players", name, class, level)
     end
 
   elseif event == "GUILD_ROSTER_UPDATE" then
@@ -136,11 +137,11 @@ libunitscan:SetScript("OnEvent", function()
     end
 
   elseif event == "WHO_LIST_UPDATE" or event == "CHAT_MSG_SYSTEM" then
-    for i = 1, C_FriendList.GetNumWhoResults() do
-      local info = C_FriendList.GetWhoInfo(i)
-      if info then
-        AddData("players", info.fullName, info.filename, info.level, nil, info.fullGuildName)
-      end
+    local name, class, level, guild, _
+    for i = 1, GetNumWhoResults() do
+      name, guild, level, _, class, _ = GetWhoInfo(i)
+      class = L["class"][class] or nil
+      AddData("players", name, class, level, nil, guild)
     end
 
   elseif event == "UPDATE_MOUSEOVER_UNIT" or event == "PLAYER_TARGET_CHANGED" or event == "NAME_PLATE_UNIT_ADDED" then

modules/autoshift.lua

@@ -2,11 +2,6 @@ pfUI:RegisterModule("autoshift", function ()
   pfUI.autoshift = CreateFrame("Frame")
   pfUI.autoshift:RegisterEvent("UI_ERROR_MESSAGE")
 
-  -- Empty stub kept only so Turtle WoW's pfUI-turtle addon doesn't error when
-  -- its obsolete moonkin autoshift module iterates pfUI.autoshift.shapeshifts.
-  -- This fork drives autoshift off GetShapeshiftFormID() and no longer uses it.
-  pfUI.autoshift.shapeshifts = {}
-
   pfUI.autoshift.scanString = string.gsub(SPELL_FAILED_ONLY_SHAPESHIFT, "%%s", "(.+)")
 
   pfUI.autoshift.errors = { SPELL_FAILED_NOT_MOUNTED, ERR_ATTACK_MOUNTED, ERR_TAXIPLAYERALREADYMOUNTED,

modules/bags.lua

@@ -972,7 +972,7 @@ pfUI:RegisterModule("bags", function ()
           frame.sort.backdrop:SetBackdropBorderColor(1,1,.25,1)
           frame.sort.texture:SetVertexColor(1,1,.25,1)
           GameTooltip:SetOwner(this, "ANCHOR_RIGHT")
-          GameTooltip:SetText(T["Sort Bags"])
+          GameTooltip:SetText("Sort Bags")
           GameTooltip:Show()
         end)
 
@@ -1085,6 +1085,21 @@ pfUI:RegisterModule("bags", function ()
           frame.search.edit:ClearFocus()
         end)
 
+        frame.search:SetScript("OnHide", function()
+          frame.search.edit:SetText(T["Search"])
+          for bag = -2, 11 do
+            if pfUI.bags[bag] then
+              local bagsize = GetContainerNumSlots(bag)
+              if bag == -2 and pfUI.bag.showKeyring == true then bagsize = GetKeyRingSize() end
+              for slot = 1, bagsize do
+                if pfUI.bags[bag] and pfUI.bags[bag].slots[slot] then
+                  pfUI.bags[bag].slots[slot].frame:SetAlpha(1)
+                end
+              end
+            end
+          end
+        end)
+
         frame.search.edit:SetScript("OnMouseUp", function()
           if arg1 == "RightButton" then
             this:ClearFocus()
@@ -1192,7 +1207,7 @@ pfUI:RegisterModule("bags", function ()
           frame.sort.backdrop:SetBackdropBorderColor(1,1,.25,1)
           frame.sort.texture:SetVertexColor(1,1,.25,1)
           GameTooltip:SetOwner(this, "ANCHOR_RIGHT")
-          GameTooltip:SetText(T["Sort Bank"])
+          GameTooltip:SetText("Sort Bank")
           GameTooltip:Show()
         end)
 

modules/bgscore.lua

@@ -32,7 +32,7 @@ pfUI:RegisterModule("bgscore", function ()
   -- Title label
   local title = mover:CreateFontString(nil, "OVERLAY")
   title:SetFont("Fonts\\FRIZQT__.TTF", 14, "OUTLINE")
-  title:SetText(T["Battleground Frames"])
+  title:SetText("Battleground Frames")
   title:SetPoint("TOP", mover, "TOP", 0, -2)
 
   -- BG score preview text

modules/buffwatch.lua

@@ -90,14 +90,12 @@ pfUI:RegisterModule("buffwatch", function ()
           if val == skill then return end
         end
         config.whitelist = config.whitelist .. "#" .. skill
-        fcache[tostring(config)] = nil -- invalidate so the new entry takes effect immediately
         DEFAULT_CHAT_FRAME:AddMessage("|cff33ffcc" .. skill .. "|r" .. T["is now whitelisted."])
       elseif IsShiftKeyDown() then
         for _, val in pairs({strsplit("#", config.blacklist)}) do
           if val == skill then return end
         end
         config.blacklist = config.blacklist .. "#" .. skill
-        fcache[tostring(config)] = nil -- invalidate so the new entry takes effect immediately
         DEFAULT_CHAT_FRAME:AddMessage("|cff33ffcc" .. skill .. "|r" .. T["is now blacklisted."])
       end
     elseif this.parent.unit == "player" then

modules/chat.lua

@@ -810,25 +810,23 @@ pfUI:RegisterModule("chat", function ()
       end
     end
 
-    -- detect the whisper prefix BEFORE prepending a timestamp; the timestamp
-    -- pushes the whisper colour code off position 1 and the find below fails
-    local isWhisper = C.chat.global.whispermod == "1" and string.find(text, wcol, 1) == 1
-
     -- show timestamp in chat
     if C.chat.text.time == "1" then
       text = timecolorhex .. tleft .. date(C.chat.text.timeformat) .. tright .. "|r " .. text
     end
 
     -- save chat history
-    if isWhisper then
+    if C.chat.global.whispermod == "1" and string.find(text, wcol, 1) == 1 then
       SaveChatHistory(frame:GetID(), string.gsub(text, wcol, ""), cr, cg, cb)
     else
       SaveChatHistory(frame:GetID(), text, a1, a2, a3)
     end
 
-    if isWhisper then
+    if C.chat.global.whispermod == "1" then
       -- patch incoming whisper string to match the colors
-      text = string.gsub(text, "|r", "|r" .. wcol)
+      if string.find(text, wcol, 1) == 1 then
+        text = string.gsub(text, "|r", "|r" .. wcol)
+      end
     end
 
     frame:HookAddMessage(text, a1, a2, a3, a4, a5, a6, a7, a8, a9, a10, a11, a12, a13, a14, a15, a16, a17)

modules/cooldown.lua

@@ -9,7 +9,7 @@ pfUI:RegisterModule("cooldown", function ()
   local parent, parent_name
   local function pfCooldownOnUpdate()
     parent = this:GetParent()
-    if not parent then this:Hide() return end
+    if not parent then this:Hide() end
     parent_name = parent:GetName()
 
     -- avoid to set cooldowns on invalid frames

modules/farmmode.lua

@@ -136,7 +136,7 @@ pfUI:RegisterModule("farmmode", function ()
     pfUI.farmmap.button.txt = pfUI.farmmap.button:CreateFontString("pfFarmMapText", "LOW", "GameFontWhite")
     pfUI.farmmap.button.txt:SetFont(pfUI.font_default, C.global.font_size, "OUTLINE")
     pfUI.farmmap.button.txt:SetPoint("CENTER", 0, 0)
-    pfUI.farmmap.button.txt:SetText("|cff33ffcc " .. T["FARM MODE"])
+    pfUI.farmmap.button.txt:SetText("|cff33ffcc FARM MODE")
   end
 
   CreateBackdrop(pfUI.farmmap)

modules/firstrun.lua

@@ -263,7 +263,7 @@ pfUI:RegisterModule("firstrun", function ()
     SkinCheckbox(f.checkbox, 18)
 
     f.NextScript = function()
-      if not pfUI.chat then message("Couldn't apply settings. Chat module is disabled.") return end
+      if not pfUI.chat then message("Couldn't apply settings. Chat module is disabled.") end
       pfUI.chat.SetupRightChat(f.checkbox:GetChecked())
     end
 
@@ -283,7 +283,7 @@ pfUI:RegisterModule("firstrun", function ()
     SkinCheckbox(f.checkbox, 18)
 
     f.NextScript = function()
-      if not pfUI.chat then message("Couldn't apply settings. Chat module is disabled.") return end
+      if not pfUI.chat then message("Couldn't apply settings. Chat module is disabled.") end
       if f.checkbox:GetChecked() then
         pfUI.chat.SetupPositions()
       end
@@ -305,7 +305,7 @@ pfUI:RegisterModule("firstrun", function ()
     SkinCheckbox(f.checkbox, 18)
 
     f.NextScript = function()
-      if not pfUI.chat then message("Couldn't apply settings. Chat module is disabled.") return end
+      if not pfUI.chat then message("Couldn't apply settings. Chat module is disabled.") end
       if f.checkbox:GetChecked() then
         pfUI.chat.SetupChannels()
       end

modules/friendnotes.lua

@@ -1,93 +0,0 @@
--- Friend notes
--- Client-side notes for friends via ClassicAPI's C_FriendList note API
--- (SetFriendNotes / GetFriendInfo(.notes)). Adds an "Edit Note" entry to the
--- friend right-click menu (the "FRIEND" UnitPopup) and shows the note in a
--- tooltip when you hover a friend in the list.
-pfUI:RegisterModule("friendnotes", function ()
-  local EDIT_TOKEN = "PFUI_FRIEND_NOTE"
-
-  -- Note editor. In 1.12 StaticPopup, OnAccept runs with `this` = the OK
-  -- button (the dialog is this:GetParent()); the friend name arrives as the
-  -- data argument passed to StaticPopup_Show.
-  StaticPopupDialogs["PFUI_FRIEND_NOTE_EDIT"] = {
-    text = SET_FRIENDNOTE_LABEL,
-    button1 = SAVE,
-    button2 = CANCEL,
-    hasEditBox = 1,
-    maxLetters = 128,
-    OnAccept = function(name)
-      C_FriendList.SetFriendNotes(name, _G[this:GetParent():GetName().."EditBox"]:GetText())
-    end,
-    EditBoxOnEnterPressed = function(name)
-      C_FriendList.SetFriendNotes(name, this:GetText())
-      this:GetParent():Hide()
-    end,
-    EditBoxOnEscapePressed = function() this:GetParent():Hide() end,
-    timeout = 0, whileDead = 1, hideOnEscape = 1,
-  }
-
-  local function OpenNoteEditor(name)
-    if not name or name == "" then return end
-    local info = C_FriendList.GetFriendInfo(name)
-    local dialog = StaticPopup_Show("PFUI_FRIEND_NOTE_EDIT", name, nil, name)
-    if not dialog then return end
-    local editbox = _G[dialog:GetName().."EditBox"]
-    editbox:SetText((info and info.notes) or "")
-    editbox:HighlightText()
-    editbox:SetFocus()
-  end
-
-  -- Add "Edit Note" to the friend right-click menu, just before Cancel.
-  UnitPopupButtons[EDIT_TOKEN] = { text = SET_NOTE, dist = 0 }
-  local friendMenu = UnitPopupMenus["FRIEND"]
-  table.insert(friendMenu, table.getn(friendMenu), EDIT_TOKEN)
-
-  UnitPopupMenus["PFUI_FRIENDNOTE"] = { EDIT_TOKEN, "CANCEL" }
-  local function OfflineNoteDropDown_Initialize()
-    UnitPopup_ShowMenu(_G[UIDROPDOWNMENU_OPEN_MENU], "PFUI_FRIENDNOTE", nil, FriendsDropDown.name)
-  end
-  local FriendsFrame_ShowDropdown_orig = FriendsFrame_ShowDropdown
-  _G.FriendsFrame_ShowDropdown = function(name, connected)
-    if connected then return FriendsFrame_ShowDropdown_orig(name, connected) end
-    HideDropDownMenu(1)
-    FriendsDropDown.initialize = OfflineNoteDropDown_Initialize
-    FriendsDropDown.displayMode = "MENU"
-    FriendsDropDown.name = name
-    ToggleDropDownMenu(1, nil, FriendsDropDown, "cursor")
-  end
-
-  -- Route our entry to the editor and close the menu; delegate the rest. A
-  -- bare assignment lands on pfUI.env, so set the global explicitly.
-  local UnitPopup_OnClick_orig = UnitPopup_OnClick
-  _G.UnitPopup_OnClick = function()
-    if this and this.value == EDIT_TOKEN then
-      local dropdown = _G[UIDROPDOWNMENU_INIT_MENU]
-      local name = dropdown and dropdown.name
-      CloseDropDownMenus()
-      OpenNoteEditor(name)
-      return
-    end
-    return UnitPopup_OnClick_orig()
-  end
-
-  -- Show the note in a tooltip while hovering a friend in the list. Friend
-  -- buttons carry their friend index via SetID (see FriendsFrame_Update).
-  local function FriendButton_OnEnter()
-    local index = this:GetID()
-    if not index or index < 1 then return end
-    local info = C_FriendList.GetFriendInfoByIndex(index)
-    if not info or not info.notes or info.notes == "" then return end
-    GameTooltip:SetOwner(this, "ANCHOR_RIGHT")
-    GameTooltip:SetText(info.name or "")
-    GameTooltip:AddLine(info.notes, 1, 1, 1, 1)
-    GameTooltip:Show()
-  end
-
-  for i = 1, FRIENDS_TO_DISPLAY do
-    local button = _G["FriendsFrameFriendButton"..i]
-    if button then
-      button:HookScript("OnEnter", FriendButton_OnEnter)
-      button:HookScript("OnLeave", GameTooltip_Hide)
-    end
-  end
-end)

modules/gui.lua

@@ -2240,6 +2240,7 @@ pfUI:RegisterModule("gui", function ()
 
         -- build config entries
         CreateConfig(U[c], T["Display Frame"] .. ": " .. t, C.unitframes[c], "visible", "checkbox")
+        CreateConfig(U[c], T["Enable Mouseover Tooltip"], C.unitframes[c], "showtooltip", "checkbox")
         CreateConfig(U[c], T["Default Transparency"], C.unitframes[c], "alpha_visible", "dropdown", pfUI.gui.dropdowns.percent_small)
         CreateConfig(U[c], T["Out Of Range Transparency"], C.unitframes[c], "alpha_outrange", "dropdown", pfUI.gui.dropdowns.percent_small)
         CreateConfig(U[c], T["Offline Transparency"], C.unitframes[c], "alpha_offline", "dropdown", pfUI.gui.dropdowns.percent_small)
@@ -2763,32 +2764,22 @@ pfUI:RegisterModule("gui", function ()
     end)
 
     CreateGUIEntry(T["Tooltip"], nil, function()
-      CreateConfig(nil, T["General"], nil, nil, "header")
       CreateConfig(nil, T["Tooltip Position"], C.tooltip, "position", "dropdown", pfUI.gui.dropdowns.tooltip_position)
       CreateConfig(nil, T["Tooltip Text Font"], C.tooltip, "font_tooltip", "dropdown", pfUI.gui.dropdowns.fonts)
       CreateConfig(nil, T["Tooltip Text Font Size"], C.tooltip, "font_tooltip_size")
       CreateConfig(nil, T["Cursor Tooltip Align"], C.tooltip, "cursoralign", "dropdown", pfUI.gui.dropdowns.tooltip_align)
       CreateConfig(nil, T["Cursor Tooltip Offset"], C.tooltip, "cursoroffset")
+      CreateConfig(nil, T["Enable Extended Guild Information"], C.tooltip, "extguild", "checkbox")
+      CreateConfig(nil, T["Always Show Health In Percent"], C.tooltip, "alwaysperc", "checkbox")
+      CreateConfig(nil, T["Show Item IDs"], C.tooltip, "itemid", "checkbox")
+      CreateConfig(nil, T["Show Movement Speed"], C.tooltip, "movespeed", "checkbox")
+      CreateConfig(nil, T["Show Aura Caster"], C.tooltip, "aurasource", "checkbox")
       CreateConfig(nil, T["Custom Transparency"], C.tooltip, "alpha")
       CreateConfig(nil, T["Status Bar Texture"], C.tooltip.statusbar, "texture", "dropdown", pfUI.gui.dropdowns.uf_bartexture)
-
-      CreateConfig(nil, T["Items"], nil, nil, "header")
       CreateConfig(nil, T["Always Show Item Comparison"], C.tooltip.compare, "showalways", "checkbox")
       CreateConfig(nil, T["Always Show Extended Vendor Values"], C.tooltip.vendor, "showalways", "checkbox")
       CreateConfig(U["questitem"], T["Show Related Quest On Questitems"], C.tooltip.questitem, "showquest", "checkbox")
       CreateConfig(U["questitem"], T["Show Required Questitem Count"], C.tooltip.questitem, "showcount", "checkbox")
-      CreateConfig(nil, T["Show Item IDs"], C.tooltip, "itemid", "checkbox")
-
-      CreateConfig(nil, T["Spells"], nil, nil, "header")
-      CreateConfig(nil, T["Show Aura Caster"], C.tooltip, "aurasource", "checkbox")
-      CreateConfig(nil, T["Show Spell IDs"], C.tooltip, "spellid", "checkbox")
-
-      CreateConfig(nil, T["Units"], nil, nil, "header")
-      CreateConfig(nil, T["Enable Extended Guild Information"], C.tooltip, "extguild", "checkbox")
-      CreateConfig(nil, T["Always Show Health In Percent"], C.tooltip, "alwaysperc", "checkbox")
-      CreateConfig(nil, T["Show Movement Speed"], C.tooltip, "movespeed", "checkbox")
-      CreateConfig(nil, T["Show Unit Buffs"], C.tooltip, "showbuffs", "checkbox")
-      CreateConfig(nil, T["Show Unit IDs"], C.tooltip, "unitid", "checkbox")
     end)
 
     CreateGUIEntry(T["Castbar"], T["General"], function()
@@ -2999,7 +2990,6 @@ pfUI:RegisterModule("gui", function ()
       CreateConfig(nil, T["Use Chat Colors for Meters"], C.thirdparty, "chatbg", "checkbox")
       CreateConfig(nil, "ShaguDPS (" .. T["Skin"] .. ")", C.thirdparty.shagudps, "skin", "checkbox")
       CreateConfig(nil, "ShaguDPS (" .. T["Dock"] .. ")", C.thirdparty.shagudps, "dock", "checkbox")
-      CreateConfig(nil, "GreedMeter (" .. T["Dock"] .. ")", C.thirdparty.greedmeter, "dock", "checkbox")
       CreateConfig(nil, "DPSMate (" .. T["Skin"] .. ")", C.thirdparty.dpsmate, "skin", "checkbox")
       CreateConfig(nil, "DPSMate (" .. T["Dock"] .. ")", C.thirdparty.dpsmate, "dock", "checkbox")
       CreateConfig(nil, "SWStats (" .. T["Skin"] .. ")", C.thirdparty.swstats, "skin", "checkbox")

modules/itemcount.lua

@@ -26,14 +26,10 @@ pfUI:RegisterModule("itemcount", function ()
     if bank < 0 then bank = 0 end
 
     frame:AddLine(" ")
-    if bags > 0     then frame:AddDoubleLine(T["Bags"] .. ":",     bags,     1, 1, 1, 1, 1, 1) end
-    if bank > 0     then frame:AddDoubleLine(T["Bank"] .. ":",     bank,     1, 1, 1, 1, 1, 1) end
-    if equipped > 0 then frame:AddDoubleLine(T["Equipped"] .. ":", equipped, 1, 1, 1, 1, 1, 1) end
-
-    local sources = (bags > 0 and 1 or 0) + (bank > 0 and 1 or 0) + (equipped > 0 and 1 or 0)
-    if sources > 1 then
-      frame:AddDoubleLine(T["Total"] .. ":", total, 1, 1, 1, 1, 1, 1)
-    end
+    if bags > 0     then frame:AddDoubleLine("Bags:",     bags,     1, 1, 1, 1, 1, 1) end
+    if bank > 0     then frame:AddDoubleLine("Bank:",     bank,     1, 1, 1, 1, 1, 1) end
+    if equipped > 0 then frame:AddDoubleLine("Equipped:", equipped, 1, 1, 1, 1, 1, 1) end
+    frame:AddDoubleLine("Total:", total, 1, 1, 1, 1, 1, 1)
     frame:Show()
   end
 

modules/loot.lua

@@ -77,7 +77,9 @@ pfUI:RegisterModule("loot", function ()
       if pfUI.loot.my_index or pfUI.loot.disenchanter_index or pfUI.loot.banker_index then
         info = wipe(info)
         info.text = T["Special Recipient"]
-        info.textR, info.textG, info.textB = NORMAL_FONT_COLOR:GetRGB()
+        info.textR = NORMAL_FONT_COLOR.r
+        info.textG = NORMAL_FONT_COLOR.g
+        info.textB = NORMAL_FONT_COLOR.b
         info.textHeight = 12
         info.hasArrow = 1
         info.notCheckable = 1
@@ -156,7 +158,9 @@ pfUI:RegisterModule("loot", function ()
     if level == 1 then
       info = wipe(info)
       info.text = T["Random"]
-      info.textR, info.textG, info.textB = NORMAL_FONT_COLOR:GetRGB()
+      info.textR = NORMAL_FONT_COLOR.r
+      info.textG = NORMAL_FONT_COLOR.g
+      info.textB = NORMAL_FONT_COLOR.b
       info.value = "PFRANDOM"
       info.textHeight = 12
       info.notCheckable = 1
@@ -166,7 +170,9 @@ pfUI:RegisterModule("loot", function ()
 
       info = wipe(info)
       info.text = T["Request Rolls"]
-      info.textR, info.textG, info.textB = NORMAL_FONT_COLOR:GetRGB()
+      info.textR = NORMAL_FONT_COLOR.r
+      info.textG = NORMAL_FONT_COLOR.g
+      info.textB = NORMAL_FONT_COLOR.b
       info.value = "PFROLLS"
       info.textHeight = 12
       info.hasArrow = 1
@@ -177,7 +183,9 @@ pfUI:RegisterModule("loot", function ()
       if UIDROPDOWNMENU_MENU_VALUE == "PFROLLS" then
         info = wipe(info)
         info.text = T["Clear Rolls"]
-        info.textR, info.textG, info.textB = NORMAL_FONT_COLOR:GetRGB()
+        info.textR = NORMAL_FONT_COLOR.r
+        info.textG = NORMAL_FONT_COLOR.g
+        info.textB = NORMAL_FONT_COLOR.b
         info.value = "PFCLEARROLLS"
         info.notCheckable = 1
         info.func = pfUI.loot.ClearRolls
@@ -185,7 +193,9 @@ pfUI:RegisterModule("loot", function ()
 
         info = wipe(info)
         info.text = T["Reroll Ties"]
-        info.textR, info.textG, info.textB = NORMAL_FONT_COLOR:GetRGB()
+        info.textR = NORMAL_FONT_COLOR.r
+        info.textG = NORMAL_FONT_COLOR.g
+        info.textB = NORMAL_FONT_COLOR.b
         info.value = "PFTIEROLL"
         info.notCheckable = 1
         info.arg1 = pfUI.loot.rollers_sorted

modules/loothistory.lua

@@ -16,6 +16,12 @@ pfUI:RegisterModule("loothistory", function ()
   local WINMARK = "Interface\\Buttons\\UI-CheckBox-Check"
   local QUESTIONMARK = "Interface\\Icons\\INV_Misc_QuestionMark"
 
+  local function ClassColor(class)
+    local c = class and RAID_CLASS_COLORS[class]
+    if c then return c.r, c.g, c.b end
+    return 1, 1, 1
+  end
+
   -- Paint an item row's icon/name/quality from a loaded Item mixin.
   local function RenderItemVisual(f, item)
     local r, g, b = 1, 1, 1
@@ -271,7 +277,7 @@ pfUI:RegisterModule("loothistory", function ()
         if wroll and wroll > 0 then f.winroll:SetText(wroll) else f.winroll:SetText("") end
         f.winroll:Show()
         f.winname:SetText(wname or UNKNOWN)
-        f.winname:SetTextColor(PFUI_CLASS_COLORS[wclass]:GetRGB())
+        f.winname:SetTextColor(ClassColor(wclass))
         f.winname:Show()
       else
         -- nobody won: everyone passed
@@ -292,7 +298,7 @@ pfUI:RegisterModule("loothistory", function ()
 
   local function RenderPlayerFrame(pf, name, class, rollType, roll, isWinner)
     pf.name:SetText(name or UNKNOWN)
-    pf.name:SetTextColor(PFUI_CLASS_COLORS[class]:GetRGB())
+    pf.name:SetTextColor(ClassColor(class))
 
     pf.rollicon:SetTexture(ROLL_TEX[rollType] or ROLL_TEX[ROLL_PASS])
     if roll and roll > 0 then pf.rolltext:SetText(roll) else pf.rolltext:SetText("") end

modules/macroicons.lua

@@ -72,7 +72,7 @@ pfUI:RegisterModule("macroicons", function ()
     end
 
     local function SaveMacroPopup()
-      local text = picker.editbox:GetText()
+      local text = strtrim(picker.editbox:GetText())
       if text == "" then return end
 
       local icon = iconPicker.GetIcon()

modules/macrotweak.lua

@@ -13,8 +13,7 @@ pfUI:RegisterModule("macrotweak", function ()
   end
 
   -- do not write macro calls into chat input history
-  -- (install once: _AddHistoryLine is our backup slot and is nil until we set it)
-  if not ChatFrameEditBox._AddHistoryLine then
+  if ChatFrameEditBox._AddHistoryLine then
     local userinput
     ChatFrameEditBox._AddHistoryLine = ChatFrameEditBox.AddHistoryLine
     ChatFrameEditBox.AddHistoryLine = function(self, text)

modules/map.lua

@@ -77,13 +77,7 @@ pfUI:RegisterModule("map", function ()
           if point == "TOPLEFT" and relpoint == "TOPLEFT" then
             offx = offx*oldscale/scale
             offy = offy*oldscale/scale
-            -- Anchor to the parent (3-arg SetPoint) rather than re-using the frame
-            -- GetPoint handed back. Re-anchoring to `rel` throws
-            -- "<unnamed> is dependent on this" as soon as anything else has anchored
-            -- itself to WorldMapFrame, which aborts the whole zoom handler. The
-            -- parent-relative form is also what LoadMovable/SaveMovable already
-            -- assume: SaveMovable stores only xpos/ypos, with no relative frame.
-            WorldMapFrame:SetPoint(point, offx, offy)
+            WorldMapFrame:SetPoint(point, rel, relpoint, offx, offy)
           end
 
           WorldMapFrame:SetScale(scale)

modules/mapreveal.lua

@@ -52,19 +52,19 @@ pfUI:RegisterModule("mapreveal", function ()
     WorldMapTooltip:AddLine(this.name, 1, 1, 1)
     WorldMapTooltip:Show()
 
-    if not explorecaches[this.area] then return end
+    if not explorecaches[this.name] then return end
     if C.appearance.worldmap.mapreveal == "0" then return end
-    for texture in pairs(explorecaches[this.area]) do
+    for texture in pairs(explorecaches[this.name]) do
       texture:SetVertexColor(1,1,1,1)
     end
   end
 
   local exploreLeave = function()
     WorldMapTooltip:Hide()
-    if not explorecaches[this.area] then return end
+    if not explorecaches[this.name] then return end
     if C.appearance.worldmap.mapreveal == "0" then return end
     local r,g,b,a = GetStringColor(C.appearance.worldmap.mapreveal_color)
-    for texture in pairs(explorecaches[this.area]) do
+    for texture in pairs(explorecaches[this.name]) do
       texture:SetVertexColor(r,g,b,a)
     end
   end
@@ -120,7 +120,6 @@ pfUI:RegisterModule("mapreveal", function ()
       explore:EnableMouse(true)
       explore:SetFrameLevel(255)
       explore.name = mapFileName .. " (" .. name .. ")"
-      explore.area = name -- cache key: explorecaches is keyed by the plain area name
       explore.tex = explore.tex or explore:CreateTexture("", "OVERLAY")
       explore.tex:SetBlendMode("ADD")
       explore.tex:SetTexCoord(.08, .92, .08, .92)

modules/nameplates.lua

@@ -283,35 +283,15 @@ pfUI:RegisterModule("nameplates", function ()
     end
   end
 
-  -- Numeric CreatureType.dbc id for the plate's unit (8 = Critter, 11 = Totem).
-  -- It's fixed per unit, so cache it and reset on plate reuse. nil while the GUID
-  -- can't resolve yet (incl. the no-SuperWoW case, where cachedGuid is empty).
-  -- Locale-proof and covers custom units for free -- no name tables needed.
-  local function CreatureType(plate)
-    local guid = plate.cachedGuid
-    if not guid or guid == "" then return nil end
-    if not plate.creatureType then
-      plate.creatureType = UnitCreatureTypeID(guid)  -- nil retries next call
+  local function TotemPlate(name)
+    if C.nameplates.totemicons == "1" then
+      for totem, icon in pairs(L["totems"]) do
+        if string.find(name, totem) then return icon end
+      end
     end
-    return plate.creatureType
   end
 
-  -- Totem icon, read straight from the game:
-  --   * Passive totems self-cast their provided buff, so they carry exactly one
-  --     aura whose icon IS the totem's icon (Totem::Summon TOTEM_PASSIVE).
-  --   * Active totems (Searing/Magma/Fire Nova) cast at enemies and hold no
-  --     self-aura, so their icon arrives via UNIT_SPELLCAST_SUCCEEDED (cached
-  --     into plate.totemIcon by the event handler); nil here until then.
-  local function TotemPlate(plate)
-    if C.nameplates.totemicons ~= "1" then return nil end
-    if CreatureType(plate) ~= 11 then return nil end
-    if plate.totemIcon then return plate.totemIcon end
-    local aura = C_UnitAuras.GetBuffDataByIndex(plate.cachedGuid, 1)
-    if aura then plate.totemIcon = aura.icon end
-    return plate.totemIcon
-  end
-
-  local function HidePlate(unittype, fullhp, target, plate)
+  local function HidePlate(unittype, name, fullhp, target)
     -- keep some plates always visible according to config
     if C.nameplates.fullhealth == "1" and not fullhp then return nil end
     if C.nameplates.target == "1" and target then return nil end
@@ -327,10 +307,14 @@ pfUI:RegisterModule("nameplates", function ()
       return true
     elseif C.nameplates.friendlyplayer == "1" and unittype == "FRIENDLY_PLAYER" then
       return true
-    elseif C.nameplates.critters == "1" and CreatureType(plate) == 8 then
-      return true
-    elseif C.nameplates.totems == "1" and CreatureType(plate) == 11 then
-      return true
+    elseif C.nameplates.critters == "1" and unittype == "NEUTRAL_NPC" then
+      for i, critter in pairs(L["critters"]) do
+        if string.lower(name) == string.lower(critter) then return true end
+      end
+    elseif C.nameplates.totems == "1" then
+      for totem in pairs(L["totems"]) do
+        if string.find(name, totem) then return true end
+      end
     end
 
     -- nothing to hide
@@ -499,7 +483,6 @@ nameplates:RegisterEvent("UNIT_SPELLCAST_START")
 nameplates:RegisterEvent("UNIT_SPELLCAST_CHANNEL_START")
 nameplates:RegisterEvent("UNIT_SPELLCAST_STOP")
 nameplates:RegisterEvent("UNIT_SPELLCAST_CHANNEL_STOP")
-nameplates:RegisterEvent("UNIT_SPELLCAST_SUCCEEDED")
 nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
   
   nameplates:SetScript("OnEvent", function()
@@ -586,8 +569,6 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
         local guid = UnitGUID(arg1)
         plate.nameplate.cachedGuid = guid
         plate.nameplate.unit = arg1
-        plate.nameplate.creatureType = nil  -- recompute for the new unit
-        plate.nameplate.totemIcon = nil
         if guid then
           plateByGuid[guid] = plate.nameplate
           -- Seed: the unit may already be mid-cast (its UNIT_SPELLCAST_START
@@ -665,22 +646,6 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
         end
       end
 
-    elseif event == "UNIT_SPELLCAST_SUCCEEDED" then
-      -- Active totems (Searing/Magma/Fire Nova) carry no self-aura, so their
-      -- attack cast is the only icon source. Capture it once per totem, gated on
-      -- creature type so a normal caster's spell never styles it as a totem.
-      if arg1 and strfind(arg1, "^nameplate") then
-        local guid = UnitGUID(arg1)
-        local plate = guid and plateByGuid[guid]
-        if plate and not plate.totemIcon and arg3 and CreatureType(plate) == 11 then
-          local tex = C_Spell.GetSpellTexture(arg3)
-          if tex then
-            plate.totemIcon = tex
-            plate.castUpdate = true  -- re-render now so the icon shows
-          end
-        end
-      end
-
     elseif event == "UNIT_AURA" then
       -- ClassicAPI: fires with arg1 == "nameplateN" when a unit's aura set
       -- changes (add/remove/modify). Flag the matching plate so OnUpdate does a
@@ -1107,11 +1072,11 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
     end
 
     -- hide frames according to the configuration
-    local TotemIcon = TotemPlate(plate)
+    local TotemIcon = TotemPlate(name)
 
     if TotemIcon then
-      -- icon resolved from the totem's aura / attack cast (already a full path)
-      plate.totem.icon:SetTexture(TotemIcon)
+      -- create totem icon
+      plate.totem.icon:SetTexture("Interface\\Icons\\" .. TotemIcon)
 
       plate.glow:Hide()
       plate.level:Hide()
@@ -1119,7 +1084,7 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
       plate.health:Hide()
       plate.guild:Hide()
       plate.totem:Show()
-    elseif HidePlate(unittype, (hpmax-hp == hpmin), target, plate) then
+    elseif HidePlate(unittype, name, (hpmax-hp == hpmin), target) then
       plate.level:SetPoint("RIGHT", plate.name, "LEFT", -3, 0)
       plate.name:SetParent(plate)
       plate.guild:SetPoint("BOTTOM", plate.name, "BOTTOM", -2, -(font_size + 2))
@@ -1154,21 +1119,12 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
       plate.level:SetText(string.format("%s%s", level, (elitestrings[elite] or "")))
     end
 
-    -- Set level color from GetDifficultyColor when using DB level.
-    -- No brightening here: adding a flat offset to all three channels
-    -- desaturates every tier toward white and collapses the boundaries
-    -- (orange reads as yellow, red reads as orange). Use Blizzard's values.
-    -- Clearing cache.levelcolor forces the sync block in OnUpdate to
-    -- re-colour once the ?? resolves and it takes ownership again.
+    -- Set level color from GetDifficultyColor when using DB level
     if levelFromDB and type(level) == "number" then
       local color = GetDifficultyColor(level)
-      plate.level:SetTextColor(color.r, color.g, color.b, 1)
-      plate.cache.levelcolor = nil
+      plate.level:SetTextColor(color.r + 0.3, color.g + 0.3, color.b + 0.3, 1)
     end
 
-    -- remember who owns the level colour so the two writers cannot fight
-    plate.cache.levelfromdb = levelFromDB or nil
-
     if guild and C.nameplates.showguildname == "1" then
       plate.guild:SetText(guild)
       if guild == myGuild then
@@ -1235,11 +1191,12 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
 
     local r, g, b, a = unpack(unitcolors[unittype])
 
-    if class and (
-      (unittype == "ENEMY_PLAYER" and C.nameplates["enemyclassc"] == "1") or
-      (unittype == "FRIENDLY_PLAYER" and C.nameplates["friendclassc"] == "1")
-    ) then
-      r, g, b, a = PFUI_CLASS_COLORS[class]:GetRGBA()
+    if class then
+      if unittype == "ENEMY_PLAYER" and C.nameplates["enemyclassc"] == "1" then
+        r, g, b, a = PFUI_CLASS_COLORS[class]:GetRGBA()
+      elseif unittype == "FRIENDLY_PLAYER" and C.nameplates["friendclassc"] == "1" then
+        r, g, b, a = PFUI_CLASS_COLORS[class]:GetRGBA()
+      end
     end
 
     if unitstr and UnitIsTapped(unitstr) and not UnitIsTappedByPlayer(unitstr) then
@@ -1260,43 +1217,15 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
       plate.cache.r, plate.cache.g, plate.cache.b = r, g, b
     end
 
-    -- Friendly player names take this colour when friendclassnamec is on.
-    -- Ownership is recorded so the OnUpdate sync below stands down rather than
-    -- racing us: both writers used to share cache.namecolor despite storing
-    -- unrelated quantities (this colour vs Blizzard's name FontString), so
-    -- either could suppress the other -- and since nameplate.cache survives
-    -- pool reuse, a recycled plate could keep the previous unit's name colour.
-    local ownname = unittype == "FRIENDLY_PLAYER" and C.nameplates["friendclassnamec"] == "1"
-      and class and true or nil
-
-    if plate.cache.ownname ~= ownname then
-      plate.cache.ownname = ownname
-      -- ownership flipped: whichever writer is now in charge must re-assert
-      plate.cache.namecolor = nil
-      plate.cache.ownnamecolor = nil
-    end
-
-    -- read the class colour directly rather than reusing the bar's r,g,b: the
-    -- bar only carries a class colour when friendclassc happens to be on, and
-    -- it also picks up the tapped-grey and barcombatstate overrides, neither of
-    -- which belongs on the name.
-    if ownname then
-      local cr, cg, cb, ca = PFUI_CLASS_COLORS[class]:GetRGBA()
-      if cr + cg + cb ~= plate.cache.ownnamecolor then
-        plate.cache.ownnamecolor = cr + cg + cb
-        plate.name:SetTextColor(cr, cg, cb, ca)
-      end
+    if r + g + b ~= plate.cache.namecolor and unittype == "FRIENDLY_PLAYER" and C.nameplates["friendclassnamec"] == "1" and class and PFUI_CLASS_COLORS[class] then
+      plate.name:SetTextColor(r, g, b, a)
+      plate.cache.namecolor = r + g + b
     end
 
+    -- update combopoints
+    for i=1, 5 do plate.combopoints[i]:Hide() end
     if target and C.nameplates.cpdisplay == "1" then
-      local cp = GetComboPoints("target")
-      if plate.cpShown ~= cp then
-        for i=1, 5 do plate.combopoints[i]:SetShown(i <= cp) end
-        plate.cpShown = cp
-      end
-    elseif plate.cpShown then
-      for i=1, 5 do plate.combopoints[i]:Hide() end
-      plate.cpShown = nil
+      for i=1, GetComboPoints("target") do plate.combopoints[i]:Show() end
     end
 
     -- update debuffs
@@ -1553,41 +1482,33 @@ nameplates:RegisterEvent("PLAYER_GUILD_UPDATE")
       update = true
     end
 
-    -- trigger update when name color changed (includes combat state check).
-    -- Skipped once OnDataChanged owns the name colour (class-coloured friendly
-    -- players), so the two writers cannot overwrite each other.
-    if not nameplate.cache.ownname then
-      local r, g, b = original.name:GetTextColor()
-      local inCombatWithPlayer = cfg.namefightcolor and UnitAffectingCombat(nameplate.unit) and UnitAffectingCombat("player")
+    -- trigger update when name color changed (includes combat state check)
+    local r, g, b = original.name:GetTextColor()
+    local inCombatWithPlayer = cfg.namefightcolor and UnitAffectingCombat(nameplate.unit) and UnitAffectingCombat("player")
+    
+    if r + g + b ~= nameplate.cache.namecolor or (cfg.namefightcolor and nameplate.cache.inCombat ~= inCombatWithPlayer) then
+      nameplate.cache.namecolor = r + g + b
+      nameplate.cache.inCombat = inCombatWithPlayer
 
-      if r + g + b ~= nameplate.cache.namecolor or (cfg.namefightcolor and nameplate.cache.inCombat ~= inCombatWithPlayer) then
-        nameplate.cache.namecolor = r + g + b
-        nameplate.cache.inCombat = inCombatWithPlayer
-
-        if cfg.namefightcolor then
-          if (r > .9 and g < .2 and b < .2) or inCombatWithPlayer then
-            nameplate.name:SetTextColor(1,0.4,0.2,1)
-          else
-            nameplate.name:SetTextColor(r,g,b,1)
-          end
+      if cfg.namefightcolor then
+        if (r > .9 and g < .2 and b < .2) or inCombatWithPlayer then
+          nameplate.name:SetTextColor(1,0.4,0.2,1)
         else
-          nameplate.name:SetTextColor(1,1,1,1)
+          nameplate.name:SetTextColor(r,g,b,1)
         end
-        update = true
+      else
+        nameplate.name:SetTextColor(1,1,1,1)
       end
+      update = true
     end
 
-    -- trigger update when level color changed. Skipped while the level came
-    -- from the database (?? plates), where OnDataChanged owns the colour --
-    -- otherwise both writers race and cache.levelcolor goes stale, which
-    -- leaves a recycled plate wearing the previous unit's colour.
-    if not nameplate.cache.levelfromdb then
-      local r, g, b = original.level:GetTextColor()
-      if r + g + b ~= nameplate.cache.levelcolor then
-        nameplate.cache.levelcolor = r + g + b
-        nameplate.level:SetTextColor(r,g,b,1)
-        update = true
-      end
+    -- trigger update when level color changed
+    local r, g, b = original.level:GetTextColor()
+    r, g, b = r + .3, g + .3, b + .3
+    if r + g + b ~= nameplate.cache.levelcolor then
+      nameplate.cache.levelcolor = r + g + b
+      nameplate.level:SetTextColor(r,g,b,1)
+      update = true
     end
 
     -- use timer based updates

modules/panel.lua

@@ -296,18 +296,18 @@ pfUI:RegisterModule("panel", function()
         local playerzone  = GetRealZoneText()
 
         for friendIndex=1, all do
-          local info = C_FriendList.GetFriendInfoByIndex(friendIndex)
-          if info and info.connected and info.classFilename and info.level then
+          local friend_name, friend_level, friend_class, friend_area, friend_connected = GetFriendInfo(friendIndex)
+          if friend_connected and friend_class and friend_level then
             if not init then
               GameTooltip_SetDefaultAnchor(GameTooltip, this)
               GameTooltip:ClearLines()
               GameTooltip:AddLine("|cff555555" .. T["Friends Online"])
               init = true
             end
-            local ccolor = PFUI_CLASS_COLORS[info.classFilename]
-            local lcolor = GetDifficultyColor(tonumber(info.level)) or { 1, 1, 1 }
-            local zcolor = info.area == playerzone and "|cff33ffcc" or "|cffcccccc"
-            GameTooltip:AddDoubleLine(ccolor:WrapTextInColorCode(info.name) .. rgbhex(lcolor) .. " [" .. info.level .. "]", zcolor .. info.area)
+            local ccolor = PFUI_CLASS_COLORS[L["class"][friend_class]] or { 1, 1, 1 }
+            local lcolor = GetDifficultyColor(tonumber(friend_level)) or { 1, 1, 1 }
+            local zcolor = friend_area == playerzone and "|cff33ffcc" or "|cffcccccc"
+            GameTooltip:AddDoubleLine(rgbhex(ccolor) .. friend_name .. rgbhex(lcolor) .. " [" .. friend_level .. "]", zcolor .. friend_area)
           end
         end
 
@@ -315,7 +315,15 @@ pfUI:RegisterModule("panel", function()
       end
       widget.Click = function() ToggleFriendsFrame(1) end
       widget:SetScript("OnEvent", function()
-        local online = C_FriendList.GetNumOnlineFriends()
+        local online = 0
+        local all = GetNumFriends()
+        for friendIndex=1, all do
+          local friend_name, friend_level, friend_class, friend_area, friend_connected = GetFriendInfo(friendIndex)
+          if ( friend_connected ) then
+            online = online + 1
+          end
+        end
+
         pfUI.panel:OutputPanel("friends", FRIENDS .. ": " .. online, widget.Tooltip, widget.Click)
       end)
     end

modules/raid.lua

@@ -4,9 +4,7 @@ pfUI:RegisterModule("raid", function ()
 
   -- tell RaidFrame.lua pfUI replaces party frames
   HookAddonOrVariable("Blizzard_RaidUI", function()
-    -- must reach _G: RaidFrame.lua reads this global, and pfUI.env has
-    -- __index but no __newindex, so a bare assignment stays in the sandbox
-    _G.GROUP_REPLACE_PARTY = "1"
+    GROUP_REPLACE_PARTY = "1"
   end)
 
   pfUI.uf.raid = CreateFrame("Frame", "pfRaidUpdater", UIParent)
@@ -125,7 +123,6 @@ pfUI:RegisterModule("raid", function ()
     local grid = self.petgrid
 
     local function place(pet, cell, id)
-      pet.label = "raidpet"
       pet.id = id
       local r, g = SlotToCoord(cell, grid.fill, grid.x, grid.y)
       pet:ClearAllPoints()
@@ -141,22 +138,6 @@ pfUI:RegisterModule("raid", function ()
       return
     end
 
-    -- Party shown as a raid grid: mirror slots 1..5 (player + party members)
-    -- into fixed cells. UpdateVisibility maps each slot to pet / partypet<N>.
-    if not IsInRaid() and IsInGroup() and C.unitframes.raidforgroup == "1" then
-      for id = 1, maxraid do
-        if self.pets[id] then
-          if id <= 5 then
-            place(self.pets[id], id, id)
-          else
-            self.pets[id].id = 0
-            self.pets[id]:Hide()
-          end
-        end
-      end
-      return
-    end
-
     if C.unitframes.raidpet.collapse == "1" then
       -- Pack the pets that exist into the leading cells, no gaps.
       local k = 0
@@ -230,13 +211,8 @@ pfUI:RegisterModule("raid", function ()
     this.tick = GetTime() + 1.0
     this.pendingUpdate = nil
 
-    -- Without a raid there is nothing to sort, but a party shown as a raid
-    -- grid still needs its pet frames placed and mapped to the party pets.
-    if not IsInRaid() then
-      this:LayoutPets()
-      this:Hide()
-      return
-    end
+    -- don't proceed without raid
+    if not IsInRaid() then return end
 
     -- clear all existing frames
     for i=1, maxraid do SetRaidIndex(pfUI.uf.raid[i], 0) end

modules/roll.lua

@@ -7,12 +7,8 @@ pfUI:RegisterModule("roll", function ()
   local LOOT_ROLL_NEED = string.gsub(LOOT_ROLL_NEED, "%%s|Hitem:%%d:%%d:%%d:%%d|h%[%%s%]|h%%s", "%%s")
   local LOOT_ROLL_PASSED = string.gsub(LOOT_ROLL_PASSED, "%%s|Hitem:%%d:%%d:%%d:%%d|h%[%%s%]|h%%s", "%%s")
 
-  -- detect the "everyone passed" subject exactly as the loot scanner will capture
-  -- it: feed a dummy item into LOOT_ROLL_ALL_PASSED and run the same LOOT_ROLL_PASSED
-  -- match. (The old strfind had no captures, so `everyone` was always nil and
-  -- "Everyone has passed on: X" got counted as a fake roller.)
-  local everyoneSample = string.gsub(LOOT_ROLL_ALL_PASSED, "%%s", "x")
-  local everyone = cmatch(everyoneSample, LOOT_ROLL_PASSED)
+  -- try to detect the everyone string
+  local _, _, everyone, _ = strfind(LOOT_ROLL_ALL_PASSED, LOOT_ROLL_PASSED)
   pfUI.roll.blacklist = { YOU, everyone }
 
   pfUI.roll.cache = {}
@@ -47,7 +43,6 @@ pfUI:RegisterModule("roll", function ()
 
     local _, _, itemLink = string.find(hyperlink, "(item:%d+:%d+:%d+:%d+)")
     local itemName = C_Item.GetItemInfo(itemLink)
-    if not itemName then return end -- uncached item: avoid cache[nil] "table index is nil"
 
     -- delete obsolete tables
     if pfUI.roll.cache[itemName] and pfUI.roll.cache[itemName]["TIMESTAMP"] < GetTime() - 60 then

modules/socialmod.lua

@@ -4,7 +4,7 @@ pfUI:RegisterModule("socialmod", function ()
   pfUI.socialmod:RegisterEvent("CHAT_MSG_SYSTEM")
   pfUI.socialmod:SetScript("OnEvent", function()
     local name = cmatch(arg1, _G.ERR_FRIEND_ONLINE_SS)
-    name = name or cmatch(arg1, _G.ERR_FRIEND_OFFLINE_S)
+    name = cmatch(arg1, _G.ERR_FRIEND_OFFLINE_S)
     if name and playerdb[name] and playerdb[name].cname then
       playerdb[name].lastseen = date("%a %d-%b-%Y")
     end
@@ -77,19 +77,20 @@ pfUI:RegisterModule("socialmod", function ()
       local off = FauxScrollFrame_GetOffset(FriendsFrameFriendsScrollFrame)
 
       for i=1, FRIENDS_TO_DISPLAY do
-        local info = C_FriendList.GetFriendInfoByIndex(off + i)
-        if not info or not info.name or info.name == _G.UNKNOWN then break end
-        local name = info.name
+        local name, level, class, zone, connected, status = GetFriendInfo(off + i)
+        if not name or name == _G.UNKNOWN then break end
         local friendName = _G["FriendsFrameFriendButton"..i.."ButtonTextName"]
         local friendLoc = _G["FriendsFrameFriendButton"..i..FRIENDS_NAME_LOCATION]
         local friendInfo = _G["FriendsFrameFriendButton"..i.."ButtonTextInfo"]
         local caption = friendName or friendLoc
 
-        if info.connected then
-          local ccolor = PFUI_CLASS_COLORS[info.classFilename]
-          local status = info.afk and CHAT_FLAG_AFK or info.dnd and CHAT_FLAG_DND or ""
-          local zone = ( info.area == playerzone and "|cffffffff" or "|cffcccccc" ) .. info.area .. "|r"
-          local cname = ccolor:WrapTextInColorCode(name)
+        if connected then
+          if not class or class == _G.UNKNOWN then break end
+          local ccolor = PFUI_CLASS_COLORS[L["class"][class]] or { 1, 1, 1 }
+          local lcolor = GetDifficultyColor(tonumber(level)) or { 1, 1, 1 }
+
+          zone = ( zone == playerzone and "|cffffffff" or "|cffcccccc" ) .. zone .. "|r"
+          local cname = rgbhex(ccolor) .. name .. "|r"
           if playerdb[name] then
             playerdb[name].lastseen = date("%a %d-%b-%Y")
             playerdb[name].cname = cname
@@ -102,7 +103,7 @@ pfUI:RegisterModule("socialmod", function ()
             friendLoc:SetText(format(TEXT(FRIENDS_LIST_TEMPLATE), cname, zone, status))
           end
 
-          friendInfo:SetText(format(TEXT(FRIENDS_LEVEL_TEMPLATE), info.level, info.className))
+          friendInfo:SetText(format(TEXT(FRIENDS_LEVEL_TEMPLATE), level, class))
           caption:SetVertexColor(1,1,1,.9)
           friendInfo:SetVertexColor(1,1,1,.9)
         else
@@ -123,73 +124,74 @@ pfUI:RegisterModule("socialmod", function ()
 
   do -- add colors to who list
     hooksecurefunc("WhoList_Update", function()
-      local num, max = C_FriendList.GetNumWhoResults()
+      local num, max = GetNumWhoResults()
       local off = FauxScrollFrame_GetOffset(WhoListScrollFrame)
 
       local playerzone  = GetRealZoneText()
       local playerrace  = UnitRace("player")
       local playerguild = GetGuildInfo("player")
 
-      if num + 1 >= MAX_WHOS_FROM_SERVER then
-        WhoFrameTotals:SetText("|cffffffff" .. format(GetText("WHO_FRAME_TOTAL_TEMPLATE", nil, num), max).."  |cffaaaaaa"..format(WHO_FRAME_SHOWN_TEMPLATE, MAX_WHOS_FROM_SERVER))
-      else
-        WhoFrameTotals:SetText("|cffffffff" .. format(GetText("WHO_FRAME_TOTAL_TEMPLATE", nil, num), num).."  |cffaaaaaa"..format(WHO_FRAME_SHOWN_TEMPLATE, num))
-      end
-
       for i=1, WHOS_TO_DISPLAY do
-        local info = C_FriendList.GetWhoInfo(off + i)
-        if info then
-          -- filename is the class token, so no L["class"] reversal is needed
-          local class = info.filename
+        local name, guild, level, race, class, zone = GetWhoInfo(off + i)
+        local displayedText = ""
 
-          _G["WhoFrameButton"..i.."Name"]:SetTextColor(NORMAL_FONT_COLOR:GetRGB())
-
-          if (UIDropDownMenu_GetSelectedID(WhoFrameDropDown) == 1) then
-            if (info.area == playerzone) then
-              _G["WhoFrameButton"..i.."Variable"]:SetTextColor(.5, 1, 1)
-            else
-              _G["WhoFrameButton"..i.."Variable"]:SetTextColor(1, 1, 1)
-            end
-
-          elseif (UIDropDownMenu_GetSelectedID(WhoFrameDropDown) == 2) then
-            if (info.fullGuildName == playerguild) then
-              _G["WhoFrameButton"..i.."Variable"]:SetTextColor(.5, 1, 1)
-            else
-              _G["WhoFrameButton"..i.."Variable"]:SetTextColor(1, 1, 1)
-            end
-
-          elseif (UIDropDownMenu_GetSelectedID(WhoFrameDropDown) == 3) then
-            if (info.raceStr == playerrace) then
-              _G["WhoFrameButton"..i.."Variable"]:SetTextColor(.5, 1, 1)
-            else
-              _G["WhoFrameButton"..i.."Variable"]:SetTextColor(1, 1, 1)
-            end
-          end
-
-          if class then
-            local classicon = _G["WhoFrameButton"..i].classicon
-            local coords = CLASS_ICON_TCOORDS[class]
-            local color = PFUI_CLASS_COLORS[class]
-
-            -- do we have classicons? (skin enabled?)
-            if classicon then
-              _G["WhoFrameButton"..i.."Class"]:SetTextColor(0,0,0,0)
-              _G["WhoFrameButton"..i.."Name"]:SetTextColor(color.r,color.g,color.b,1)
-
-              if coords then
-                classicon:Show()
-                classicon:SetTexCoord(unpack(coords))
-              else
-                classicon:Hide()
-              end
-            else
-              _G["WhoFrameButton"..i.."Class"]:SetTextColor(color.r,color.g,color.b,1)
-            end
-          end
-
-          local color = GetDifficultyColor(info.level)
-          _G["WhoFrameButton"..i.."Level"]:SetTextColor(color.r, color.g, color.b)
+        if num + 1 >= MAX_WHOS_FROM_SERVER then
+          displayedText = format(WHO_FRAME_SHOWN_TEMPLATE, MAX_WHOS_FROM_SERVER)
+          WhoFrameTotals:SetText("|cffffffff" .. format(GetText("WHO_FRAME_TOTAL_TEMPLATE", nil, num), max).."  |cffaaaaaa"..displayedText)
+        else
+          displayedText = format(WHO_FRAME_SHOWN_TEMPLATE, num)
+          WhoFrameTotals:SetText("|cffffffff" .. format(GetText("WHO_FRAME_TOTAL_TEMPLATE", nil, num), num).."  |cffaaaaaa"..displayedText)
         end
+
+        class = L["class"][class]
+
+        _G["WhoFrameButton"..i.."Name"]:SetTextColor(NORMAL_FONT_COLOR.r, NORMAL_FONT_COLOR.g, NORMAL_FONT_COLOR.b)
+
+        if (UIDropDownMenu_GetSelectedID(WhoFrameDropDown) == 1) then
+          if (zone == playerzone) then
+            _G["WhoFrameButton"..i.."Variable"]:SetTextColor(.5, 1, 1)
+          else
+            _G["WhoFrameButton"..i.."Variable"]:SetTextColor(1, 1, 1)
+          end
+
+        elseif (UIDropDownMenu_GetSelectedID(WhoFrameDropDown) == 2) then
+          if (guild == playerguild) then
+            _G["WhoFrameButton"..i.."Variable"]:SetTextColor(.5, 1, 1)
+          else
+            _G["WhoFrameButton"..i.."Variable"]:SetTextColor(1, 1, 1)
+          end
+
+        elseif (UIDropDownMenu_GetSelectedID(WhoFrameDropDown) == 3) then
+          if (race == playerrace) then
+            _G["WhoFrameButton"..i.."Variable"]:SetTextColor(.5, 1, 1)
+          else
+            _G["WhoFrameButton"..i.."Variable"]:SetTextColor(1, 1, 1)
+          end
+        end
+
+        if class then
+          local classicon = _G["WhoFrameButton"..i].classicon
+          local coords = CLASS_ICON_TCOORDS[class]
+          local color = PFUI_CLASS_COLORS[class]
+
+          -- do we have classicons? (skin enabled?)
+          if classicon then
+            _G["WhoFrameButton"..i.."Class"]:SetTextColor(0,0,0,0)
+            _G["WhoFrameButton"..i.."Name"]:SetTextColor(color.r,color.g,color.b,1)
+
+            if coords then
+              classicon:Show()
+              classicon:SetTexCoord(unpack(coords))
+            else
+              classicon:Hide()
+            end
+          else
+            _G["WhoFrameButton"..i.."Class"]:SetTextColor(color.r,color.g,color.b,1)
+          end
+        end
+
+        local color = GetDifficultyColor(level)
+        _G["WhoFrameButton"..i.."Level"]:SetTextColor(color.r, color.g, color.b)
       end
     end)
   end

modules/superwow.lua

@@ -75,10 +75,7 @@ pfUI:RegisterModule("superwow", function ()
     DEFAULT_CHAT_FRAME:AddMessage("-> https://github.com/balakethelock/SuperWoW/releases/")
   end
 
-  -- compare numerically: an exact string match silently drops this on any
-  -- SuperWoW release past 1.5
-  local swVersion = tonumber(SUPERWOW_VERSION)
-  if swVersion and swVersion >= 1.5 then
+  if SUPERWOW_VERSION == "1.5" then
     QueueFunction(function()
       local pfCombatText_AddMessage = _G.CombatText_AddMessage
       _G.CombatText_AddMessage = function(message, a, b, c, d, e, f)

modules/swingtimer.lua

@@ -284,9 +284,7 @@ pfUI:RegisterModule("swingtimer", function ()
   UpdateMovable(pfUI.swingtimer.ranged)
 
   -- OH weapon detection
-  -- OH weapon detection: slot 17 (off hand) accepts one-hand (13) and off-hand
-  -- weapons (22); main-hand-only (21) can never sit there.
-  local OH_WEAPON_TYPES = { [13]=true, [22]=true }
+  local OH_WEAPON_TYPES = { [13]=true, [21]=true }
   local function HasOffhandWeapon()
     local l = GetInventoryItemLink("player", 17)
     if not l then return false end

modules/thirdparty.lua

@@ -277,103 +277,6 @@ pfUI:RegisterModule("thirdparty", function()
     end
   end)
 
-  -- GreedMeter Damage Meter
-  -- Vanilla: https://github.com/iGreed/GreedMeter
-  -- GreedMeter builds its windows lazily and supports several of them, so the
-  -- frames are resolved at call time. The primary window takes the "damage"
-  -- dock slot; a second window (GreedMeter.UI.frames[2]) takes the "threat"
-  -- slot so the two tile the panel side-by-side. Docking triggers a
-  -- LayoutBars/RefreshFrame so the bars refit the new width.
-  HookAddonOrVariable("GreedMeter", function()
-    local UI = GreedMeter.UI
-
-    local function frameAt(i)
-      return UI and UI.frames and UI.frames[i]
-    end
-    local function primary()
-      return (UI and UI.mainFrame) or getglobal("GreedMeterFrame1")
-    end
-    local function relayout(frame)
-      if not UI then return end
-      if UI.LayoutBars then UI:LayoutBars(frame) end
-      if UI.RefreshFrame then UI:RefreshFrame(frame) end
-    end
-
-    -- Builds a docktable for a window resolved through getframe(). left picks
-    -- the left half in split mode; the primary window uses the right half.
-    local function BuildDock(name, getframe, left)
-      return { "greedmeter", "GreedMeter", name,
-        function() -- single: fill the panel
-          local frame = getframe()
-          if not frame then return end
-          frame:ClearAllPoints()
-          frame:SetAllPoints(pfUI.chat.right)
-          frame:SetWidth(pfUI.chat.right:GetWidth())
-          relayout(frame)
-        end,
-        function() -- dual: take one half
-          local frame = getframe()
-          if not frame then return end
-          frame:ClearAllPoints()
-          if left then
-            frame:SetPoint("TOPLEFT", pfUI.chat.right, "TOPLEFT", 0, 0)
-            frame:SetPoint("BOTTOMRIGHT", pfUI.chat.right, "BOTTOM", 0, 0)
-          else
-            frame:SetPoint("TOPLEFT", pfUI.chat.right, "TOP", 0, 0)
-            frame:SetPoint("BOTTOMRIGHT", pfUI.chat.right, "BOTTOMRIGHT", 0, 0)
-          end
-          frame:SetWidth(pfUI.chat.right:GetWidth() / 2)
-          relayout(frame)
-        end,
-        function() -- show
-          local frame = getframe()
-          if frame then frame:Show() relayout(frame) end
-        end,
-        function() -- hide
-          local frame = getframe()
-          if frame then frame:Hide() end
-        end
-      }
-    end
-
-    local threatdock = BuildDock("GreedMeterFrame2", function() return frameAt(2) end, true)
-    pfUI.thirdparty.meters:RegisterMeter("damage", BuildDock("GreedMeterFrame1", primary, false))
-
-    -- pfUI's dock decides fill-vs-split by whether the threat slot is filled,
-    -- not by counting windows, so reconcile that slot whenever GreedMeter adds
-    -- or removes one. Only the first two windows fit; any beyond that float.
-    local function SyncSecond()
-      if not pfUI.thirdparty.meters.damage then return end -- dock disabled
-      local has2 = frameAt(2) ~= nil
-      if has2 and not pfUI.thirdparty.meters.threat then
-        pfUI.thirdparty.meters.threat = threatdock
-      elseif not has2 and pfUI.thirdparty.meters.threat == threatdock then
-        pfUI.thirdparty.meters.threat = nil
-      end
-      pfUI.thirdparty.meters:Resize()
-
-      -- match the second window's visibility to the panel (the primary tracks it)
-      local f1, f2 = primary(), frameAt(2)
-      if f2 then
-        if f1 and f1:IsShown() then f2:Show() relayout(f2) else f2:Hide() end
-      end
-    end
-
-    -- AddFrame and the login-time layout restore both go through
-    -- CreateMeterFrame; RemoveFrame handles closing a window.
-    local origCreate = UI.CreateMeterFrame
-    UI.CreateMeterFrame = function(self, isPrimary, copyFrom)
-      local frame = origCreate(self, isPrimary, copyFrom)
-      SyncSecond()
-      return frame
-    end
-    local origRemove = UI.RemoveFrame
-    UI.RemoveFrame = function(self, frame)
-      origRemove(self, frame)
-      SyncSecond()
-    end
-  end)
-
   -- DPSMate Damage Meter
   -- Vanilla: https://github.com/Geigerkind/DPSMate
   -- TBC: https://github.com/Geigerkind/DPSMateTBC

modules/tooltip.lua

@@ -185,102 +185,14 @@ pfUI:RegisterModule("tooltip", function ()
   GameTooltipStatusBar.SetStatusBarColor_orig = GameTooltipStatusBar.SetStatusBarColor
   GameTooltipStatusBar.SetStatusBarColor = function() return end
 
-  -- A row of buff icons anchored above the tooltip for mouseover units.
-  -- Icons come from a ClassicAPI object pool: ReleaseAll hides the whole
-  -- set each refresh, then we Acquire and re-anchor left-to-right.
-  local BUFF_SIZE, BUFF_SPACING, BUFF_MAX = 20, 2, 32
-  pfUI.tooltip.buffs = CreateFrame("Frame", "pfTooltipBuffs", GameTooltip)
-  pfUI.tooltip.buffs:SetPoint("BOTTOMLEFT", GameTooltipStatusBar, "TOPLEFT", 0, default_border + 2)
-  pfUI.tooltip.buffs:SetHeight(BUFF_SIZE)
-
-  local function CreateBuffIcon()
-    local icon = CreateFrame("Frame", nil, pfUI.tooltip.buffs)
-    icon:SetSize(BUFF_SIZE, BUFF_SIZE)
-    icon.texture = icon:CreateTexture(nil, "BACKGROUND")
-    icon.texture:SetTexCoord(.07, .93, .07, .93)
-    icon.texture:SetAllPoints(icon)
-    CreateBackdrop(icon)
-
-    icon.stacks = icon:CreateFontString(nil, "OVERLAY", "GameFontNormal")
-    icon.stacks:SetPoint("BOTTOMRIGHT", icon, "BOTTOMRIGHT", 1, 0)
-    icon.stacks:SetFont(C.tooltip.font_tooltip, C.tooltip.font_tooltip_size, "OUTLINE")
-    icon.stacks:SetTextColor(1, 1, 1, 1)
-
-    icon.timer = icon:CreateFontString(nil, "OVERLAY", "GameFontNormal")
-    icon.timer:SetPoint("CENTER", icon, "CENTER", 0, 0)
-    icon.timer:SetFont(C.tooltip.font_tooltip, C.tooltip.font_tooltip_size, "OUTLINE")
-    icon.timer:SetTextColor(1, 1, 1, 1)
-    return icon
-  end
-
-  pfUI.tooltip.buffpool = CreateObjectPool(CreateBuffIcon, function(_, icon)
-    icon:Hide()
-    icon:ClearAllPoints()
-  end)
-
-  -- Tick the remaining-time text on the visible icons. OnUpdate only fires
-  -- while the row is shown, so it stops as soon as the tooltip hides.
-  pfUI.tooltip.buffs:SetScript("OnUpdate", function()
-    local now = GetTime()
-    if (this.tick or 0) > now then return end
-    this.tick = now + 0.1
-    for icon in pfUI.tooltip.buffpool:EnumerateActive() do
-      local timeleft = icon.expirationTime and icon.expirationTime > 0 and (icon.expirationTime - now) or 0
-      icon.timer:SetText(timeleft > 0 and GetColoredTimeString(timeleft) or "")
-    end
-  end)
-
-  function pfUI.tooltip:UpdateBuffs(unit)
-    pfUI.tooltip.buffpool:ReleaseAll()
-
-    if C.tooltip.showbuffs ~= "1" or not unit or unit == "none" then
-      pfUI.tooltip.buffs:Hide()
-      return
-    end
-
-    local prev, count = nil, 0
-    for i = 1, BUFF_MAX do
-      local aura = C_UnitAuras.GetBuffDataByIndex(unit, i)
-      if not aura then break end
-
-      count = count + 1
-      local icon = pfUI.tooltip.buffpool:Acquire()
-      icon.texture:SetTexture(aura.icon)
-      icon.stacks:SetText(aura.applications and aura.applications > 1 and aura.applications or "")
-
-      icon.expirationTime = aura.expirationTime
-      local timeleft = aura.expirationTime and aura.expirationTime > 0 and (aura.expirationTime - GetTime()) or 0
-      icon.timer:SetText(timeleft > 0 and GetColoredTimeString(timeleft) or "")
-
-      if prev then
-        icon:SetPoint("LEFT", prev, "RIGHT", BUFF_SPACING, 0)
-      else
-        icon:SetPoint("BOTTOMLEFT", pfUI.tooltip.buffs, "BOTTOMLEFT", 0, 0)
-      end
-      icon:Show()
-      prev = icon
-    end
-
-    if count > 0 then
-      pfUI.tooltip.buffs:SetWidth(count * BUFF_SIZE + (count - 1) * BUFF_SPACING)
-      pfUI.tooltip.buffs:Show()
-    else
-      pfUI.tooltip.buffs:Hide()
-    end
-  end
-
   function pfUI.tooltip:Update()
       local unit = pfUI.tooltip:GetUnit()
-      pfUI.tooltip:UpdateBuffs(unit)
       if unit == "none" then
+        -- process item tooltips
         if C.tooltip.itemid == "1" and GameTooltip:HasItem() then
           local _, _, itemID = GameTooltip:GetItem()
           GameTooltip:AddLine(T["ItemID"] .. ": " .. itemID, .25,.5,1)
           GameTooltip:Show()
-        elseif C.tooltip.spellid == "1" and GameTooltip:HasSpell() then
-          local _, _, spellID = GameTooltip:GetSpell()
-          GameTooltip:AddLine(T["SpellID"] .. ": " .. spellID, .25,.5,1)
-          GameTooltip:Show()
         end
 
         return
@@ -347,13 +259,6 @@ pfUI:RegisterModule("tooltip", function ()
         end
       end
 
-      if C.tooltip.unitid == "1" and not UnitIsPlayer(unit) then
-        local npcID = C_CreatureInfo.GetCreatureID(UnitGUID(unit))
-        if npcID then
-          GameTooltip:AddLine(T["UnitID"] .. ": " .. npcID, .25,.5,1)
-        end
-      end
-
       if hp and hpm then
         if hp >= 1000 then hp = round(hp / 1000, 1) .. "k" end
         if hpm >= 1000 then hpm = round(hpm / 1000, 1) .. "k" end
@@ -362,31 +267,16 @@ pfUI:RegisterModule("tooltip", function ()
       GameTooltip:Show()
     end
 
-  if C.tooltip.aurasource == "1" or C.tooltip.spellid == "1" then
+  if C.tooltip.aurasource == "1" then
     hooksecurefunc(GameTooltip, "SetUnitAura", function(self, ...)
       local aura = C_UnitAuras.GetAuraDataByIndex(unpack(arg))
       if not aura then return end
-      if C.tooltip.aurasource == "1" then
-        local caster = aura.sourceUnit and UnitName(aura.sourceUnit)
-        if not caster and aura.sourceGUID then
-          caster = UnitNameFromGUID(aura.sourceGUID)
-        end
-        if caster and caster ~= "" then
-          local classToken
-          if aura.sourceUnit and UnitIsPlayer(aura.sourceUnit) then
-            classToken = UnitClassBase(aura.sourceUnit)
-          elseif aura.sourceGUID then
-            classToken = select(2, GetPlayerInfoByGUID(aura.sourceGUID))
-          end
-          if classToken then
-            caster = PFUI_CLASS_COLORS[classToken]:WrapTextInColorCode(caster)
-          end
-          self:AddLine(T["Cast by"] .. ": " .. caster, .25,.5,1)
-        end
-      end
-      if C.tooltip.spellid == "1" then
-        self:AddLine(T["SpellID"] .. ": " .. aura.spellId, .25,.5,1)
+      local caster = aura.sourceUnit and UnitName(aura.sourceUnit)
+      if not caster and aura.sourceGUID then
+        caster = UnitNameFromGUID(aura.sourceGUID)
       end
+      if not caster or caster == "" then return end
+      self:AddLine(T["Cast by"] .. ": " .. caster, .7, .7, 1)
       self:Show()
     end)
   end

modules/unitxp.lua

@@ -28,7 +28,7 @@ pfUI:RegisterModule("unitxp", function ()
       behindFrame.text:SetFont(pfUI.font_default, fontSize, "OUTLINE")
       behindFrame.text:SetPoint("RIGHT", behindFrame, "RIGHT", -1, 0)
       behindFrame.text:SetTextColor(0.3, 1, 0.3, 1)
-      behindFrame.text:SetText(T["BEHIND"])
+      behindFrame.text:SetText("BEHIND")
       behindFrame.text:Hide()
 
       local lastCheck = 0
@@ -62,7 +62,7 @@ pfUI:RegisterModule("unitxp", function ()
       losFrame.text:SetFont(pfUI.font_default, fontSize, "OUTLINE")
       losFrame.text:SetPoint("RIGHT", losFrame, "RIGHT", -1, -slot)
       losFrame.text:SetTextColor(1, 0.3, 0.3, 1)
-      losFrame.text:SetText(T["NO LOS"])
+      losFrame.text:SetText("NO LOS")
       losFrame.text:Hide()
 
       local lastCheck = 0
@@ -169,9 +169,6 @@ pfUI:RegisterModule("unitxp", function ()
 
           local throttle = 0
           local scanner = CreateFrame("Frame")
-          -- expose the poller frame so PLAYER_LOGOUT can stop its UnitXP OnUpdate
-          -- (it lives on this separate frame, not on distanceIndicator) -> crash 132
-          pfUI.uf.target.distanceScanner = scanner
           scanner:SetScript("OnUpdate", function()
             throttle = throttle + arg1
             if throttle < 0.05 then return end
@@ -239,10 +236,6 @@ pfUI:RegisterModule("unitxp", function ()
         if pfUI.uf.target.distanceIndicator then
           pfUI.uf.target.distanceIndicator:SetScript("OnUpdate", nil)
         end
-        -- free-frame mode polls from its own scanner frame, not the indicator
-        if pfUI.uf.target.distanceScanner then
-          pfUI.uf.target.distanceScanner:SetScript("OnUpdate", nil)
-        end
       end
       return
     end
@@ -275,9 +268,7 @@ pfUI:RegisterModule("unitxp", function ()
     -- Also notify on BG queue pop
     local origBattlefieldPortShow = BattlefieldFrame_Show
     if origBattlefieldPortShow then
-      -- pfUI.env has __index but no __newindex, so a bare global assignment
-      -- inside a RegisterModule closure never leaves the sandbox
-      _G.BattlefieldFrame_Show = function()
+      BattlefieldFrame_Show = function()
         pcall(UnitXP, "notify", "taskbarIcon")
         pcall(UnitXP, "notify", "systemSound")
         return origBattlefieldPortShow()

pfUI.lua

@@ -23,7 +23,7 @@ do
   -- ClassicAPI dependency check.
   -- pfUI relies pervasively on the modern C_* / SuperWoW / nameplate / focus
   -- API surface that ClassicAPI polyfills, so presence is required.
-  local PFUI_CLASSIC_API_MIN     = 10906  -- (X*10000 + Y*100 + Z)
+  local PFUI_CLASSIC_API_MIN     = 10905  -- (X*10000 + Y*100 + Z)
   local PFUI_CLASSIC_API_LATEST  = PFUI_CLASSIC_API_MIN
   local PFUI_CLASSIC_API_WEBSITE = "https://github.com/brues-code/ClassicAPI"
   local PFUI_CLASSIC_API_LATEST_URL = PFUI_CLASSIC_API_WEBSITE .. "/releases/latest"
@@ -110,14 +110,6 @@ pfUI.movables = {}
 pfUI.version = {}
 pfUI.env = {}
 
--- Capability flag: this fork's pfActionBar buttons correctly handle the modern
--- HookScript widget method, so ClassicAPI's AddOnCompat shim must NOT shadow
--- HookScript during actionbar load (older forks branch on `if button.HookScript`
--- and take a modern path they were never written for). This describes behavior,
--- not identity: a fork that keeps the HookScript-correct actionbar code keeps
--- the flag; one that lacks it should drop the flag and get the safe fallback.
-pfUI.handlesHookScript = true
-
 if not pfUI.disabled then
   pfUI.events = Mixin({}, CallbackRegistryMixin)
   pfUI.events:OnLoad()
@@ -300,21 +292,6 @@ function pfUI:GetEnvironment()
   pfUI.env.C = pfUI_config
   pfUI.env.pfUI_throttle = _G.pfUI_throttle
   pfUI.env.L = (pfUI_locale[GetLocale()] or pfUI_locale["enUS"])
-  pfUI.env.L["class"] = pfUI.env.L["class"] or tInvert(LOCALIZED_CLASS_NAMES_MALE)
-  if not pfUI.env.L["race"] then
-      pfUI.env.L["race"] = {}
-      local i = 1
-      local raceInfo = C_CreatureInfo.GetRaceInfo(i)
-      while raceInfo ~= nil do
-          pfUI.env.L["race"][raceInfo.clientFileString] = {
-              raceName = raceInfo.raceName,
-              raceID = raceInfo.raceID,
-              faction = C_CreatureInfo.GetFactionInfo(i).groupTag,
-          }
-          i = i + 1
-          raceInfo = C_CreatureInfo.GetRaceInfo(i)
-      end
-   end
 
   return pfUI.env
 end

pfUI.toc

@@ -6,6 +6,7 @@
 ## Version: @project-version@
 ## SavedVariables: pfUI_profiles, pfUI_addon_profiles, pfUI_cache, pfUI_throttle
 ## SavedVariablesPerCharacter: pfUI_config, pfUI_init, pfUI_playerDB
+## Dependencies: !!!ClassicAPI
 ## X-Website: https://github.com/brues-code/pfUI
 
 pfUI.lua