QA에서 발견한 버그 개선 및 MyPage -> Player 이동

android/app/src/main/AndroidManifest.xml

@@ -15,7 +15,8 @@
         android:usesCleartextTraffic="true">
         <activity
             android:name=".feature.login.LoginActivity"
-            android:exported="true">
+            android:exported="true"
+            android:screenOrientation="portrait">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />
 
@@ -25,7 +26,8 @@
 
         <activity
             android:name=".MainActivity"
-            android:exported="true" />
+            android:exported="true"
+            android:screenOrientation="portrait" />
 
         <service
             android:name=".mediasession.PlaybackService"

android/core/domain/src/main/java/com/ohdodok/catchytape/core/domain/utils/CoroutineUtils.kt

@@ -0,0 +1,21 @@
+package com.ohdodok.catchytape.core.domain.utils
+
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.flow
+
+fun <T> Flow<T>.throttleFist(windowDuration: Long): Flow<T> = flow {
+    var windowStartTime = System.currentTimeMillis()
+    var emitted = false
+    collect { value ->
+        val currentTime = System.currentTimeMillis()
+        val delta = currentTime - windowStartTime
+        if (delta >= windowDuration) {
+            windowStartTime += delta / windowDuration * windowDuration
+            emitted = false
+        }
+        if (!emitted) {
+            emit(value)
+            emitted = true
+        }
+    }
+}

android/core/ui/src/main/java/com/ohdodok/catchytape/core/ui/BaseFragment.kt

@@ -17,6 +17,9 @@ import androidx.navigation.fragment.findNavController
 import com.google.android.material.appbar.MaterialToolbar
 import com.google.android.material.snackbar.Snackbar
 import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.channels.awaitClose
+import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.flow.callbackFlow
 import kotlinx.coroutines.launch
 
 abstract class BaseFragment<VB : ViewDataBinding>(
@@ -41,7 +44,7 @@ abstract class BaseFragment<VB : ViewDataBinding>(
         _binding = null
     }
 
-    protected fun setupBackStack(toolbar: MaterialToolbar){
+    protected fun setupBackStack(toolbar: MaterialToolbar) {
         toolbar.setNavigationOnClickListener {
             findNavController().popBackStack()
         }
@@ -57,4 +60,9 @@ abstract class BaseFragment<VB : ViewDataBinding>(
         Snackbar.make(this.requireView(), messageId, Snackbar.LENGTH_LONG).show()
     }
 
+    protected fun clicksFlow(view: View): Flow<Unit> = callbackFlow {
+        view.setOnClickListener { trySend(Unit) }
+        awaitClose { view.setOnClickListener(null) }
+    }
+
 }

android/core/ui/src/main/res/layout/bottom_sheet_playlist.xml

@@ -4,14 +4,15 @@
     xmlns:tools="http://schemas.android.com/tools">
 
     <data>
+
         <variable
             name="viewModel"
             type="com.ohdodok.catchytape.core.ui.PlaylistBottomSheetViewModel" />
     </data>
 
     <androidx.constraintlayout.widget.ConstraintLayout
         android:layout_width="match_parent"
-        android:layout_height="wrap_content"
+        android:layout_height="match_parent"
         android:background="@color/surface_bright">
 
         <TextView
@@ -43,11 +44,9 @@
             android:layout_height="wrap_content"
             android:layout_marginTop="8dp"
             android:layout_marginBottom="8dp"
-            android:maxHeight="400dp"
             android:orientation="vertical"
             android:paddingHorizontal="@dimen/margin_horizontal"
             app:layoutManager="androidx.recyclerview.widget.LinearLayoutManager"
-            app:layout_constraintBottom_toBottomOf="parent"
             app:layout_constraintEnd_toEndOf="parent"
             app:layout_constraintStart_toStartOf="parent"
             app:layout_constraintTop_toBottomOf="@id/divider"

android/feature/home/src/main/java/com/ohdodok/catchytape/feature/home/HomeFragment.kt

@@ -69,5 +69,5 @@ private fun NavController.navigateToPlayerScreen() {
         NavDeepLinkRequest.Builder.fromUri("android-app://com.ohdodok.catchytape/player_fragment".toUri())
             .build()
 
-    this.navigate(request)
+    navigate(request)
 }

android/feature/mypage/src/main/java/com/ohdodok/catchytape/feature/mypage/MyPageFragment.kt

@@ -5,6 +5,7 @@ import android.view.View
 import androidx.core.net.toUri
 import androidx.core.view.ViewCompat
 import androidx.fragment.app.viewModels
+import androidx.navigation.NavController
 import androidx.navigation.NavDeepLinkRequest
 import androidx.navigation.fragment.findNavController
 import com.ohdodok.catchytape.core.ui.BaseFragment
@@ -37,6 +38,9 @@ class MyPageFragment : BaseFragment<FragmentMyPageBinding>(R.layout.fragment_my_
                     is MyPageEvent.ShowMessage -> {
                         showMessage(event.error.toMessageId())
                     }
+                    is MyPageEvent.NavigateToPlayerScreen -> {
+                        findNavController().navigateToPlayerScreen()
+                    }
                 }
             }
         }
