Skip to content

fix: correct BlockEntityVersion range for MC 26.2 to fix dependency resolution#263

Open
mjiangmc wants to merge 1 commit into
Elytrium:masterfrom
mjiangmc:fix/block-entity-range-26.2
Open

fix: correct BlockEntityVersion range for MC 26.2 to fix dependency resolution#263
mjiangmc wants to merge 1 commit into
Elytrium:masterfrom
mjiangmc:fix/block-entity-range-26.2

Conversation

@mjiangmc

Copy link
Copy Markdown

Fixed an issue where downstream plugins (such as LimboAuth) fail to depend on LimboAPI when running on Minecraft 26.2.

This was caused by the BlockEntityVersion range being outdated for 26.2. This PR updates the version range to ensure that dependency resolution works correctly for dependent plugins on this Minecraft version.

…esolution

Fixed an issue where downstream plugins (such as `LimboAuth`) fail to depend on LimboAPI when running on Minecraft 26.2. 

This was caused by the `BlockEntityVersion` range being outdated for 26.2. This PR updates the version range to ensure that dependency resolution works correctly for dependent plugins on this Minecraft version.

@UserNugget UserNugget left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Absolutely forget about BlockEntity implementation, thanks for PR.

MINECRAFT_1_21_9(EnumSet.of(ProtocolVersion.MINECRAFT_1_21_9)),
MINECRAFT_1_21_11(EnumSet.of(ProtocolVersion.MINECRAFT_1_21_11)),
MINECRAFT_26_1(EnumSet.of(ProtocolVersion.MINECRAFT_26_1));
MINECRAFT_26_1(EnumSet.range(ProtocolVersion.MINECRAFT_26_1, ProtocolVersion.MAXIMUM_VERSION));

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
MINECRAFT_26_1(EnumSet.range(ProtocolVersion.MINECRAFT_26_1, ProtocolVersion.MAXIMUM_VERSION));
MINECRAFT_26_1(EnumSet.of(ProtocolVersion.MINECRAFT_26_1)),
MINECRAFT_26_2(EnumSet.range(ProtocolVersion.MINECRAFT_26_2, ProtocolVersion.MAXIMUM_VERSION));

26.2 removed bed from block entities, so we need to add it as separate version.

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.

2 participants