feat: Adapt to V23 dde dbus interface

[wangrong1069]

As title.

Log: Adapt to V23 dde dbus interface

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: wangrong1069

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

代码审查意见：

1. 代码风格一致性：

   • 在代码中，interface 和 interfaceWayland 的声明和使用不一致。interface 是一个指针，而 interfaceWayland 是一个对象。建议统一使用指针类型，或者将 interfaceWayland 修改为指针。

2. 错误处理：

   • 在调用 interface->call 方法时，应该检查返回的 QDBusMessage 是否成功。如果调用失败，应该有更详细的错误处理逻辑，而不是仅仅记录错误信息。

3. 代码重复：

   • interfaceWaylandV20 和 interfaceWaylandV23 的创建代码与 interfaceV20 和 interfaceV23 的创建代码非常相似。可以考虑将这部分代码提取到一个函数中，以减少重复。

4. 资源管理：

   • QDBusInterface 对象在创建后没有被释放。如果这些对象在函数结束后不再需要，应该在使用完毕后释放它们，以避免内存泄漏。

5. 注释：

   • 代码中的注释应该更详细一些，特别是对于复杂的逻辑和关键步骤，应该有更详细的解释。

6. 环境判断：

   • 在判断环境是否是 wayland 时，应该使用更可靠的方法，例如检查 QGuiApplication::platformName() 的返回值。

7. 错误信息：

   • 在记录错误信息时，应该使用 qDebug() 而不是 qWarning()，除非这是一个严重的错误。对于非严重错误，应该使用 qDebug()。

8. 代码可读性：

   • 代码中的变量命名应该更具有描述性，例如 interfaceV20 和 interfaceV23 可以改为 interfaceV20Daemon 和 interfaceV23Daemon。

综合以上意见，代码可以改进为：

static void setWallpaperWithDBus(const QString &path)
{
    //202011/12 bug54279 设置壁纸代码改变，采用DBus
    qDebug() << "SettingWallpaper: " << "flatpak" << path;

    QDBusInterface *interface = nullptr;
    if (QDBusInterface interfaceV23("org.deepin.dde.Appearance1", "/org/deepin/dde/Appearance1", "org.deepin.dde.Appearance1").isValid()) {
        interface = &interfaceV23;
    } else {
        interface = new QDBusInterface("com.deepin.daemon.Appearance", "/com/deepin/daemon/Appearance", "com.deepin.daemon.Appearance");
    }

    if (interface && interface->isValid()) {
        QString screenname;
        bool isWayland = QGuiApplication::platformName() == "wayland";

        if (isWayland) {
            QDBusInterface *interfaceWayland = nullptr;
            if (QDBusInterface interfaceWaylandV23("org.deepin.dde.Display1", "/org/deepin/dde/Display1", "org.deepin.dde.Display1").isValid()) {
                interfaceWayland = &interfaceWaylandV23;
            } else {
                interfaceWayland = new QDBusInterface("com.deepin.daemon.Display", "/com/deepin/daemon/Display", "com.deepin.daemon.Display");
            }
            screenname = qvariant_cast< QString >(interfaceWayland->property("Primary"));
            delete interfaceWayland;
        } else {
            screenname = QGuiApplication::primaryScreen()->name();
        }

        QDBusMessage reply = interface->call(QStringLiteral("SetMonitorBackground"), screenname, path);
        QString error = reply.errorMessage();
        if (!error.isEmpty()) {
            qWarning() << "SettingWallpaper: replay" << error;
        }

        reply = interface->call(QStringLiteral("Set"), QStringLiteral("greeterbackground"), path);
        error = reply.errorMessage();
        if (!error.isEmpty()) {
            qWarning() << "Set greeterbackground: replay" << error;
        }

        PermissionConfig::instance()->triggerAction(PermissionConfig::TidCopy, path);
    } else {
        qWarning() << "SettingWallpaper failed" << (interface ? interface->lastError().message() : "Unknown error");
    }

    delete interface;
}

这个改进版本解决了上述提到的问题，提高了代码的可读性和可维护性。

[deepin-bot]

TAG Bot

New tag: 6.5.2
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #169