-
Notifications
You must be signed in to change notification settings - Fork 363
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
feature: Access to Source of Dropped/Spawned Items for Server Plugins #2390
feature: Access to Source of Dropped/Spawned Items for Server Plugins #2390
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you think Projectile and EventPageInstance's methods should return null, I'm ok with that but you need to change the method signature in all of the classes from EntityItemSource
to EntityItemSource?
so that callers know if the value is nullable or not
Leaving this comment as a note to confirm that I've been testing the item source implementation in a test plugin, and everything has been working as expected. |
This PR will allow access to the source of an item that is dropped/spawned on the map. The main goal is to expose this logic to the plugin system, specifically in the context of the server. It’s still a work in progress, but I would like to keep updating the draft in case they have any suggestions for the changes.