Skip to content

Update to 1.21.11#40

Merged
enjarai merged 9 commits into
enjarai:1.21.11/devfrom
cputnam-a11y:1.21.11
Jan 4, 2026
Merged

Update to 1.21.11#40
enjarai merged 9 commits into
enjarai:1.21.11/devfrom
cputnam-a11y:1.21.11

Conversation

@cputnam-a11y
Copy link
Copy Markdown

No description provided.

@cputnam-a11y cputnam-a11y changed the base branch from 1.20.5/dev to 1.21/dev January 1, 2026 23:30
public class ModelProvider$BlockStateSuppliers {
@ModifyExpressionValue(method = "validate", at = @At(value = "INVOKE", target = "Ljava/util/stream/Stream;filter(Ljava/util/function/Predicate;)Ljava/util/stream/Stream;"))
private static Stream<RegistryEntry.Reference<Block>> patchOpenBlockOutOfValidation(Stream<RegistryEntry.Reference<Block>> original) {
return original.filter(it -> !ModBlocks.OPEN_BOX_BLOCK.equals(it.value()));
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

What is this for?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Validation doesn't pass if all blocks don't have a registered model, so I had to exclude this one for now

Comment thread src/main/java/nl/enjarai/omnihopper/blocks/hopper/HopperBlock.java
@enjarai
Copy link
Copy Markdown
Owner

enjarai commented Jan 2, 2026

It seems like something changed with the blockstate model spec, cause some orientations of hopper are no longer rendering correctly
image

@cputnam-a11y
Copy link
Copy Markdown
Author

Hmm. They seemed to work in testing but I'll test more extensively

@cputnam-a11y
Copy link
Copy Markdown
Author

cputnam-a11y commented Jan 2, 2026

okay, resolved the orientation bug

@cputnam-a11y cputnam-a11y requested a review from enjarai January 2, 2026 15:52
@enjarai enjarai changed the base branch from 1.21/dev to 1.21.11/dev January 4, 2026 09:50
@enjarai enjarai merged commit a24596f into enjarai:1.21.11/dev Jan 4, 2026
0 of 2 checks passed
@vico93 vico93 mentioned this pull request Jan 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants