github/Bose-SoundTouch
1
0
Fork 0
mirror of https://github.com/gesellix/Bose-SoundTouch.git synced 2026-09-07 15:07:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(tunein): consolidate the Items/body fallback into one helper

Browse Source 
TuneInSearch, TuneInSearchNext, and TuneInNavigateProfile each
duplicated the identical "Items" (v1.3)-then-"body" (legacy) fallback.
Extracted tuneInRawItems, used by all three. Pure refactor, no
behavior change.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
Tobias Gesellchen
2026-09-05 14:24:10 +02:00
co-authored by Claude Sonnet 5
parent cb6929ebec
commit c56fc6189c
1 changed files with 15 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/service/bmx/tunein.go
+15 -14
View File
@@ -99,6 +99,18 @@ func isTuneInOpmlURI(rawURL string) bool {
return strings.EqualFold(u.Hostname(), "opml.radiotime.com")
}
// tuneInRawItems extracts a response's item list, trying "Items" (the
// v1.3 search/profiles API shape) first, then falling back to "body"
// (the legacy/OPML shape).
func tuneInRawItems(data map[string]interface{}) []interface{} {
items, ok := data["Items"].([]interface{})
if !ok {
items, _ = data["body"].([]interface{})
}
return items
}
// tuneInRenderJSONURI returns the URL with render=json set as a query parameter,
// replacing any existing render value instead of appending a duplicate.
func tuneInRenderJSONURI(rawURL string) string {
@@ -350,11 +362,7 @@ func TuneInSearch(query string) (*models.BmxNavResponse, error) {
Layout: "classic",
}
// Try "Items" (v1.3) first, then "body" (legacy)
items, ok := data["Items"].([]interface{})
if !ok {
items, _ = data["body"].([]interface{})
}
items := tuneInRawItems(data)
for idx, item := range items {
m, ok := item.(map[string]interface{})
@@ -479,10 +487,7 @@ func TuneInSearchNext(encodedCursor string) (*models.BmxNavResponse, error) {
return nil, err
}
rawItems, ok := data["Items"].([]interface{})
if !ok {
rawItems, _ = data["body"].([]interface{})
}
rawItems := tuneInRawItems(data)
navItems := make([]models.BmxNavItem, 0, len(rawItems))
for _, raw := range rawItems {
@@ -662,11 +667,7 @@ func TuneInNavigateProfile(encodedURI string) (*models.BmxNavResponse, error) {
return nil, err
}
// "Items" (v1.3 profiles/search API) first, then "body" (legacy/OPML).
rawItems, ok := contents["Items"].([]interface{})
if !ok {
rawItems, _ = contents["body"].([]interface{})
}
rawItems := tuneInRawItems(contents)
// The Contents pivot doesn't return playable items directly: each
// top-level entry is a "Container" (identified by a "ContainerType"