xfl03 / MCCustomSkinLoader

Customize Skin, Cape and Elytra in Minecraft 1.8-1.21
GNU General Public License v3.0
506 stars 34 forks source link

[Bug] [OtherModCompatibility] Essentials在CustomSkinLoader.log内刷屏 #243

Closed 8MiYile closed 1 year ago

8MiYile commented 1 year ago

描述问题(中英两版)

我的客户端 安装了 Essential和CustomSkinLoader 想要试着用用Essential的动态披风 发现用不了 于是看了一下CustomSkinLoader.log 好家伙 刷屏呢 日志内 EssentialBot 这个ID 最为抢眼 我猜测皮肤和披风的事件全被CSL抢走了(草) (查看了一下工单系统 没有Essential这个关键词 我就直接提交了)

My client has installed Essential and CustomSkinLoader I wanted to try to use Essential's dynamic cloak and found that it didn't work So I looked at the CustomSkinLoader.log. EssentialBot ID in the log is the most eye-catching I guess the skin and cloak events were all taken by CSL (grass) (Checked the work order system without the keyword Essential, so I submitted it directly)

复现步骤

前言:Essential [下载地址](https://essential.gg/)
1. 安装Essential和CustomSkinLoader
2. 进入服务器
3. 使用Essential的免费披风

崩溃报告查看地址

无崩溃

CustomSkinLoader.log 查看地址

https://mclo.gs/mqBAB0D

模组版本

14.16-SNAPSHOT-26

Minecraft游戏版本

1.19.3

模组加载器

Quilt

操作系统

Linux

在提交错误报告之前,有没有做到...

8MiYile commented 1 year ago

两种解决方向

  1. 直接兼容动态的披风,额外增加允许其他Mod调用的API,以兼容其他模组直接调用并替换
  2. 在披风这块,检测到Essential或者类似模组修改披风,直接放行,CSL不做任何关于披风的处理

Two solution directions

  1. Directly compatible with Dynamic Cape, additionally add API that allows other mods to call to be compatible with other mods to call and replace directly
  2. In Cape, Essential or similar Mod modifications to Cape are detected and released directly, CSL does not do anything about Cape
8MiYile commented 1 year ago

image