Skip to content

fix(instance-setup): 调整 Reload 中 ServerLogin 的调用时机 - #3634

Merged
LuLu-ling merged 1 commit into
devfrom
fix/server-login-reload-order
Sep 26, 2026
Merged

LuLu-ling merged 1 commit into
devfrom
fix/server-login-reload-order

Conversation

@LuLu-ling

@LuLu-ling LuLu-ling commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

Sourcery 摘要

错误修复:

  • 修正 Reload 流程,确保在加载完所有服务器身份验证字段后再处理服务器登录。
Original summary in English

Summary by Sourcery

Bug Fixes:

  • Correct the Reload sequence so server login handling runs after all server authentication fields are loaded.

@LuLu-ling
LuLu-ling requested a review from a team September 26, 2026 06:22
@pcl-ce-automation pcl-ce-automation Bot added 🛠️ 等待审查 Pull Request 已完善,等待维护者或负责人进行代码审查 size: XS PR 大小评估:微型 labels Sep 26, 2026
@sourcery-ai

sourcery-ai Bot commented Sep 26, 2026 •

Copy link
Copy Markdown
Contributor
审查者指南(小型 PR 中折叠显示)

审查者指南

将 Reload 中 ServerLogin 的调用延后至服务器认证配置字段完成加载之后,避免登录初始化逻辑读取到未更新的认证信息。

重新排序后的服务器登录初始化时序图

sequenceDiagram
    participant Reload
    participant Config
    participant ServerLogin
    participant AuthFields

    Reload->>Config: Load LoginRequirementSolution
    Reload->>AuthFields: Load AuthServerAddress
    Reload->>AuthFields: Load AuthServerDisplayName
    Reload->>AuthFields: Load AuthRegisterAddress
    Reload->>ServerLogin: ServerLogin(SelectedIndex)
Loading

文件级变更

变更 详情 文件
调整服务器登录设置初始化顺序,确保登录方式切换逻辑在相关服务器认证字段加载完成后执行。
  • 将 ServerLogin 调用移至服务器认证地址、名称和注册链接配置加载之后。
  • 保持登录需求下拉框及其历史值的读取逻辑不变。
Plain Craft Launcher 2/Pages/PageInstance/PageInstanceSetup.xaml.cs

可能关联的问题

  • #未提供:PR 调整 Reload 中 ServerLogin 调用顺序,直接涉及实例设置重载,可能修复修改 Minecraft 版本时的错误。
  • #实例设置中刷新所有服务器失败:PR 将 ServerLogin 移至认证配置加载之后,针对刷新服务器触发的初始化顺序错误。

提示和命令

与 Sourcery 交互

  • 触发新的审查: 在拉取请求中评论 @sourcery-ai review。
  • 继续讨论: 直接回复 Sourcery 的审查评论。
  • 从审查评论生成 GitHub issue: 回复审查评论,请 Sourcery 根据该评论创建 issue。你也可以回复审查评论并使用 @sourcery-ai issue,根据该评论创建 issue。
  • 生成拉取请求标题: 在拉取请求标题的任意位置写入 @sourcery-ai,即可随时生成标题。你也可以在拉取请求中评论 @sourcery-ai title,随时重新生成标题。
  • 生成拉取请求摘要: 在拉取请求正文中任意位置写入 @sourcery-ai summary,即可在指定位置随时生成 PR 摘要。你也可以在拉取请求中评论 @sourcery-ai summary,随时重新生成摘要。
  • 生成审查者指南: 在拉取请求中评论 @sourcery-ai guide,即可随时重新生成审查者指南。
  • 解决所有 Sourcery 评论: 在拉取请求中评论 @sourcery-ai resolve,即可解决所有 Sourcery 评论。如果你已经处理完所有评论且不想再看到它们,此功能会非常有用。
  • 忽略所有 Sourcery 审查: 在拉取请求中评论 @sourcery-ai dismiss,即可忽略所有现有的 Sourcery 审查。如果你想从新的审查开始,这一功能尤其有用——别忘了评论 @sourcery-ai review 以触发新的审查!

自定义你的体验

访问你的控制面板以:

  • 启用或禁用审查功能,例如 Sourcery 生成的拉取请求摘要、审查者指南等。
  • 更改审查语言。
  • 添加、移除或编辑自定义审查说明。
  • 调整其他审查设置。

获取帮助

Original review guide in English
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

将 Reload 中 ServerLogin 的调用延后至服务器认证配置字段完成加载之后,避免登录初始化逻辑读取到未更新的认证信息。

Sequence diagram for reordered server login initialization

sequenceDiagram
    participant Reload
    participant Config
    participant ServerLogin
    participant AuthFields

    Reload->>Config: Load LoginRequirementSolution
    Reload->>AuthFields: Load AuthServerAddress
    Reload->>AuthFields: Load AuthServerDisplayName
    Reload->>AuthFields: Load AuthRegisterAddress
    Reload->>ServerLogin: ServerLogin(SelectedIndex)
Loading

File-Level Changes

Change Details Files
调整服务器登录设置初始化顺序,确保登录方式切换逻辑在相关服务器认证字段加载完成后执行。
  • 将 ServerLogin 调用移至服务器认证地址、名称和注册链接配置加载之后。
  • 保持登录需求下拉框及其历史值的读取逻辑不变。
Plain Craft Launcher 2/Pages/PageInstance/PageInstanceSetup.xaml.cs

Possibly linked issues

  • #未提供: PR 调整 Reload 中 ServerLogin 调用顺序,直接涉及实例设置重载,可能修复修改 Minecraft 版本时的错误。
  • #实例设置中刷新所有服务器失败: PR 将 ServerLogin 移至认证配置加载之后,针对刷新服务器触发的初始化顺序错误。

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你好——我已经审阅了你的更改,看起来很棒!


Sourcery 对开源项目免费——如果你喜欢我们的审阅结果,请考虑分享它们 ✨
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨

@LuLu-ling LuLu-ling changed the title fix(instance-setup):调整 Reload 中 ServerLogin 的调用时机 fix(instance-setup): 调整 Reload 中 ServerLogin 的调用时机 Sep 26, 2026
@pcl-ce-automation pcl-ce-automation Bot added 🕑 等待合并 已处理完毕,正在等待代码合并入主分支 and removed 🛠️ 等待审查 Pull Request 已完善,等待维护者或负责人进行代码审查 labels Sep 26, 2026
@LuLu-ling
LuLu-ling merged commit ce9dc83 into dev Sep 26, 2026
3 checks passed
@pcl-ce-automation pcl-ce-automation Bot added 👌 完成 相关问题已修复或功能已实现,计划在下次版本更新时正式上线 and removed 🕑 等待合并 已处理完毕,正在等待代码合并入主分支 labels Sep 26, 2026
@LuLu-ling
LuLu-ling deleted the fix/server-login-reload-order branch September 26, 2026 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: XS PR 大小评估:微型 👌 完成 相关问题已修复或功能已实现,计划在下次版本更新时正式上线

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants