Skip to content
GitLab
Explore
Sign in
Primary navigation
Search or go to…
Project
DLCM-Backend
Manage
Activity
Members
Labels
Plan
Issues
Issue boards
Milestones
Code
Merge requests
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Deploy
Releases
Help
Help
Support
GitLab documentation
Compare GitLab plans
Community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
DLCM
Community
DLCM-Backend
Merge requests
!1178
fix: correct to avoid NPE + add missing message
Code
Review changes
Check out branch
Download
Patches
Plain diff
Closed
fix: correct to avoid NPE + add missing message
hbo_fix_notifcation_service_missing_message
into
master
Overview
1
Commits
1
Changes
2
Closed
Homada.Boumedane
requested to merge
hbo_fix_notifcation_service_missing_message
into
master
1 year ago
Overview
1
Commits
1
Changes
2
Expand
refactor code to avoid NPE + add missing message.
0
0
Merge request reports
Compare
master
master (base)
and
latest version
latest version
ee382a93
1 commit,
1 year ago
2 files
+
3
−
1
Inline
Compare changes
Side-by-side
Inline
Show whitespace changes
Show one file at a time
Files
2
Search (e.g. *.vue) (Ctrl+P)
DLCM-Admin/src/main/java/ch/dlcm/business/NotificationService.java
+
2
−
1
Options
@@ -240,7 +240,8 @@ public class NotificationService extends ResourceService<Notification> {
SolidifyEventPublisher
.
getPublisher
().
publishEvent
(
Objects
.
requireNonNull
(
emailMessage
));
}
}
if
(
item
.
getNotificationType
().
getNotificationCategory
().
equals
(
NotificationCategory
.
INFO
))
{
if
(
item
.
getNotificationType
().
getNotificationCategory
()
!=
null
&&
item
.
getNotificationType
().
getNotificationCategory
()
.
equals
(
NotificationCategory
.
INFO
))
{
item
.
setNotificationStatus
(
NotificationStatus
.
NON_APPLICABLE
);
}
return
super
.
save
(
item
);
Loading