ultrasonic/src/main/kotlin/org/moire/ultrasonic/playback/AutoMediaBrowserCallback.kt

@@ -18,7 +18,6 @@ import androidx.media3.common.MediaMetadata.FOLDER_TYPE_MIXED
 import androidx.media3.common.MediaMetadata.FOLDER_TYPE_PLAYLISTS
 import androidx.media3.common.MediaMetadata.FOLDER_TYPE_TITLES
 import androidx.media3.common.Rating
-import androidx.media3.common.StarRating
 import androidx.media3.session.CommandButton
 import androidx.media3.session.LibraryResult
 import androidx.media3.session.MediaLibraryService
@@ -297,11 +296,7 @@ class AutoMediaBrowserCallback(val libraryService: MediaLibraryService) :
     ): ListenableFuture<SessionResult> {
         val mediaItem = session.player.currentMediaItem
         if (mediaItem != null) {
-            if (rating is HeartRating) {
+            mediaItem.toTrack().starred = (rating as HeartRating).isHeart
-                mediaItem.toTrack().starred = rating.isHeart
-            } else if (rating is StarRating) {
-                mediaItem.toTrack().userRating = rating.starRating.toInt()
-            }
             return onSetRating(
                 session,
                 controller,

ultrasonic/src/main/kotlin/org/moire/ultrasonic/service/DownloadTask.kt

@@ -92,8 +92,8 @@ class DownloadTask(
         // Attempt partial HTTP GET, appending to the file if it exists.
         val (inStream, isPartial) = musicService.getDownloadInputStream(
             downloadTrack.track, fileLength,
-            if (downloadTrack.pinned) Settings.maxBitRatePinning else Settings.maxBitRate,
+            Settings.maxBitRate,
-            downloadTrack.pinned && Settings.pinWithHighestQuality
+            downloadTrack.pinned
         )
 
         inputStream = inStream

ultrasonic/src/main/kotlin/org/moire/ultrasonic/util/Settings.kt

@@ -33,23 +33,18 @@ object Settings {
     val maxBitRate: Int
         get() {
             return if (Util.isNetworkRestricted()) {
-                maxBitRateMobile
+                maxMobileBitRate
             } else {
-                maxBitRateWifi
+                maxWifiBitRate
             }
         }
 
-    private var maxBitRateWifi
+    private var maxWifiBitRate
         by StringIntSetting(getKey(R.string.setting_key_max_bitrate_wifi))
 
-    private var maxBitRateMobile
+    private var maxMobileBitRate
         by StringIntSetting(getKey(R.string.setting_key_max_bitrate_mobile))
 
-    var maxBitRatePinning
-        by StringIntSetting(getKey(R.string.setting_key_max_bitrate_pinning))
-    val pinWithHighestQuality: Boolean
-        get() = (maxBitRatePinning == 0)
-
     @JvmStatic
     val preloadCount: Int
         get() {

ultrasonic/src/main/res/values/setting_keys.xml

@@ -12,7 +12,6 @@
     <string name="setting_key.show_track_number" translatable="false">showTrackNumber</string>
     <string name="setting_key.max_bitrate_wifi" translatable="false">maxBitrateWifi</string>
     <string name="setting_key.max_bitrate_mobile" translatable="false">maxBitrateMobile</string>
-    <string name="setting_key.max_bitrate_pinning" translatable="false">maxBitratePinning</string>
     <string name="setting_key.cache_size" translatable="false">cacheSize</string>
     <string name="setting_key.custom_cache_location" translatable="false">customCacheLocation</string>
     <string name="setting_key.cache_location" translatable="false">cacheLocation</string>

ultrasonic/src/main/res/values/strings.xml

@@ -233,7 +233,6 @@
     <string name="settings.max_bitrate_mobile">Max Bitrate - Mobile</string>
     <string name="settings.max_bitrate_unlimited">Unlimited</string>
     <string name="settings.max_bitrate_wifi">Max Bitrate - Wi-Fi</string>
-    <string name="settings.max_bitrate_pinning">Max Bitrate - When pinning a song permanently</string>
     <string name="settings.max_songs">Max Songs</string>
     <string name="settings.media_button_summary">Respond to phone, headset and Bluetooth media buttons</string>
     <string name="settings.media_button_title">Media Buttons</string>

ultrasonic/src/main/res/xml/settings.xml

@@ -167,25 +167,18 @@
         a:title="@string/settings.network_title"
         app:iconSpaceReserved="false">
         <ListPreference
-            a:defaultValue="256"
+            a:defaultValue="0"
-            a:entries="@array/maxBitrateNames"
-            a:entryValues="@array/maxBitrateValues"
-            a:key="@string/setting_key.max_bitrate_mobile"
-            a:title="@string/settings.max_bitrate_mobile"
-            app:iconSpaceReserved="false"/>
-        <ListPreference
-            a:defaultValue="320"
             a:entries="@array/maxBitrateNames"
             a:entryValues="@array/maxBitrateValues"
             a:key="@string/setting_key.max_bitrate_wifi"
             a:title="@string/settings.max_bitrate_wifi"
             app:iconSpaceReserved="false"/>
         <ListPreference
-            a:defaultValue="320"
+            a:defaultValue="0"
             a:entries="@array/maxBitrateNames"
             a:entryValues="@array/maxBitrateValues"
-            a:key="@string/setting_key.max_bitrate_pinning"
+            a:key="@string/setting_key.max_bitrate_mobile"
-            a:title="@string/settings.max_bitrate_pinning"
+            a:title="@string/settings.max_bitrate_mobile"
             app:iconSpaceReserved="false"/>
         <CheckBoxPreference
             a:defaultValue="false"