Skip to content

Fix versioncheck - #66

Open
HotWaterFlask wants to merge 4 commits into
yuttyann:masterfrom
HotWaterFlask:fix-versioncheck
Open

Fix versioncheck#66
HotWaterFlask wants to merge 4 commits into
yuttyann:masterfrom
HotWaterFlask:fix-versioncheck

Conversation

@HotWaterFlask

Copy link
Copy Markdown

解决McVersion 静态初始化时调用 Version.of(...) 解析当前 MC 版本字符串,Integer.parseInt("") 抛异常,导致整个类初始化失败(ExceptionInInitializerError)。也就是说它解析 "26.2" 这种新版号格式时取到了空字符串(老插件通常按 1.x.x 三段式硬解析,26.2 只有两段,某段下标越界取到空的问题。

【McVersion 静的初期化時に Version.of(...) を呼び出して現在の MC バージョン文字列を解析する際、Integer.parseInt("") が例外を投げ、クラス全体の初期化が失敗する(ExceptionInInitializerError)。つまり、"26.2" のような新しいバージョン番号形式を解析する際に空文字列を取得してしまう(古いプラグインは通常 1.x.x の3セグメント形式でハードコード解析するが、26.2 は2セグメントしかないため、あるセグメントの添字が範囲外になり空文字を取得する)】問題が解決された。

Minecraft 26.2 version support
修复内容(3 个文件)
NetMinecraft.java#L305-L306 — 新增枚举:
Java



/** {@code net.minecraft.world.entity.monster.cubemob} */
WORLD_ENTITY_MONSTER_CUBEMOB(WORLD_ENTITY_MONSTER, "cubemob"),
PaperRemappedAccessor_v1_21_6.java#L66-L77 — 新增 getMagmaCubeClass() 辅助方法,优先尝试旧包,失败后 fallback 到 cubemob 新包;两处 WORLD_ENTITY_MONSTER.getClass("MagmaCube") 替换为 getMagmaCubeClass()
PaperRemappedAccessorLegacy.java#L65-L76 — 同样的 fallback 处理
兼容性说明
GlowEntity(ScriptViewer 发光可视化)使用 Bukkit API 的 org.bukkit.entity.MagmaCube,该接口在 26.2 中依然存在(setSize 从 Slime 移到 AbstractCubeMob),无需修改
旧版本(≤1.21.x)会命中 WORLD_ENTITY_MONSTER 路径,新版本(26.2+)命中 cubemob 路径,两者兼容
…t.resources.Identifier(MC 1.21.11 起的全局改名),导致方案 2 反射失败。

更新 PaperRemappedAccessor_v1_21_6.java#L350-L365 和 PaperRemappedAccessorLegacy.java#L349-L364:

Java



// 先尝试 Identifier(1.21.11+ / 26.x)
identifierClass = Utils.getClassForName("net.minecraft.resources.Identifier");
// 失败再回退 ResourceLocation(1.21.10 及更早)
现在的完整 fallback 链
EntityType.MAGMA_CUBE 静态字段(≤1.21.10)
BuiltInRegistries.ENTITY_TYPE.getValue(Identifier.parse("minecraft:magma_cube"))(26.x,Identifier)
BuiltInRegistries.ENTITY_TYPE.getValue(ResourceLocation.parse(...))(1.21.11 前兜底)
CraftEntityType.bukkitToMinecraft(EntityType.MAGMA_CUBE)(CraftBukkit 桥接兜底)
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.

1 participant