Shadowlands Pre-Patch errors
Meebsy opened this issue · 14 comments
Message: ...ce\AddOns\PetTracker\addons\main\features\gossip.lua:44: hooksecurefunc(): SelectGossipOption is not a function Time: Wed Oct 14 09:49:32 2020 Count: 1 Stack: ...ce\AddOns\PetTracker\addons\main\features\gossip.lua:44: hooksecurefunc(): SelectGossipOption is not a function
Version 8.3.8
Message: ...ddOns\PetTracker\addons\main\features\objectives.lua:78: attempt to index field 'BlocksFrame' (a nil value) Time: Wed Oct 14 09:49:34 2020 Count: 1 Stack: ...ddOns\PetTracker\addons\main\features\objectives.lua:78: attempt to index field 'BlocksFrame' (a nil value)
Here's another one from 9.0.1:
Message: ...Ons\PetTracker\addons\main\classes\maps\rivalPin.lua:37: attempt to call global 'IsQuestFlaggedCompleted' (a nil value)
Time: Wed Oct 14 17:43:40 2020
Count: 10
Stack: ...Ons\PetTracker\addons\main\classes\maps\rivalPin.lua:37: attempt to call global 'IsQuestFlaggedCompleted' (a nil value)
I believe there's "C_QuestLog." missing in front of the funtion name.
I tested the fixes for the other two errors and they seem to work.
this should fix it, BUT it need to be tested!
replace "...\Interface\AddOns\PetTracker\addons\main\features\gossip.lua"
hooksecurefunc('SelectGossipOption', function(index)
with
hooksecurefunc(C_GossipInfo, 'SelectOption', function(index)
this should fix the first error!
also replace "...\Interface\AddOns\PetTracker\addons\main\features\objectives.lua"
local height = DEFAULT_OBJECTIVE_TRACKER_MODULE.BlocksFrame.contentsHeight or 0
with
local height = ObjectiveTrackerFrame.BlocksFrame.contentsHeight or 0
this should fix the second error
And also change in all PetTracker TOC files:
## Interface: 80300
to this:
## Interface: 90001
That's an issue with Sushi-3.1, a library which is used in PetTracker:
I've been fixing other bugs lately, I can probably tackle PetTracker and that one if I can get you all as testers.
How can you make PetTracker work for you?
Until the author returns, make these changes yourself:
I think you can simply download my fork and unzip it overtop of your installation, but I haven't tried that.
Otherwise, you can edit all the individual files:
eb82e4f
8feca83
c485ad9
2cc249c
Jaliborc/Sushi-3.2@d307f68 (I personally just deleted Libs/Sushi-3.1
and re-downloaded it).
Jaliborc/CustomTutorials-2.1@d732865 (not confirmed by me)
comments
I think this is all, but I haven't done anything pet-related in two expansions so I can't even remember all of what this addon does. 😝
If you find future problems, please open a new ticket https://github.com/Jaliborc/PetTracker/issues/new/choose
You can @spiralofhope
in there if you think I can help (like if you found a solution!), but I'm not a programmer (no, really)
outstanding issues
And this one:
Message: ...terface\AddOns\PetTracker\addons\battle\listener.lua:101: attempt to call field 'GetDate' (a nil value)
Time: Wed Oct 14 18:07:11 2020
Count: 1
Stack: ...terface\AddOns\PetTracker\addons\battle\listener.lua:101: attempt to call field 'GetDate' (a nil value)
Don't know if it helps but I'm seeing the same error and have more, hopefully useful information:
165x ...ddOns\PetTracker\addons\main\features\objectives.lua:78: attempt to index field 'BlocksFrame' (a nil value)
[string "@PetTracker\addons\main\features\objectives.lua"]:78: in function GetUsedHeight' [string "@PetTracker\addons\main\features\objectives.lua"]:48: in function <...ddOns\PetTracker\addons\main\features\objectives.lua:47> [string "=[C]"]: in function
ObjectiveTracker_Update'
[string "@Blizzard_ObjectiveTracker\Blizzard_ObjectiveTracker.lua"]:937: in function <...zzard_ObjectiveTracker\Blizzard_ObjectiveTracker.lua:936>
[string "=[C]"]: in function GetHeight' [string "@Blizzard_ObjectiveTracker\Blizzard_ObjectiveTracker.lua"]:1339: in function <...zzard_ObjectiveTracker\Blizzard_ObjectiveTracker.lua:1325> [string "=[C]"]: in function
ObjectiveTracker_Update'
[string "@Blizzard_ObjectiveTracker\Blizzard_ObjectiveTracker.lua"]:917: in function <...zzard_ObjectiveTracker\Blizzard_ObjectiveTracker.lua:835>
Locals:
self = {
0 =
Lines =
}
Index = 1
RegisterMessage = defined @datastore\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:90
UnregisterMessage = defined @datastore\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:153
Update = defined @PetTracker\addons\main\features\objectives.lua:69
Anchor = {
}
SendMessage = defined @datastore\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:59
UnregisterSignal = defined @PetTracker\libs\WildAddon-1.0\WildAddon-1.0.lua:78
RegisterEvent = defined @datastore\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:90
UnregisterAllMessages = defined @datastore\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:174
MaxEntries = 0
SendSignal = defined @PetTracker\libs\WildAddon-1.0\WildAddon-1.0.lua:82
GetUsedHeight = defined @PetTracker\addons\main\features\objectives.lua:77
OnEnable = defined @PetTracker\addons\main\features\objectives.lua:35
Header = {
}
RegisterSignal = defined @PetTracker\libs\WildAddon-1.0\WildAddon-1.0.lua:74
NewModule = defined @PetTracker\libs\WildAddon-1.0\WildAddon-1.0.lua:68
UnregisterAllEvents = defined @datastore\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:174
UnregisterEvent = defined @datastore\libs\CallbackHandler-1.0\CallbackHandler-1.0.lua:153
Tag = "PETTRACKER_"
}
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index field 'BlocksFrame' (a nil value)"
Another Error that breaks (AGAIN and again) another addon (Handynotes: Vision's of N'Zoth)...
only because of the "Sushi" s h i t dropdown menu... -.-
1x ...PetTracker\libs\Sushi-3.1-4\classes\drops\Dropdown.lua:136: Invalid data provided for `:SetBackdrop`
[string "=[C]"]: in function `assert'
[string "@PetTracker\libs\Sushi-3.1-4\classes\drops\Dropdown.lua"]:136: in function `SetBackdrop'
[string "@PetTracker\libs\Sushi-3.1-4\classes\drops\Dropdown.lua"]:48: in function `New'
[string "@PetTracker\libs\Sushi-3.1-4\classes\drops\Dropdown.lua"]:57: in function `Toggle'
[string "@PetTracker\addons\main\features\mapSearch.lua"]:124: in function `ToggleTrackingTypes'
[string "@PetTracker\addons\main\features\mapSearch.lua"]:71: in function <PetTracker\addons\main\features\mapSearch.lua:71>
Locals:
(*temporary) = false
(*temporary) = "Invalid data provided for `:SetBackdrop`"
Okay gang, things seem to be working well. I'm happy to have helped, but I'm burnt out. :)
It's all packaged into one big pull request for the author.
1st, Another LUA error!
How to Reproduce:
Show "Species" on the WorldMap and just click on any Species Icon on the WorldMap
This will open the Pet Journal and triggers the LUA Error here:
1x ...s\PetTracker\addons\main\classes\abstract\specie.lua:26: attempt to index a nil value
[string "@PetTracker\addons\main\classes\abstract\specie.lua"]:26: in function `GetID'
[string "@PetTracker\addons\main\classes\abstract\pet-Pet.lua"]:33: in function `Display'
[string "@PetTracker\addons\main\classes\maps\speciePin.lua"]:67: in function <...ns\PetTracker\addons\main\classes\maps\speciePin.lua:65>
Locals:
self = <table> {
specie = 267
}
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index a nil value"
2nd, The Search bar:
The "Search Bar" stay in actual state (Visible or Invisible) when toggle "Species" on the WorldMap.
But it is visible if "Species" is enabled or invisible of disabled and you Reload the UI!
3rd, Another thing is, the "Search Bar" overlaps the new Map-Pin Button and is non Interactive at all.
Also the "Filter Text" overlaps the actual Search Text!
Seems there is something wrong in function MapSearch:UpdateFrames()
and another one...
How to Reproduce:
ESC - Interface -> trigger this here:
1x ...ker\libs\CustomTutorials-2.1\CustomTutorials-2.1-10.lua:150: attempt to call method 'SetBackdrop' (a nil value)
[string "@PetTracker\libs\CustomTutorials-2.1\CustomTutorials-2.1-10.lua"]:150: in function <...ker\libs\CustomTutorials-2.1\CustomTutorials-2.1.lua:125>
[string "@PetTracker\libs\CustomTutorials-2.1\CustomTutorials-2.1-10.lua"]:181: in function `RegisterTutorials'
[string "@PetTracker\addons\config\tutorials.lua"]:43: in function `Load'
[string "@PetTracker\addons\config\tutorials.lua"]:27: in function `?'
[string "@PetTracker\libs\WildAddon-1.0\WildAddon-1.0-1.lua"]:20: in function <...dOns\PetTracker\libs\WildAddon-1.0\WildAddon-1.0.lua:18>
[string "@PetTracker\libs\WildAddon-1.0\WildAddon-1.0-1.lua"]:56: in function <...dOns\PetTracker\libs\WildAddon-1.0\WildAddon-1.0.lua:53>
[string "=[C]"]: ?
[string "@TradeSkillMaster\External\EmbeddedLibs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:29: in function <...ddedLibs\CallbackHandler-1.0\CallbackHandler-1.0.lua:25>
[string "@TradeSkillMaster\External\EmbeddedLibs\CallbackHandler-1.0\CallbackHandler-1.0-7.lua"]:64: in function `Fire'
[string "@ArkInventory\Externals\Ace3\AceEvent\AceEvent-3.0-4.lua"]:120: in function <...rkInventory\Externals\Ace3\AceEvent\AceEvent-3.0.lua:119>
[string "=[C]"]: ?
[string "=[C]"]: in function `LoadAddOn'
[string "@PetTracker\addons\main\main.lua"]:44: in function <PetTracker\addons\main\main.lua:44>
[string "=[C]"]: in function `Show'
[string "@FrameXML\UIParent.lua"]:2893: in function `SetUIPanel'
[string "@FrameXML\UIParent.lua"]:2699: in function `ShowUIPanel'
[string "@FrameXML\UIParent.lua"]:2606: in function <FrameXML\UIParent.lua:2602>
[string "=[C]"]: in function `SetAttribute'
[string "@FrameXML\UIParent.lua"]:3457: in function <FrameXML\UIParent.lua:3436>
[string "=[C]"]: in function `ShowUIPanel'
[string "@FrameXML\UIParent.lua"]:3482: in function `ShowOptionsPanel'
[string "*:OnClick"]:2: in function <[string "*:OnClick"]:1>
local height = ObjectiveTrackerFrame.BlocksFrame.contentsHeight or 0
im not seeing the second fix in the lua this one
also replace "...\Interface\AddOns\PetTracker\addons\main\features\objectives.lua"
local height = DEFAULT_OBJECTIVE_TRACKER_MODULE.BlocksFrame.contentsHeight or 0
with
local height = ObjectiveTrackerFrame.BlocksFrame.contentsHeight or 0
this should fix the second error
here is my objectives.lua
--[[
Copyright 2012-2020 João Cardoso
PetTracker is distributed under the terms of the GNU General Public License (Version 3).
As a special exception, the copyright holders of this addon do not give permission to
redistribute and/or modify it.
This addon is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
GNU General Public License for more details.
You should have received a copy of the GNU General Public License
along with the addon. If not, see http://www.gnu.org/licenses/gpl-3.0.txt.
This file is part of PetTracker.
--]]
if IsAddOnLoaded('Carbonite.Quests') then
return
end
local ADDON, Addon = ...
local Parent, HeaderButton = ObjectiveTrackerBlocksFrame, ObjectiveTrackerFrame.HeaderMenu
local Objectives = Addon:NewModule('Objectives', Addon.Tracker(Parent))
do
OBJECTIVE_TRACKER_ADDONS = OBJECTIVE_TRACKER_ADDONS or {}
tinsert(OBJECTIVE_TRACKER_ADDONS, 0)
Objectives.Index = #OBJECTIVE_TRACKER_ADDONS
end
--[[ Startup ]]--
function Objectives:OnEnable()
local header = CreateFrame('Button', nil, self, 'ObjectiveTrackerHeaderTemplate')
header:SetScript('OnClick', self.ToggleDropdown)
header:RegisterForClicks('anyUp')
header:SetPoint('TOPLEFT')
header.Text:SetText(PETS)
header:Show()
self.Anchor:SetPoint('TOPLEFT', header, 'BOTTOMLEFT', -4, -10)
self.Anchor:SetScript('OnMouseDown', self.ToggleOptions)
self.Header = header
hooksecurefunc('ObjectiveTracker_Update', function()
local off = self:GetUsedHeight()
local availableEntries = floor(((Parent.maxHeight or 0) - off - 45) / 20)
if availableEntries ~= self.MaxEntries then
self.MaxEntries = availableEntries
self:Update()
end
self:SetPoint('TOPLEFT', Parent, -10, -off)
end)
HeaderButton:HookScript('OnHide', function()
if self:IsShown() then
HeaderButton:Show()
end
end)
end
--[[ API Override ]]--
function Objectives:Update()
self:GetClass().Update(self)
self:SetShown(Addon.sets.trackPets and self.Anchor:IsShown())
HeaderButton:SetShown(Parent.currentBlock or self:IsShown())
OBJECTIVE_TRACKER_ADDONS[self.Index] = self:IsShown() and self:GetHeight() or 0
end
function Objectives:GetUsedHeight()
local height = Parent.contentsHeight or 0
for i = 1, self.Index-1 do
height = height + OBJECTIVE_TRACKER_ADDONS[i]
end
if height > 0 then
height = height + 15
end
return height
end
I just updated it and have pre shadowlands. i was told in my ticket on my lua errors this was post was my fix
ty for any help
it is also not fixed when click on a Pet-Pin on the Worldmap!
1x ...s\PetTracker\addons\main\classes\abstract\specie.lua:26: attempt to index a nil value
[string "@PetTracker\addons\main\classes\abstract\specie.lua"]:26: in function `GetID'
[string "@PetTracker\addons\main\classes\abstract\pet-Pet.lua"]:33: in function `Display'
[string "@PetTracker\addons\main\classes\maps\speciePin.lua"]:67: in function <...ns\PetTracker\addons\main\classes\maps\speciePin.lua:65>
Locals:
self = <table> {
specie = 389
}
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index a nil value"
but -> "issue closed"...
For anyone still having errors, please
- Make sure you update (even if you have to download it manually and unzip it overtop of your current
PetTracker
folder. - Then reproduce it.
- Then if it persists, open a new ticket.
This will keep things clean for the developer.