PandasWS / Pandas

熊猫模拟器 - 基于 rAthena 构建的中文仙境传说模拟器(欢迎加入QQ交流群:928171346)
https://pandas.ws
GNU General Public License v3.0
8 stars 2 forks source link

修复配置BUG: 读取公会图标配置失败 #657

Closed lyy289065406 closed 1 year ago

CairoLee commented 1 year ago

根据最新的程序调整,这两个战斗配置选项应该都被挪动到了 inter_athena.conf 中。 请问你是在使用过程中遇到什么问题吗?

之所以需要从 client.conf 移动到 inter_athena.conf 是因为在新的客户端中,公会图标是通过 web_server.exe(WEB 接口服务器)来负责接收客户端传来的公会图标的(以前是地图服务器负责)。

因此,这些选项之前在 client.conf 中只能被地图服务器读取,为了能够跨服务器读取到这些配置,选项从“战斗配置选项(Battle Configure)”被移动到“内部通讯配置文件 ”

CairoLee commented 1 year ago

请问还有更多的补充吗?如果没有的话,明天这个问题先关闭了哟

lyy289065406 commented 1 year ago

根据最新的程序调整,这两个战斗配置选项应该都被挪动到了 inter_athena.conf 中。 请问你是在使用过程中遇到什么问题吗?

之所以需要从 client.conf 移动到 inter_athena.conf 是因为在新的客户端中,公会图标是通过 web_server.exe(WEB 接口服务器)来负责接收客户端传来的公会图标的(以前是地图服务器负责)。

因此,这些选项之前在 client.conf 中只能被地图服务器读取,为了能够跨服务器读取到这些配置,选项从“战斗配置选项(Battle Configure)”被移动到“内部通讯配置文件 ”

1.1.18 版本加载不到这两个配置项,在启动时会报错找不到配置项,我修改后是 Ok 的。如果未来 1.1.19 版本是根据你说明的变更了的话,无视这个 PR 即可

CairoLee commented 1 year ago

出于严谨考虑,能否请你确认一下,你本地的代码是否合并完整?我提供一下相关线索:

rAthena 在 https://github.com/PandasWS/Pandas/commit/0ac52d0bb49e858f7ff7774685bb0d18af6d1cf7 中将上述两个配置移动了位置;熊猫在 https://github.com/PandasWS/Pandas/pull/647 中将这部分改动汇入了程序

最终这部分改动在 v1.1.17 版本中发布,差异对比记录:https://github.com/PandasWS/Pandas/compare/v1.1.16...v1.1.17

CairoLee commented 1 year ago

可以检查一下,你的程序 battle.cpp 中是否已经去掉了关于这两个选项的定义

https://github.com/PandasWS/Pandas/commit/0ac52d0bb49e858f7ff7774685bb0d18af6d1cf7#diff-e9da83c5add2e830d737c917e5a40115323bdd83189435c7bbc32534a7717213L10092-L10093

如果合并正确,这两行已经被移除了~

image

lyy289065406 commented 1 year ago

可以检查一下,你的程序 battle.cpp 中是否已经去掉了关于这两个选项的定义

0ac52d0#diff-e9da83c5add2e830d737c917e5a40115323bdd83189435c7bbc32534a7717213L10092-L10093

如果合并正确,这两行已经被移除了~

image

检查了一下确实被我保留了,回去再全部确认下,谢谢。 那这个 PR 应该可以关闭了