illiath / EnhancedFlightMap

Flight addon for WoW
Apache License 2.0
3 stars 7 forks source link

/efm report errors #18

Closed beren12 closed 2 years ago

beren12 commented 3 years ago
Message: Interface\AddOns\EnhancedFlightMap\message.lua:66: SendChatMessage(): Whisper message missing target player!
Time: Fri May 14 15:52:53 2021
Count: 2
Stack: Interface\AddOns\EnhancedFlightMap\message.lua:66: SendChatMessage(): Whisper message missing target player!
[string "=[C]"]: in function `SendChatMessage'
[string "@Interface\AddOns\EnhancedFlightMap\message.lua"]:66: in function `EFM_Report_Flight'
[string "@Interface\AddOns\EnhancedFlightMap\EnhancedFlightMap.lua"]:140: in function `EFM_SlashCommandHandler'
[string "@Interface\AddOns\EnhancedFlightMap\EnhancedFlightMap.lua"]:24: in function `?'
[string "@Interface\FrameXML\ChatFrame.lua"]:4825: in function `ChatEdit_ParseText'
[string "@Interface\FrameXML\ChatFrame.lua"]:4488: in function `ChatEdit_SendText'
[string "@Interface\FrameXML\ChatFrame.lua"]:4524: in function `ChatEdit_OnEnterPressed'
[string "*:OnEnterPressed"]:1: in function <[string "*:OnEnterPressed"]:1>

Locals: (*temporary) = "EFM: Heading to Lakeshire, Redridge, estimated to arrive in 4:48."
(*temporary) = "WHISPER"
(*temporary) = "Common"
(*temporary) = ""
beren12 commented 3 years ago

change

        elseif (messageDest ~= nil) then
            local chanNum = tonumber(reportTo);

to

        elseif (messageDest ~= "") then
            local chanNum = tonumber(reportTo);
beren12 commented 3 years ago

should also change the max number of channels to 99 i have toons in 17 channels...

beren12 commented 3 years ago

is there a reason to have a max number at all?

beren12 commented 3 years ago

https://github.com/beren12/EnhancedFlightMap/commit/7b6039a634f0eb26747a8dbf5b8c351b64d4e05c