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

Adding things to Player.cs #233

Open
wants to merge 2 commits into
base: 13.1
Choose a base branch
from

Conversation

SrLicht
Copy link
Contributor

@SrLicht SrLicht commented Oct 23, 2023

Fixed an issue with "Collection was modified" error when using Player.Items in a foreach loop and removing an object. Now, I use ToList() to create a new temporary list.

  • Moved methods from the extensions of my private plugin to the Player.cs API so that everyone can use them.

  • IsInventoryFul now uses Inventory.MaxSlot in case the number of inventory slots for the player changes at some point.

  • Fixed some typos.

Fixed an issue with "Collection was modified" error when using Player.Items in a foreach loop and removing an object. Now, I use `ToList()` to create a new temporary list.

* Moved methods from the extensions of my private plugin to the Player.cs API so that everyone can use them.

* `IsInventoryFul`l now uses `Inventory.MaxSlot` in case the number of inventory slots for the player changes at some point.

* Fixed some typos.
@SrLicht
Copy link
Contributor Author

SrLicht commented Nov 4, 2023

yes i use Github for resolving that.

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.

1 participant