From 6a4e9474e3d4c5b8da0408fe182c5d56c206679b Mon Sep 17 00:00:00 2001
From: Anil Kumar Beesetti <66936649+anilbeesetti@users.noreply.github.com>
Date: Wed, 2 Aug 2023 13:18:05 +0530
Subject: [PATCH] Refactor: Rename Directory model to Folder model (#483)

* refactor: get sorted video use case to use repository function instead of filtering

* refactor: rename Directory model to Folder model

* lint: run KtlintFormat
---
 .../mappers/DirectoryEntityToDirectory.kt     |  4 +--
 .../data/repository/LocalMediaRepository.kt   | 12 ++++---
 .../core/data/repository/MediaRepository.kt   | 28 +++++++--------
 .../repository/fake/FakeMediaRepository.kt    | 10 ++++--
 ...sUseCase.kt => GetSortedFoldersUseCase.kt} | 16 ++++-----
 .../core/domain/GetSortedVideosUseCase.kt     | 36 +++++++++----------
 .../core/model/{Directory.kt => Folder.kt}    |  4 +--
 .../MediaLibraryPreferencesViewModel.kt       |  6 ++--
 .../videopicker/MediaPickerScreenTest.kt      |  8 ++---
 .../videopicker/composables/FolderItem.kt     | 16 ++++-----
 .../videopicker/composables/MediaContent.kt   | 16 ++++-----
 .../feature/videopicker/screens/MediaState.kt |  4 +--
 .../screens/media/MediaPickerViewModel.kt     |  6 ++--
 13 files changed, 87 insertions(+), 79 deletions(-)
 rename core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/{GetSortedDirectoriesUseCase.kt => GetSortedFoldersUseCase.kt} (82%)
 rename core/model/src/main/java/dev/anilbeesetti/nextplayer/core/model/{Directory.kt => Folder.kt} (89%)

