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

Add warning that xPlayer must be checked for nil value before indexing #109

Closed
pablordgez opened this issue Apr 22, 2024 · 3 comments
Closed
Assignees

Comments

@pablordgez
Copy link

I've had to fix so many scripts that fail because they try to iterate xPlayers and try to get some information about a player without checking whether it's nil. A warning somewhere in the documentation would be nice

@Arctos2win
Copy link
Contributor

so many scripts that fail because they try to iterate xPlayers and try to get some information a

What function are you talking about ?

@pablordgez
Copy link
Author

Any function that iterates through xPlayers without checking whether xPlayer is nil is prone to fail, and I've seen this countless times, there should be a warning somewhere in the documentation that this must be checked

@Gellipapa Gellipapa moved this from 🆕 New to docs in ESX Roadmap May 3, 2024
@NikartikTheBear NikartikTheBear self-assigned this May 20, 2024
@Gellipapa
Copy link
Contributor

Gellipapa commented May 20, 2024

Hi! @pablordgez Will be included in the docs, thanks for your suggestion.

@github-project-automation github-project-automation bot moved this from docs to ✅ Done in ESX Roadmap May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

4 participants