Merge branch 'Plasma/5.16'

applets/notifications/package/contents/ui/FullRepresentation.qml

@@ -399,6 +399,7 @@ ColumnLayout{
                     NotificationHeader {
                         applicationName: model.applicationName
                         applicationIconSource: model.applicationIconName
+                        originName: model.originName || ""
 
                         // don't show timestamp for group
 

libnotificationmanager/notificationgroupingproxymodel.cpp

@@ -43,8 +43,12 @@ bool NotificationGroupingProxyModel::appsMatch(const QModelIndex &a, const QMode
     const QString aDesktopEntry = a.data(Notifications::DesktopEntryRole).toString();
     const QString bDesktopEntry = b.data(Notifications::DesktopEntryRole).toString();
 
+    const QString aOriginName = a.data(Notifications::OriginNameRole).toString();
+    const QString bOriginName = a.data(Notifications::OriginNameRole).toString();
+
     return !aName.isEmpty() && aName == bName
-            && aDesktopEntry == bDesktopEntry;
+            && aDesktopEntry == bDesktopEntry
+            && aOriginName == bOriginName;
 }
 
 bool NotificationGroupingProxyModel::isGroup(int row) const
@@ -471,23 +475,16 @@ QVariant NotificationGroupingProxyModel::data(const QModelIndex &proxyIndex, int
             return false;
 
         case Notifications::DesktopEntryRole:
-            for (int i = 0; i < rowCount(proxyIndex); ++i) {
-                const QString desktopEntry = index(i, 0, proxyIndex).data(Notifications::DesktopEntryRole).toString();
-                if (!desktopEntry.isEmpty()) {
-                    return desktopEntry;
-                }
-            }
-            return QString();
         case Notifications::NotifyRcNameRole:
+        case Notifications::OriginNameRole:
             for (int i = 0; i < rowCount(proxyIndex); ++i) {
-                const QString notifyRcName = index(i, 0, proxyIndex).data(Notifications::NotifyRcNameRole).toString();
+                const QString stringData = index(i, 0, proxyIndex).data(role).toString();
-                if (!notifyRcName.isEmpty()) {
+                if (!stringData.isEmpty()) {
-                    return notifyRcName;
+                    return stringData;
                 }
             }
             return QString();
 
-
         case Notifications::ConfigurableRole: // if there is any configurable child item
             for (int i = 0; i < rowCount(proxyIndex); ++i) {
                 if (index(i, 0, proxyIndex).data(Notifications::ConfigurableRole).toBool()) {