8000 LUA error when mouseover a character when some BNet friends from another area are logged in · Issue #3 · tmg-pub/crossrp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

LUA error when mouseover a character when some BNet friends from another area are logged in #3

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.

Sign up for GitHub

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

Jump to bottom
Closed
LenweSaralonde opened this issue Aug 15, 2022 · 1 comment
Closed

LUA error when mouseover a character when some BNet friends from another area are logged in #3

LenweSaralonde opened this issue Aug 15, 2022 · 1 comment

Comments

@LenweSaralonde
Copy link
Contributor

Steps:

  1. Have a BNet account with friends in Area 1 (ie EU)
  2. Create a trial WoW account in another Area 2 (ie US)
  3. Create character on the trial account in Area 2 (ie Moonguard)
  4. Wait approx 20s
  5. Mouse over a character

The following LUA error occurs:
Interface\AddOns\CrossRP\CrossRP.lua:732: attempt to index field 'realmName' (a nil value)

The root cause is the object returned by the C_BattleNet.GetFriendGameAccountInfo( friend, account_index ) function is missing the realmName property.

Since there is no way to interact with players from other areas, we could just ignore those who realmName that is nil.
LenweSaralonde added a commit to LenweSaralonde/crossrp that referenced this issue Aug 15, 2022
@LenweSaralonde
LUA error with BNet friends from another area tmg-pub#3
79c53a3
tmg-pub added a commit that referenced this issue Aug 17, 2022
@tmg-pub
Merge pull request #4 from LenweSaralonde/main
8f2d90e 
Fix for LUA error when mouseover a character when some BNet friends from another area are logged in #3
@tmg-pub
Copy link
Owner
tmg-pub commented Aug 17, 2022

Merged your changes. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@LenweSaralonde @tmg-pub
0