From d4a8f411ba4547bc8d1c7194d08ad7c3180d04ed Mon Sep 17 00:00:00 2001 From: Andreas Svanberg <andreass@dsv.su.se> Date: Wed, 13 Nov 2024 14:51:16 +0100 Subject: [PATCH] Added missing @Transactional --- .../se/su/dsv/scipro/notifications/NotificationServiceImpl.java | 2 ++ 1 file changed, 2 insertions(+) diff --git a/core/src/main/java/se/su/dsv/scipro/notifications/NotificationServiceImpl.java b/core/src/main/java/se/su/dsv/scipro/notifications/NotificationServiceImpl.java index 6c460a0741..5bbe68093e 100755 --- a/core/src/main/java/se/su/dsv/scipro/notifications/NotificationServiceImpl.java +++ b/core/src/main/java/se/su/dsv/scipro/notifications/NotificationServiceImpl.java @@ -50,6 +50,7 @@ public class NotificationServiceImpl extends AbstractServiceImpl<Notification,Lo } @Override + @Transactional public void setRead(User user, NotificationEvent notificationEvent, boolean read) { Iterable<Notification> notifications = findAll( QNotification.notification.notificationEvent.eq(notificationEvent) @@ -62,6 +63,7 @@ public class NotificationServiceImpl extends AbstractServiceImpl<Notification,Lo } @Override + @Transactional @SuppressWarnings("unchecked") // This looks so wrong but a List<Notification> is what we actually get // during runtime, see SCIPRO-167935. Reflection so awesome...