fix: Use Notification Service Layer rather than Storage Layer - MEED-7279 - Meeds-io/MIPs#134

boubaker · boubaker · commit f457a51829e1 · 2024-07-23T17:12:39.000+01:00
diff --git a/component/notification/src/main/java/org/exoplatform/social/notification/impl/RelationshipNotificationImpl.java b/component/notification/src/main/java/org/exoplatform/social/notification/impl/RelationshipNotificationImpl.java
@@ -129,7 +129,6 @@ private void updateNotification(NotificationInfo notification) {
         notification.setRead(false);
         notification.setOnPopOver(true);
         getWebNotificationService().save(notification);
-        WebNotificationSender.sendJsonMessage(notification.getTo(), new MessageInfo());
       } catch (Exception e) {
         LOG.error("Can not update relationship notification.", e.getMessage());
       }
diff --git a/component/notification/src/main/java/org/exoplatform/social/notification/impl/SpaceNotificationImpl.java b/component/notification/src/main/java/org/exoplatform/social/notification/impl/SpaceNotificationImpl.java
@@ -148,7 +148,6 @@ private void updateNotification(NotificationInfo notification) {
         notification.setRead(false);
         notification.setOnPopOver(true);
         getWebNotificationService().save(notification);
-        WebNotificationSender.sendJsonMessage(notification.getTo(), new MessageInfo());
       } catch (Exception e) {
         LOG.error("Can not update space invitation notification.", e.getMessage());
       }
diff --git a/component/notification/src/main/java/org/exoplatform/social/notification/plugin/SocialNotificationUtils.java b/component/notification/src/main/java/org/exoplatform/social/notification/plugin/SocialNotificationUtils.java

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,6 @@ private void updateNotification(NotificationInfo notification) {`
`129`	`129`	`notification.setRead(false);`
`130`	`130`	`notification.setOnPopOver(true);`
`131`	`131`	`getWebNotificationService().save(notification);`
`132`		`- WebNotificationSender.sendJsonMessage(notification.getTo(), new MessageInfo());`
`133`	`132`	`} catch (Exception e) {`
`134`	`133`	`LOG.error("Can not update relationship notification.", e.getMessage());`
`135`	`134`	`}`
Original file line number	Diff line number	Diff line change
`@@ -148,7 +148,6 @@ private void updateNotification(NotificationInfo notification) {`
`148`	`148`	`notification.setRead(false);`
`149`	`149`	`notification.setOnPopOver(true);`
`150`	`150`	`getWebNotificationService().save(notification);`
`151`		`- WebNotificationSender.sendJsonMessage(notification.getTo(), new MessageInfo());`
`152`	`151`	`} catch (Exception e) {`
`153`	`152`	`LOG.error("Can not update space invitation notification.", e.getMessage());`
`154`	`153`	`}`