fix flac artwork bug

Author: sergcam

app/src/main/java/dev/secam/simpletag/ui/editor/EditorViewModel.kt

@@ -37,6 +37,7 @@ import dev.secam.simpletag.data.MusicData
 import dev.secam.simpletag.data.SimpleTagField
 import dev.secam.simpletag.data.preferences.PreferencesRepo
 import dev.secam.simpletag.data.preferences.UserPreferences
+import dev.secam.simpletag.util.setArtworkField
 import kotlinx.coroutines.CoroutineExceptionHandler
 import kotlinx.coroutines.Deferred
 import kotlinx.coroutines.Dispatchers
@@ -91,10 +92,10 @@ class EditorViewModel @Inject constructor(
         }!!
     }
 
-    fun isCompatible(ext: String): Boolean {
-        val types = listOf("mp3", "wav", "wave", "dsf", "aiff", "aif", "aifc", "wma", "ogg", "mp4", "m4a", "m4p", "flac")
-        return types.contains(ext)
-    }
+//    fun isCompatible(ext: String): Boolean {
+//        val types = listOf("mp3", "wav", "wave", "dsf", "aiff", "aif", "aifc", "wma", "ogg", "mp4", "m4a", "m4p", "flac")
+//        return types.contains(ext)
+//    }
     fun getArtworkFromUri(contentResolver: ContentResolver, uri: Uri): Artwork?{
         var path: String? = null
         val projection = arrayOf(
@@ -127,7 +128,13 @@ class EditorViewModel @Inject constructor(
                 }
                 val tag = file.tag
                 tag.deleteArtworkField()
-                if (artwork != null) tag.setField(artwork)
+                if (artwork != null) {
+                    if(tag.javaClass == FlacTag().javaClass) {
+                        (tag as FlacTag).setArtworkField(artwork)
+                    }
+                    else tag.setField(artwork)
+
+                }
 
                 for (field in fields) {
                     tag.setField(field.key.fieldKey, field.value.text as String)
@@ -252,6 +259,7 @@ class EditorViewModel @Inject constructor(
         _uiState.update { it.copy(artworkChanged = artworkChanged) }
     }
 
+
 }
 
 data class EditorUiState(
@@ -263,4 +271,5 @@ data class EditorUiState(
     val showBackDialog: Boolean = false,
     val showSaveDialog: Boolean = false,
     val savedFields: List<String> = listOf()
-)
+)
+

app/src/main/java/dev/secam/simpletag/util/SetFlacArtwork.kt

@@ -0,0 +1,37 @@
+/*
+ * Copyright (C) 2025  Sergio Camacho
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ */
+
+package dev.secam.simpletag.util
+
+import org.jaudiotagger.tag.flac.FlacTag
+import org.jaudiotagger.tag.images.Artwork
+import org.jaudiotagger.tag.reference.PictureTypes
+
+fun FlacTag.setArtworkField(artwork: Artwork) {
+    setField(
+        createArtworkField(
+            /* imageData = */ artwork.binaryData,
+            /* pictureType = */ PictureTypes.DEFAULT_ID,
+            /* mimeType = */ artwork.mimeType,
+            /* description = */ "cover",
+            /* width = */ artwork.width,
+            /* height = */ artwork.height,
+            /* colourDepth = */ 24,
+            /* indexedColouredCount = */ 0
+        )
+    )
+}