fix: turning off 'Unlocking is required to wake up the computer' does…

zsien commented 3 days ago

… not work

判断 sleep-lock 状态

Bug: https://pms.uniontech.com/bug-view-287959.html

ECQZXC commented 1 day ago

/approved

deepin-ci-robot commented 1 day ago

deepin pr auto review

关键摘要:

在LockWorker构造函数中，m_powerGsettings的初始化放在了if语句中，这可能导致m_powerGsettings在某些情况下未被初始化。
在doPowerAction函数中，m_model的setIsBlackMode和setVisible方法被调用，但没有检查m_model是否为nullptr，可能会导致空指针解引用。
doPowerAction函数中的delayTime变量被硬编码为500，建议将其定义为常量或从配置文件中读取。
AuthInterface类中新增了m_powerGsettings成员变量，但没有相应的注释说明其用途。

是否建议立即修改: 是

deepin-ci-robot commented 1 day ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fly602, zsien

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

Needs approval from an approver in each of these files: - **[debian/deepin/OWNERS](https://github.com/linuxdeepin/dde-session-shell/blob/master/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

linuxdeepin / dde-session-shell

fix: turning off 'Unlocking is required to wake up the computer' does… #387

deepin pr auto review