DMC/scipro
7
0
Fork 0
Code Issues 31 Pull Requests 3 Actions Projects 1 Wiki Activity

Fix some remaining issues from the Spring migration #20

Merged
niat8586 merged 14 commits from release-firefighting into develop 2024-11-20 12:56:22 +01:00
Conversation 1 Commits 14 Files Changed 14 +2
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit d4a8f411ba - Show all commits

2
core/src/main/java/se/su/dsv/scipro/notifications/NotificationServiceImpl.java
View File

@ -50,6 +50,7 @@ public class NotificationServiceImpl extends AbstractServiceImpl<Notification,Lo
} }
@Override @Override
@Transactional
public void setRead(User user, NotificationEvent notificationEvent, boolean read) { public void setRead(User user, NotificationEvent notificationEvent, boolean read) {
Iterable<Notification> notifications = findAll( Iterable<Notification> notifications = findAll(
QNotification.notification.notificationEvent.eq(notificationEvent) QNotification.notification.notificationEvent.eq(notificationEvent)
@ -62,6 +63,7 @@ public class NotificationServiceImpl extends AbstractServiceImpl<Notification,Lo
} }
@Override @Override
@Transactional
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
// This looks so wrong but a List<Notification> is what we actually get // This looks so wrong but a List<Notification> is what we actually get
// during runtime, see SCIPRO-167935. Reflection so awesome... // during runtime, see SCIPRO-167935. Reflection so awesome...