Aloxaf / cdda-itembrowser

CDDA 物品浏览器
MIT License
8 stars 2 forks source link

[BUG] copy-from 的处理不完全 #4

Open Aloxaf opened 4 years ago

Aloxaf commented 4 years ago

1. 没有考虑到 mod 中同 id 的物品会导致覆盖的情况

比如 https://cdda-trunk.aloxaf.cn/mutation/CANNIBAL 中 CRIT 扩展的食人族并没有正确继承原版的属性。

这个问题并不能简单地通过只记录首个出现的 id 来解决,举例来说,A copy-from B,而 MOD 覆盖了 B,如果只记录原版的 B,就会导致 A 继承的属性错误,比如 https://cdda-trunk.aloxaf.cn/308

2. copy-from 应该处理嵌套的结构,而不是简单判断键值存不存在

比如 https://cdda-trunk.aloxaf.cn/itemgroup/homebooks 中大魔法的物品组没有正确继承原版物品组中的物品。

Aloxaf commented 3 years ago

对于问题 1,应该永远只让 MOD 覆盖原版物品 否则不同 MOD 加载顺序导致的问题完全无法解决