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

QueryUserInfo bugs #30

Open
MikhailSorokin opened this issue Jul 9, 2020 · 2 comments
Open

QueryUserInfo bugs #30

MikhailSorokin opened this issue Jul 9, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@MikhailSorokin
Copy link
Contributor

Environment

Project Version: v0.3
EOS SDK Version: v1.7
Unreal Engine Version: 4.24.3

Other software/plugins that *might* be the cause of the issue?

N/A

Steps to reproduce

If you do a simple query command for a local player OnlineUserInterface::QueryUserInfo in the master branch, nothing happens and attempting to get the name shortly after fails since it relies on that function to work.

I noticed this issue when trying to design for the friends interface myself because I noticed the number of desired cases was never reaching the number of completed queries.

Expected behaviour

What I am expecting is the number of queries inputted - which you could have many passed in - to reach a desired number of completed queries and have that be a non-blocking call with other queries that may be happening at different timestamps

Actual behaviour

The number of completed queries is 0, the TriggerOnQueryUserInfoComplete delegate never gets triggered and no subsequent logs in QueryUserInfo get triggered. I tested this using a derived local player class and created a QueryUserInfoComplete callback. OnQueryUserInfoCompleteDelegateHandle = OnlineUserInterface->AddOnQueryUserInfoCompleteDelegate_Handle(GetControllerId(), FOnQueryUserInfoCompleteDelegate::CreateUObject(this, &ThisClass::HandleQueryUserInfoComplete)); which would be called on the login callback that I made. Also, the call to get the unique net id fails.
I am going to make a PR shortly that fixes this issue entirely.

@MikhailSorokin MikhailSorokin added the bug Something isn't working label Jul 9, 2020
@MikhailSorokin MikhailSorokin mentioned this issue Jul 9, 2020
7 tasks
@MikhailSorokin
Copy link
Contributor Author

Closed with PR #31

@MikhailSorokin
Copy link
Contributor Author

MikhailSorokin commented Jul 20, 2020

Would like to reopen this as there are some bugs I caused from an earlier PR, but those of which I fixed in the UserInterface class in a new branch I plan to soon merge in as a feature: #32 with regards to further information needed to remove the right query index.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant