Skip to content

Commit 452cd40

Browse files
authored
fix: don’t treat future months as later this week (#1542)
1 parent e351751 commit 452cd40

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

shared/src/commonMain/kotlin/com/mbta/tid/mbta_app/model/AlertSummary.kt

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -187,9 +187,11 @@ public data class AlertSummary(
187187
return Timeframe.LaterDate(endTime)
188188
}
189189

190-
private fun laterThisWeek(onDate: LocalDate, endDate: LocalDate): Boolean =
191-
onDate.dayOfWeek.isoDayNumber < endDate.dayOfWeek.isoDayNumber &&
192-
endDate.minus(onDate).days < 7
190+
private fun laterThisWeek(onDate: LocalDate, endDate: LocalDate): Boolean {
191+
if (onDate.dayOfWeek.isoDayNumber >= endDate.dayOfWeek.isoDayNumber) return false
192+
val difference = endDate.minus(onDate)
193+
return difference.years == 0 && difference.months == 0 && difference.days < 7
194+
}
193195

194196
private fun alertLocation(
195197
alert: Alert,

0 commit comments

Comments
 (0)