Upstream/Plan
1
0
Fork 0
mirror of https://github.com/plan-player-analytics/Plan.git synced 2024-09-16 08:46:33 +02:00
Code Issues Projects Releases Wiki Activity

Fixed TimeAmountFormatter 0s formatting

Browse Source
This commit is contained in:
Rsl1122 2019-07-19 13:02:45 +03:00
parent 93600b74ed
commit ad7f8c5631
3 changed files with 14 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Plan/common/src/main/java/com/djrapitops/plan/utilities/formatting/time/TimeAmountFormatter.java
View File

@ -107,6 +107,7 @@ public class TimeAmountFormatter implements Formatter<Long> {
}
private void appendSeconds(StringBuilder builder, long seconds, long minutes, long hours, String fHours, String fMinutes, String fSeconds) {
if (seconds != 0 || fSeconds.contains(ZERO_PH)) {
String s = fSeconds.replace(SECONDS_PH, String.valueOf(seconds));
if (minutes == 0 && s.contains(MINUTES_PH)) {
if (hours == 0 && fMinutes.contains(HOURS_PH)) {
@ -120,6 +121,7 @@ public class TimeAmountFormatter implements Formatter<Long> {
}
builder.append(s);
}
}
private void appendDays(StringBuilder builder, long days) {
String singular = config.get(FormatSettings.DAY);

2
Plan/common/src/test/java/com/djrapitops/plan/data/store/mutators/formatting/TimeAmountFormatterDefaultTest.java
View File

@ -114,7 +114,7 @@ class TimeAmountFormatterDefaultTest {
@Test
void exampleZero() {
String expected = "-";
String expected = "0s";
long ms = 0L;
String result = underTest.apply(ms);

2
Plan/common/src/test/java/com/djrapitops/plan/data/store/mutators/formatting/TimeAmountFormatterExtraZerosTest.java
View File

@ -112,7 +112,7 @@ class TimeAmountFormatterExtraZerosTest {
@Test
void exampleZero() {
String expected = "-";
String expected = "00:00:00";
long ms = 0L;
String result = underTest.apply(ms);