Skip to content

Fix Inventory#addItem javadoc mismatch with behaviour#11782

Merged
lynxplay merged 2 commits into
PaperMC:mainfrom
masmc05:fix-add-item
Dec 23, 2024
Merged

Fix Inventory#addItem javadoc mismatch with behaviour#11782
lynxplay merged 2 commits into
PaperMC:mainfrom
masmc05:fix-add-item

Conversation

@masmc05
Copy link
Copy Markdown
Contributor

@masmc05 masmc05 commented Dec 23, 2024

Spigot recently changed the behavior of Inventory#addItem by not allowing making overstacked items in inventory with it, but didn't update the javadocs, making it inconsistent with the implementation

@masmc05 masmc05 requested a review from a team as a code owner December 23, 2024 10:40
Comment thread paper-api/src/main/java/org/bukkit/inventory/Inventory.java Outdated
@kennytv kennytv added the type: bug Something doesn't work as it was intended to. label Dec 23, 2024
@lynxplay lynxplay merged commit 62d4130 into PaperMC:main Dec 23, 2024
@masmc05 masmc05 deleted the fix-add-item branch December 23, 2024 19:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: bug Something doesn't work as it was intended to.

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

4 participants