Skip to content

Registry API for supported Mob Variants#12364

Closed
BertTowne wants to merge 4 commits into
PaperMC:update/1.21.5from
BertTowne:update/1.21.5
Closed

Registry API for supported Mob Variants#12364
BertTowne wants to merge 4 commits into
PaperMC:update/1.21.5from
BertTowne:update/1.21.5

Conversation

@BertTowne
Copy link
Copy Markdown
Contributor

#12350 Currently only implemented Chicken variants as a way to gather feedback before adding the rest. I'm especially wanting opinions on the best way to implement SpawnConditions to the API, as those have intentionally been left out of this, but right now custom mob variants are given SpawnPrioritySelectors.EMPTY.

ModelAndTexture has been separated into Model and Texture for API purposes, as the vanilla-defined models are not entirely the same across each mob and are unfortunately stored as enums. assetId is required, and model will default to NORMAL.

@Lulu13022002
Copy link
Copy Markdown
Contributor

Changes between generated comments will be lost. Registration is now done in RegistryEntries in the generator module. I have some plan later to move that over a json file but for now that should do it. The registry don't need to be delayed too.

@BertTowne
Copy link
Copy Markdown
Contributor Author

Thank you, I have made the proposed changes and went forward with implementing the rest of the data-driven mob variants in a similar way to that of the Chicken Variants from before.

@kennytv kennytv deleted the branch PaperMC:update/1.21.5 April 12, 2025 15:30
@kennytv kennytv closed this Apr 12, 2025
@github-project-automation github-project-automation Bot moved this from Awaiting review to Closed in Paper PR Queue Apr 12, 2025
@kennytv
Copy link
Copy Markdown
Member

kennytv commented Apr 12, 2025

Oop, please reopen a PR targeting main again

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: api type: feature Request for a new Feature.

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

4 participants