Skip to content

feat: Prohibit checking for updates when automatic restore is enabled#121

Merged
mhduiy merged 1 commit into
linuxdeepin:masterfrom
mhduiy:rec
Jun 18, 2025
Merged

feat: Prohibit checking for updates when automatic restore is enabled#121
mhduiy merged 1 commit into
linuxdeepin:masterfrom
mhduiy:rec

Conversation

@mhduiy

@mhduiy mhduiy commented Jun 18, 2025

Copy link
Copy Markdown
Contributor

After enabling automatic restore, updates are invalid, so checking for updates is directly prohibited

pms: TASK-378055

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, mhduiy

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

Details Needs approval from an approver in each of these files:

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

After enabling automatic restore, updates are invalid, so checking for updates is directly prohibited

pms: TASK-378055
@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

代码审查意见:

  1. UpdateDBusProxy类中新增的immutableAutoRecovery方法,应该检查m_managerInter是否为空,以避免潜在的空指针异常。
  2. UpdateModel类的refreshIsUpdateDisabled方法中,m_immutableAutoRecovery变量被用来判断是否禁用更新,但是这个变量在UpdateModel类中并没有被定义。应该检查这个变量的定义是否正确。
  3. UpdateModel类的setImmutableAutoRecovery方法中,应该添加对m_immutableAutoRecovery的检查,以避免不必要的信号发射。
  4. UpdateWorker类的activate方法中,m_model->setImmutableAutoRecovery(m_updateInter->immutableAutoRecovery());这一行代码可能会抛出异常,如果m_updateInter为空的话。应该添加对m_updateInter的检查。
  5. UpdateDisable.qml文件中,D.Labeltext属性应该使用tr函数进行翻译,以支持多语言。
  6. 在所有的.ts文件中,对于新增的翻译文本,应该检查是否已经正确翻译,并且是否已经添加到相应的语言文件中。
  7. UpdateDBusProxy类的updateStatus方法中,应该添加对m_managerInter的检查,以避免潜在的空指针异常。
  8. UpdateDBusProxy类的hardwareId方法中,应该添加对m_managerInter的检查,以避免潜在的空指针异常。
  9. UpdateDBusProxy类的immutableAutoRecovery方法中,应该添加对m_managerInter的检查,以避免潜在的空指针异常。
  10. UpdateDBusProxy类的updateStatus方法中,应该添加对返回值的检查,以避免潜在的异常。
  11. UpdateDBusProxy类的hardwareId方法中,应该添加对返回值的检查,以避免潜在的异常。
  12. UpdateDBusProxy类的immutableAutoRecovery方法中,应该添加对返回值的检查,以避免潜在的异常。
  13. UpdateModel类的setImmutableAutoRecovery方法中,应该添加对value的检查,以避免潜在的异常。
  14. UpdateWorker类的initConnect方法中,应该添加对m_updateInter的检查,以避免潜在的空指针异常。
  15. UpdateWorker类的activate方法中,应该添加对m_updateInter的检查,以避免潜在的空指针异常。

以上是代码审查的意见,希望能够帮助到您。

@mhduiy mhduiy merged commit bdcf2e9 into linuxdeepin:master Jun 18, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants