From 93aae20f85bc9390082e95e84b019dcaf5880693 Mon Sep 17 00:00:00 2001
From: brreynie <bram.reyniers@ugent.be>
Date: Sat, 13 May 2023 13:22:12 +0200
Subject: [PATCH] #109 archiving subject archives all tasks of subject

---
 .../java/be/ugent/sel/studeez/domain/SubjectDAO.kt |  2 ++
 .../domain/implementation/FireBaseSubjectDAO.kt    | 14 ++++++++++++++
 .../screens/subjects/form/SubjectFormScreen.kt     |  7 ++++++-
 .../screens/subjects/form/SubjectFormViewModel.kt  |  6 ++++--
 4 files changed, 26 insertions(+), 3 deletions(-)

diff --git a/app/src/main/java/be/ugent/sel/studeez/domain/SubjectDAO.kt b/app/src/main/java/be/ugent/sel/studeez/domain/SubjectDAO.kt
index c9f6a8a..d887ef5 100644
--- a/app/src/main/java/be/ugent/sel/studeez/domain/SubjectDAO.kt
+++ b/app/src/main/java/be/ugent/sel/studeez/domain/SubjectDAO.kt
@@ -13,6 +13,8 @@ interface SubjectDAO {
 
     fun updateSubject(newSubject: Subject)
 
+    suspend fun archiveSubject(subject: Subject)
+
     fun getTaskCount(subject: Subject): Flow<Int>
     fun getCompletedTaskCount(subject: Subject): Flow<Int>
     fun getStudyTime(subject: Subject): Flow<Int>
diff --git a/app/src/main/java/be/ugent/sel/studeez/domain/implementation/FireBaseSubjectDAO.kt b/app/src/main/java/be/ugent/sel/studeez/domain/implementation/FireBaseSubjectDAO.kt
index e8931ed..66815dc 100644
--- a/app/src/main/java/be/ugent/sel/studeez/domain/implementation/FireBaseSubjectDAO.kt
+++ b/app/src/main/java/be/ugent/sel/studeez/domain/implementation/FireBaseSubjectDAO.kt
@@ -1,8 +1,10 @@
 package be.ugent.sel.studeez.domain.implementation
 
+import android.util.Log
 import be.ugent.sel.studeez.data.local.models.task.Subject
 import be.ugent.sel.studeez.data.local.models.task.SubjectDocument
 import be.ugent.sel.studeez.data.local.models.task.Task
+import be.ugent.sel.studeez.data.local.models.task.TaskDocument
 import be.ugent.sel.studeez.domain.AccountDAO
 import be.ugent.sel.studeez.domain.SubjectDAO
 import be.ugent.sel.studeez.domain.TaskDAO
@@ -45,6 +47,18 @@ class FireBaseSubjectDAO @Inject constructor(
         currentUserSubjectsCollection().document(newSubject.id).set(newSubject)
     }
 
+    override suspend fun archiveSubject(subject: Subject) {
+        currentUserSubjectsCollection().document(subject.id).update(SubjectDocument.archived, true)
+        currentUserSubjectsCollection().document(subject.id)
+            .collection(FireBaseCollections.TASK_COLLECTION)
+            .taskNotArchived()
+            .get().await()
+            .documents
+            .forEach {
+                it.reference.update(TaskDocument.archived, true)
+            }
+    }
+
     override fun getTaskCount(subject: Subject): Flow<Int> {
         return taskDAO.getTasks(subject)
             .map(List<Task>::count)
diff --git a/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormScreen.kt b/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormScreen.kt
index 19e6816..bd81c8c 100644
--- a/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormScreen.kt
+++ b/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormScreen.kt
@@ -6,6 +6,7 @@ import androidx.compose.material.OutlinedTextField
 import androidx.compose.material.Text
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.getValue
+import androidx.compose.runtime.rememberCoroutineScope
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.graphics.Color
 import androidx.compose.ui.res.stringResource
@@ -16,6 +17,7 @@ import be.ugent.sel.studeez.common.composable.SecondaryScreenTemplate
 import be.ugent.sel.studeez.common.ext.basicButton
 import be.ugent.sel.studeez.common.ext.fieldModifier
 import be.ugent.sel.studeez.resources
+import kotlinx.coroutines.launch
 import be.ugent.sel.studeez.R.string as AppText
 
 @Composable
@@ -42,6 +44,7 @@ fun SubjectEditRoute(
     viewModel: SubjectEditFormViewModel,
 ) {
     val uiState by viewModel.uiState
+    val coroutineScope = rememberCoroutineScope()
     SubjectForm(
         title = AppText.edit_subject,
         goBack = goBack,
@@ -51,7 +54,9 @@ fun SubjectEditRoute(
         onColorChange = {},
     ) {
         DeleteButton(text = AppText.delete_subject) {
-            viewModel.onDelete(openAndPopUp)
+            coroutineScope.launch {
+                viewModel.onDelete(openAndPopUp)
+            }
         }
     }
 }
diff --git a/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormViewModel.kt b/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormViewModel.kt
index 7c2f601..84162d0 100644
--- a/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormViewModel.kt
+++ b/app/src/main/java/be/ugent/sel/studeez/screens/subjects/form/SubjectFormViewModel.kt
@@ -6,6 +6,7 @@ import be.ugent.sel.studeez.data.SelectedSubject
 import be.ugent.sel.studeez.data.local.models.task.Subject
 import be.ugent.sel.studeez.domain.LogService
 import be.ugent.sel.studeez.domain.SubjectDAO
+import be.ugent.sel.studeez.domain.TaskDAO
 import be.ugent.sel.studeez.navigation.StudeezDestinations
 import be.ugent.sel.studeez.screens.StudeezViewModel
 import dagger.hilt.android.lifecycle.HiltViewModel
@@ -59,6 +60,7 @@ class SubjectCreateFormViewModel @Inject constructor(
 @HiltViewModel
 class SubjectEditFormViewModel @Inject constructor(
     subjectDAO: SubjectDAO,
+    private val taskDAO: TaskDAO,
     selectedSubject: SelectedSubject,
     logService: LogService,
 ) : SubjectFormViewModel(subjectDAO, selectedSubject, logService) {
@@ -69,8 +71,8 @@ class SubjectEditFormViewModel @Inject constructor(
         )
     )
 
-    fun onDelete(openAndPopUp: (String, String) -> Unit) {
-        subjectDAO.updateSubject(selectedSubject().copy(archived = true))
+    suspend fun onDelete(openAndPopUp: (String, String) -> Unit) {
+        subjectDAO.archiveSubject(selectedSubject())
         openAndPopUp(StudeezDestinations.SUBJECT_SCREEN, StudeezDestinations.EDIT_SUBJECT_FORM)
     }