Skip to content

fix(mod): 修复 NeoForge 在线玩家统计查询失败 - #17

Merged
zkhssb merged 3 commits into
easybot-team:developfrom
BaizhouziYou:fix/statistics-async-output
Oct 2, 2026
Merged

zkhssb merged 3 commits into
easybot-team:developfrom
BaizhouziYou:fix/statistics-async-output

Conversation

@BaizhouziYou

@BaizhouziYou BaizhouziYou commented Oct 2, 2026 •

Copy link
Copy Markdown

NeoForge 的 Mixin 声明误写为 [[mixin]],导致 ServerStatsCounterAccessor 未注入,查询在线玩家统计时抛出 ClassCastException。

将声明修正为 [[mixins]],恢复在线统计 JSON 导出和 %ez-statistic_jump% 等统计占位符查询。

CI 构建时将 Gradle 下载地址临时切换至官方源,保持 9.7.0 版本和本地镜像配置;该改动与 PR #20 完全一致。

验证

  • :1.21-neoforge:shadowJar 构建通过
  • :26.3-neoforge:compileJava 编译通过
  • NeoForge 21.1.252 / Minecraft 1.21.1 实服:测试客户端登录成功,ServerStatsCounterAccessor 注入及在线统计 JSON 导出正常
  • 将测试玩家跳跃统计设置为 7,%ez-statistic_jump% 查询返回 7
  • git diff --check 通过

关联 issue

easybot-team/easybot-issues#88

@BaizhouziYou
BaizhouziYou marked this pull request as ready for review October 2, 2026 04:30
@BaizhouziYou
BaizhouziYou force-pushed the fix/statistics-async-output branch from 0e908a7 to 85e7c96 Compare October 2, 2026 04:51

@Ftimever Ftimever 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.

不赞同此类异步收集方法,时间没法界定,有的长有的短,体验都不好

- 将 NeoForge Mixin 声明修正为 [[mixins]]
- 恢复 ServerStatsCounterAccessor 注入,避免在线统计查询抛出 ClassCastException
@BaizhouziYou
BaizhouziYou force-pushed the fix/statistics-async-output branch from 85e7c96 to 9048e6a Compare October 2, 2026 13:16
@BaizhouziYou BaizhouziYou changed the title fix(mod): 修复 NeoForge 在线统计查询及异步命令输出丢失 fix(mod): 修复 NeoForge 在线玩家统计查询失败 Oct 2, 2026
@BaizhouziYou

Copy link
Copy Markdown
Author

不赞同此类异步收集方法,时间没法界定,有的长有的短,体验都不好

回退掉了,等后续主程序修吧

@BaizhouziYou
BaizhouziYou requested a review from Ftimever October 2, 2026 13:17
@zkhssb
zkhssb merged commit 8ebd104 into easybot-team:develop Oct 2, 2026
1 check passed
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