Use one single definition of the date/time format in the config file Change-Id: Ieccde4467405fbef85ab94a77cb339f72ce8ae36
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 c04b9d9..5fbbed1 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/GetMessage.java +++ b/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/GetMessage.java
@@ -33,7 +33,7 @@ private static final String KEY_STARTS_AT = "startsAt"; private static final String KEY_EXPIRES_AT = "expiresAt"; - private static final DateTimeFormatter DATE_FORMAT = DateTimeFormatter.ofPattern("yyyyMMdd:HHmm"); + static final DateTimeFormatter DATE_FORMAT = DateTimeFormatter.ofPattern("yyyyMMdd:HHmm"); private static final DateTimeFormatter REST_RESPONSE_FORMAT = DateTimeFormatter.ofPattern("MMM d, yyyy h:mm:ss a", Locale.ENGLISH);
diff --git a/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/SetMessage.java b/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/SetMessage.java index 3fc8898..ad36a09 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/SetMessage.java +++ b/src/main/java/com/googlesource/gerrit/plugins/messageoftheday/SetMessage.java
@@ -14,6 +14,8 @@ package com.googlesource.gerrit.plugins.messageoftheday; +import static com.googlesource.gerrit.plugins.messageoftheday.GetMessage.DATE_FORMAT; + import com.google.common.base.Strings; import com.google.common.flogger.FluentLogger; import com.google.gerrit.extensions.restapi.AuthException; @@ -104,11 +106,7 @@ + INPUT_DATE_FORMAT_PATTERN + "' format"); } - cfg.setString( - SECTION_MESSAGE, - null, - KEY_EXPIRES_AT, - time.format(DateTimeFormatter.ofPattern("yyyyMMdd:HHmm"))); + cfg.setString(SECTION_MESSAGE, null, KEY_EXPIRES_AT, time.format(DATE_FORMAT)); } else { String expiredAt = cfg.getString(SECTION_MESSAGE, null, KEY_EXPIRES_AT); if (expiredAt == null) {