refactor: remove debug panel and legacy DBus code

1. Removed DataPanel.qml and related test functionality from
NotifyHeader.qml
2. Removed legacy DBus notification handling code in
notificationcenterpanel.cpp
3. Cleaned up NotifyAccessor by removing debug/test methods (addNotify,
fetchDataInfo)
4. Simplified notification removal methods to only use new dataUpdater
interface
5. Improved database error logging in DBAccessor
6. Enhanced notification logging in NotificationManager

These changes remove unused debug/test code and legacy DBus interfaces
that were replaced by the new dataUpdater system. The cleanup improves
code maintainability and reduces potential security surface by removing
unused code paths. The logging improvements provide better debugging
information while removing redundant debug tools.

refactor: 移除调试面板和遗留DBus代码

1. 移除DataPanel.qml及NotifyHeader.qml中的相关测试功能
2. 删除notificationcenterpanel.cpp中的遗留DBus通知处理代码
3. 清理NotifyAccessor中的调试/测试方法(addNotify, fetchDataInfo)
4. 简化通知移除方法，仅使用新的dataUpdater接口
5. 在DBAccessor中改进数据库错误日志记录
6. 在NotificationManager中增强通知日志记录

这些变更移除了未使用的调试/测试代码和被新dataUpdater系统取代的遗留DBus接
口。清理工作提高了代码可维护性，并通过移除未使用的代码路径减少了潜在的安
全风险。日志记录的改进在移除冗余调试工具的同时提供了更好的调试信息。

Author: 18202781743

panels/notification/center/CMakeLists.txt

@@ -19,7 +19,6 @@ qt_add_qml_module(notificationcenterpanel
         NotifySettingMenu.qml
         AnimationSettingButton.qml
         BoundingRectangle.qml
-        DataPanel.qml
         NotifyHeaderTitleText.qml
     SOURCES
         notificationcenterpanel.h

panels/notification/center/DataPanel.qml

@@ -13,28 +13,6 @@ FocusScope {
     id: root
 
     required property NotifyModel notifyModel
-    signal headerClicked()
-
-    // test
-    onHeaderClicked: function () {
-        dataPanelLoader.active = !dataPanelLoader.active
-        NotifyAccessor.fetchDataInfo()
-        dataPanelLoader.item.show()
-    }
-    Loader {
-        id: dataPanelLoader
-        active: false
-        sourceComponent: Window {
-            id: dataPanel
-            width: 360
-            height: 600
-            x: dataPanel.transientParent.x + root.Window.width + 10
-            y: dataPanel.transientParent.y
-            DataPanel {
-                notifyModel: root.notifyModel
-            }
-        }
-    }
 
     RowLayout {
         anchors.fill: parent
@@ -43,12 +21,6 @@ FocusScope {
             Layout.alignment: Qt.AlignLeft
             Layout.leftMargin: 18
             tFont: DTK.fontManager.t4
-            MouseArea {
-                anchors.fill: parent
-                onDoubleClicked: {
-                    root.headerClicked()
-                }
-            }
         }
 
         Item {

panels/notification/center/NotifyHeader.qml

@@ -88,14 +88,10 @@ bool NotificationCenterPanel::init()
                                  Qt::QueuedConnection);
         notifycenter::NotifyAccessor::instance()->setDataUpdater(server);
         notifycenter::NotifyAccessor::instance()->setEnabled(visible());
-    } else {
-        // old interface by dbus
-        auto connection = QDBusConnection::sessionBus();
-        valid = connection.connect(DDENotifyDBusServer, DDENotifyDBusPath, DDENotifyDBusInterface,
-                                   "RecordAdded", this, SLOT(onReceivedRecord(const QString &)));
     }
     if (!valid) {
         qWarning(notifyLog) << "NotifyConnection is invalid, and can't receive RecordAdded signal.";
+        return false;
     }
 
     return true;

panels/notification/center/notificationcenterpanel.cpp

@@ -139,35 +139,21 @@ QStringList NotifyAccessor::fetchApps(int maxCount) const
 void NotifyAccessor::removeEntity(qint64 id)
 {
     qDebug(notifyLog) << "Remove notify" << id;
-    if (m_dataUpdater) {
-        QMetaObject::invokeMethod(m_dataUpdater, "removeNotification", Qt::DirectConnection,
-                                  Q_ARG(qint64, id));
-    } else {
-        m_accessor->removeEntity(id);
-    }
+    QMetaObject::invokeMethod(m_dataUpdater, "removeNotification", Qt::DirectConnection, Q_ARG(qint64, id));
 }
 
 void NotifyAccessor::removeEntityByApp(const QString &appName)
 {
     qDebug(notifyLog) << "Remove notifies for the application" << appName;
 
-    if (m_dataUpdater) {
-        QMetaObject::invokeMethod(m_dataUpdater, "removeNotifications", Qt::DirectConnection,
-                                  Q_ARG(const QString &, appName));
-    } else {
-        m_accessor->removeEntityByApp(appName);
-    }
+    QMetaObject::invokeMethod(m_dataUpdater, "removeNotifications", Qt::DirectConnection, Q_ARG(const QString &, appName));
 }
 
 void NotifyAccessor::clear()
 {
     qDebug(notifyLog) << "Remove all notify";
 
-    if (m_dataUpdater) {
-        QMetaObject::invokeMethod(m_dataUpdater, "removeNotifications", Qt::DirectConnection);
-    } else {
-        m_accessor->clear();
-    }
+    QMetaObject::invokeMethod(m_dataUpdater, "removeNotifications", Qt::DirectConnection);
 }
 
 void NotifyAccessor::closeNotify(const NotifyEntity &entity, NotifyEntity::ClosedReason reason)
@@ -240,34 +226,6 @@ void NotifyAccessor::openNotificationSetting()
     }
 }
 
