Skip to content

fix: 修改版本名时未验证是否有不同大小写字母的文件夹名 #6087

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

wyc-26
Copy link
Contributor

@wyc-26 wyc-26 commented Apr 7, 2025

fix #6083

@wyc-26 wyc-26 closed this Apr 7, 2025
@wyc-26
Copy link
Contributor Author

wyc-26 commented Apr 7, 2025

wait...

@HexDragon-Bot HexDragon-Bot added the 忽略 因为各种杂项原因而关闭 label Apr 7, 2025
@wyc-26 wyc-26 changed the title fix(ModValidate): If 写反了导致修改版本名时未验证是否有不同大小写字母的文件夹名 fix: 修改版本名时未验证是否有不同大小写字母的文件夹名 Apr 7, 2025
@wyc-26 wyc-26 reopened this Apr 7, 2025
@wyc-26
Copy link
Contributor Author

wyc-26 commented Apr 7, 2025

嘶,等等别审先,还没搞完

@Open-KFC Open-KFC added · Bug 社区处理中 社区正在调查或处理该项 and removed 忽略 因为各种杂项原因而关闭 labels Apr 7, 2025
@Open-KFC

This comment was marked as resolved.

@wyc-26
Copy link
Contributor Author

wyc-26 commented Apr 7, 2025

一个非常不干净利落的写法,并且还会导致重命名版本时填入相同的名称时也可重命名,虽然不会报错就是了
我是 .Net 小白

Co-authored-by: 薄奚梦灵 <boximengling@outlook.com>
@Open-KFC Open-KFC added 等待确认 已经过社区确认,等待开发者确认 and removed 社区处理中 社区正在调查或处理该项 labels Apr 8, 2025
@shimoranla shimoranla requested a review from LTCatt April 8, 2025 11:59
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 and removed 等待确认 已经过社区确认,等待开发者确认 labels Apr 9, 2025
@Open-KFC
Copy link
Collaborator

Open-KFC commented Apr 11, 2025

感觉这样写太难读了,改了一下,在 Open-KFC:fix6083
顺便删了 IgnoreCase ,毕竟文件系统不支持区分大小写,留着也没用;
给 PathIgnore 加了初始值以免去对空值的判断;
把 PathIgnore 名字改成了 PathDirs ,不知道是不是我英语不好所以觉得原命名看不懂……

Co-authored-by: Open-KFC <168712411+Open-KFC@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· Bug 处理中 开发者正在调查或处理该项 🟩 低 优先度:低
Projects
None yet
Development

Successfully merging this pull request may close these issues.

修改版本名时未验证是否有不同大小写字母的文件夹名
5 participants