merge dev

app/src/main/java/be/ugent/sel/studeez/screens/home/HomeScreen.kt

@@ -11,26 +11,21 @@ import be.ugent.sel.studeez.R
 import be.ugent.sel.studeez.common.composable.BasicButton
 import be.ugent.sel.studeez.common.composable.PrimaryScreenTemplate
 import be.ugent.sel.studeez.common.composable.drawer.DrawerActions
-import be.ugent.sel.studeez.common.composable.drawer.DrawerViewModel
-import be.ugent.sel.studeez.common.composable.drawer.getDrawerActions
 import be.ugent.sel.studeez.common.composable.navbar.NavigationBarActions
-import be.ugent.sel.studeez.common.composable.navbar.NavigationBarViewModel
-import be.ugent.sel.studeez.common.composable.navbar.getNavigationBarActions
 import be.ugent.sel.studeez.common.ext.basicButton
 import be.ugent.sel.studeez.resources
 
 @Composable
 fun HomeRoute(
     open: (String) -> Unit,
-    openAndPopUp: (String, String) -> Unit,
     viewModel: HomeViewModel,
-    drawerViewModel: DrawerViewModel,
-    navBarViewModel: NavigationBarViewModel,
+    drawerActions: DrawerActions,
+    navigationBarActions: NavigationBarActions,
 ) {
     HomeScreen(
         onStartSessionClick = { viewModel.onStartSessionClick(open) },
-        drawerActions = getDrawerActions(drawerViewModel, open, openAndPopUp),
-        navigationBarActions = getNavigationBarActions(navBarViewModel, open),
+        drawerActions = drawerActions,
+        navigationBarActions = navigationBarActions,
     )
 }
 
