Skip to content

fix:为可能出错的部分加个 Try Catch#5912

Merged
LTCatt merged 4 commits intoMeloong-Git:mainfrom
LuoYun-Team:uac
Apr 25, 2025
Merged

fix:为可能出错的部分加个 Try Catch#5912
LTCatt merged 4 commits intoMeloong-Git:mainfrom
LuoYun-Team:uac

Conversation

@LinQingYuu
Copy link
Collaborator

Resolve #5910

@LinQingYuu LinQingYuu requested a review from LTCatt March 22, 2025 01:15
@LinQingYuu LinQingYuu added · Bug 等待确认 已经过社区确认,等待开发者确认 labels Mar 22, 2025
@Open-KFC
Copy link
Collaborator

Open-KFC commented Mar 22, 2025

这种用户取消的东西还要弹窗提示吗,我觉得 Hint 一下就够了吧

            Try
                Process.Start(Info)
            Catch ex As ComponentModel.Win32Exception When ex.NativeErrorCode = 1223 'ERROR_CANCELLED
                Hint(ex.Message)
            End Try

而且自定义事件打开文件是在 ModEvent.vb 里的逻辑啊(

直接改 ModBase.vb 的逻辑也不妥吧,例如在 FormMain#EndProgramForce 中调用 ShellOnly 时就没有进行任何向用户发提示的条件,应该在外部调 ShellOnly 的时候包裹 Try 。

@LinQingYuu
Copy link
Collaborator Author

这种用户取消的东西还要弹窗提示吗,我觉得 Hint 一下就够了吧

            Try
                Process.Start(Info)
            Catch ex As ComponentModel.Win32Exception When ex.NativeErrorCode = 1223 'ERROR_CANCELLED
                Hint(ex.Message)
            End Try

而且自定义事件打开文件是在 ModEvent.vb 里的逻辑啊(

直接改 ModBase.vb 的逻辑也不妥吧,例如在 FormMain#EndProgramForce 中调用 ShellOnly 时就没有进行任何向用户发提示的条件,应该在外部调 ShellOnly 的时候包裹 Try 。

现在怎么样?

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 and removed 等待确认 已经过社区确认,等待开发者确认 labels Mar 22, 2025
@Open-KFC
Copy link
Collaborator

@LinQingYuu
Copy link
Collaborator Author

LinQingYuu commented Mar 22, 2025

ModEvent 不会出现崩启动器的情况,应该没有必要改,问题只是出现在拉起日志时在 UAC 授权确认窗口点击取消才会出现,具体可以看 Issue 的示例

Issue 提到的崩溃显示的是 Windows 自带的弹窗....

@Open-KFC
Copy link
Collaborator

哦哦,我还以为 issue 反馈的情况就是指的自定义主页事件,那没事了

@LinQingYuu LinQingYuu changed the title fix(ModBase):加个 Try Catch fix:为可能出错的部分加个 Try Catch Apr 4, 2025
@LTCatt LTCatt merged commit 31434a9 into Meloong-Git:main Apr 25, 2025
2 checks passed
@HexDragon-Bot HexDragon-Bot added 完成 已被处理,将在下次更新之后生效 and removed 处理中 开发者正在调查或处理该项 labels Apr 25, 2025
@LinQingYuu LinQingYuu deleted the uac branch April 25, 2025 12:23
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.

拉起高权限进程时点击 UAC 授权窗口的取消按钮导致启动器崩溃

4 participants