-void NotifyAccessor::addNotify(const QString &appName, const QString &content)
-{
-    qDebug(notifyLog) << "Add notify" << appName;
-    static int id = 10000;
-    NotifyEntity entity(id++, appName);
-    entity.setBody(content);
-    m_accessor->addEntity(entity);
-
-    if (auto entity = fetchLastEntity(appName); entity.isValid()) {
-        entityReceived(entity.id());
-    }
-}
-
-void NotifyAccessor::fetchDataInfo()
-{
-    QStringList info;
-    auto entityCount = fetchEntityCount(DataAccessor::AllApp());
-    auto apps = fetchApps();
-    info.append(QString("notifyCount: %1, appCount: %2").arg(entityCount).arg(apps.size()));
-    for (auto item : apps) {
-        info.append(QString("%1 -> %2").arg(item).arg(fetchEntityCount(item)));
-    }
-    QString ret = info.join("\n");
-    m_dataInfo = ret;
-    dataInfoChanged();
-    appsChanged();
-}
-
 void NotifyAccessor::onNotificationStateChanged(qint64 id, int processedType)
 {
     if (!enabled())
@@ -285,16 +243,6 @@ void NotifyAccessor::onReceivedRecord(const QString &id)
     emit entityReceived(id.toLongLong());
 }
 
-QString NotifyAccessor::dataInfo() const
-{
-    return m_dataInfo;
-}
-
-QStringList NotifyAccessor::apps() const
-{
-    return m_apps;
-}
-
 bool NotifyAccessor::debugging() const
 {
     return m_debugging;

panels/notification/center/notifyaccessor.cpp

@@ -25,8 +25,6 @@ class NotifyAccessor : public QObject
     Q_OBJECT
     QML_ELEMENT
     QML_SINGLETON
-    Q_PROPERTY(QString dataInfo READ dataInfo NOTIFY dataInfoChanged FINAL)
-    Q_PROPERTY(QStringList apps READ apps NOTIFY appsChanged FINAL)
     Q_PROPERTY(bool debugging READ debugging NOTIFY debuggingChanged)
 public:
     static NotifyAccessor *instance();
@@ -60,13 +58,6 @@ class NotifyAccessor : public QObject
     void stagingEntityReceived(qint64 id);
     void stagingEntityClosed(qint64 id);
 
-public slots:
-    void addNotify(const QString &appName, const QString &content);
-    void fetchDataInfo();
-
-signals:
-    void dataInfoChanged();
-    void appsChanged();
     void debuggingChanged();
 
 private slots:
@@ -76,17 +67,13 @@ private slots:
 private:
     explicit NotifyAccessor(QObject *parent = nullptr);
 
-    QString dataInfo() const;
-    QStringList apps() const;
     bool debugging() const;
 
 private:
     DataAccessor *m_accessor = nullptr;
     QObject *m_dataUpdater = nullptr;
     QStringList m_pinnedApps;
-    QStringList m_apps;
     bool m_debugging = false;
-    QString m_dataInfo;
     bool m_enabled = false;
 };
 }

panels/notification/center/notifyaccessor.h

@@ -173,7 +173,11 @@ bool DBAccessor::open(const QString &dataPath)
 bool DBAccessor::isValid() const
 {
     QMutexLocker locker(&m_mutex);
-    return !m_connection.lastError().isValid();
+    if (m_connection.lastError().isValid()) {
+        qWarning(notifyLog) << "Database error" << m_connection.lastError().text();
+        return false;
+    }
+    return true;
 }
 
 qint64 DBAccessor::addEntity(const NotifyEntity &entity)

panels/notification/common/dbaccessor.cpp

@@ -180,15 +180,9 @@ uint NotificationManager::Notify(const QString &appName, uint replacesId, const
                                  const QString &body, const QStringList &actions, const QVariantMap &hints,
                                  int expireTimeout)
 {
-    qDebug(notifyLog) << "Notify"
-            << ", appName:" << appName
-            << ", summary:" << summary
-            << ", appIcon:" << appIcon
-            << ", body size:" << body.size()
-            << ", actions:" << actions
-            << ", hint: " << hints
-            << ", replaceId:" << replacesId
-            << ", expireTimeout:" << expireTimeout;
+    qInfo(notifyLog) << "Notify"
+                     << ", appName:" << appName << ", summary:" << summary << ", appIcon:" << appIcon << ", body size:" << body.size()
+                     << ", actions:" << actions << ", hint: " << hints << ", replaceId:" << replacesId << ", expireTimeout:" << expireTimeout;
 
     if (calledFromDBus() && m_setting->systemValue(NotificationSetting::CloseNotification).toBool()) {
         qDebug(notifyLog) << "Notify has been disabled by CloseNotification setting.";