From b80f14cc217ee1a0e00725e084624654d8b756a9 Mon Sep 17 00:00:00 2001 From: Alex Baker Date: Fri, 16 Oct 2020 10:55:03 -0500 Subject: [PATCH] Fetch all tasks to reschedule at once --- .../java/org/tasks/dialogs/DateTimePicker.kt | 28 +++++++++---------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/app/src/main/java/org/tasks/dialogs/DateTimePicker.kt b/app/src/main/java/org/tasks/dialogs/DateTimePicker.kt index e15225f40..f75c17dc1 100644 --- a/app/src/main/java/org/tasks/dialogs/DateTimePicker.kt +++ b/app/src/main/java/org/tasks/dialogs/DateTimePicker.kt @@ -272,20 +272,20 @@ class DateTimePicker : BottomSheetDialogFragment() { targetFragment?.onActivityResult(targetRequestCode, RESULT_OK, intent) } else { lifecycleScope.launch(NonCancellable) { - taskIds.forEach { taskId -> - taskDao.fetch(taskId)?.let { - it.setDueDateAdjustingHideUntil(when { - selectedDay == MULTIPLE_DAYS -> - it.dueDate.toDateTime().withMillisOfDay(selectedTime).millis - selectedDay == NO_DAY -> 0L - selectedTime == MULTIPLE_TIMES -> - selectedDay.toDateTime().withMillisOfDay(it.dueDate.millisOfDay()).millis - selectedTime == NO_TIME -> selectedDay - else -> selectedDay.toDateTime().withMillisOfDay(selectedTime).millis - }) - taskDao.save(it) - } - } + taskDao + .fetch(taskIds.toList()) + .forEach { + it.setDueDateAdjustingHideUntil(when { + selectedDay == MULTIPLE_DAYS -> + it.dueDate.toDateTime().withMillisOfDay(selectedTime).millis + selectedDay == NO_DAY -> 0L + selectedTime == MULTIPLE_TIMES -> + selectedDay.toDateTime().withMillisOfDay(it.dueDate.millisOfDay()).millis + selectedTime == NO_TIME -> selectedDay + else -> selectedDay.toDateTime().withMillisOfDay(selectedTime).millis + }) + taskDao.save(it) + } } } }