search

Minecraft-AMS / Carpet-AMS-Addition

A fabric-carpet extension mod. It offers some interesting and useful features.
https://minecraft-ams.github.io/carpetamsaddition/
GNU Lesser General Public License v3.0
57 stars 7 forks source link

最大方块交互距离更改时交互了两次 #46

Closed cdqtzrc closed 10 months ago

cdqtzrc commented 10 months ago

Bug description

最大方块交互距离启用时，拿着刷怪蛋点一下地面，生成了两只生物

Steps to reproduce

1.拿一个刷怪蛋 2.将最大方块交互距离改成一个较大值（最好大于6） 3.拿刷怪蛋点一下近处的地面 或 1.找到一个活板门方块 2.点击一下活板门

Expected behavior

生成一只生物 或 活板门打开

Actual behavior

生成了两只生物 或 活板门打开后立即关闭

Relevant logs

No response

Minecraft version

1.20.1

TemplateMod version

Carpet-AMS-Addition v2.14.1

Other information

解决方案：

屏幕截图 2023-11-25 194310

interactBlock()方法调用了两次，一次是距离小于64.0时正常调用的，另一次是距离小于AmsServerSettings.maxBlockInteractionDistance时调用的

Check list

1024-byteeeee commented 10 months ago

十分感谢，v2.14.1更新之后服里有人发现跟我说了然后更了v2.14.2🤣，不过我是这样解决的： 1（1）

cdqtzrc commented 10 months ago

但是还有一个小问题，在世界最高处以上放置方块时，会显示一条世界高度限制是319的警告信息，但是修改了交互距离后这条警告没有了

1024-byteeeee commented 10 months ago

但是还有一个小问题，在世界最高处以上放置方块时，会显示一条世界高度限制是319的警告信息，但是修改了交互距离后这条警告没有了

这应该不是一个值得注意的问题我感觉，因为建筑高度本身就是319，你怎么看。🤔

cdqtzrc commented 10 months ago

不想改就算了，毕竟这个bug并不严重

1024-byteeeee commented 10 months ago

不想改就算了，毕竟这个bug并不严重

是的，org carpet有计划做低版本支持吗，有的话如下部分我也要做对应修改....

1(1)

cdqtzrc commented 10 months ago

目前没有，估计以后也不会有

1024-byteeeee commented 10 months ago

目前没有，估计以后也不会有

那我暂时先不动了，虽然它有兼容性上的问题但是我感觉这里用 @ModifyConstant 比 @Inject 更好。

cdqtzrc commented 10 months ago

还有件事，在提交这个bug之前我是看了更新日志的，但是你似乎没有把问题描述清除

image
1024-byteeeee commented 10 months ago

还有件事，在提交这个bug之前我是看了更新日志的，但是你似乎没有把问题描述清除 image

是的，现在看来我的描述应该是有错误的，这并不属于兼容性问题（因为成员发现后我随便放了两个方块只是快速放置会放两次，正常对地板放没有问题），我现在修改日志。