@@ -56,11 +60,20 @@ class MyPageFragment : BaseFragment<FragmentMyPageBinding>(R.layout.fragment_my_
     }
 
     private fun setupRecyclerView() {
-        binding.rvMusics.adapter = MusicAdapter(Orientation.VERTICAL)
+        binding.rvMusics.adapter = MusicAdapter(
+            musicItemOrientation = Orientation.VERTICAL,
+            listener = viewModel
+        )
     }
 
     override fun onStart() {
         super.onStart()
         viewModel.fetchMyMusics(count = 3)
     }
+}
+
+private fun NavController.navigateToPlayerScreen() {
+    val request =
+        NavDeepLinkRequest.Builder.fromUri("android-app://com.ohdodok.catchytape/player_fragment".toUri()).build()
+    navigate(request)
 }

android/feature/mypage/src/main/java/com/ohdodok/catchytape/feature/mypage/MyPageViewModel.kt

@@ -6,6 +6,8 @@ import com.ohdodok.catchytape.core.domain.model.CtErrorType
 import com.ohdodok.catchytape.core.domain.model.CtException
 import com.ohdodok.catchytape.core.domain.model.Music
 import com.ohdodok.catchytape.core.domain.repository.MusicRepository
+import com.ohdodok.catchytape.core.domain.usecase.player.CurrentPlaylistUseCase
+import com.ohdodok.catchytape.core.ui.MusicAdapter
 import dagger.hilt.android.lifecycle.HiltViewModel
 import kotlinx.coroutines.CoroutineExceptionHandler
 import kotlinx.coroutines.flow.MutableSharedFlow
@@ -28,7 +30,8 @@ data class MyPageUiState(
 @HiltViewModel
 class MyPageViewModel @Inject constructor(
     private val musicRepository: MusicRepository,
-) : ViewModel() {
+    private val currentPlaylistUseCase: CurrentPlaylistUseCase,
+) : ViewModel(), MusicAdapter.Listener {
 
     private val _uiState = MutableStateFlow(MyPageUiState())
     val uiState: StateFlow<MyPageUiState> = _uiState.asStateFlow()
@@ -55,8 +58,16 @@ class MyPageViewModel @Inject constructor(
             }
             .launchIn(viewModelScopeWithExceptionHandler)
     }
+
+    override fun onClick(music: Music) {
+        currentPlaylistUseCase.playMusic(music)
+        viewModelScope.launch {
+            _events.emit(MyPageEvent.NavigateToPlayerScreen)
+        }
+    }
 }
 
 sealed interface MyPageEvent {
     data class ShowMessage(val error: CtErrorType) : MyPageEvent
+    data object NavigateToPlayerScreen : MyPageEvent
 }

android/feature/player/src/main/java/com/ohdodok/catchytape/feature/player/PlayerFragment.kt

@@ -5,13 +5,17 @@ import android.view.View
 import android.widget.SeekBar
 import androidx.core.view.ViewCompat
 import androidx.fragment.app.activityViewModels
+import androidx.lifecycle.lifecycleScope
 import androidx.media3.exoplayer.ExoPlayer
 import androidx.navigation.NavController
 import androidx.navigation.fragment.findNavController
+import com.ohdodok.catchytape.core.domain.utils.throttleFist
 import com.ohdodok.catchytape.core.ui.BaseFragment
 import com.ohdodok.catchytape.core.ui.RootViewInsetsCallback
 import com.ohdodok.catchytape.feature.player.databinding.FragmentPlayerBinding
 import dagger.hilt.android.AndroidEntryPoint
+import kotlinx.coroutines.flow.launchIn
+import kotlinx.coroutines.flow.onEach
 import javax.inject.Inject
 
 const val millisecondsPerSecond = 1000
@@ -68,18 +72,21 @@ class PlayerFragment : BaseFragment<FragmentPlayerBinding>(R.layout.fragment_pla
         }
 
         binding.btnAddToPlaylist.setOnClickListener {
-            findNavController().showPlaylistBottomSheet()
+            clicksFlow(view = it).throttleFist(500)
+                .onEach {
+                    findNavController().showPlaylistBottomSheet()
+                }.launchIn(viewLifecycleOwner.lifecycleScope)
         }
     }
 
     private fun NavController.showPlaylistBottomSheet() {
         val musicId = viewModel.uiState.value.currentMusic?.id ?: return
-
         val action = PlayerFragmentDirections.actionPlayerFragmentToPlaylistBottomSheet(musicId = musicId)
-        this.navigate(action)
+        navigate(action)
     }
 }
 
 fun NavController.navigateToPlayer() {
-    this.navigate(R.id.player_nav_graph)
+    navigate(R.id.player_nav_graph)
 }
