Skip to content

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

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.

Sign up for GitHub

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

Jump to bottom
Open
1 task done
AL-1830 opened this issue Apr 7, 2025 · 6 comments · May be fixed by #6087
Open
1 task done

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

AL-1830 opened this issue Apr 7, 2025 · 6 comments · May be fixed by #6087
Labels
· Bug 可合并 已有 Pull Request 处理了该 Issue

Comments

@AL-1830
Copy link

AL-1830 commented Apr 7, 2025

检查项

描述

在修改版本名时若同一目录内有文件名字母相同但大小写不同时的版本时会出现报错
（原因：Windows文件夹名称不区分大小写）
日志为复现例子，从1669行开始

重现步骤

例：原版本文件夹列表内有两版本：Version和Version2
1、进入Version2的设置
2、然后点击修改版本名，修改为VERSION（字母相同，但大小写不同）
3、出现报错，Version2的版本名变为VERSION_temp

日志与附件

Log1.txt
@AL-1830 AL-1830 added · Bug 新提交 需要社区初步确认其有效的新提交 labels Apr 7, 2025
@Open-KFC
Copy link
Collaborator

Open-KFC commented Apr 7, 2025

问题在这一行，为了允许只更改名称大小写而加上了IgnoreCase:=False

PCL2/Plain Craft Launcher 2/Pages/PageVersion/PageVersionOverall.xaml.vb

Line 112 in 16e09c7

Dim NewName As String = MyMsgBoxInput("重命名版本", "", OldName, New ObjectModel.Collection(Of Validate) From {New ValidateFolderName(PathMcFolder & "versions", IgnoreCase:=False)})

@Open-KFC Open-KFC added 等待确认 已经过社区确认，等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Apr 7, 2025
@wyc-26
Copy link
Contributor

wyc-26 commented Apr 7, 2025

感觉这里龙猫好像写反了
IgnoreCase 的意思是忽略大小写，但龙猫这里写的是 True 时不忽略，False 时忽略🤣

@Open-KFC
Copy link
Collaborator

Open-KFC commented Apr 7, 2025

感觉这里龙猫好像写反了 IgnoreCase 的意思是忽略大小写，但龙猫这里写的是 True 时不忽略，False 时忽略🤣

他要这么写的原因我已经说过了……

@wyc-26
Copy link
Contributor

wyc-26 commented Apr 7, 2025

感觉这里龙猫好像写反了 IgnoreCase 的意思是忽略大小写，但龙猫这里写的是 True 时不忽略，False 时忽略🤣

他要这么写的原因我已经说过了……

？你看看 ValidateExceptSame 去，不是 ValidateFolderName
ValidateFolderName 里面调用了 ValidateExceptSame

@wyc-26 wyc-26 linked a pull request Apr 7, 2025 that will close this issue
Open
@Open-KFC
Copy link
Collaborator

Open-KFC commented Apr 7, 2025

如果不写忽略大小写:=False的话用户是不能把名为ForgeABC的版本重命名为Forgeabc的。

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度：低 可合并 已有 Pull Request 处理了该 Issue and removed 等待确认 已经过社区确认，等待开发者确认 处理中 开发者正在调查或处理该项 🟩 低 优先度：低 labels Apr 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
· Bug 可合并 已有 Pull Request 处理了该 Issue
Projects
None yet
Milestone
No milestone
4 participants
@LTCatt @AL-1830 @wyc-26 @Open-KFC