Fix NullPointerExceptions
Change-Id: Ia92f5973beb2d7187f6e04c9f5caf561482576f3
diff --git a/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/GetMessage.java b/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/GetMessage.java
index 85cb1b1..255bb90 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/GetMessage.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/GetMessage.java
@@ -74,14 +74,14 @@
motd.id = cfg.getString(SECTION_MESSAGE, null, KEY_ID);
if (Strings.isNullOrEmpty(motd.id)) {
log.warn("id not defined, no message will be shown");
- return null;
+ return Response.none();
}
try {
motd.expiresAt = DATE_FORMAT.parse(cfg.getString(SECTION_MESSAGE, null, KEY_EXPIRES_AT));
} catch (ParseException | NullPointerException e) {
log.warn("expiresAt not defined, no message will be shown");
- return null;
+ return Response.none();
}
try {
@@ -90,7 +90,7 @@
log.warn(
String.format(
"No HTML-file was found for message %s, no message will be shown", motd.id));
- return null;
+ return Response.none();
}
try {
@@ -101,7 +101,7 @@
}
if (motd.startsAt.compareTo(new Date()) > 0 || motd.expiresAt.compareTo(new Date()) < 0) {
- return null;
+ return Response.none();
}
motd.redisplay = getRedisplay();