Skip to content

Add ItemStack getter to FurnaceExactEvent#13569

Merged
Warriorrrr merged 6 commits intoPaperMC:mainfrom
notsneakybeaky:fix/furnace-itemstack
Feb 26, 2026
Merged

Add ItemStack getter to FurnaceExactEvent#13569
Warriorrrr merged 6 commits intoPaperMC:mainfrom
notsneakybeaky:fix/furnace-itemstack

Conversation

@notsneakybeaky
Copy link
Contributor

@notsneakybeaky notsneakybeaky commented Jan 19, 2026

Converting Material to ItemStack in FurnaceExtractEvent for #13547

@notsneakybeaky notsneakybeaky requested a review from a team as a code owner January 19, 2026 21:47
@github-project-automation github-project-automation bot moved this to Awaiting review in Paper PR Queue Jan 19, 2026
@Lulu13022002 Lulu13022002 linked an issue Jan 19, 2026 that may be closed by this pull request
@mbax
Copy link
Contributor

mbax commented Jan 19, 2026

This doesn't appear to actually provide the itemstack, though?

@notsneakybeaky
Copy link
Contributor Author

as in having a getter method? I can add that

@Privatech38
Copy link
Contributor

Why doesn't this deprecate the getItemType()?

@notsneakybeaky
Copy link
Contributor Author

Why doesn't this deprecate the getItemType()?

It doesn't use the deprecated Material API

@Warriorrrr Warriorrrr changed the title Fix/furnace itemstack Add ItemStack getter to FurnaceExactEvent Feb 4, 2026
@Warriorrrr
Copy link
Member

Updated it to clone the item when accessed, don't think it's necessary to mention that it's a clone

@Warriorrrr Warriorrrr merged commit 7e80cef into PaperMC:main Feb 26, 2026
4 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting review to Merged in Paper PR Queue Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

ItemStack reference in FurnaceExtractEvent

6 participants