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

fix(server/main): Cannot find account #57

Closed
wants to merge 1 commit into from
Closed

fix(server/main): Cannot find account #57

wants to merge 1 commit into from

Conversation

SamShanks1
Copy link

GetPlayerObject requires source and is currently getting ID

@uShifty
Copy link
Collaborator

uShifty commented Aug 15, 2023

Hey, we have a update in the works currently so pull requests will not be merged as it is a LUA rewrite with UI improvements.

Also this is not a fix, GetPlayerObject has 2 parameters a source which will accept the initiators source. This is only used to notify the player that the player they are looking for cannot be found if that is the case. Their also is the id parameter which can be a player source or a player identifier. which is why it will then call GetPlayerObjectFromID if Player is not defined.

The changes will always return the initiators player object which is not intended.

@uShifty uShifty closed this Aug 15, 2023
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

Successfully merging this pull request may close these issues.

2 participants