library: S2 artwork API misses names with curly apostrophes and Unicode dashes

Aperta Adatta ai principianti
#1,101 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
kotlin

Direzione di ricerca

Inizia da S2ArtworkApi.albumArtworkUrl e artistArtworkUrl in android/mediaprovider/core/src/commonMain/.../S2ArtworkApi.kt, poi esamina i test unitari in :android:mediaprovider:core. Verifica come i nomi vengono trasformati in URL di ricerca e aggiungi test per gli apostrofi, le virgolette e i trattini Unicode elencati. Il lavoro è completato quando questi nomi producono ricerche normalizzate e i test unitari del core passano.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug impact:U2 P2 reach:R2 size:S

Artist/album names with typographic punctuation miss in the S2 artwork API, so those items show placeholders when the media server has no image either.

  • GET /v1/artwork?artist=DMA's&album=How Many Dreams? → 200; the same with DMA’s (U+2019) → 404.
  • S2ArtworkApi.albumArtworkUrl / artistArtworkUrl (android/mediaprovider/core/src/commonMain/.../S2ArtworkApi.kt:21-23) pass the tag text through verbatim. Shared by Android (S2 artwork sources) and iOS (ArtworkUrls).
  • Seen on the owner's iPhone 2026-11-05: 7 S2 API lookups answered 404 during one Home screen load; DMA's tile shows the placeholder. That the library's tag uses U+2019 is (unverified): the App Store build's container and request URLs aren't readable.

Fix: normalise the lookup names before building the URL (‘ ’ ʼ → ', “ ” → ", Unicode hyphens/dashes → -), with unit tests in :android:mediaprovider:core. Cache keys change for affected items only, which is fine (misses aren't cached).

Impact U2 (visible missing covers) · Reach R2 (device repro, one library) · Implemented: no · Design fit: fits · Complexity: S

Lingua principale
Kotlin
Stelle
234
Fork
28
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di timusus/Shuttle2

Tutte le issue di timusus/Shuttle2

Issue simili

Altre issue su Kotlin

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.