Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mod更新按钮不会在获取Mod更新信息后刷新 #4377

Closed
1 task done
USTC-XeF2 opened this issue Jul 30, 2024 · 4 comments
Closed
1 task done

Mod更新按钮不会在获取Mod更新信息后刷新 #4377

USTC-XeF2 opened this issue Jul 30, 2024 · 4 comments
Labels
· Bug 完成 已被处理,将在下次更新之后生效 🟩 低 优先度:低

Comments

@USTC-XeF2
Copy link

检查项

描述

在版本Mod界面若在Mod版本加载之前全选,则需要更新的Mod并不会提示更新
f0770f179c25f17350c51a2cc80d44b

重现步骤

1、点击版本设置-Mod管理
2、在加载好前快速全选Mod
注:若Mod已加载可以先清理启动器缓存

日志与附件

Log3.txt
Log1.txt
Log2.txt
在Log1与Log3中两次出现了此Bug
解决方案:可以在获取Mod更新信息后增加一次刷新按钮的操作

@USTC-XeF2 USTC-XeF2 added · Bug 新提交 需要社区初步确认其有效的新提交 labels Jul 30, 2024
@JingHai-Lingyun
Copy link
Collaborator

在 Mod 更新信息加载好之前全选 Mod 导致无法一键更新全部 Mod

@JingHai-Lingyun JingHai-Lingyun added 等待确认 已经过社区确认,等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Jul 30, 2024
@wuliaodexiaoluo
Copy link
Contributor

wuliaodexiaoluo commented Jul 30, 2024

我的看法(可能不是很友好)(点击展示)

在加载好前快速全选Mod

无意冒犯,但是我觉得有点卡时机的意味 ,Issue 被龙猫拒掉的概率 +10%

注:若Mod已加载可以先清理启动器缓存

无意冒犯,但是特殊条件 +1 ,Issue 被龙猫拒掉的概率 +20%

在版本Mod界面若在Mod版本加载之前全选,则需要更新的Mod并不会提示更新

这应该是预期行为,如果反复加载信息会导致一瞬间的 IO 爆炸和 CPU 占用的显著提高,如果没回收干净就算内存泄漏,文件被使用、损坏了会报错,热加载可能导致死锁或者类似 #3092 那样的 UI Bug,信息没刷新就不可能更新,就像你不可能让小学生做高中奥数,因为根本做不来 (当然也不排除有神童能做出来(OFFTOPIC))

Issue 被龙猫拒掉的概率 +20%

解决方案:可以在获取Mod更新信息后增加一次刷新按钮的操作

如果重新选中能使此 Bug 的到解决,那这个按钮不太可能加上去
所以其实没啥用

Issue 被龙猫拒掉的概率 +40%

综合来看,龙猫拒掉的可能性高达 90%,约等于拒掉,而且就算打上了处理中的 Label 也不太可能是做 Issue 提到的修复方案,我觉得最多是阻止在加载完毕前选中 Mod (这也不太可能)

如果是选中后也不能更新,那这个 Issue 也只能是修掉在加载完成前选中后再次选中不能更新

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 and removed 等待确认 已经过社区确认,等待开发者确认 labels Jul 30, 2024
@USTC-XeF2
Copy link
Author

我的看法(可能不是很友好)(点击展示)

所以你应该好奇了这个奇怪的Bug是如何被发现的。事实上是因为本人电脑游戏等经常多开,比较着急,每次打开PCL都先去检查Mod更新(Ctrl+A)然后就切到别的界面等待加载。今天这次切回来看没有要更新的Mod就进入游戏了,结果发现模组菜单提示有更新...于是机缘巧合发现了这个Bug

不过我也没想到这个Bug能过

@USTC-XeF2
Copy link
Author

我的看法(可能不是很友好)(点击展示)

哦对还有占用问题,其实在获取Mod资源之后如果有更新在每个Mod处的更新小箭头是有刷新显示的,那应该也可以把全部更新的按钮刷新放到同一个回调函数里吧(.net我不太懂,如果不行那就算了)

@LTCatt LTCatt closed this as completed Aug 13, 2024
@HexDragon-Bot HexDragon-Bot added 完成 已被处理,将在下次更新之后生效 and removed 处理中 开发者正在调查或处理该项 labels Aug 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· Bug 完成 已被处理,将在下次更新之后生效 🟩 低 优先度:低
Projects
None yet
Development

No branches or pull requests

5 participants