Auctionator / Journalator

World of Warcraft addon. Keep track of your gold making on the AH, crafting orders and with vendors
https://www.curseforge.com/wow/addons/journalator
MIT License
10 stars 6 forks source link

Journalator hangs on opening #61

Closed sjtyejopemlzn closed 1 year ago

sjtyejopemlzn commented 1 year ago

Brief Description of Bug

Slash command '/jnr' produces no response. Clicked on the Journalator minimap icon to access add-on. Journalator hangs on opening, no functionality available. Progress bar is shown, but no progress occurs.

Steps to Reproduce

Click Journalator minimap icon.

Loaded Addons

Auctionator 10.1.7 Journalator 1.43 BugGrabber 10.1.0 BugSack 10.1.2

Back Trace (LUA Error Output)

1x ...aceJournalator/Source/Archiving/GetRange.lua:11: bad argument #1 to 'ipairs' (table expected, got nil) [string "=[C]"]: in function ipairs' [string "@Journalator/Source/Archiving/GetRange.lua"]:11: in functionGetRange' [string "@Journalator_Display/Source/GetCharactersAndRealms.lua"]:15: in function GetCharactersAndRealms' [string "@Journalator_Display/Source/Display/FiltersContainer.lua"]:49: in functionUpdateRealmsAndCharacters' [string "@Journalator_Display/Source/Display/FiltersContainer.lua"]:40: in function <...rnalator_Display/Source/Display/FiltersContainer.lua:39> [string "=[C]"]: in function SetShown' [string "@Journalator_Display/Source/Initialize.lua"]:9: in functionToggleView' [string "@Journalator_Display/Source/MinimapIcon.lua"]:39: in function `OnClick' [string "@BugSack/Libs/LibDBIcon-1.0-51/LibDBIcon-1.0.lua"]:170: in function <BugSack/Libs/LibDBIcon-1.0/LibDBIcon-1.0.lua:168>

Locals: (temporary) = nil (temporary) = "table expected, got nil" = defined =[C]:-1

6x ...aceJournalator/Source/Archiving/LoadUpTo.lua:64: attempt to compare nil with number [string "@Journalator/Source/Archiving/LoadUpTo.lua"]:64: in function IsLoadedUpTo' [string "@Journalator_Display/Source/MinimapIcon.lua"]:46: in functionOnTooltipShow' [string "@BugSack/Libs/LibDBIcon-1.0-51/LibDBIcon-1.0.lua"]:73: in function <BugSack/Libs/LibDBIcon-1.0/LibDBIcon-1.0.lua:59>

Locals: minTime = 1685631599 (temporary) = nil (temporary) = "attempt to compare nil with number" frame = nil

Screenshot (79)

plusmouse commented 1 year ago

Can you attach your Journalator saved variables file (found at Wow directory\_retail_\WTF\some numbers\SavedVariables\Journalator.lua)?

sjtyejopemlzn commented 1 year ago

I don't know if it makes a difference, but I found the saved variable file in this path, instead of the one you suggested:

World of Warcraft_retail_\WTF\Account\some numbers\SavedVariables\ Journalator.lua

On Mon, Jun 12, 2023 at 12:23 PM Dan Reno @.***> wrote:

On Sun, Jun 11, 2023 at 10:57 PM plusmouse @.***> wrote:

Can you attach your Journalator saved variables file (found at Wow directory_retail_\WTF\some numbers\SavedVariables\Journalator.lua)?

— Reply to this email directly, view it on GitHub https://github.com/Auctionator/Journalator/issues/61#issuecomment-1586534753, or unsubscribe https://github.com/notifications/unsubscribe-auth/BAPXQPJQOPNLR75YVKBEUODXK2HT5ANCNFSM6AAAAAAZCU3QXM . You are receiving this because you authored the thread.Message ID: @.***>

plusmouse commented 1 year ago

That location is fine, I mistyped the location, forgot the Account

plusmouse commented 1 year ago

Any updates on supplying the saved variables file?

Can you also try disabling all other add-ons (other than Journalator and Auctionator) and see if it's still broken?

sjtyejopemlzn commented 1 year ago

I attached the lua file via email reply on Mon 12 Jun:

[image: 1f749b09-01d1-4319-9308-02c1a17e2a7c.png]

I'll try attaching the file to this message. If it doesn't go through, I can try resending, or use the github platform to send it to you.

I disabled all addons other than Journalator 1.43 and Auctionator 10.1.7, and the errors still persist. I upgraded to Journalator 1.44, and I am also getting the errors.

To refine what I'm seeing on my end - I am able to click on the various tabs of Journalator, but I'm not seeing any information, just a static 'progress' bar in the middle of the screen:

[image: Screenshot (84).png] [image: Screenshot (85).png] [image: Screenshot (86).png]

Thank you.

On Wed, Jun 14, 2023 at 3:10 PM plusmouse @.***> wrote:

Any updates on supplying the saved variables file?

Can you also try disabling all other add-ons (other than Journalator and Auctionator) and see if it's still broken?

— Reply to this email directly, view it on GitHub https://github.com/Auctionator/Journalator/issues/61#issuecomment-1591904950, or unsubscribe https://github.com/notifications/unsubscribe-auth/BAPXQPIMFARZRMNDYK5CDT3XLILD5ANCNFSM6AAAAAAZCU3QXM . You are receiving this because you authored the thread.Message ID: @.***>

plusmouse commented 1 year ago

The email replies don't seem to include any attachments or screenshots. Can you try adding the attachments and screenshots to using the Github interface for the issue?

Can you do the following chat commands and report the output for each one? One of these should output a different value to what I'd expect.

/dump JOURNALATOR_LOGS
/dump JOURNALATOR_ARCHIVE_TIMES
/dump Journalator.State.MinTimeLoaded
/dump time()
sjtyejopemlzn commented 1 year ago

Okay, here are the screenshots I've been trying to send to you:

Screenshot (84) Screenshot (85) Screenshot (86)

Saved variable file:

Journalator.txt

Four /dump commands:

JOURNALATOR_LOGS.txt JOURNALATOR_ARCHIVE_TIMES.txt Journalator.State.MinTimeLoaded.txt time().txt

plusmouse commented 1 year ago

Thanks for the files and screenshots I've found the cause of the problem and fixed it. You can now update to the newest alpha build and it will open correctly Journalator-1.44-6-g1967373.zip

plusmouse commented 1 year ago

The issue came from the data being old and the compatibility code having broken since it was written in 2021, I've fixed the compatibility code now.

sjtyejopemlzn commented 1 year ago

Thank you for your quick replies and quick fix. Thanks also for your patience with sorting out the issue with the attachments.

On Sat, Jun 17, 2023 at 1:55 PM plusmouse @.***> wrote:

The issue came from the data being old and the compatibility code having broken since it was written in 2021, I've fixed the compatibility code now.

— Reply to this email directly, view it on GitHub https://github.com/Auctionator/Journalator/issues/61#issuecomment-1595830827, or unsubscribe https://github.com/notifications/unsubscribe-auth/BAPXQPPUZSIOKDNXZEKP35TXLX4TPANCNFSM6AAAAAAZCU3QXM . You are receiving this because you authored the thread.Message ID: @.***>