diff --git a/Sources/Shared/Views/ScheduleView/Components/ScheduleDetailView.swift b/Sources/Shared/Views/ScheduleView/Components/ScheduleDetailView.swift index 5ecd1d9..6a7f965 100644 --- a/Sources/Shared/Views/ScheduleView/Components/ScheduleDetailView.swift +++ b/Sources/Shared/Views/ScheduleView/Components/ScheduleDetailView.swift @@ -54,8 +54,8 @@ struct ScheduleDetailView: View { //1st or 2nd lunch revolving periods, 2nd out of 3 UI sections var lunchPeriods: LunchBlock? { let periods = scheduleDay?.periods ?? [] - let firstIndex = scheduleDay?.periods?.firstIndex{$0.periodCategory.isLunchRevolving} - let lastIndex = scheduleDay?.periods?.lastIndex{$0.periodCategory.isLunchRevolving} //Last instance 1st/2nd nutrition block + let firstIndex = scheduleDay?.periods.firstIndex{$0.periodCategory.isLunchRevolving} + let lastIndex = scheduleDay?.periods.lastIndex{$0.periodCategory.isLunchRevolving} //Last instance 1st/2nd nutrition block if let firstIndex = firstIndex, let lastIndex = lastIndex { let lunchPeriods = Array(periods[firstIndex...lastIndex]) @@ -66,12 +66,9 @@ struct ScheduleDetailView: View { //Periods after lunch, 3rd out of 3 UI sections var postLunchPeriods: [ClassPeriod] { - let lastIndex = scheduleDay?.periods?.lastIndex{$0.periodCategory.isLunchRevolving} - if let lastIndex = lastIndex, - let scheduleDay = scheduleDay, - let removingPeriod8 = scheduleDay.periods?.filter({$0.periodNumber != 8}) - { - + let lastIndex = scheduleDay?.periods.lastIndex{$0.periodCategory.isLunchRevolving} + if let lastIndex = lastIndex, let scheduleDay = scheduleDay { + let removingPeriod8 = scheduleDay.periods.filter {$0.periodNumber != 8} //lastIndex + 1 to shorten array, remove unwanted return Array(removingPeriod8.suffix(from: lastIndex + 1)) } @@ -79,7 +76,7 @@ struct ScheduleDetailView: View { } var period8: ClassPeriod? { - scheduleDay?.periods?.filter {$0.periodNumber == 8}.first + scheduleDay?.periods.filter {$0.periodNumber == 8}.first } var scheduleDateDescription: String { @@ -93,7 +90,7 @@ struct ScheduleDetailView: View { var shouldFallback: Bool { return userSettings.preferLegacySchedule || developerScheduleOn || - scheduleDay?.periods?.isEmpty ?? true + scheduleDay?.periods.isEmpty ?? true } var horizontalPadding = true