Skip to content

fix: update lastore config paths to new dde namespace#94

Merged
mhduiy merged 1 commit into
linuxdeepin:masterfrom
mhduiy:dialog
May 29, 2025
Merged

fix: update lastore config paths to new dde namespace#94
mhduiy merged 1 commit into
linuxdeepin:masterfrom
mhduiy:dialog

Conversation

@mhduiy

@mhduiy mhduiy commented May 29, 2025

Copy link
Copy Markdown
Contributor
  1. Changed config paths from "org.deepin.lastore" to "org.deepin.dde.lastore" in both recoverydialog.cpp and updateworker.cpp
  2. Updated corresponding log messages to reflect new namespace
  3. This change aligns with the new DDE namespace convention for consistency across the project

fix: 更新lastore配置路径到新的dde命名空间

  1. 在recoverydialog.cpp和updateworker.cpp中将配置路径 从"org.deepin.lastore"改为"org.deepin.dde.lastore"
  2. 更新了相应的日志消息以反映新的命名空间
  3. 此更改符合项目新的DDE命名空间约定,保持一致性

pms: BUG-313665

1. Changed config paths from "org.deepin.lastore" to
"org.deepin.dde.lastore" in both recoverydialog.cpp and updateworker.cpp
2. Updated corresponding log messages to reflect new namespace
3. This change aligns with the new DDE namespace convention for
consistency across the project

fix: 更新lastore配置路径到新的dde命名空间

1. 在recoverydialog.cpp和updateworker.cpp中将配置路径
从"org.deepin.lastore"改为"org.deepin.dde.lastore"
2. 更新了相应的日志消息以反映新的命名空间
3. 此更改符合项目新的DDE命名空间约定,保持一致性

pms: BUG-313665
@deepin-ci-robot

Copy link
Copy Markdown

deepin pr auto review

代码审查意见:

  1. 配置文件路径更改

    • recoverydialog.cpp文件中,配置文件路径从org.deepin.lastore更改为org.deepin.dde.lastore。请确认这一更改是否与配置文件的实际路径一致,并且相关的配置项是否已经更新。
  2. 日志信息

    • recoverydialog.cpp文件中,日志信息从org.deepin.lastore更改为org.deepin.dde.lastore。这一更改应该与配置文件路径的更改保持一致,以确保日志信息与实际操作相对应。
  3. 错误处理

    • recoverydialog.cpp文件中,当JSON解析出错时,错误信息被记录为org.deepin.lastore upgrade-status, json parse error。这一信息应该与配置文件路径的更改保持一致,即更改为org.deepin.dde.lastore upgrade-status, json parse error
  4. 日志级别

    • recoverydialog.cpp文件中,当JSON解析出错时,使用qWarning()记录错误信息。建议使用qCritical()qFatal()来记录严重的错误,以便于快速定位问题。
  5. 代码重复

    • updateworker.cpp文件中,获取lastore-daemon-status的代码与recoverydialog.cpp文件中的代码非常相似。建议将这部分代码提取到一个公共函数中,以减少代码重复。
  6. 常量定义

    • updateworker.cpp文件中,IS_UPDATE_READY常量被定义为1。建议使用宏定义或枚举来定义常量,以提高代码的可读性和可维护性。
  7. 日志信息

    • updateworker.cpp文件中,日志信息记录了Lastore daemon status。建议添加更多的上下文信息,以便于调试和问题定位。
  8. 注释

    • recoverydialog.cpp文件中,注释// running, 代表在更新过程中被中断应该使用中文,以保持代码的一致性。

总体来说,代码的更改看起来是合理的,但是需要注意日志信息的准确性,以及代码的重复和可维护性。

@deepin-ci-robot

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

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

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

@mhduiy mhduiy merged commit 22eed95 into linuxdeepin:master May 29, 2025
6 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