Updated ratings to use the local icon first then fall back to server side icons

RadicalMuffinMan · RadicalMuffinMan · commit c4689ba68fb4 · 2026-02-19T10:51:55.000-05:00
diff --git a/app/src/main/java/org/jellyfin/androidtv/data/repository/RatingIconProvider.kt b/app/src/main/java/org/jellyfin/androidtv/data/repository/RatingIconProvider.kt
@@ -10,12 +10,17 @@ object RatingIconProvider {
 	}
 
 	fun getIcon(baseUrl: String?, source: String, scorePercent: Int? = null): RatingIcon? {
+		// Prefer local drawable when available (always works, no plugin dependency)
+		val localIcon = getLocalFallbackIcon(source, scorePercent)
+		if (localIcon != null) return localIcon
+
+		// Fall back to server-hosted icon (requires Moonfin plugin)
 		if (baseUrl != null) {
 			getServerIconFile(source, scorePercent)?.let { file ->
 				return RatingIcon.ServerUrl("$baseUrl/Moonfin/Assets/$file")
 			}
 		}
-		return getLocalFallbackIcon(source, scorePercent)
+		return null
 	}
 
 	private fun getServerIconFile(source: String, scorePercent: Int?): String? = when (source) {

Original file line number	Diff line number	Diff line change
`@@ -10,12 +10,17 @@ object RatingIconProvider {`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`fun getIcon(baseUrl: String?, source: String, scorePercent: Int? = null): RatingIcon? {`
	`13`	`+ // Prefer local drawable when available (always works, no plugin dependency)`
	`14`	`+ val localIcon = getLocalFallbackIcon(source, scorePercent)`
	`15`	`+ if (localIcon != null) return localIcon`
	`16`	`+`
	`17`	`+ // Fall back to server-hosted icon (requires Moonfin plugin)`
`13`	`18`	`if (baseUrl != null) {`
`14`	`19`	`getServerIconFile(source, scorePercent)?.let { file ->`
`15`	`20`	`return RatingIcon.ServerUrl("$baseUrl/Moonfin/Assets/$file")`
`16`	`21`	`}`
`17`	`22`	`}`
`18`		`- return getLocalFallbackIcon(source, scorePercent)`
	`23`	`+ return null`
`19`	`24`	`}`
`20`	`25`
`21`	`26`	`private fun getServerIconFile(source: String, scorePercent: Int?): String? = when (source) {`