diff --git a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/mappers/DirectoryEntityToDirectory.kt b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/mappers/DirectoryEntityToDirectory.kt
index c5ad8ea97..73b3e360f 100644
--- a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/mappers/DirectoryEntityToDirectory.kt
+++ b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/mappers/DirectoryEntityToDirectory.kt
@@ -2,9 +2,9 @@ package dev.anilbeesetti.nextplayer.core.data.mappers
 
 import dev.anilbeesetti.nextplayer.core.common.Utils
 import dev.anilbeesetti.nextplayer.core.database.entities.DirectoryEntity
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 
-fun DirectoryEntity.toDirectory() = Directory(
+fun DirectoryEntity.toFolder() = Folder(
     name = name,
     path = path,
     mediaSize = size,
diff --git a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/LocalMediaRepository.kt b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/LocalMediaRepository.kt
index ee5d2618b..3aaf87947 100644
--- a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/LocalMediaRepository.kt
+++ b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/LocalMediaRepository.kt
@@ -7,7 +7,7 @@ import androidx.activity.result.IntentSenderRequest
 import dagger.hilt.android.qualifiers.ApplicationContext
 import dev.anilbeesetti.nextplayer.core.common.di.ApplicationScope
 import dev.anilbeesetti.nextplayer.core.common.extensions.deleteFiles
-import dev.anilbeesetti.nextplayer.core.data.mappers.toDirectory
+import dev.anilbeesetti.nextplayer.core.data.mappers.toFolder
 import dev.anilbeesetti.nextplayer.core.data.mappers.toVideo
 import dev.anilbeesetti.nextplayer.core.data.mappers.toVideoState
 import dev.anilbeesetti.nextplayer.core.data.models.VideoState
@@ -15,7 +15,7 @@ import dev.anilbeesetti.nextplayer.core.database.dao.DirectoryDao
 import dev.anilbeesetti.nextplayer.core.database.dao.MediumDao
 import dev.anilbeesetti.nextplayer.core.database.entities.DirectoryEntity
 import dev.anilbeesetti.nextplayer.core.database.entities.MediumEntity
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.model.Video
 import javax.inject.Inject
 import kotlinx.coroutines.CoroutineScope
@@ -36,8 +36,12 @@ class LocalMediaRepository @Inject constructor(
         return mediumDao.getAll().map { it.map(MediumEntity::toVideo) }
     }
 
-    override fun getDirectoriesFlow(): Flow<List<Directory>> {
-        return directoryDao.getAll().map { it.map(DirectoryEntity::toDirectory) }
+    override fun getVideosFlowFromFolderPath(folderPath: String): Flow<List<Video>> {
+        return mediumDao.getAllFromDirectory(folderPath).map { it.map(MediumEntity::toVideo) }
+    }
+
+    override fun getFoldersFlow(): Flow<List<Folder>> {
+        return directoryDao.getAll().map { it.map(DirectoryEntity::toFolder) }
     }
 
     override suspend fun getVideoState(path: String): VideoState? {
diff --git a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/MediaRepository.kt b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/MediaRepository.kt
index edb407cac..46c215e04 100644
--- a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/MediaRepository.kt
+++ b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/MediaRepository.kt
@@ -3,7 +3,7 @@ package dev.anilbeesetti.nextplayer.core.data.repository
 import androidx.activity.result.ActivityResultLauncher
 import androidx.activity.result.IntentSenderRequest
 import dev.anilbeesetti.nextplayer.core.data.models.VideoState
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.model.Video
 import kotlinx.coroutines.flow.Flow
 
@@ -14,7 +14,18 @@ interface MediaRepository {
      */
     fun getVideosFlow(): Flow<List<Video>>
 
-    fun getDirectoriesFlow(): Flow<List<Directory>>
+    /**
+     * Get list of [Video]s as flow for a specific directory
+     * @param folderPath the path the directory from which the videos should be returned
+     * @return flow of list of video items
+     */
+    fun getVideosFlowFromFolderPath(folderPath: String): Flow<List<Video>>
+
+    /**
+     * Get list of [Folder]s as flow
+     * @return flow of list of folder items
+     */
+    fun getFoldersFlow(): Flow<List<Folder>>
 
     /**
      * Save video state
@@ -25,19 +36,8 @@ interface MediaRepository {
      * -1 to disable track
      * null to not change track
      */
-    suspend fun saveVideoState(
-        path: String,
-        position: Long,
-        audioTrackIndex: Int?,
-        subtitleTrackIndex: Int?,
-        playbackSpeed: Float?
-    )
+    suspend fun saveVideoState(path: String, position: Long, audioTrackIndex: Int?, subtitleTrackIndex: Int?, playbackSpeed: Float?)
 
-    /**
-     * Get video state
-     * @param path path of the video
-     * @return [VideoState] of the video
-     */
     suspend fun getVideoState(path: String): VideoState?
 
     suspend fun deleteVideos(videoUris: List<String>, intentSenderLauncher: ActivityResultLauncher<IntentSenderRequest>)
diff --git a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/fake/FakeMediaRepository.kt b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/fake/FakeMediaRepository.kt
index 94e59746e..fe12e714d 100644
--- a/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/fake/FakeMediaRepository.kt
+++ b/core/data/src/main/java/dev/anilbeesetti/nextplayer/core/data/repository/fake/FakeMediaRepository.kt
@@ -4,7 +4,7 @@ import androidx.activity.result.ActivityResultLauncher
 import androidx.activity.result.IntentSenderRequest
 import dev.anilbeesetti.nextplayer.core.data.models.VideoState
 import dev.anilbeesetti.nextplayer.core.data.repository.MediaRepository
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.model.Video
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.flowOf
@@ -12,13 +12,17 @@ import kotlinx.coroutines.flow.flowOf
 class FakeMediaRepository : MediaRepository {
 
     val videos = mutableListOf<Video>()
-    val directories = mutableListOf<Directory>()
+    val directories = mutableListOf<Folder>()
 
     override fun getVideosFlow(): Flow<List<Video>> {
         return flowOf(videos)
     }
 
-    override fun getDirectoriesFlow(): Flow<List<Directory>> {
+    override fun getVideosFlowFromFolderPath(folderPath: String): Flow<List<Video>> {
+        return flowOf(videos)
+    }
+
+    override fun getFoldersFlow(): Flow<List<Folder>> {
         return flowOf(directories)
     }
 
diff --git a/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedDirectoriesUseCase.kt b/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedFoldersUseCase.kt
similarity index 82%
rename from core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedDirectoriesUseCase.kt
rename to core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedFoldersUseCase.kt
index c59e27268..5e8c489b3 100644
--- a/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedDirectoriesUseCase.kt
+++ b/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedFoldersUseCase.kt
@@ -2,9 +2,9 @@ package dev.anilbeesetti.nextplayer.core.domain
 
 import dev.anilbeesetti.nextplayer.core.common.Dispatcher
 import dev.anilbeesetti.nextplayer.core.common.NextDispatchers
-import dev.anilbeesetti.nextplayer.core.data.repository.LocalMediaRepository
+import dev.anilbeesetti.nextplayer.core.data.repository.MediaRepository
 import dev.anilbeesetti.nextplayer.core.data.repository.PreferencesRepository
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.model.SortBy
 import dev.anilbeesetti.nextplayer.core.model.SortOrder
 import javax.inject.Inject
@@ -13,19 +13,19 @@ import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.combine
 import kotlinx.coroutines.flow.flowOn
 
-class GetSortedDirectoriesUseCase @Inject constructor(
-    private val videoRepository: LocalMediaRepository,
+class GetSortedFoldersUseCase @Inject constructor(
+    private val mediaRepository: MediaRepository,
     private val preferencesRepository: PreferencesRepository,
     @Dispatcher(NextDispatchers.Default) private val defaultDispatcher: CoroutineDispatcher
 ) {
 
-    operator fun invoke(): Flow<List<Directory>> {
+    operator fun invoke(): Flow<List<Folder>> {
         return combine(
-            videoRepository.getDirectoriesFlow(),
+            mediaRepository.getFoldersFlow(),
             preferencesRepository.applicationPreferences
-        ) { directories, preferences ->
+        ) { folders, preferences ->
 
-            val nonExcludedDirectories = directories.filterNot {
+            val nonExcludedDirectories = folders.filterNot {
                 it.path in preferences.excludeFolders
             }
 
diff --git a/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedVideosUseCase.kt b/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedVideosUseCase.kt
index 51376d1a3..b4f326e06 100644
--- a/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedVideosUseCase.kt
+++ b/core/domain/src/main/java/dev/anilbeesetti/nextplayer/core/domain/GetSortedVideosUseCase.kt
@@ -21,39 +21,39 @@ class GetSortedVideosUseCase @Inject constructor(
 ) {
 
     operator fun invoke(folderPath: String? = null): Flow<List<Video>> {
+        val videosFlow = if (folderPath != null) {
+            mediaRepository.getVideosFlowFromFolderPath(folderPath)
+        } else {
+            mediaRepository.getVideosFlow()
+        }
+
         return combine(
-            mediaRepository.getVideosFlow(),
+            videosFlow,
             preferencesRepository.applicationPreferences
         ) { videoItems, preferences ->
 
-            val filteredVideos = videoItems.filter {
-                if (preferences.groupVideosByFolder) {
-                    folderPath == null || it.parentPath == folderPath
-                } else {
-                    true
-                }
-            }.filterNot {
+            val nonExcludedVideos = videoItems.filterNot {
                 it.parentPath in preferences.excludeFolders
             }
 
             when (preferences.sortOrder) {
                 SortOrder.ASCENDING -> {
                     when (preferences.sortBy) {
-                        SortBy.TITLE -> filteredVideos.sortedBy { it.displayName.lowercase() }
-                        SortBy.LENGTH -> filteredVideos.sortedBy { it.duration }
-                        SortBy.PATH -> filteredVideos.sortedBy { it.path.lowercase() }
-                        SortBy.SIZE -> filteredVideos.sortedBy { it.size }
-                        SortBy.DATE -> filteredVideos.sortedBy { it.dateModified }
+                        SortBy.TITLE -> nonExcludedVideos.sortedBy { it.displayName.lowercase() }
+                        SortBy.LENGTH -> nonExcludedVideos.sortedBy { it.duration }
+                        SortBy.PATH -> nonExcludedVideos.sortedBy { it.path.lowercase() }
+                        SortBy.SIZE -> nonExcludedVideos.sortedBy { it.size }
+                        SortBy.DATE -> nonExcludedVideos.sortedBy { it.dateModified }
                     }
                 }
 
                 SortOrder.DESCENDING -> {
                     when (preferences.sortBy) {
-                        SortBy.TITLE -> filteredVideos.sortedByDescending { it.displayName.lowercase() }
-                        SortBy.LENGTH -> filteredVideos.sortedByDescending { it.duration }
-                        SortBy.PATH -> filteredVideos.sortedByDescending { it.path.lowercase() }
-                        SortBy.SIZE -> filteredVideos.sortedByDescending { it.size }
-                        SortBy.DATE -> filteredVideos.sortedByDescending { it.dateModified }
+                        SortBy.TITLE -> nonExcludedVideos.sortedByDescending { it.displayName.lowercase() }
+                        SortBy.LENGTH -> nonExcludedVideos.sortedByDescending { it.duration }
+                        SortBy.PATH -> nonExcludedVideos.sortedByDescending { it.path.lowercase() }
+                        SortBy.SIZE -> nonExcludedVideos.sortedByDescending { it.size }
+                        SortBy.DATE -> nonExcludedVideos.sortedByDescending { it.dateModified }
                     }
                 }
             }
diff --git a/core/model/src/main/java/dev/anilbeesetti/nextplayer/core/model/Directory.kt b/core/model/src/main/java/dev/anilbeesetti/nextplayer/core/model/Folder.kt
similarity index 89%
rename from core/model/src/main/java/dev/anilbeesetti/nextplayer/core/model/Directory.kt
rename to core/model/src/main/java/dev/anilbeesetti/nextplayer/core/model/Folder.kt
index 15a4cc998..3b813d8bc 100644
--- a/core/model/src/main/java/dev/anilbeesetti/nextplayer/core/model/Directory.kt
+++ b/core/model/src/main/java/dev/anilbeesetti/nextplayer/core/model/Folder.kt
@@ -1,6 +1,6 @@
 package dev.anilbeesetti.nextplayer.core.model
 
-data class Directory(
+data class Folder(
     val name: String,
     val path: String,
     val mediaSize: Long,
@@ -10,7 +10,7 @@ data class Directory(
 ) {
 
     companion object {
-        val sample = Directory(
+        val sample = Folder(
             name = "Folder 1",
             path = "/storage/emulated/0/DCIM/Camera/Live Photos",
             mediaSize = 1024,
diff --git a/feature/settings/src/main/java/dev/anilbeesetti/nextplayer/settings/screens/medialibrary/MediaLibraryPreferencesViewModel.kt b/feature/settings/src/main/java/dev/anilbeesetti/nextplayer/settings/screens/medialibrary/MediaLibraryPreferencesViewModel.kt
index 14cbf5fcc..2f6c732a0 100644
--- a/feature/settings/src/main/java/dev/anilbeesetti/nextplayer/settings/screens/medialibrary/MediaLibraryPreferencesViewModel.kt
+++ b/feature/settings/src/main/java/dev/anilbeesetti/nextplayer/settings/screens/medialibrary/MediaLibraryPreferencesViewModel.kt
@@ -6,7 +6,7 @@ import dagger.hilt.android.lifecycle.HiltViewModel
 import dev.anilbeesetti.nextplayer.core.data.repository.MediaRepository
 import dev.anilbeesetti.nextplayer.core.data.repository.PreferencesRepository
 import dev.anilbeesetti.nextplayer.core.model.ApplicationPreferences
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import javax.inject.Inject
 import kotlinx.coroutines.flow.SharingStarted
 import kotlinx.coroutines.flow.map
@@ -19,7 +19,7 @@ class MediaLibraryPreferencesViewModel @Inject constructor(
     private val preferencesRepository: PreferencesRepository
 ) : ViewModel() {
 
-    val uiState = mediaRepository.getDirectoriesFlow()
+    val uiState = mediaRepository.getFoldersFlow()
         .map { FolderPreferencesUiState.Success(it) }
         .stateIn(
             scope = viewModelScope,
@@ -52,5 +52,5 @@ class MediaLibraryPreferencesViewModel @Inject constructor(
 sealed interface FolderPreferencesUiState {
     object Loading : FolderPreferencesUiState
 
-    data class Success(val directories: List<Directory>) : FolderPreferencesUiState
+    data class Success(val directories: List<Folder>) : FolderPreferencesUiState
 }
diff --git a/feature/videopicker/src/androidTest/java/dev/anilbeesetti/nextplayer/feature/videopicker/MediaPickerScreenTest.kt b/feature/videopicker/src/androidTest/java/dev/anilbeesetti/nextplayer/feature/videopicker/MediaPickerScreenTest.kt
index 14a172e6d..b5df8320a 100644
--- a/feature/videopicker/src/androidTest/java/dev/anilbeesetti/nextplayer/feature/videopicker/MediaPickerScreenTest.kt
+++ b/feature/videopicker/src/androidTest/java/dev/anilbeesetti/nextplayer/feature/videopicker/MediaPickerScreenTest.kt
@@ -8,7 +8,7 @@ import androidx.compose.ui.test.onNodeWithTag
 import androidx.compose.ui.test.onNodeWithText
 import androidx.compose.ui.test.onParent
 import dev.anilbeesetti.nextplayer.core.model.ApplicationPreferences
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.model.Video
 import dev.anilbeesetti.nextplayer.core.ui.R
 import dev.anilbeesetti.nextplayer.feature.videopicker.screens.FoldersState
@@ -84,7 +84,7 @@ class MediaPickerScreenTest {
 
     /**
      * This test is to check if the folder items are displayed
-     * when the [FoldersState.Success] is passed with list of [Directory],
+     * when the [FoldersState.Success] is passed with list of [Folder],
      * along with [ApplicationPreferences].groupVideosByFolder = true
      */
     @Test
@@ -206,14 +206,14 @@ val videoItemsTestData = listOf(
 )
 
 val foldersTestData = listOf(
-    Directory(
+    Folder(
         name = "Folder 1",
         path = "/storage/emulated/0/DCIM/Camera/Folder 1",
         mediaCount = 1,
         mediaSize = 1000,
         dateModified = 1000
     ),
-    Directory(
+    Folder(
         name = "Folder 2",
         path = "/storage/emulated/0/DCIM/Camera/Folder 2",
         mediaCount = 2,
diff --git a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/FolderItem.kt b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/FolderItem.kt
index bc4a00c0b..06e6965b0 100644
--- a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/FolderItem.kt
+++ b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/FolderItem.kt
@@ -20,7 +20,7 @@ import androidx.compose.ui.text.style.TextOverflow
 import androidx.compose.ui.unit.Dp
 import androidx.compose.ui.unit.dp
 import androidx.compose.ui.unit.offset
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.ui.R
 import dev.anilbeesetti.nextplayer.core.ui.designsystem.NextIcons
 import dev.anilbeesetti.nextplayer.core.ui.preview.DayNightPreview
@@ -29,7 +29,7 @@ import dev.anilbeesetti.nextplayer.core.ui.theme.NextPlayerTheme
 @OptIn(ExperimentalLayoutApi::class)
 @Composable
 fun FolderItem(
-    directory: Directory,
+    folder: Folder,
     modifier: Modifier = Modifier
 ) {
     ListItem(
@@ -47,7 +47,7 @@ fun FolderItem(
         },
         headlineContent = {
             Text(
-                text = directory.name,
+                text = folder.name,
                 maxLines = 2,
                 style = MaterialTheme.typography.titleMedium,
                 overflow = TextOverflow.Ellipsis
@@ -55,7 +55,7 @@ fun FolderItem(
         },
         supportingContent = {
             Text(
-                text = directory.path,
+                text = folder.path,
                 maxLines = 2,
                 style = MaterialTheme.typography.bodySmall,
                 overflow = TextOverflow.Ellipsis,
@@ -67,15 +67,15 @@ fun FolderItem(
                 verticalAlignment = Alignment.CenterVertically
             ) {
                 InfoChip(
-                    text = "${directory.mediaCount} ${
+                    text = "${folder.mediaCount} ${
                     stringResource(
-                        id = R.string.video.takeIf { directory.mediaCount == 1 } ?: R.string.videos
+                        id = R.string.video.takeIf { folder.mediaCount == 1 } ?: R.string.videos
                     )
                     }",
                     modifier = Modifier.padding(vertical = 5.dp)
                 )
                 InfoChip(
-                    text = directory.formattedMediaSize,
+                    text = folder.formattedMediaSize,
                     modifier = Modifier.padding(vertical = 5.dp)
                 )
             }
@@ -88,7 +88,7 @@ fun FolderItem(
 @Composable
 fun FolderItemPreview() {
     NextPlayerTheme {
-        FolderItem(directory = Directory.sample)
+        FolderItem(folder = Folder.sample)
     }
 }
 
diff --git a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/MediaContent.kt b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/MediaContent.kt
index 6480c0dc1..603418c48 100644
--- a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/MediaContent.kt
+++ b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/composables/MediaContent.kt
@@ -43,7 +43,7 @@ import androidx.compose.ui.text.style.TextAlign
 import androidx.compose.ui.text.style.TextOverflow
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.unit.dp
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.model.Video
 import dev.anilbeesetti.nextplayer.core.ui.R
 import dev.anilbeesetti.nextplayer.core.ui.components.CancelButton
@@ -181,8 +181,8 @@ fun FoldersListFromState(
     onDeleteFolderClick: (String) -> Unit
 ) {
     val haptic = LocalHapticFeedback.current
-    var showDirectoryActionsFor: Directory? by rememberSaveable { mutableStateOf(null) }
-    var deleteAction: Directory? by rememberSaveable { mutableStateOf(null) }
+    var showFolderActionsFor: Folder? by rememberSaveable { mutableStateOf(null) }
+    var deleteAction: Folder? by rememberSaveable { mutableStateOf(null) }
     val scope = rememberCoroutineScope()
     val bottomSheetState = rememberModalBottomSheetState(skipPartiallyExpanded = true)
 
@@ -194,12 +194,12 @@ fun FoldersListFromState(
             MediaLazyList {
                 items(foldersState.data, key = { it.path }) {
                     FolderItem(
-                        directory = it,
+                        folder = it,
                         modifier = Modifier.combinedClickable(
                             onClick = { onFolderClick(it.path) },
                             onLongClick = {
                                 haptic.performHapticFeedback(HapticFeedbackType.LongPress)
-                                showDirectoryActionsFor = it
+                                showFolderActionsFor = it
                             }
                         )
                     )
@@ -208,10 +208,10 @@ fun FoldersListFromState(
         }
     }
 
-    showDirectoryActionsFor?.let {
+    showFolderActionsFor?.let {
         OptionsBottomSheet(
             title = it.name,
-            onDismiss = { showDirectoryActionsFor = null }
+            onDismiss = { showFolderActionsFor = null }
         ) {
             BottomSheetItem(
                 text = stringResource(R.string.delete),
@@ -219,7 +219,7 @@ fun FoldersListFromState(
                 onClick = {
                     deleteAction = it
                     scope.launch { bottomSheetState.hide() }.invokeOnCompletion {
-                        if (!bottomSheetState.isVisible) showDirectoryActionsFor = null
+                        if (!bottomSheetState.isVisible) showFolderActionsFor = null
                     }
                 }
             )
diff --git a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/MediaState.kt b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/MediaState.kt
index 36d79cb71..daadad8b7 100644
--- a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/MediaState.kt
+++ b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/MediaState.kt
@@ -1,6 +1,6 @@
 package dev.anilbeesetti.nextplayer.feature.videopicker.screens
 
-import dev.anilbeesetti.nextplayer.core.model.Directory
+import dev.anilbeesetti.nextplayer.core.model.Folder
 import dev.anilbeesetti.nextplayer.core.model.Video
 
 sealed interface VideosState {
@@ -10,5 +10,5 @@ sealed interface VideosState {
 
 sealed interface FoldersState {
     object Loading : FoldersState
-    data class Success(val data: List<Directory>) : FoldersState
+    data class Success(val data: List<Folder>) : FoldersState
 }
diff --git a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/media/MediaPickerViewModel.kt b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/media/MediaPickerViewModel.kt
index 1ff09d184..0897bbaf1 100644
--- a/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/media/MediaPickerViewModel.kt
+++ b/feature/videopicker/src/main/java/dev/anilbeesetti/nextplayer/feature/videopicker/screens/media/MediaPickerViewModel.kt
@@ -7,7 +7,7 @@ import androidx.lifecycle.viewModelScope
 import dagger.hilt.android.lifecycle.HiltViewModel
 import dev.anilbeesetti.nextplayer.core.data.repository.MediaRepository
 import dev.anilbeesetti.nextplayer.core.data.repository.PreferencesRepository
-import dev.anilbeesetti.nextplayer.core.domain.GetSortedDirectoriesUseCase
+import dev.anilbeesetti.nextplayer.core.domain.GetSortedFoldersUseCase
 import dev.anilbeesetti.nextplayer.core.domain.GetSortedVideosUseCase
 import dev.anilbeesetti.nextplayer.core.model.ApplicationPreferences
 import dev.anilbeesetti.nextplayer.feature.videopicker.screens.FoldersState
@@ -21,7 +21,7 @@ import kotlinx.coroutines.launch
 @HiltViewModel
 class MediaPickerViewModel @Inject constructor(
     getSortedVideosUseCase: GetSortedVideosUseCase,
-    getSortedDirectoriesUseCase: GetSortedDirectoriesUseCase,
+    getSortedFoldersUseCase: GetSortedFoldersUseCase,
     private val mediaRepository: MediaRepository,
     private val preferencesRepository: PreferencesRepository
 ) : ViewModel() {
@@ -34,7 +34,7 @@ class MediaPickerViewModel @Inject constructor(
             initialValue = VideosState.Loading
         )
 
-    val foldersState = getSortedDirectoriesUseCase.invoke()
+    val foldersState = getSortedFoldersUseCase.invoke()
         .map { FoldersState.Success(it) }
         .stateIn(
             scope = viewModelScope,