Skip to content

Add BlockType/ItemType getters and setters#12427

Closed
ImTheLion wants to merge 1 commit into
PaperMC:mainfrom
ImTheLion:feature/TypeGetters
Closed

Add BlockType/ItemType getters and setters#12427
ImTheLion wants to merge 1 commit into
PaperMC:mainfrom
ImTheLion:feature/TypeGetters

Conversation

@ImTheLion
Copy link
Copy Markdown

Adds getters/setters in some classes, making BlockType and ItemType API even more useable without converting from/to material.

@ImTheLion ImTheLion requested a review from a team as a code owner April 13, 2025 12:36
@github-project-automation github-project-automation Bot moved this to Awaiting review in Paper PR Queue Apr 13, 2025
@lynxplay
Copy link
Copy Markdown
Contributor

Hi, thank you for the PR and welcome to paper 🥳 !
I am semi certain we are not planning on doing it this way as Item/BlockType will simply replace the existing Material parameters and return types when we retire Material.

I'll check in with the team on that tho.

@electronicboy
Copy link
Copy Markdown
Member

I believe that this kinda thing was intended as a core team sorta task given that we also just wanted to start purging material at the same time and have bytecode translations for it

@lynxplay
Copy link
Copy Markdown
Contributor

Okay, yea that is indeed the plan.
Thank you for the PR but we have different plans for the integration of those type :)

@lynxplay lynxplay closed this Apr 13, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting review to Closed in Paper PR Queue Apr 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

3 participants