@@ -40,12 +35,11 @@ fun HomeScreen(
     drawerActions: DrawerActions,
     navigationBarActions: NavigationBarActions,
 ) {
-
     PrimaryScreenTemplate(
         title = resources().getString(R.string.home),
         drawerActions = drawerActions,
         navigationBarActions = navigationBarActions,
-        action = { FriendsAction() }
+        barAction = { FriendsAction() }
     ) {
         BasicButton(R.string.start_session, Modifier.basicButton()) {
             onStartSessionClick()
@@ -69,6 +63,6 @@ fun HomeScreenPreview() {
     HomeScreen(
         onStartSessionClick = {},
         drawerActions = DrawerActions({}, {}, {}, {}, {}),
-        navigationBarActions = NavigationBarActions({}, {}, {}, {})
+        navigationBarActions = NavigationBarActions({ false }, {}, {}, {}, {})
     )
 }

app/src/main/java/be/ugent/sel/studeez/screens/profile/ProfileScreen.kt

@@ -11,15 +11,12 @@ import androidx.compose.runtime.mutableStateOf
 import androidx.compose.runtime.remember
 import androidx.compose.runtime.setValue
 import androidx.compose.ui.tooling.preview.Preview
-import androidx.hilt.navigation.compose.hiltViewModel
 import be.ugent.sel.studeez.R
 import be.ugent.sel.studeez.common.composable.Headline
 import be.ugent.sel.studeez.common.composable.PrimaryScreenTemplate
-import be.ugent.sel.studeez.resources
 import be.ugent.sel.studeez.common.composable.drawer.DrawerActions
-import be.ugent.sel.studeez.common.composable.drawer.getDrawerActions
 import be.ugent.sel.studeez.common.composable.navbar.NavigationBarActions
-import be.ugent.sel.studeez.common.composable.navbar.getNavigationBarActions
+import be.ugent.sel.studeez.resources
 import kotlinx.coroutines.CoroutineScope
 import be.ugent.sel.studeez.R.string as AppText
 
@@ -41,13 +38,14 @@ fun getProfileActions(
 @Composable
 fun ProfileRoute(
     open: (String) -> Unit,
-    openAndPopUp: (String, String) -> Unit,
     viewModel: ProfileViewModel,
+    drawerActions: DrawerActions,
+    navigationBarActions: NavigationBarActions,
 ) {
     ProfileScreen(
         profileActions = getProfileActions(viewModel, open),
-        drawerActions = getDrawerActions(hiltViewModel(), open, openAndPopUp),
-        navigationBarActions = getNavigationBarActions(hiltViewModel(), open),
+        drawerActions = drawerActions,
+        navigationBarActions = navigationBarActions,
     )
 }
 
@@ -65,7 +63,7 @@ fun ProfileScreen(
         title = resources().getString(AppText.profile),
         drawerActions = drawerActions,
         navigationBarActions = navigationBarActions,
-        action = { EditAction(onClick = profileActions.onEditProfileClick) }
+        barAction = { EditAction(onClick = profileActions.onEditProfileClick) }
     ) {
         Headline(text = (username ?: resources().getString(R.string.no_username)))
     }
@@ -90,6 +88,6 @@ fun ProfileScreenPreview() {
     ProfileScreen(
         profileActions = ProfileActions({ null }, {}),
         drawerActions = DrawerActions({}, {}, {}, {}, {}),
-        navigationBarActions = NavigationBarActions({}, {}, {}, {})
+        navigationBarActions = NavigationBarActions({ false }, {}, {}, {}, {})
     )
 }

app/src/main/java/be/ugent/sel/studeez/screens/session/SessionRoute.kt

@@ -14,15 +14,18 @@ data class SessionActions(
     val getTask: () -> String,
     val startMediaPlayer: () -> Unit,
     val releaseMediaPlayer: () -> Unit,
+    val endSession: () -> Unit
 )
 
 private fun getSessionActions(
     viewModel: SessionViewModel,
+    openAndPopUp: (String, String) -> Unit,
     mediaplayer: MediaPlayer,
 ): SessionActions {
     return SessionActions(
         getTimer = viewModel::getTimer,
         getTask = viewModel::getTask,
+        endSession = { viewModel.endSession(openAndPopUp) },
         startMediaPlayer = mediaplayer::start,
         releaseMediaPlayer = mediaplayer::release,
     )
@@ -31,6 +34,7 @@ private fun getSessionActions(
 @Composable
 fun SessionRoute(
     open: (String) -> Unit,
+    openAndPopUp: (String, String) -> Unit,
     viewModel: SessionViewModel,
 ) {
     val context = LocalContext.current
@@ -47,6 +51,6 @@ fun SessionRoute(
 
     sessionScreen(
         open = open,
-        sessionActions = getSessionActions(viewModel, mediaplayer)
+        sessionActions = getSessionActions(viewModel, openAndPopUp, mediaplayer)
     )
 }

app/src/main/java/be/ugent/sel/studeez/screens/session/SessionViewModel.kt

@@ -1,20 +1,21 @@
 package be.ugent.sel.studeez.screens.session
 
+import be.ugent.sel.studeez.data.SelectedTimerState
+import be.ugent.sel.studeez.data.SessionReportState
 import be.ugent.sel.studeez.data.local.models.timer_functional.FunctionalTimer
 import be.ugent.sel.studeez.domain.LogService
+import be.ugent.sel.studeez.navigation.StudeezDestinations
 import be.ugent.sel.studeez.screens.StudeezViewModel
-import be.ugent.sel.studeez.data.SelectedTimerState
-import be.ugent.sel.studeez.data.local.models.timer_functional.FunctionalPomodoroTimer
 import dagger.hilt.android.lifecycle.HiltViewModel
 import javax.inject.Inject
 
 @HiltViewModel
 class SessionViewModel @Inject constructor(
     private val selectedTimerState: SelectedTimerState,
+    private val sessionReportState: SessionReportState,
     logService: LogService
 ) : StudeezViewModel(logService) {
 
-    private val timer: FunctionalTimer = FunctionalPomodoroTimer(15, 5, 3)
     private val task : String = "No task selected" // placeholder for tasks implementation
 
     fun getTimer() : FunctionalTimer {
@@ -24,4 +25,9 @@ class SessionViewModel @Inject constructor(
     fun getTask(): String {
         return task
     }
+
+    fun endSession(openAndPopUp: (String, String) -> Unit) {
+        sessionReportState.sessionReport = getTimer().getSessionReport()
+        openAndPopUp(StudeezDestinations.SESSION_RECAP, StudeezDestinations.SESSION_SCREEN)
+    }
 }

app/src/main/java/be/ugent/sel/studeez/screens/session/sessionScreens/AbstractSessionScreen.kt

@@ -19,7 +19,6 @@ import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.sp
 import be.ugent.sel.studeez.data.local.models.timer_functional.FunctionalEndlessTimer
-import be.ugent.sel.studeez.navigation.StudeezDestinations
 import be.ugent.sel.studeez.screens.session.SessionActions
 import kotlinx.coroutines.delay
 import kotlin.time.Duration.Companion.seconds
@@ -45,8 +44,7 @@ abstract class AbstractSessionScreen {
                 TextButton(
                     onClick = {
                         sessionActions.releaseMediaPlayer
-                        open(StudeezDestinations.HOME_SCREEN)
-                        // Vanaf hier ook naar report gaan als "end session" knop word ingedrukt
+                        sessionActions.endSession()
                     },
                     modifier = Modifier
                         .padding(horizontal = 20.dp)
@@ -136,5 +134,5 @@ fun TimerPreview() {
         override fun callMediaPlayer() {}
 
     }
-    sessionScreen.Timer(sessionActions = SessionActions({ FunctionalEndlessTimer() }, { "Preview" }, {}, {}))
+    sessionScreen.Timer(sessionActions = SessionActions({ FunctionalEndlessTimer() }, { "Preview" }, {}, {}, {}))
 }

app/src/main/java/be/ugent/sel/studeez/screens/session_recap/SessionRecapScreen.kt

@@ -0,0 +1,65 @@
+package be.ugent.sel.studeez.screens.session_recap
+
+import androidx.compose.foundation.layout.Column
+import androidx.compose.material.ButtonDefaults
+import androidx.compose.material.Text
+import androidx.compose.runtime.Composable
+import androidx.compose.ui.Modifier
+import androidx.compose.ui.graphics.Color
+import be.ugent.sel.studeez.R
+import be.ugent.sel.studeez.common.composable.BasicButton
+import be.ugent.sel.studeez.common.ext.basicButton
+import be.ugent.sel.studeez.data.local.models.SessionReport
+import be.ugent.sel.studeez.data.local.models.timer_functional.HoursMinutesSeconds
+import be.ugent.sel.studeez.data.local.models.timer_functional.Time
+
+data class SessionRecapActions(
+    val getSessionReport: () -> SessionReport,
+    val saveSession: () -> Unit,
+    val discardSession: () -> Unit
+)
+
+fun getSessionRecapActions(
+    viewModel: SessionRecapViewModel,
+    openAndPopUp: (String, String) -> Unit,
+): SessionRecapActions {
+    return SessionRecapActions(
+        viewModel::getSessionReport,
+        {viewModel.saveSession(openAndPopUp)},
+        {viewModel.discardSession(openAndPopUp)}
+    )
+}
+
+@Composable
+fun SessionRecapRoute(
+    openAndPopUp: (String, String) -> Unit,
+    modifier: Modifier = Modifier,
+    viewModel: SessionRecapViewModel,
+) {
+    SessionRecapScreen(
+        modifier = modifier,
+        getSessionRecapActions(viewModel, openAndPopUp)
+    )
+}
+
+@Composable
+fun SessionRecapScreen(modifier: Modifier, sessionRecapActions: SessionRecapActions) {
+    val sessionReport: SessionReport = sessionRecapActions.getSessionReport()
+    val studyTime: Int = sessionReport.studyTime
+    val hms: HoursMinutesSeconds = Time(studyTime).getAsHMS()
+    Column {
+        Text(text = "You studied: ${hms.hours} : ${hms.minutes} : ${hms.seconds}")
+
+        BasicButton(
+            R.string.save, Modifier.basicButton()
+        ) {
+            sessionRecapActions.saveSession()
+        }
+        BasicButton(
+            R.string.discard, Modifier.basicButton(),
+            colors = ButtonDefaults.buttonColors(backgroundColor = Color.Red)
+        ) {
+            sessionRecapActions.discardSession()
+        }
+    }
+}

app/src/main/java/be/ugent/sel/studeez/screens/session_recap/SessionRecapViewModel.kt

@@ -0,0 +1,33 @@
+package be.ugent.sel.studeez.screens.session_recap
+
+import be.ugent.sel.studeez.data.SessionReportState
+import be.ugent.sel.studeez.data.local.models.SessionReport
+import be.ugent.sel.studeez.domain.LogService
+import be.ugent.sel.studeez.domain.SessionDAO
+import be.ugent.sel.studeez.navigation.StudeezDestinations
+import be.ugent.sel.studeez.screens.StudeezViewModel
+import dagger.hilt.android.lifecycle.HiltViewModel
+import javax.inject.Inject
+
+@HiltViewModel
+class SessionRecapViewModel @Inject constructor(
+    private val sessionReportState: SessionReportState,
+    private val sessionDAO: SessionDAO,
+    logService: LogService
+) : StudeezViewModel(logService) {
+
+    private val report: SessionReport = sessionReportState.sessionReport!!
+
+    fun getSessionReport(): SessionReport {
+        return report
+    }
+
+    fun saveSession(open: (String, String) -> Unit) {
+        sessionDAO.saveSession(getSessionReport())
+        open(StudeezDestinations.HOME_SCREEN, StudeezDestinations.SESSION_RECAP)
+    }
+
+    fun discardSession(open: (String, String) -> Unit) {
+        open(StudeezDestinations.HOME_SCREEN, StudeezDestinations.SESSION_RECAP)
+    }
+}

app/src/main/java/be/ugent/sel/studeez/screens/timer_overview/TimerOverviewScreen.kt

@@ -1,6 +1,5 @@
 package be.ugent.sel.studeez.screens.timer_overview
 
-import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
@@ -8,18 +7,12 @@ import androidx.compose.runtime.Composable
 import androidx.compose.runtime.collectAsState
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.tooling.preview.Preview
-import androidx.compose.ui.unit.dp
 import be.ugent.sel.studeez.R
 import be.ugent.sel.studeez.common.composable.BasicButton
-import be.ugent.sel.studeez.common.composable.PrimaryScreenTemplate
+import be.ugent.sel.studeez.common.composable.DrawerScreenTemplate
 import be.ugent.sel.studeez.common.composable.StealthButton
 import be.ugent.sel.studeez.common.composable.TimerEntry
 import be.ugent.sel.studeez.common.composable.drawer.DrawerActions
-import be.ugent.sel.studeez.common.composable.drawer.DrawerViewModel
-import be.ugent.sel.studeez.common.composable.drawer.getDrawerActions
-import be.ugent.sel.studeez.common.composable.navbar.NavigationBarActions
-import be.ugent.sel.studeez.common.composable.navbar.NavigationBarViewModel
-import be.ugent.sel.studeez.common.composable.navbar.getNavigationBarActions
 import be.ugent.sel.studeez.common.ext.basicButton
 import be.ugent.sel.studeez.data.local.models.timer_info.CustomTimerInfo
 import be.ugent.sel.studeez.data.local.models.timer_info.TimerInfo
@@ -45,38 +38,29 @@ fun getTimerOverviewActions(
 
 @Composable
 fun TimerOverviewRoute(
-    open: (String) -> Unit,
-    openAndPopUp: (String, String) -> Unit,
     viewModel: TimerOverviewViewModel,
-    drawerViewModel: DrawerViewModel,
-    navBarViewModel: NavigationBarViewModel,
+    drawerActions: DrawerActions,
 ) {
     TimerOverviewScreen(
         timerOverviewActions = getTimerOverviewActions(viewModel),
-        drawerActions = getDrawerActions(drawerViewModel, open, openAndPopUp),
-        navigationBarActions = getNavigationBarActions(navBarViewModel, open),
+        drawerActions = drawerActions,
     )
 }
 
 @Composable
 fun TimerOverviewScreen(
     timerOverviewActions: TimerOverviewActions,
-    drawerActions: DrawerActions,
-    navigationBarActions: NavigationBarActions,
+    drawerActions: DrawerActions
 ) {
 
     val timers = timerOverviewActions.getUserTimers().collectAsState(initial = emptyList())
 
-    // TODO moet geen primary screen zijn: geen navbar nodig
-    PrimaryScreenTemplate(
+    DrawerScreenTemplate(
         title = resources().getString(R.string.timers),
-        drawerActions = drawerActions,
-        navigationBarActions = navigationBarActions,
+        drawerActions = drawerActions
     ) {
         Column {
-            LazyColumn(
-                verticalArrangement = Arrangement.spacedBy(7.dp)
-            ) {
+            LazyColumn {
                 // Default Timers, cannot be edited
                 items(timerOverviewActions.getDefaultTimers()) {
                     TimerEntry(timerInfo = it) {}
@@ -112,7 +96,6 @@ fun TimerOverviewPreview() {
             { flowOf() },
             { listOf(customTimer, customTimer) },
             {}),
-        drawerActions = DrawerActions({}, {}, {}, {}, {}),
-        navigationBarActions = NavigationBarActions({}, {}, {}, {})
+        drawerActions = DrawerActions({}, {}, {}, {}, {})
     )
 }

app/src/main/java/be/ugent/sel/studeez/screens/timer_selection/TimerSelectionScreen.kt

@@ -1,22 +1,14 @@
 package be.ugent.sel.studeez.screens.timer_selection
 
-import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.items
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.collectAsState
 import androidx.compose.ui.tooling.preview.Preview
-import androidx.compose.ui.unit.dp
 import be.ugent.sel.studeez.R
-import be.ugent.sel.studeez.common.composable.PrimaryScreenTemplate
+import be.ugent.sel.studeez.common.composable.SecondaryScreenTemplate
 import be.ugent.sel.studeez.common.composable.StealthButton
 import be.ugent.sel.studeez.common.composable.TimerEntry
-import be.ugent.sel.studeez.common.composable.drawer.DrawerActions
-import be.ugent.sel.studeez.common.composable.drawer.DrawerViewModel
-import be.ugent.sel.studeez.common.composable.drawer.getDrawerActions
-import be.ugent.sel.studeez.common.composable.navbar.NavigationBarActions
-import be.ugent.sel.studeez.common.composable.navbar.NavigationBarViewModel
-import be.ugent.sel.studeez.common.composable.navbar.getNavigationBarActions
 import be.ugent.sel.studeez.data.local.models.timer_info.TimerInfo
 import be.ugent.sel.studeez.resources
 import kotlinx.coroutines.flow.Flow
@@ -40,41 +32,37 @@ fun getTimerSelectionActions(
 @Composable
 fun TimerSelectionRoute(
     open: (String) -> Unit,
-    openAndPopUp: (String, String) -> Unit,
+    popUp: () -> Unit,
     viewModel: TimerSelectionViewModel,
-    drawerViewModel: DrawerViewModel,
-    navBarViewModel: NavigationBarViewModel,
 ) {
     TimerSelectionScreen(
         timerSelectionActions = getTimerSelectionActions(viewModel, open),
-        drawerActions = getDrawerActions(drawerViewModel, open, openAndPopUp),
-        navigationBarActions = getNavigationBarActions(navBarViewModel, open),
+        popUp = popUp
     )
 }
 
 @Composable
 fun TimerSelectionScreen(
     timerSelectionActions: TimerSelectionActions,
-    drawerActions: DrawerActions,
-    navigationBarActions: NavigationBarActions,
+    popUp: () -> Unit
 ) {
     val timers = timerSelectionActions.getAllTimers().collectAsState(initial = emptyList())
-    PrimaryScreenTemplate(
+    SecondaryScreenTemplate(
         title = resources().getString(R.string.timers),
-        drawerActions = drawerActions,
-        navigationBarActions = navigationBarActions,
+        popUp = popUp
     ) {
-        LazyColumn(verticalArrangement = Arrangement.spacedBy(7.dp)) {
+        LazyColumn {
             // All timers
             items(timers.value) { timerInfo ->
                 TimerEntry(
                     timerInfo = timerInfo,
-                ) {
-                    StealthButton(
-                        text = R.string.start,
-                        onClick = { timerSelectionActions.startSession(timerInfo) }
-                    )
-                }
+                    leftButton = {
+                        StealthButton(
+                            text = R.string.start,
+                            onClick = { timerSelectionActions.startSession(timerInfo) }
+                        )
+                    }
+                )
             }
         }
     }
@@ -85,7 +73,6 @@ fun TimerSelectionScreen(
 fun TimerSelectionPreview() {
     TimerSelectionScreen(
         timerSelectionActions = TimerSelectionActions({ flowOf() }, {}),
-        drawerActions = DrawerActions({}, {}, {}, {}, {}),
-        navigationBarActions = NavigationBarActions({}, {}, {}, {}),
+        popUp = {}
     )
 }