bug fixes

Libs/LibOpenRaid/LibOpenRaid.lua

@@ -43,10 +43,10 @@ TODO:
     - add into gear info how many tier set parts the player has
     - raid lockouts normal-heroic-mythic
     - soulbind character (covenant choise) - probably not used in 10.0
-    
+
 BUGS:
     - after a /reload, it is not starting new tickers for spells under cooldown
-    
+
 --]=]
 
 local versionString, revision, launchDate, gameVersion = GetBuildInfo()
@@ -63,9 +63,11 @@ if (WOW_PROJECT_ID ~= WOW_PROJECT_MAINLINE and not isExpansion_Dragonflight()) t
 end
 
 local major = "LibOpenRaid-1.0"
-local CONST_LIB_VERSION = 59
+local CONST_LIB_VERSION = 60
 LIB_OPEN_RAID_CAN_LOAD = false
 
+local unpack = table.unpack or _G.unpack
+
 --declae the library within the LibStub
     local libStub = _G.LibStub
     local openRaidLib = libStub:NewLibrary(major, CONST_LIB_VERSION)
@@ -322,15 +324,15 @@ LIB_OPEN_RAID_CAN_LOAD = false
     openRaidLib.Schedules = {
         registeredUniqueTimers = {}
     }
-    
+
     --run a scheduled function with its payload
     local triggerScheduledTick = function(tickerObject)
         local payload = tickerObject.payload
         local callback = tickerObject.callback
-    
-        local result, errortext = pcall(callback, _G.unpack(payload))
+
+        local result, errortext = xpcall(callback, geterrorhandler(), unpack(payload))
         if (not result) then
-            sendChatMessage("openRaidLib: error on scheduler:", tickerObject.scheduleName, tickerObject.stack, errortext)
+            sendChatMessage("openRaidLib: error on scheduler:", tickerObject.scheduleName, tickerObject.stack)
         end
 
         if (tickerObject.isUnique) then
@@ -455,15 +457,14 @@ LIB_OPEN_RAID_CAN_LOAD = false
             local func = addonObject[functionName]
 
             if (func) then
-                --using pcall at the moment, should get a better caller in the future
-                local okay, errorMessage = pcall(func, ...)
+                local okay, errorMessage = xpcall(func, geterrorhandler(), ...)
                 if (not okay) then
-                    sendChatMessage("error:", errorMessage)
+                    sendChatMessage("error on callback for event:", event)
                 end
             end
         end
     end
-    
+
     function openRaidLib.RegisterCallback(addonObject, event, callbackMemberName)
         --check of integrity
         local integrity = checkRegisterDataIntegrity(addonObject, event, callbackMemberName)
@@ -1211,7 +1212,7 @@ openRaidLib.internalCallback.RegisterCallback("onLeaveCombat", openRaidLib.UnitI
         weaponEnchant = 0,
         noGems = {},
         noEnchants = {},
-    }    
+    }
 
     function openRaidLib.GetAllUnitsGear()
         return openRaidLib.GearManager.GetAllUnitsGear()
@@ -1376,7 +1377,7 @@ openRaidLib.internalCallback.RegisterCallback("onLeaveCombat", openRaidLib.UnitI
         --[2] int durability
         --[3] int weapon enchant
         --[4] table with integers of equipSlot without enchant
-        --[5] table with integers of equipSlot which has a gem slot but the slot is empty            
+        --[5] table with integers of equipSlot which has a gem slot but the slot is empty
 
         local dataToSend = CONST_COMM_GEARINFO_FULL_PREFIX .. ","
         local playerGearInfo = openRaidLib.GearManager.GetPlayerFullGearInfo()