deps: upgrade deps

aaaaaaa
coil -> 2.0.0-rc03
material -> 1.7.0-alpha01
leakcanary -> 2.9.1
navigation -> 2.4.2
kotlin -> 1.6.21
agp -> 7.1.3

app/build.gradle

@@ -84,7 +84,7 @@ dependencies {
 
     // Media
     // TODO: Dumpster this for Media3
-    implementation "androidx.media:media:1.5.0"
+    implementation "androidx.media:media:1.6.0"
 
     // Preferences
     implementation "androidx.preference:preference-ktx:1.2.0"
@@ -99,13 +99,13 @@ dependencies {
     implementation fileTree(dir: "libs", include: ["extension-*.aar"])
 
     // Image loading
-    implementation "io.coil-kt:coil:2.0.0-rc02"
+    implementation "io.coil-kt:coil:2.0.0-rc03"
 
     // Material
-    implementation "com.google.android.material:material:1.6.0-beta01"
+    implementation "com.google.android.material:material:1.7.0-alpha01"
 
     // LeakCanary
-    debugImplementation "com.squareup.leakcanary:leakcanary-android:2.8.1"
+    debugImplementation "com.squareup.leakcanary:leakcanary-android:2.9.1"
 }
 
 spotless {

app/src/main/java/org/oxycblt/auxio/MainActivity.kt

@@ -46,9 +46,11 @@ import org.oxycblt.auxio.util.systemBarInsetsCompat
  *
  * TODO: Rework some fragments to use listeners *even more*
  *
- * TODO: Unify all member variables under an m prefix
+ * TODO: Phase out m for _
  *
  * TODO: Fix how selection works in the RecyclerViews (doing it poorly right now)
+ *
+ * TODO: Rework padding ethos
  */
 class MainActivity : AppCompatActivity() {
     private val playbackModel: PlaybackViewModel by viewModels()

app/src/main/java/org/oxycblt/auxio/music/Indexer.kt

@@ -88,6 +88,9 @@ import org.oxycblt.auxio.util.logD
  * I wish I was born in the neolithic.
  *
  * @author OxygenCobalt
+ *
+ * TODO: Experiment with making the exoplayer metadata system more effificent so that we could
+ * leverage it here (maybe)
  */
 object Indexer {
     /**

app/src/main/java/org/oxycblt/auxio/music/Music.kt

@@ -21,7 +21,6 @@ import android.content.ContentUris
 import android.content.Context
 import android.net.Uri
 import android.provider.MediaStore
-import android.text.TextUtils.isDigitsOnly
 import androidx.core.text.isDigitsOnly
 import org.oxycblt.auxio.R
 import org.oxycblt.auxio.ui.Item

app/src/main/java/org/oxycblt/auxio/playback/PlaybackPanelFragment.kt

@@ -49,6 +49,8 @@ import org.oxycblt.auxio.util.textSafe
  * @author OxygenCobalt
  *
  * TODO: Handle RTL correctly in the playback buttons
+ *
+ * TODO: Make seek thumb grow when selected
  */
 class PlaybackPanelFragment :
     ViewBindingFragment<FragmentPlaybackPanelBinding>(),

app/src/main/java/org/oxycblt/auxio/playback/state/PlaybackStateDatabase.kt

@@ -102,7 +102,7 @@ class PlaybackStateDatabase(context: Context) :
 
     /**
      * Read the stored [SavedState] from the database, if there is one.
-     * @param musicStore Required to transform database songs/parents into actual instances
+     * @param library Required to transform database songs/parents into actual instances
      * @return The stored [SavedState], null if there isn't one.
      */
     fun readState(library: MusicStore.Library): SavedState? {

app/src/main/java/org/oxycblt/auxio/playback/system/MediaSessionComponent.kt

@@ -127,7 +127,7 @@ class MediaSessionComponent(private val context: Context, private val player: Pl
         onSongChanged(playbackManager.song)
     }
 
-    fun onSongChanged(song: Song?) {
+    private fun onSongChanged(song: Song?) {
         if (song == null) {
             mediaSession.setMetadata(emptyMetadata)
             return

app/src/main/java/org/oxycblt/auxio/playback/system/PlaybackService.kt

@@ -166,9 +166,9 @@ class PlaybackService :
         unregisterReceiver(systemReceiver)
 
         serviceJob.cancel()
-        player.release()
         mediaSessionComponent.release()
         widgets.release()
+        player.release()
 
         playbackManager.removeCallback(this)
         settingsManager.removeCallback(this)
@@ -385,7 +385,6 @@ class PlaybackService :
     private fun stopAndSave() {
         stopForeground(true)
         isForeground = false
-
         saveScope.launch { playbackManager.saveStateToDatabase(this@PlaybackService) }
     }
 
@@ -406,8 +405,7 @@ class PlaybackService :
                 // a non-starter since both require me to display a permission prompt
                 // 4. Some weird internal framework thing that also handles bluetooth headsets???
                 //
-                // They should have just stopped at ACTION_HEADSET_PLUG. Just use 1 and 2 so that
+                // They should have just stopped at ACTION_HEADSET_PLUG.
-                // *something* fills in the role.
                 AudioManager.ACTION_HEADSET_PLUG -> {
                     when (intent.getIntExtra("state", -1)) {
                         0 -> pauseFromPlug()

app/src/main/java/org/oxycblt/auxio/settings/SettingsListFragment.kt

@@ -45,6 +45,9 @@ import org.oxycblt.auxio.util.systemBarInsetsCompat
 /**
  * The actual fragment containing the settings menu. Inherits [PreferenceFragmentCompat].
  * @author OxygenCobalt
+ *
+ * TODO: Add option to restore state
+ * TODO: Add option to not restore state
  */
 @Suppress("UNUSED")
 class SettingsListFragment : PreferenceFragmentCompat() {

app/src/main/java/org/oxycblt/auxio/settings/SettingsManager.kt

@@ -247,7 +247,6 @@ class SettingsManager private constructor(context: Context) :
      */
     interface Callback {
         fun onLibTabsUpdate(libTabs: Array<Tab>) {}
-        fun onColorizeNotifUpdate(doColorize: Boolean) {}
         fun onNotifActionUpdate(useAltAction: Boolean) {}
         fun onShowCoverUpdate(showCovers: Boolean) {}
         fun onQualityCoverUpdate(doQualityCovers: Boolean) {}

app/src/main/java/org/oxycblt/auxio/widgets/Forms.kt

@@ -26,6 +26,8 @@ import org.oxycblt.auxio.playback.system.PlaybackService
 import org.oxycblt.auxio.util.newBroadcastIntent
 import org.oxycblt.auxio.util.newMainIntent
 
+// TODO: Still need to change the default cover here
+
 /**
  * The default widget is displayed whenever there is no music playing. It just shows the message "No
  * music playing".

app/src/main/res/values/integers.xml

@@ -2,7 +2,7 @@
 <resources>
     <integer name="detail_app_bar_title_anim_duration">150</integer>
 
-    <!-- FIXME: Unify this with the integer table object by simply defining a class for each dependency. Will reduce bugs. -->
+    <!-- FIXME: Unify this with the integer table object by simply defining a class for each setting. -->
     <!-- Preference values -->
     <string-array name="entries_theme">
         <item>@string/set_theme_auto</item>

build.gradle

@@ -1,7 +1,7 @@
 // Top-level build file where you can add configuration options common to all sub-projects/modules.
 buildscript {
-    ext.kotlin_version = '1.6.10'
+    ext.kotlin_version = '1.6.21'
-    ext.navigation_version = "2.4.1"
+    ext.navigation_version = "2.4.2"
 
     repositories {
         google()
@@ -9,7 +9,7 @@ buildscript {
     }
 
     dependencies {
-        classpath 'com.android.tools.build:gradle:7.1.2'
+        classpath 'com.android.tools.build:gradle:7.1.3'
         classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
         classpath "androidx.navigation:navigation-safe-args-gradle-plugin:$navigation_version"
         classpath "com.diffplug.spotless:spotless-plugin-gradle:6.3.0"