Todoloo-WoW / Todoloo

The Todoloo task manager addon for World of Warcraft
MIT License
1 stars 0 forks source link

Loading order issue result in null pointer (TaskManager) #54

Open Nils89 opened 2 weeks ago

Nils89 commented 2 weeks ago

Version
11.0.5.1

Describe the bug
5x ...s/Todoloo/Source/Display/TaskTracker/TaskTracker.lua:50: attempt to index field 'TaskManager' (a nil value) [string "@Todoloo/Source/Display/TaskTracker/TaskTracker.lua"]:50: in function BuildGroupInfos' [string "@Todoloo/Source/Display/TaskTracker/TaskTracker.lua"]:70: in functionEnumGroupData' [string "@Todoloo/Source/Display/TaskTracker/TaskTracker.lua"]:233: in function Update' [string "@Todoloo/Source/Display/TaskTracker/FloatingTracker.lua"]:837: in functionTodolooTracker_Update' [string "@Todoloo/Source/Display/TaskTracker/FloatingTracker.lua"]:695: in function <...doloo/Source/Display/TaskTracker/FloatingTracker.lua:689>

To Reproduce
Just login on for me

I check the code and the Part where the Task manager get init is loaded ofter the first use and that result in this null pointer

detilium commented 2 weeks ago

Hey.

Thank you for the bug report.

Does this happen every time you login?

Nils89 commented 2 weeks ago

Yes happens everytime. Maybe could also be a Mac client issue which was introduced with 11.0.5

detilium commented 2 weeks ago

Oh that may very well be. I haven't tested on Mac and I can't seem to replicate the issue on a Windows.

Nils89 commented 2 weeks ago

Have just added the Task Manager Init above the error and and that solves the issue. Only if the Task Manager is Nil

detilium commented 2 weeks ago

You're welcome to make a PR if you've managed resolved the issue :)

Nils89 commented 2 weeks ago

Sure, will do that next week :)

detilium commented 2 weeks ago

I'm about to release a new version in which the Todoloo table gets fully exposed for use in other addons.

If you can tell me exactly what you did and on what lines, I'd be able to include it in the release?