Add offset modes (direct/lrc tag)

Lambada10 · Oct 29, 2024 · 412fafc · 412fafc
1 parent e0d0732
commit 412fafc
Show file tree

Hide file tree

Showing 8 changed files with 66 additions and 10 deletions.
diff --git a/app/src/main/java/pl/lambada/songsync/data/remote/UserSettingsController.kt b/app/src/main/java/pl/lambada/songsync/data/remote/UserSettingsController.kt
@@ -52,6 +52,9 @@ class UserSettingsController(private val dataStore: DataStore<Preferences>) {
     var showPath by mutableStateOf(dataStore.get(showPathKey, false))
         private set
 
+    var directlyModifyTimestamps by mutableStateOf(dataStore.get(directlyModifyTimestampsKey, false))
+        private set
+
     var sortOrder by mutableStateOf(
         SortOrders.entries
             .find { it.queryName == dataStore.get(sortOrderKey, SortOrders.ASCENDING.queryName) }!!
@@ -119,6 +122,11 @@ class UserSettingsController(private val dataStore: DataStore<Preferences>) {
         showPath = to
     }
 
+    fun updateDirectlyModifyTimestamps(to: Boolean) {
+        dataStore.set(directlyModifyTimestampsKey, to)
+        directlyModifyTimestamps = to
+    }
+
     fun updateSortOrder(to: SortOrders) {
         dataStore.set(sortOrderKey, to.queryName)
         sortOrder = to
@@ -142,4 +150,5 @@ private val pureBlackKey = booleanPreferencesKey("pure_black")
 private val sdCardPathKey = stringPreferencesKey("sd_card_path")
 private val showPathKey = booleanPreferencesKey("show_path")
 private val sortOrderKey = stringPreferencesKey("sort_order")
-private val sortByKey = stringPreferencesKey("sort_by")
+private val sortByKey = stringPreferencesKey("sort_by")
+private val directlyModifyTimestampsKey = booleanPreferencesKey("directly_modify_timestamps")
diff --git a/app/src/main/java/pl/lambada/songsync/ui/screens/about/AboutScreen.kt b/app/src/main/java/pl/lambada/songsync/ui/screens/about/AboutScreen.kt
@@ -26,6 +26,7 @@ import pl.lambada.songsync.ui.screens.about.components.CreditsSection
 import pl.lambada.songsync.ui.screens.about.components.ExternalLinkSection
 import pl.lambada.songsync.ui.screens.about.components.MarqueeSwitch
 import pl.lambada.songsync.ui.screens.about.components.MultiPersonSwitch
+import pl.lambada.songsync.ui.screens.about.components.OffsetModeSwitch
 import pl.lambada.songsync.ui.screens.about.components.PureBlackThemeSwitch
 import pl.lambada.songsync.ui.screens.about.components.SdCardPathSetting
 import pl.lambada.songsync.ui.screens.about.components.ShowPathSwitch
@@ -105,6 +106,13 @@ fun AboutScreen(
                 )
             }
 
+            item {
+                OffsetModeSwitch(
+                    selected = userSettingsController.directlyModifyTimestamps,
+                    onToggle = { userSettingsController.updateDirectlyModifyTimestamps(it) }
+                )
+            }
+
             if (Build.VERSION.SDK_INT < Build.VERSION_CODES.R) {
                 item {
                     SdCardPathSetting(

diff --git a/app/src/main/java/pl/lambada/songsync/ui/screens/about/components/OffsetModeSwitch.kt b/app/src/main/java/pl/lambada/songsync/ui/screens/about/components/OffsetModeSwitch.kt
@@ -0,0 +1,21 @@
+package pl.lambada.songsync.ui.screens.about.components
+
+import androidx.compose.runtime.Composable
+import androidx.compose.ui.res.stringResource
+import pl.lambada.songsync.R
+import pl.lambada.songsync.ui.components.AboutItem
+import pl.lambada.songsync.ui.components.SwitchItem
+
+@Composable
+fun OffsetModeSwitch(
+    selected: Boolean,
+    onToggle: (Boolean) -> Unit
+) {
+    AboutItem(label = stringResource(R.string.offset_mode)) {
+        SwitchItem(
+            label = stringResource(R.string.offset_mode_summary),
+            selected = selected,
+            onClick = { onToggle(!selected) }
+        )
+    }
+}
diff --git a/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/LyricsFetchScreen.kt b/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/LyricsFetchScreen.kt
@@ -152,6 +152,7 @@ fun SharedTransitionScope.LyricsFetchScreen(
                             viewModel.lrcOffset = 0
                             viewModel.queryState = QueryStatus.NotSubmitted
                         },
+                        directOffset = viewModel.userSettingsController.directlyModifyTimestamps,
                         offset = viewModel.lrcOffset,
                         onSetOffset = { viewModel.lrcOffset = it },
                         onSaveLyrics = {

diff --git a/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/LyricsFetchViewModel.kt b/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/LyricsFetchViewModel.kt
@@ -86,7 +86,7 @@ class LyricsFetchViewModel(
         context: Context,
         generatedUsingString: String
     ) {
-        val lrcContent = generateLrcContent(song, lyrics, generatedUsingString, lrcOffset)
+        val lrcContent = generateLrcContent(song, lyrics, generatedUsingString, lrcOffset, userSettingsController.directlyModifyTimestamps)
         val file = newLyricsFilePath(filePath, song)
 
         if (!isLegacyFileAccessRequired(filePath)) {
@@ -127,7 +127,7 @@ class LyricsFetchViewModel(
         context: Context,
         song: SongInfo
     ) {
-        val lrcContent = generateLrcContent(song, lyrics, context.getString(R.string.generated_using), lrcOffset)
+        val lrcContent = generateLrcContent(song, lyrics, context.getString(R.string.generated_using), lrcOffset, userSettingsController.directlyModifyTimestamps)
 
         runCatching {
             embedLyricsInFile(

diff --git a/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/components/SuccessContent.kt b/app/src/main/java/pl/lambada/songsync/ui/screens/lyricsFetch/components/SuccessContent.kt
@@ -38,6 +38,7 @@ fun SharedTransitionScope.SuccessContent(
     result: SongInfo,
     onTryAgain: () -> Unit,
     onEdit: () -> Unit,
+    directOffset: Boolean,
     offset: Int,
     onSetOffset: (Int) -> Unit,
     onSaveLyrics: (String) -> Unit,
@@ -103,7 +104,13 @@ fun SharedTransitionScope.SuccessContent(
                 LyricsFetchState.NotSubmitted -> { /* nothing */ }
 
                 is LyricsFetchState.Success -> LyricsSuccessContent(
-                    lyrics = applyOffsetToLyrics(it.lyrics, offset),
+                    lyrics = it.lyrics.let {
+                        if (offset != 0 && directOffset) {
+                            applyOffsetToLyrics(it, offset)
+                        } else {
+                            it
+                        }
+                    },
                     offset = offset,
                     onSetOffset = onSetOffset,
                     onSaveLyrics = { onSaveLyrics(it.lyrics) },

diff --git a/app/src/main/java/pl/lambada/songsync/util/LyricsUtils.kt b/app/src/main/java/pl/lambada/songsync/util/LyricsUtils.kt
@@ -26,12 +26,17 @@ fun generateLrcContent(
     lyrics: String,
     generatedUsingString: String,
     offset: Int = 0,
+    directOffset: Boolean
 ): String {
-        return ("[ti:${song.songName}]\n" +
+    val offsetSign = if (offset >= 0) "+" else ""
+    val offsetStr = if (!directOffset) "[offset:${offsetSign}${offset}]\n" else ""
+    val lyrics = if (directOffset && offset != 0) applyOffsetToLyrics(lyrics, offset) else lyrics
+
+    return "[ti:${song.songName}]\n" +
         "[ar:${song.artistName}]\n" +
-        "[by:$generatedUsingString]\n" + lyrics).let {
-            if (offset != 0) applyOffsetToLyrics(it, offset) else it
-        }
+        offsetStr +
+        "[by:$generatedUsingString]\n" +
+        lyrics
 }
 
 fun newLyricsFilePath(filePath: String?, song: SongInfo): File {
@@ -251,7 +256,8 @@ private suspend fun downloadLyricsForSong(
                         context,
                         viewModel.userSettingsController.sdCardPath,
                         songInfo,
-                        it
+                        it,
+                        viewModel.userSettingsController.directlyModifyTimestamps
                     )
 
                     onLyricsSaved()
@@ -265,13 +271,15 @@ private fun formatAndSaveLyricsForSong(
     sdCardPath: String?,
     songInfo: SongInfo,
     lyrics: String,
+    directOffset: Boolean
 ) {
     val targetFile = song.filePath.toLrcFile()
 
     val lrcContent = generateLrcContent(
         songInfo,
         lyrics,
-        context.getString(R.string.generated_using)
+        context.getString(R.string.generated_using),
+        directOffset = directOffset
     )
 
     writeLyricsToFile(targetFile, lrcContent, context, song, sdCardPath)

diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
@@ -138,4 +138,6 @@
     <string name="ascending">Ascending</string>
     <string name="descending">Descending</string>
     <string name="sort">Sort</string>
+    <string name="offset_mode">Offset mode</string>
+    <string name="offset_mode_summary">Directly modify the lyrics timestamps instead of using offset LRC tag</string>
 </resources>