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

Expand PlayerItemMendEvent #7382

Merged

Conversation

Machine-Maker
Copy link
Member

@Machine-Maker Machine-Maker commented Jan 21, 2022

Resolves #7313
Resolves #7449
Closes #7815

Adds a getter/setter for the operation used to turn the amount of durability repaired, into an XP value to be subtracted away from the total value of the experience orb.

@Machine-Maker Machine-Maker requested a review from a team as a code owner January 21, 2022 02:13
@Machine-Maker
Copy link
Member Author

Rebased for 1.18.2

Now resolves 2 open issues.

@stale
Copy link

stale bot commented May 31, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@rafaelsms
Copy link

Now resolves 2 open issues.

Now resolves 3 open issues.

@stale stale bot removed the resolution: stale label Jun 3, 2022
Copy link
Member

@kennytv kennytv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd write out "Operation" in the getter and setter, even if it does make it pretty long, but otherwise fine

patches/server/0884-Expand-PlayerItemMendEvent.patch Outdated Show resolved Hide resolved
@Machine-Maker Machine-Maker force-pushed the feature/expand-PlayerItemMendEvent branch 2 times, most recently from 5cd7615 to 97a78c4 Compare June 3, 2022 18:51
@Machine-Maker Machine-Maker linked an issue Jun 3, 2022 that may be closed by this pull request
@Machine-Maker
Copy link
Member Author

Ok, tested and this fixes the recursion issue, and the new API works as expected. I'll wait for a 2nd approval since this is new API and not a simple fix.

@oddlama
Copy link

oddlama commented Aug 23, 2022

Any updates on this PR? It seems to be fully implemented and functional already - is there anything that is blocking this from moving forward?

@Machine-Maker Machine-Maker force-pushed the feature/expand-PlayerItemMendEvent branch from 97a78c4 to 49a0fbc Compare March 20, 2023 08:23
@Machine-Maker Machine-Maker force-pushed the feature/expand-PlayerItemMendEvent branch 2 times, most recently from 39ee9e6 to a2e44e2 Compare May 31, 2023 16:18
@Machine-Maker Machine-Maker force-pushed the feature/expand-PlayerItemMendEvent branch from a2e44e2 to 2da9441 Compare May 31, 2023 23:53
@Machine-Maker Machine-Maker merged commit 2eda177 into PaperMC:master May 31, 2023
@Machine-Maker Machine-Maker deleted the feature/expand-PlayerItemMendEvent branch May 31, 2023 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
6 participants