+

android/feature/player/src/main/java/com/ohdodok/catchytape/feature/player/PlayerViewModel.kt

@@ -1,6 +1,7 @@
 package com.ohdodok.catchytape.feature.player
 
 import androidx.lifecycle.ViewModel
+
 import androidx.lifecycle.viewModelScope
 import com.ohdodok.catchytape.core.domain.model.CtErrorType
 import com.ohdodok.catchytape.core.domain.model.CtException
@@ -28,7 +29,6 @@ data class PlayerState(
     val duration: Int = 0,
     val isNextEnable: Boolean = false,
     val isPreviousEnable: Boolean = false
-
 ) {
     val isPlayEnable: Boolean
         get() = currentMusic != null

android/feature/playlist/src/main/java/com/ohdodok/catchytape/feature/playlist/PlaylistDetailFragment.kt

@@ -53,5 +53,5 @@ private fun NavController.navigateToPlayerScreen() {
     val targetUri = "android-app://com.ohdodok.catchytape/player_fragment".toUri()
     val request = NavDeepLinkRequest.Builder.fromUri(targetUri).build()
 
-    this.navigate(request)
+    navigate(request)
 }

android/feature/playlist/src/main/java/com/ohdodok/catchytape/feature/playlist/PlaylistsFragment.kt

@@ -5,14 +5,18 @@ import android.view.View
 import androidx.core.view.ViewCompat
 import androidx.fragment.app.DialogFragment
 import androidx.fragment.app.viewModels
+import androidx.lifecycle.lifecycleScope
 import androidx.navigation.NavController
 import androidx.navigation.fragment.findNavController
+import com.ohdodok.catchytape.core.domain.utils.throttleFist
 import com.ohdodok.catchytape.core.ui.BaseFragment
 import com.ohdodok.catchytape.core.ui.PlaylistAdapter
 import com.ohdodok.catchytape.core.ui.RootViewInsetsCallback
 import com.ohdodok.catchytape.core.ui.cterror.toMessageId
 import com.ohdodok.catchytape.feature.playlist.databinding.FragmentPlaylistsBinding
 import dagger.hilt.android.AndroidEntryPoint
+import kotlinx.coroutines.flow.launchIn
+import kotlinx.coroutines.flow.onEach
 
 @AndroidEntryPoint
 class PlaylistsFragment : BaseFragment<FragmentPlaylistsBinding>(R.layout.fragment_playlists),
@@ -28,11 +32,16 @@ class PlaylistsFragment : BaseFragment<FragmentPlaylistsBinding>(R.layout.fragme
         viewModel.fetchPlaylists()
 
         observeEvents()
-        val newPlaylistDialog = NewPlaylistDialog()
+        setupButton(NewPlaylistDialog())
+    }
+
+    private fun setupButton(newPlaylistDialog: NewPlaylistDialog) {
         binding.fabNewPlaylist.setOnClickListener {
-            newPlaylistDialog.show(childFragmentManager, NewPlaylistDialog.TAG)
+            clicksFlow(view = it).throttleFist(500)
+                .onEach {
+                    newPlaylistDialog.show(childFragmentManager, NewPlaylistDialog.TAG)
+                }.launchIn(viewLifecycleOwner.lifecycleScope)
         }
-
     }
 
     private fun observeEvents() {

android/feature/search/src/main/java/com/ohdodok/catchytape/feature/search/SearchFragment.kt

@@ -19,19 +19,15 @@ import dagger.hilt.android.AndroidEntryPoint
 
 @AndroidEntryPoint
 class SearchFragment : BaseFragment<FragmentSearchBinding>(R.layout.fragment_search) {
-
     private val viewModel: SearchViewModel by viewModels()
 
     override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
         super.onViewCreated(view, savedInstanceState)
 
         ViewCompat.setOnApplyWindowInsetsListener(binding.root, RootViewInsetsCallback())
-
         binding.viewModel = viewModel
-
         observeEvents()
         setupRecyclerView()
-
     }
 
     private fun setupRecyclerView() {
@@ -65,5 +61,5 @@ private fun NavController.navigateToPlayerScreen() {
         NavDeepLinkRequest.Builder.fromUri("android-app://com.ohdodok.catchytape/player_fragment".toUri())
             .build()
 
-    this.navigate(request)
+    navigate(request)
 }

android/feature/upload/src/main/res/layout/fragment_upload.xml

@@ -113,6 +113,8 @@
             <com.google.android.material.textfield.TextInputEditText
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
+                android:imeOptions="actionDone"
+                android:inputType="text"
                 android:onTextChanged="@{(text, start, before, count) -> viewModel.updateMusicTitle(text)}" />
 
         </com.google.android.material.textfield.TextInputLayout>