Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error when hovering over people in LFG tool #260

Open
FabianKool opened this issue Nov 19, 2023 · 10 comments
Open

Error when hovering over people in LFG tool #260

FabianKool opened this issue Nov 19, 2023 · 10 comments

Comments

@FabianKool
Copy link

Hi! Since the new Season, I keep getting very often Lua errors, when I try to hover over someone to see their rio stats. If this happens, a reload won’t help either and it litterly happens for everyone signing via the LFG tool. I get the following Lua message:

Message: Interface/FrameXML/LFGList.lua:1877: bad argument #1 to 'format' (string expected, got nil)
Time: Sun Nov 19 22:15:09 2023
Count: 1
Stack: Interface/FrameXML/LFGList.lua:1877: bad argument #1 to 'format' (string expected, got nil)
[string "=[C]"]: in function `format'
[string "@Interface/FrameXML/LFGList.lua"]:1877: in function <Interface/FrameXML/LFGList.lua:1851>

Locals: (*temporary) = "%s %s"
(*temporary) = nil
(*temporary) = "Mage"
(*temporary) = "string expected, got nil"

@hollo6
Copy link

hollo6 commented Nov 24, 2023

This is happening every time when you hover over someone who applied as part of a group.
Tested with all other addons disabled.

@WFrancois
Copy link
Member

Hey!

From what I tried, I get this issue with every addon disabled (including RaiderIO), so I don't think it's related to us? :/

@hollo6
Copy link

hollo6 commented Nov 26, 2023

Hey!

From what I tried, I get this issue with every addon disabled (including RaiderIO), so I don't think it's related to us? :/

I have tried and if this addon is disabled, I don't get the error.

It comes from this call:
CLUB_FINDER_LOOKING_FOR_CLASS_SPEC:format(PlayerUtil.GetSpecNameBySpecID(specID), classSpecializationName);

And I guess it's because they changed C_LFGList.GetApplicantMemberInfo to return one more variable.

From what I understood from a brief look at the latest commit, this might be a bug but even if it is, god knows when they intend to deploy the fix and I am sure there is a workaround that this addon can do in the meanitme.

@WFrancois
Copy link
Member

WFrancois commented Nov 26, 2023

Is it the same error as in the original post?

Make sure to show Lua error with /console scriptErrors 1

@hollo6
Copy link

hollo6 commented Nov 26, 2023

Is it the same error as in the original post?

Make sure to show Lua error with /console scriptErrors 1

It is 100% the same every time. Every temporary variable, even the Mage.

@FabianKool
Copy link
Author

Yeaa, I really hope this addon can find a temp fix, cause blizz aint gonna fix if for a long time

@WFrancois
Copy link
Member

WFrancois commented Nov 26, 2023

@FabianKool I wonder, do you get the error when RaiderIO is disabled? Or is it like hollo6?
Make sure to run /console scriptErrors 1 as well just in case

@WFrancois
Copy link
Member

Ok, so a sum up of our search (thanks @Vladinator for the help!)

Here's the issue we get:
image

We also get this when hovering on an Evoker:
image

This is related to around this line of code: https://github.com/Gethe/wow-ui-source/blob/live/Interface/FrameXML/LFGList.lua#L1877 (blizzard code here)

To debug some more, we ran:

/run T={} local r,k hooksecurefunc(C_LFGList,"GetApplicantMemberInfo",function(i,j)if r then return end r=1 k=format("%d_%d",i,j)T[k]={C_LFGList.GetApplicantMemberInfo(i,j)}r=nil end)

Which allowed us to see in detail the variables returned by C_LFGList.GetApplicantMemberInfo

Here's what we see for a Paladin:
image

Numbers 14 is factionId, 15 classId and 16 specId, and it returns 0.

For the Evoker, we see this:
image

And we can see that the specId is 3201340112.


All this to say, I don't think RaiderIO trigger the issue, and I'm not sure how we could fix it?

@WFrancois
Copy link
Member

This is this issue that was created on the WoWUIBugs repo: Stanzilla/WoWUIBugs#502

@StFreve
Copy link

StFreve commented Nov 27, 2023

Created WeakAura with temporary fix if anyone wants: Fix Error LFGList.lua:1877. It makes GetSpecNameBySpecID to return empty string instead of nil when no real specID is passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants