Validate notification fire date before scheduling

[ivan-magda]

Задача: #APPS-2165

Описание:
Добавили проверку даты перед планированием нотификации.
Из отызва пользователя:

При запуске приложения непрерывно выпадают сверху напоминания от всех активных курсов и модулей.

[kvld]

Без отрицания?

guard self.isFireDateValid(contentProvider.fireDate) else {
    return Promise(error: Error.badFireDate) 
}

[kvld]

Кстати, возможно, стоит прокомментировать, что localNotificationScheduleNotification возвращает промис, чтобы была совместимость с асинхронным методом в iOS 10.0+
Или написать тело так (кажется, так чуть понятнее):

return Promise { seal in
    guard self.isFireDateValid(contentProvider.fireDate) else {
        throw Error.badFireDate
    }

    // ...

    seal.fulfill(())
}

[kvld]

^ потому что приходится смотреть туда, где этот метод вызывается, так как непонятно, зачем там синхронный код обернут в промис

[ivan-magda]

Да, для разных версий системы разное апи. Смотри как пытаются добиться единообразия https://stackoverflow.com/a/51052127

[kvld]

Аналогично, отрицание

[kvld]

Возможно, так понятнее

let nextTriggerDate: Date? = {
    switch notificationTrigger {
    case let timeIntervalTrigger as UNTimeIntervalNotificationTrigger:
        return timeIntervalTrigger.nextTriggerDate()
    case let calendarTrigger as UNCalendarNotificationTrigger:
        return calendarTrigger.nextTriggerDate()
    default:
        return nil
    }
}()