Thread Tools Display Modes
05/11/22, 05:04 PM   #1
Mr.Envy
 
Mr.Envy's Avatar
Join Date: Apr 2020
Posts: 20
CombatCloud help

Hi guys, so, I'm using CombatCloud to show the dmg I am dealing above my reticle, I really like the way it looks/works, the only problem is that it is a bit outdated and the author no longer supports it, I'd like help in solving a LUA error that keeps popping up in an abusive way, for example, after completing a dungeon I have like 40+ notifications regarding this LUA error:

bad argument #1 to 'unpack' (table/struct expected, got nil)
stack traceback:
[C]: in function 'unpack'
user:/AddOns/CombatCloud/Core/EventViewers/_Base.lua:173: in function 'CombatCloud_EventViewer:PrepareLabel'
user:/AddOns/CombatCloud/Core/EventViewers/Combat_Hybrid.lua:58: in function 'CombatCloud_CombatHybridEventViewer:View'
user:/AddOns/CombatCloud/Core/EventViewers/Combat_Hybrid.lua:47: in function 'CombatCloud_CombatHybridEventViewer:ViewFromEventBuffer'
user:/AddOns/CombatCloud/Core/EventViewers/Combat_Hybrid.lua:34: in function 'func'
/EsoUI/Libraries/Globals/globalapi.lua:244: in function '(anonymous)'

It doesn't break my game or anything, CombatCloud works as intended, but spams the sh*t out of me with that error and it's becoming annoying. Anyone knows how I might make this disappear?
  Reply With Quote
05/12/22, 04:28 AM   #2
Baertram
Super Moderator
 
Baertram's Avatar
WoWInterface Super Mod
AddOn Author - Click to view addons
Join Date: Mar 2014
Posts: 4,912
Edit the file /Core/EventViewers/_base.lua with a text editor
Add this line at line 173
if not color then return end

The function CombatCloud_EventViewer:PrepareLabel looks like this in the end
Lua Code:
  1. function CombatCloud_EventViewer:PrepareLabel(label, fontSize, color, text)
  2.     label:SetText(text)
  3.     if not color then return end
  4.     label:SetColor(unpack(color))
  5.     label:SetFont(format('%s|%d|%s', self.LMP:Fetch('font', CombatCloudSettings.fontFace), fontSize, CombatCloudSettings.fontOutline))
  6. end

Hope it fixes the spam, as the unpack function says it cannot use the table "color" properly as it is nil. So adding a check before should abort before the error occurs.
  Reply With Quote
05/12/22, 07:39 AM   #3
Mr.Envy
 
Mr.Envy's Avatar
Join Date: Apr 2020
Posts: 20
It did fix my problem, just completed a dungeon just to test it out and not a single error popped up. Thanks a lot Baertram! If it is ok for you I would like to post this fix in the addons' comments section so others can use it, giving the credits to you, ofc. ^^ Thank you once again, I really appreciate your help, all the best.

Nevermind about posting, just noticed that you already did, awesome!
  Reply With Quote
05/12/22, 09:50 AM   #4
Baertram
Super Moderator
 
Baertram's Avatar
WoWInterface Super Mod
AddOn Author - Click to view addons
Join Date: Mar 2014
Posts: 4,912

You are welcome, glad it works
  Reply With Quote

ESOUI » AddOns » AddOn Help/Support » CombatCloud help

Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off