Skip to content

Change some registries to use resource locations, and move some static maps to be registries#4936

Open
gustovafing wants to merge 9 commits into
1.20.1from
gus/registration-changes
Open

Change some registries to use resource locations, and move some static maps to be registries#4936
gustovafing wants to merge 9 commits into
1.20.1from
gus/registration-changes

Conversation

@gustovafing

Copy link
Copy Markdown
Member

What

  • Switch tag prefixes, material icon sets and placeholders to be stored in registries.

  • Convert element registry to a resource location registry

  • Methods for getting/registering content which is now stored in a registry have been marked as deprecated.

  • Methods for registering using string keys will default to GT namespace

AI Usage

  • No AI driven tools were used for this pull request.

Potential Compatibility Issues

  • All existing code should work fine, as all existing IDs which use strings will default to the gt namespace.

@gustovafing gustovafing requested a review from a team as a code owner June 16, 2026 14:30
@github-actions github-actions Bot added 1.20.1 Tests: Passed Game Tests have passed on this PR labels Jun 16, 2026
@jurrejelle

Copy link
Copy Markdown
Contributor

I imagine this will be breaking to some packs, maybe add a line or two about it to v8.0.0.md?

@jurrejelle

Copy link
Copy Markdown
Contributor

I imagine this will be breaking to some packs, maybe add a line or two about it to v8.0.0.md?

oh you updated the methods to be the same 🤔 I think this should be fine in 90% of cases? thre are some packs that might still be referencing it directly but I'll leave it up to you if you wanna add docs for it

@gustovafing

Copy link
Copy Markdown
Member Author

All the api methods are same, I don't think we need a note about the removal of the static maps, etc since they aren't supposed to be accessed externally to begin with.

@screret screret left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are material icon types intentionally left as is?

Comment thread src/main/java/com/gregtechceu/gtceu/api/addon/IGTAddon.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/client/renderer/block/OreBlockRenderer.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/common/data/GTElements.java
Comment thread src/main/java/com/gregtechceu/gtceu/integration/kjs/GregTechKubeJSPlugin.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/integration/kjs/GTRegistryInfo.java Outdated
Comment thread src/main/java/com/gregtechceu/gtceu/integration/kjs/GTRegistryInfo.java Outdated
@gustovafing

Copy link
Copy Markdown
Member Author

Material icon types are weird, they don't actually store anything it's just a list of strings.

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

Labels

1.20.1 Tests: Passed Game Tests have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants