fix: Fifth round of bug fixes for private update & update transfer & …

[ut005973]

…HTTP speed limit online configuration features

Fixed the following issues:
[V25 2500U1][Update Management System][Enterprise Update Management System][Sixth Round Testing] When the server supports P2P and the terminal successfully connects, the transfer optimization switch in Control Center should be grayed out and non-closable [V25 2500U1][Update Management System][Control Center Update Client][Sixth Round Testing][Public Network] After checking the upload speed limit and download speed limit checkboxes of transfer optimization sub-items, unchecking them again causes the speed limit value to change to 104857 [V25 2500U1][Update Management System][Control Center Update Client][Fourth Round Testing] After modifying the status or value of transfer optimization upload/download speed limits, reopening Control Center causes the transfer optimization upload/download speed limit positions to change [V25 2500U1][Update Management System][Control Center Update Client][Sixth Round Testing][Public Network] The English translation of transfer optimization does not match requirements [V25 2500U1][Update Management System][Control Center Update Client][Sixth Round Testing][System Update][Public Network] When enabling or disabling transfer optimization settings fails, a retry dialog appears with prompt information and buttons inconsistent with requirements [V25 2500U1][Update Management System][Control Center Update Client][Sixth Round Testing][Public Network] When enabling or disabling transfer optimization sub-item settings fails, the dialog display does not match requirements [V25 2500U1][Update Management System][Control Center Update Client][Sixth Round Testing][Public Network] When enabling transfer optimization settings fails, the dialog needs to support closing with Esc key [V25 2500U1][Update Management System][Control Center Update Client][Sixth Round Testing][Public Network] When enabling or disabling transfer optimization sub-item settings fails, the dialog needs to support closing with Esc key [V25 2500U1][Update Management System][Control Center Update Client][Sixth Round Testing][System Update][Public Network] The display names of transfer optimization sub-items need to match requirements

Log: As above
Bug: https://pms.uniontech.com/bug-view-357875.html https://pms.uniontech.com/bug-view-358203.html
https://pms.uniontech.com/bug-view-357291.html
https://pms.uniontech.com/bug-view-358231.html
https://pms.uniontech.com/bug-view-358257.html
https://pms.uniontech.com/bug-view-358321.html
https://pms.uniontech.com/bug-view-358303.html
https://pms.uniontech.com/bug-view-358357.html
https://pms.uniontech.com/bug-view-358365.html
Influence: Affects private update & update transfer & HTTP speed limit online configuration features

fix: 私有化更新&更新传递&http限速在线配置功能第五轮bug修复
修复以下问题：
【V25 2500U1】【更新管理系统】【企业级更新管理系统】【第六轮测试】服务端支持P2P，终端接入成功后，控制中心-传递优化开关应该置灰并不可关闭 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第六轮测试】【公网】勾选传递优化子配置项的上传限速和下载限速复选框，再次去勾选复选框，限速值变成了104857 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第四轮测试】修改传递优化上传下载限速的状态或者值，重新打开控制中心，传递优化上传下载限速位置发生变化 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第六轮测试】【公网】传递优化英文翻译与需求不一致 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第六轮测试】【系统更新】【公网】传递优化设置项开启或关闭失败，出现重试弹窗，提示信息和按钮与需求不一致 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第六轮测试】【公网】传递优化设置项子配置项开启或关闭失败，弹窗展示与需求不一致 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第六轮测试】【公网】传递优化设置项开启失败，弹窗需要支持Esc键关闭弹窗 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第六轮测试】【公网】传递优化设置项子配置项开启或关闭失败，弹窗需要支持esc关闭操作 【V25 2500U1】【更新管理系统】【控制中心更新客户端】【第六轮测试】【系统更新】【公网】传递优化子配置项展示名称需要与需求一致

Log：如上所述
Bug：https://pms.uniontech.com/bug-view-357875.html https://pms.uniontech.com/bug-view-358203.html
https://pms.uniontech.com/bug-view-357291.html
https://pms.uniontech.com/bug-view-358231.html
https://pms.uniontech.com/bug-view-358257.html
https://pms.uniontech.com/bug-view-358321.html
https://pms.uniontech.com/bug-view-358303.html
https://pms.uniontech.com/bug-view-358357.html
https://pms.uniontech.com/bug-view-358365.html
Influence：影响私有化更新&更新传递&http限速在线配置功能

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ut005973

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[deepin-ci-robot]

deepin pr auto review

代码审查报告

我已对提供的git diff进行了全面审查，主要涉及更新插件的传递优化功能修改。以下是我的详细分析：

1. 语法逻辑审查

updatedatastructs.h 中的修改

if (lastoreDeliveryConfig.isOnlineSpeedLimit) {
    lastoreDeliveryConfig.limitSpeed = QString::number(UpgradeSpeedLimitConfig::fromJson(deliveryConfig.toUtf8()).currentRate);
} else {
    lastoreDeliveryConfig.limitSpeed = QString::number(UpgradeSpeedLimitConfig::fromJson(deliveryConfig.toUtf8()).limitRate);
}

问题：

• 重复调用 UpgradeSpeedLimitConfig::fromJson(deliveryConfig.toUtf8()) 3次，效率低下
• 应该只解析一次JSON并重用结果

建议改进：

auto speedConfig = UpgradeSpeedLimitConfig::fromJson(deliveryConfig.toUtf8());
lastoreDeliveryConfig.isOnlineSpeedLimit = speedConfig.ifInOnlineLimit();
lastoreDeliveryConfig.speedLimitEnabled = speedConfig.shouldLimitRate();
lastoreDeliveryConfig.limitSpeed = QString::number(lastoreDeliveryConfig.isOnlineSpeedLimit ? 
                                                   speedConfig.currentRate : 
                                                   speedConfig.limitRate);

UpdateSetting.qml 中的修改

• 删除了 pendingUpgradeDeliveryEnabled 属性，直接使用 dccData.model().upgradeDeliveryEnable
• 修改了错误对话框的按钮文本和逻辑

问题：

• 在 upgradeDeliverySetEnableFailedDialog 中，重试按钮逻辑改为 !dccData.model().upgradeDeliveryEnable，这看起来是尝试切换当前状态，但逻辑可能不够清晰
• 缺少对 isPrivateUpdate 属性的初始化和变更监听

2. 代码质量审查

优点

1. 界面文本更一致，统一使用"Delivery Optimization"术语
2. 简化了错误对话框的提示信息，不再区分启用/禁用失败
3. 添加了 ESC 键关闭对话框的功能，提升了用户体验

问题

1. UpdateSetting.qml 中多处使用 dccData.model().isPrivateUpdate 来控制可见性，但缺少对该属性变更的响应
2. advancedSetting.showDetails 的初始化和销毁逻辑不一致：

   property bool showDetails: dccData.model().isPrivateUpdate
   Component.onDestruction: {
       advancedSetting.showDetails = false || dccData.model().isPrivateUpdate
   }

   销毁时的逻辑看起来不必要，因为组件即将被销毁

3. 代码性能审查

1. updatedatastructs.h 中重复解析JSON是主要的性能问题，已在上文指出
2. UpdateSetting.qml 中多次访问 dccData.model().isPrivateUpdate，可以考虑使用局部变量缓存
3. 翻译文件中添加了大量新的翻译条目，但未删除旧的翻译条目，可能导致翻译文件膨胀

4. 代码安全审查

1. updatedatastructs.h 中对JSON解析结果没有错误检查，如果JSON格式不正确可能导致未定义行为
2. pluginupdateplugin.cpp 中新增的 m_isPrivateUpdate 成员变量没有初始化检查
3. UpdateSetting.qml 中直接使用 dccData.model().upgradeDeliveryEnable 而没有验证其有效性

5. 其他建议

1. 命名一致性：

   • 代码中混合使用了 "Upgrade Delivery" 和 "Delivery Optimization" 术语，虽然这次修改统一了大部分，但还有一些地方可能遗漏
   • 建议在代码注释中也使用一致的术语

2. 国际化：

   • 翻译文件中有 type="unfinished" 的条目，需要确保所有语言都完成翻译
   • 建议添加翻译完整性检查

3. 文档：

   • 建议为 isPrivateUpdate 属性添加文档说明其用途和行为
   • 建议为 transferDeliveryConfigToLastoreDeliveryConfig 函数添加更详细的注释

4. 测试：

   • 建议添加对 isPrivateUpdate 状态切换的测试用例
   • 建议添加对限速配置切换的测试用例

总结

这次修改主要是为了改进传递优化功能的用户体验和内部逻辑，整体方向是正确的。主要问题在于性能优化（重复JSON解析）和一些逻辑清晰度问题。建议在合并前解决这些问题，特别是JSON解析的性能问题和错误处理。